Request : Minor Table API Change

Vaadin 6 / 7 / 8 Framework UI
1

Hi,

Would it be possible to change the accessibilit of com.vaadin.ui.Table#resetPageBuffer and com.vaadin.ui.Table#refreshRenderedCells from
private
to
protected
?

We are working with a extension class to Table, which allows us to register column formatters for dates and amounts and so on, which needs to be i18n aware : if the locale changes, I want to refresh all of the “rendered” cells (including those that are in the page buffer). Currently, I am “frigging” this by doing

...
super.valueChange(new Property.ValueChangeEvent() {
      public Property getProperty() {
        return null;
      }
    })
...

which works, but is frankly “icky”.

Cheers,

Charles.

2

Hi,

What you really would like is a method to call to refresh all contents of the Table? I do not think opening up two internal Table methods is the best solution for this. There are other cases where it could be beneficial to be able to trigger a refresh from the code so I created
http://dev.vaadin.com/ticket/3822
.

3

Hi,

Well, it depends on what you mean by “refresh
all
 contents”! I want to force the table to re-render all of the visible (and cached) cells, but not to force the container to get any more data : I am using a PagingContainer, i.e. one that loads and caches pages of data on request - there could be many thousands of rows represented by the table, but only a handful displayed by the table.

IFYSWIM - I am pretty sure you do, and have interpreted my request correctly, but just want to be sure.

Cheers,

Charles.

4

Yes, we are talking about the same thing. Refreshing the data in the table, not in the container and not loading extra data from the container to the table. That is refresh the data that is cached in the Table and shown to the user.

I created
#3822
 for this.

5

Hey i have the same problem did you find a solution ?

6

A new method was added in v6.7.2; call
Table#refreshRowCache()

Cheers,

Charles.

7

Thanks for the help