[ 
https://issues.apache.org/jira/browse/WICKET-6347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15943505#comment-15943505
 ] 

Pedro Santos commented on WICKET-6347:
--------------------------------------

Again (I pointed the semantic error in your point earlier today), there's no 
temporary value being detached in this implementation. This empty method is 
rather a placeholder than a default implementation.

Yes it's forcing all renderers to communicate that they fulfill the contract: 
"I can be detached", regardless if they really do, which is another semantic 
error.

> IChoiceRenderer implements IDetachable
> --------------------------------------
>
>                 Key: WICKET-6347
>                 URL: https://issues.apache.org/jira/browse/WICKET-6347
>             Project: Wicket
>          Issue Type: Improvement
>          Components: wicket
>    Affects Versions: 8.0.0-M4
>            Reporter: Sven Meier
>            Assignee: Sven Meier
>            Priority: Trivial
>
> IChoiceRenderer (and other renderers ) don't implement IDetachable currently.
> I've came across situations where this would have been handy, and apparently 
> other users would also welcome such an improvement:
>     
> http://stackoverflow.com/questions/42933128/automatically-detaching-choice-renderer-in-wicket
> I've made the required changes locally and there's no negative impact as far 
> as I can see.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to