Are you sure? From a technical point of view it prevents a NPE if
getModel() == null.
Juergen
On Sun, 02 Jan 2005 20:44:42 +0100, Eelco Hillenius
<[EMAIL PROTECTED]> wrote:
> This (in Component.invalidateModel)
>
> // If the components have the same equals
> identity (which is
> // assumed to be implemented in terms of
> database identity)
> // and component is accessing the same
> property of the model
> if (current.getModel() != null
> && getModel() != null
> && current.getModel().equals(getModel())
> && current.getName().equals(getName()))
> {
> // then make the page holding the
> component stale
> currentPage.setStale(true);
> }
>
> should probably be:
>
> // If the components have the same equals
> identity (which is
> // assumed to be implemented in terms of
> database identity)
> // and component is accessing the same
> property of the model
> if (current.getModelObject() != null
> && getModelObject() != null
> && current.getModel().equals(getModel())
> && current.getName().equals(getName()))
> {
> // then make the page holding the
> component stale
> currentPage.setStale(true);
> }
>
> Agreed?
>
> Eelco
>
> -------------------------------------------------------
> The SF.Net email is sponsored by: Beat the post-holiday blues
> Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek.
> It's fun and FREE -- well, almost....http://www.thinkgeek.com/sfshirt
> _______________________________________________
> Wicket-develop mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/wicket-develop
>
-------------------------------------------------------
The SF.Net email is sponsored by: Beat the post-holiday blues
Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek.
It's fun and FREE -- well, almost....http://www.thinkgeek.com/sfshirt
_______________________________________________
Wicket-develop mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/wicket-develop
