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.
