[ 
https://issues.apache.org/jira/browse/WICKET-2968?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Pedro Santos closed WICKET-2968.
--------------------------------

    Resolution: Fixed

sorry, it is duplicated, when I press the submit the site complained that I'm 
not logged, but it seems that the issue was committed anyway

> Throw an IllegalStateException for developed onInitialize methods
> -----------------------------------------------------------------
>
>                 Key: WICKET-2968
>                 URL: https://issues.apache.org/jira/browse/WICKET-2968
>             Project: Wicket
>          Issue Type: Improvement
>          Components: wicket
>    Affects Versions: 1.4.9
>            Reporter: Pedro Santos
>             Fix For: 1.4.9
>
>
> As discussed on the devs mail list, the oninitialize method may already be 
> implemented by some user. The Component on the next release can throw an 
> IllegalStateException  for those situations. IMO it is better than have this 
> method called twice. I'm sending an patch with the implementation and an test 
> for it.
> On a related note, the onInitialize javadoc says:
>      * Overrides must call supe...@link #onInitialize()}. Usually this should 
> be the first thing an
>      * override does, much like a constructor.
> I just put that super call on the onInitialize overriding methods at the 
> ComponentInitializationTest.

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

Reply via email to