Changeset 1924 in josm


Ignore:
Timestamp:
Aug 7, 2009 11:22:31 PM (4 years ago)
Author:
jttt
Message:

Made OsmPrimitive.keys deprecated, removed remaining references in JOSM

Location:
trunk/src/org/openstreetmap/josm
Files:
10 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/actions/JoinNodeWayAction.java

    r1915 r1924  
    5959        for (Map.Entry<Way, List<Integer>> insertPoint : insertPoints.entrySet()) { 
    6060            Way w = insertPoint.getKey(); 
    61             List<Node> nodesToAdd = new ArrayList<Node>(w.getNodes()); 
     61            List<Node> nodesToAdd = w.getNodes(); 
    6262            List<Integer> is = insertPoint.getValue(); 
    6363            pruneSuccsAndReverse(is); 

  • trunk/src/org/openstreetmap/josm/actions/PasteTagsAction.java

    r1847 r1924  
    3939        for (Iterator<? extends OsmPrimitive> it = pasteBufferSubset.iterator(); it.hasNext();) { 
    4040            OsmPrimitive osm = it.next(); 
    41             Map<String, String> m = osm.keys; 
    42             if(m == null) { 
    43                 continue; 
    44             } 
    4541 
    46             for (String key : m.keySet()) { 
     42            for (String key : osm.keySet()) { 
    4743                clist.add(new ChangePropertyCommand(selectionSubset, key, osm.get(key))); 
    4844            } 

  • trunk/src/org/openstreetmap/josm/actions/SplitWayAction.java

    r1898 r1924  
    281281        while (chunkIt.hasNext()) { 
    282282            Way wayToAdd = new Way(); 
    283             if (selectedWay.keys != null) { 
    284                 wayToAdd.keys = new HashMap<String, String>(selectedWay.keys); 
    285             } 
     283            wayToAdd.setKeys(selectedWay.getKeys()); 
    286284            newWays.add(wayToAdd); 
    287285            wayToAdd.setNodes(chunkIt.next()); 

  • trunk/src/org/openstreetmap/josm/actions/UnGlueAction.java

    r1912 r1924  
    152152 
    153153        Node c = new Node(selectedNode); 
    154         c.keys = null; 
     154        c.removeAll(); 
    155155        c.setSelected(false); 
    156156        cmds.add(new ChangeCommand(selectedNode, c)); 

  • trunk/src/org/openstreetmap/josm/actions/search/SearchCompiler.java

    r1899 r1924  
    236236                return osm.get(key) != null; 
    237237            case ANY_KEY: 
    238                 for (String v:osm.keys.values()) { 
     238                for (String v:osm.getKeys().values()) { 
    239239                    if (v.equals(value)) 
    240240                        return true; 
    … …  
    244244                return value.equals(osm.get(key)); 
    245245            case ANY_KEY_REGEXP: 
    246                 for (String v:osm.keys.values()) { 
     246                for (String v:osm.getKeys().values()) { 
    247247                    if (valuePattern.matcher(v).matches()) 
    248248                        return true; 

  • trunk/src/org/openstreetmap/josm/command/DeleteCommand.java

    r1898 r1924  
    394394 
    395395            Way wnew2 = new Way(); 
    396             if (wnew.keys != null) { 
    397                 wnew2.keys = new HashMap<String, String>(wnew.keys); 
    398             } 
     396            wnew2.setKeys(wnew.getKeys()); 
    399397            wnew2.setNodes(n2); 
    400398            cmds.add(new AddCommand(wnew2)); 

  • trunk/src/org/openstreetmap/josm/data/osm/OsmPrimitive.java

    r1899 r1924  
    236236    /** 
    237237     * The key/value list for this primitive. 
    238      */ 
     238     * @deprecated This field will became private or protected in future, use api instead 
     239     */ 
     240    @Deprecated 
    239241    public Map<String, String> keys; 
     242 
     243    /** 
     244     * 
     245     * @return Keys of this primitive. Changes made in returned map are not mapped 
     246     * back to the primitive, use setKeys() to modify the keys 
     247     * @since 1924 
     248     */ 
     249    public Map<String, String> getKeys() { 
     250        // TODO More effective map 
     251        return new HashMap<String, String>(keys); 
     252    } 
     253 
     254    /** 
     255     * 
     256     * @since 1924 
     257     */ 
     258    public void setKeys(Map<String, String> keys) { 
     259        if (keys == null) { 
     260            this.keys = null; 
     261        } else { 
     262            this.keys = new HashMap<String, String>(keys); 
     263        } 
     264    } 
    240265 
    241266    /** 
    … …  
    269294 
    270295    /** 
    271      * Added in revision 1843 
    272      * Please do not use in plug-ins until this version becomes JOSM tested 
     296     * 
     297     * @since 1843 
    273298     */ 
    274299    public final void removeAll() { 
    … …  
    294319 
    295320    /** 
    296      * Added in revision 1843 
    297      * Please do not use in plug-ins until this version becomes JOSM tested 
     321     * 
     322     * @since 1843 
    298323     */ 
    299324    public final boolean hasKeys() { 

  • trunk/src/org/openstreetmap/josm/gui/mappaint/ElemStyle.java

    r1747 r1924  
    22 
    33import java.util.Collection; 
    4 import java.util.Map; 
    54 
    65import org.openstreetmap.josm.data.osm.OsmPrimitive; 
    … …  
    3130        return code; 
    3231    } 
    33     public boolean check(Map<String, String> keys) 
     32    public boolean check(OsmPrimitive primitive) 
    3433    { 
    3534        if(rules == null) 
    … …  
    3736        for(Rule r : rules) 
    3837        { 
    39             String k = keys.get(r.key); 
     38            String k = primitive.get(r.key); 
    4039            String bv = OsmUtils.getNamedOsmBoolean(r.boolValue); 
    4140            if(k == null || (r.value != null && !k.equals(r.value)) 

  • trunk/src/org/openstreetmap/josm/gui/mappaint/ElemStyles.java

    r1843 r1924  
    77import java.util.LinkedList; 
    88import java.util.List; 
    9 import java.util.Map; 
     9import java.util.Map.Entry; 
    1010 
    1111import org.openstreetmap.josm.Main; 
    … …  
    3737            areasList = new LinkedList<AreaElemStyle>(); 
    3838        } 
    39         private IconElemStyle getNode(Map<String, String> keys) 
     39        private IconElemStyle getNode(OsmPrimitive primitive) 
    4040        { 
    4141            IconElemStyle ret = null; 
    42             Iterator<String> iterator = keys.keySet().iterator(); 
    43             while(iterator.hasNext()) 
    44             { 
    45                 String key = iterator.next(); 
    46                 String val = keys.get(key); 
     42            for (Entry<String, String> entry:primitive.entrySet()) { 
     43                String key = entry.getKey(); 
     44                String val = entry.getValue(); 
    4745                IconElemStyle style; 
    4846                if((style = icons.get("n" + key + "=" + val)) != null) 
    … …  
    6462            for(IconElemStyle s : iconsList) 
    6563            { 
    66                 if((ret == null || s.priority > ret.priority) && s.check(keys)) 
     64                if((ret == null || s.priority > ret.priority) && s.check(primitive)) 
    6765                    ret = s; 
    6866            } 
    6967            return ret; 
    7068        } 
    71         private ElemStyle get(Map<String, String> keys, boolean noclosed) 
     69        private ElemStyle get(OsmPrimitive primitive, boolean noclosed) 
    7270        { 
    7371            AreaElemStyle retArea = null; 
    … …  
    7573            String linestring = null; 
    7674            HashMap<String, LineElemStyle> over = new HashMap<String, LineElemStyle>(); 
    77             Iterator<String> iterator = keys.keySet().iterator(); 
    78             while(iterator.hasNext()) 
    79             { 
    80                 String key = iterator.next(); 
    81                 String val = keys.get(key); 
     75            for (Entry<String, String> entry:primitive.entrySet()) { 
     76                String key = entry.getKey(); 
     77                String val = entry.getValue(); 
    8278                AreaElemStyle styleArea; 
    8379                LineElemStyle styleLine; 
    … …  
    125121            { 
    126122                if((retArea == null || s.priority > retArea.priority) 
    127                 && (!noclosed || !s.closed) && s.check(keys)) 
     123                && (!noclosed || !s.closed) && s.check(primitive)) 
    128124                    retArea = s; 
    129125            } 
    … …  
    131127            { 
    132128                if((retLine == null || s.priority > retLine.priority) 
    133                 && s.check(keys)) 
     129                && s.check(primitive)) 
    134130                    retLine = s; 
    135131            } 
    136132            for(LineElemStyle s : modifiersList) 
    137133            { 
    138                 if(s.check(keys)) 
     134                if(s.check(primitive)) 
    139135                    over.put(s.getCode(), s); 
    140136            } 
    … …  
    158154        public ElemStyle get(OsmPrimitive osm) 
    159155        { 
    160             return (osm.keys == null) ? null : 
    161             ((osm instanceof Node) ? getNode(osm.keys) : get(osm.keys, 
     156            return (!osm.hasKeys()) ? null : 
     157            ((osm instanceof Node) ? getNode(osm) : get(osm, 
    162158            osm instanceof Way && !((Way)osm).isClosed())); 
    163159        } 
    … …  
    165161        public ElemStyle getArea(Way osm) 
    166162        { 
    167             if(osm.keys != null) 
     163            if(osm.hasKeys()) 
    168164            { 
    169165                /* force area mode also for unclosed ways */ 
    170                 ElemStyle style = get(osm.keys, false); 
     166                ElemStyle style = get(osm, false); 
    171167                if(style != null && style instanceof AreaElemStyle) 
    172168                    return style; 
    … …  
    177173        public IconElemStyle getIcon(OsmPrimitive osm) 
    178174        { 
    179             return (osm.keys == null) ? null : getNode(osm.keys); 
     175            return osm.hasKeys() ? getNode(osm): null; 
    180176        } 
    181177 
    182178        public boolean isArea(OsmPrimitive o) 
    183179        { 
    184             if(o.keys != null && !(o instanceof Node)) 
     180            if(o.hasKeys() && !(o instanceof Node)) 
    185181            { 
    186182                boolean noclosed = o instanceof Way && !((Way)o).isClosed(); 
    … …  
    200196                for(AreaElemStyle s : areasList) 
    201197                { 
    202                     if(!(s.closed && noclosed) && s.check(o.keys)) 
     198                    if(!(s.closed && noclosed) && s.check(o)) 
    203199                        return true; 
    204200                } 

  • trunk/src/org/openstreetmap/josm/io/OsmReader.java

    r1910 r1924  
    8585        public void copyTo(OsmPrimitive osm) { 
    8686            osm.id = id; 
    87             osm.keys = keys; 
     87            osm.setKeys(keys); 
    8888            osm.modified = modified; 
    8989            osm.setSelected(selected); 
Note: See TracChangeset for help on using the changeset viewer.