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

Rik van der Kleij commented on WICKET-1766:
-------------------------------------------

Yes, I see the problem but I don't understand the solution. Consider the 
situation that during isVisible(return getRowCount()>0) 1 item exists. After 
that the only one is removed before rendering. So the table is visible but 
contains no item.

To mine opinion it is better to have a consistent state than a ambiguous state 
during one request. Cache one time for a request. Data can always change so you 
have to take a snapshot in time.



> Method size() of a data provider is invoked twice.
> --------------------------------------------------
>
>                 Key: WICKET-1766
>                 URL: https://issues.apache.org/jira/browse/WICKET-1766
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket
>    Affects Versions: 1.3.4
>         Environment: Ubuntu
>            Reporter: Rik van der Kleij
>            Assignee: Igor Vaynberg
>            Priority: Minor
>
> Method size() of a data provider is invoked twice during one request. This 
> happens when navigating to a next or latest page in a data table. During 
> debugging I noticed that the onDetach() of AbstractPageableView is invoked an 
> extra time. Implementations of size() normally query the database so it will 
> slow down the performance.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to