|
Implementation looks good but theres a problem.
On any operation with pulse view we get multiple messages in log:
WARNING: You are using IndexedContainerProperty.toString() instead of getValue() to get the value for a IndexedContainerProperty. This will not be supported starting from Vaadin 7.1 (your debugger might call toString() and cause this message to appear).
May 24, 2013 11:33:45 AM com.vaadin.data.util.IndexedContainer$IndexedContainerProperty toString
This occurs whether running in debug mode or not, so i dont think its a debugger issue.
I tested on master and dont get these warnings - so it must be something introduced in the branch.
Ideally the warnings would not be generated at all. But if there is a good argument, then at the least we need a new ticket with explicit notice on what change in this branch is causing the warnings. I spent a while trying to track down the change that introduced them but could not.
Maybe its a clue that when a column is resorted - you get two of these warnings for each message in the list.
|