[ https://issues.apache.org/jira/browse/PIVOT-875?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13470782#comment-13470782 ]
Roger Whitcomb commented on PIVOT-875: -------------------------------------- Also had to fix the JSONSerializerTest because it was relying on the old (IMHO broken) behavior and checking the value of "count" from a serialized JSON map. "ant test" now runs clean again. [trunk] Sending core\test\org\apache\pivot\json\test\JSONSerializerTest.java Transmitting file data . Committed revision 1394858. [branches/2.0.x] Sending . Sending core\test\org\apache\pivot\json\test\JSONSerializerTest.java Transmitting file data . Committed revision 1394859. > JSON should support using pivot.collections.Map directly > -------------------------------------------------------- > > Key: PIVOT-875 > URL: https://issues.apache.org/jira/browse/PIVOT-875 > Project: Pivot > Issue Type: Improvement > Components: core-json > Affects Versions: 2.0.2 > Environment: All > Reporter: Roger Whitcomb > Priority: Trivial > Labels: JSON > Fix For: 2.0.3 > > > When supplying a pivot.collections.Map as the data object for a TableView, > the data is fetched for each row using JSON.get(...). However, this method > will default to using the BeanAdapter for anything other than a > java.util.Map, which results in values like "class", "capacity", "count" and > "comparator" fetching the values for the Map object instead of using the > "get" method of the actual map to get the values. > This fix will likely speed things up (a bit) because the first "if" in the > chain will now succeed for Pivot Map-based objects, instead of having to fail > and do three other tests before getting to the "dictionary.get" for these > objects. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira