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)