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

Florian Müller resolved CMIS-1041.
----------------------------------
       Resolution: Fixed
    Fix Version/s: OpenCMIS 1.2.0

> Commons Properties API inconsistence between List and Map
> ---------------------------------------------------------
>
>                 Key: CMIS-1041
>                 URL: https://issues.apache.org/jira/browse/CMIS-1041
>             Project: Chemistry
>          Issue Type: Bug
>          Components: opencmis-commons
>    Affects Versions: OpenCMIS 1.1.0
>            Reporter: Sascha Homeier
>            Assignee: Florian Müller
>            Priority: Minor
>             Fix For: OpenCMIS 1.2.0
>
>
> _org.apache.chemistry.opencmis.commons.impl.dataobjects.PropertiesImpl,_ the 
> provided implementation of the _MutableProperties_ Interface in Commons API 
> is backed by a List and Map to provide these two different views via 
> _getPropertyList_ and_ getProperties_.
> If a Property with the same ID is added twice via _addProperty_ the Map 
> replaces the old property while the List appends the new one.
> Junit Test:
>         {code}@Test
>       public void testProperties_OpenCMISImpl() throws Exception
>       {
>               MutableProperties properties = new PropertiesImpl();
>               PropertyData<String> propertyOne = new 
> PropertyStringImpl("my:test", "testOne");
>               PropertyData<String> propertyTwo = new 
> PropertyStringImpl("my:test", "testTwo");
>               properties.addProperty(propertyOne);
>               Assert.assertEquals(1, properties.getProperties().size());
>               Assert.assertEquals(1, properties.getPropertyList().size());
>               properties.addProperty(propertyTwo);
>               // map replaces old property by new one, so only one entry here
>               Assert.assertEquals(1, properties.getProperties().size());
>               // list appends at the end, so two entries here
>               Assert.assertEquals(2, properties.getPropertyList().size());
>       }{code}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to