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

Martin Grigorov commented on WICKET-5265:
-----------------------------------------

I remember there was onAdd() in 1.2.x but it has been removed for some reason. 
Ask in dev@ and hopefully someone from the other devs will give more details.

I think having onInitialize() and onAdd() will lead to more confusion.
You can think of onInitialize() as a delayed constructor but I see it as a 
callback method which is called when the component has access to bot its parent 
and its markup. The extra bonus is that you can call overridable methods there, 
like AjaxLink's newAjaxEventBehavior(String event) and be sure that the 
AjaxLink component is fully initialized.

By having onAdd() additionally the application developer will be confused 
because it won't be clear which callback is called first and which second and 
what semantics they provide.

Better ask in dev@. Very few devs are subscribed to JIRA notifications.

> FencedFeedbackPanel is broken with RefreshingView(and it's implementations)
> ---------------------------------------------------------------------------
>
>                 Key: WICKET-5265
>                 URL: https://issues.apache.org/jira/browse/WICKET-5265
>             Project: Wicket
>          Issue Type: Bug
>            Reporter: vineet semwal
>            Assignee: Carl-Eric Menzel
>              Labels: DataView, FencedFeedbackPanel, RefreshingView
>         Attachments: 0001-WICKET-5265-recreate-fence-mark-when-needed.patch, 
> WICKET-5265.patch, second.patch
>
>
> FencedFeedbackPanel doesn't work correctly if inner form(s) are in 
> RefreshingView(or it's implementations)..
> in this case outerform feedbackpanel just starts including messages meant for 
> inner feedbackpanel.
> with ListView FencedFeedbackPanel works correctly..
> actually one user(Mike Dundee) created this issue in quickview 
> https://github.com/vineetsemwal/quickview/issues/19
>  so in that link he has described his problem and pasted the code you can use 
> to reproduce ...
> there i have also explained why it's broken with RefreshingView and it's 
> implementations currently(it's a little complex so i am trying to avoid 
> explaining all again ,also english is not my first language :-) ) 
> thank you !



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to