[jira] Updated: (WICKET-579) Need for DataTable not to require size

["Kurt Roekle (JIRA)" <ji...@apache.org>]

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

Kurt Roekle updated WICKET-579:
-------------------------------

    Attachment: wicket-toolbar.jar

source code for my fix

> Need for DataTable not to require size
> --------------------------------------
>
>                 Key: WICKET-579
>                 URL: https://issues.apache.org/jira/browse/WICKET-579
>             Project: Wicket
>          Issue Type: Improvement
>          Components: wicket, wicket-extensions
>    Affects Versions: 1.3.0-beta1
>         Environment: 1.3 snapshot dated 4/25
>            Reporter: Kurt Roekle
>         Attachments: wicket-toolbar.jar
>
>
> Requiring a size query for the DataProvider passed to the DataTable hurts performance and scalability (not to mention, it wont be accurate in a system when a lot of updates/deletes are taking place).  I had hoped to be able to code just a new toolbar and DataProvider that would fix this, but I found I needed changes in DataTable and  AbstractPageableView due to the fact AbstractPageableView caches the row count.  I've made fixes for DataTable and AbstractPageableView and I've included one new interface that would be required.  I've also added an implementation of a new toolbar and DataProvider that will enable DataTable to work without size (I've also included a modified NavigationToolbar (and friends) that could be implemented instead of a new toolbar.  I've tested these changes with my limited knowledge of wicket and they seem to not break any existing code.  I currently can't give you a diff, but the line #'s that changed are as follows:
>   
> AbstractPageableView: 70,97,137,139,142-149
> DataTable: 120-122
> NavigationToolbar: 39,49,60,64,88,104
> PagingNavigator: 48,62-66,81,97,110
> NavigatorLabel: 42,54,74

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