Hi Martin,

many thanks for your feedback.

I think org.apache.wicket.Component#canCallListenerInterface() should not
receive a Method as parameter anymore, since it could be just one method

the method was called with null in some places previously. Yes, it does fit to 
remove the parameter now.

Same for Behavior.

Dito.

I think we should preserve #onSelectionChanged()
#onSelectionChanged() is more meaningful and overridden in many apps.
...
Here #onSelectionChanged() is final!
Does it make sense to keep its current body in #onRequest() (again final)
and then call overridable #onSelectionChanged() ?

Actually #onSelectionChanged() was marked final in 3 of 5 cases, 
#onSelectionChanged(Boolean) was the actual method to override.
So I think it's ok to drop the parameter-less method.

I'll continue working on the test.

Regards
Sven

Reply via email to