[richfaces-issues] [JBoss JIRA] Updated: (RF-3488) DataScrollerPhaseListener searches component tree when not using datascroller component

Nick Belaevski (JIRA) jira-events at lists.jboss.org
Sat Dec 6 13:29:36 EST 2008


     [ https://jira.jboss.org/jira/browse/RF-3488?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Nick Belaevski updated RF-3488:
-------------------------------

    Fix Version/s: Future


Current version checks datascroller component only.

> DataScrollerPhaseListener searches component tree when not using datascroller component
> ---------------------------------------------------------------------------------------
>
>                 Key: RF-3488
>                 URL: https://jira.jboss.org/jira/browse/RF-3488
>             Project: RichFaces
>          Issue Type: Bug
>    Affects Versions: 3.2.0
>         Environment: RichFaces 3.2.0.SR1 + Seam 2.0.1.GA + JBoss AS 4.2.2.GA
>            Reporter: Peter Brewer
>            Assignee: Nick Belaevski
>            Priority: Minor
>             Fix For: Future
>
>
> The org.richfaces.component.DataScrollerPhaseListener adds a phase listener that always searches through the component tree looking for a DataScroller component (to reset the pagination I think), evaluating the isRendered flag on each component.
> I would suggest that, if possible, this should only occur when a DataScroller component is actually present on the page, avoiding the overhead of searching the entire component tree on each request. (Could the DataScroller component store its id in the view root attributes to switch the searching on/ off?)
> Also, the evaluation of the rendered attribute by the phase listener ought to be documented to aid developers.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        



More information about the richfaces-issues mailing list