[
https://issues.jboss.org/browse/JBSEAM-3251?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Marek Novotny closed JBSEAM-3251.
---------------------------------
Resolution: Done
I have moved call renderChildren(context, component); up, this is related to 1.
point. Second point is not changed wrt Pete's comment.
> ValidateAllRendererBase not robust enough for dynamic components
> ----------------------------------------------------------------
>
> Key: JBSEAM-3251
> URL: https://issues.jboss.org/browse/JBSEAM-3251
> Project: Seam
> Issue Type: Bug
> Components: JSF Controls
> Affects Versions: 2.0.2.SP1, 2.0.3.CR1, 2.1.0.A1
> Environment: Any
> Reporter: Richard Kennard
> Assignee: Marek Novotny
> Priority: Minor
> Fix For: 2.2.1.CR3
>
> Attachments: ValidateAllRendererBase.java
>
> Original Estimate: 30 minutes
> Remaining Estimate: 30 minutes
>
> org.jboss.seam.ui.renderkit.ValidateAllRendererBase has two minor problems
> (with easy fixes) that stop it working with highly dynamic components:
> 1. doEncodeChildren attaches its ModelValidators before calling
> renderChildren. This misses any subcomponents the children may themselves
> dynamically create during their encodeBegin phase. Moving renderChildren down
> fixes this
> 2. A ModelValidator is only attached if the component has no existing
> Validators. This seems a bit of a blunt instrument. Presumably the idea is to
> stop attaching multiple ModelValidators. If the component happens to have,
> say, a LengthValidator that shouldn't stop ModelValidator entirely.
> I have attached an updated ValidateAllRendererBase. It does not break any
> existing unit tests.
> Disclaimer: my vested interest in this is because of Metawidget
> (http://www.metawidget.org).
--
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
_______________________________________________
seam-issues mailing list
[email protected]
https://lists.jboss.org/mailman/listinfo/seam-issues
