You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by "Alexander Keul (JIRA)" <ji...@apache.org> on 2014/05/16 13:24:55 UTC

[jira] [Updated] (WICKET-5593) AjaxFormValidatingBehavior attempts to update non-visible feedback panels

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

Alexander Keul updated WICKET-5593:
-----------------------------------

    Attachment: foo.rar

Attaching a simple page demonstrating how it would fail. Feedback panel is a child of a WebMarkupContainer with setVisible(false);

> AjaxFormValidatingBehavior attempts to update non-visible feedback panels
> -------------------------------------------------------------------------
>
>                 Key: WICKET-5593
>                 URL: https://issues.apache.org/jira/browse/WICKET-5593
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket
>    Affects Versions: 6.14.0
>            Reporter: Alexander Keul
>            Priority: Minor
>         Attachments: foo.rar
>
>
> AjaxFormValidatingBehavior#addFeedbackPanels adds all feedback panels on the page to the AjaxRequestTarget, without any check for visibility in the hierarchy or lack of output MarkupId.
> As the behaviour is so broad it should probably have an added check:
> if ( component.isVisibleInHierarchy()) {
>   target.add(component);
> } else {
>   visit.dontGoDeeper();
> }



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