MapContent.LayerList.set(int, Layer) method is not safe
-------------------------------------------------------

                 Key: GEOT-3918
                 URL: https://jira.codehaus.org/browse/GEOT-3918
             Project: GeoTools
          Issue Type: Bug
          Components: render
    Affects Versions: 8.0-M2, 2.7.3
            Reporter: Michael Bedward
            Assignee: Michael Bedward
             Fix For: 2.7.4, 8.0-M3


In version 8, the MapContent.LayerList.set(int, Layer) method replaces the 
element at the specified list position with the given layer and then fires a 
"layer moved" event. The method thus effectively allows a layer to be removed 
and another added while by-passing the usual events associated with these 
operations. Unless there is an obvious need for this method it should be 
changed to throw an UnsupportedOperationException.

(In version 2.7.x the same issue applies in the anonymous list class returned 
by the MapContent.layers() method)

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

------------------------------------------------------------------------------
All the data continuously generated in your IT infrastructure contains a
definitive record of customers, application performance, security
threats, fraudulent activity and more. Splunk takes this data and makes
sense of it. Business sense. IT sense. Common sense.
http://p.sf.net/sfu/splunk-d2d-oct
_______________________________________________
Geotools-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to