Changeset 1924 in josm for trunk/src


Ignore:
Timestamp:
2009-08-07T23:22:31+02:00 (15 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.