You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by "Eelco Hillenius (JIRA)" <ji...@apache.org> on 2007/08/03 23:13:52 UTC

[jira] Commented: (WICKET-815) add setTemporary(boolean) method to AbstractBehavior class

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

Eelco Hillenius commented on WICKET-815:
----------------------------------------

The problem with that is that it is more state. And we're using lots of behaviors in pages typically, so it makes enough of a difference to consider. The alternative to this setter is what we have today: just override the behavior and method isTemporary. Imho, this is good enough.

> add setTemporary(boolean) method to AbstractBehavior class
> ----------------------------------------------------------
>
>                 Key: WICKET-815
>                 URL: https://issues.apache.org/jira/browse/WICKET-815
>             Project: Wicket
>          Issue Type: Improvement
>          Components: wicket
>    Affects Versions: 1.3.0-beta1, 1.3.0-beta2
>            Reporter: Sean Sullivan
>            Priority: Minor
>
> Currently, org.apache.wicket.behavior.AbstractBehavior does not have a setTemporary(boolean) method.
> I'd like to see setTemporary(boolean) in this class.   It would allow me to do this in my application:
>   		AttributeAppender aa = new AttributeAppender("class", new Model(CSS_CLASS_NAME), " ");
>                 aa.setTemporary(true);
> 		c.add(aa);
>    

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.