Ticket #2302: tagged.patch

File tagged.patch, 12.5 KB (added by jttt, 17 years ago)

Patch to change hasDirectionKeys and isTagged to read only properties

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

     
    253253            Way wayToAdd = new Way();
    254254            if (selectedWay.keys != null) {
    255255                wayToAdd.keys = new HashMap<String, String>(selectedWay.keys);
    256                 wayToAdd.checkTagged();
    257                 wayToAdd.checkDirectionTagged();
    258256            }
    259257            newWays.add(wayToAdd);
    260258            wayToAdd.nodes.addAll(chunkIt.next());

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

     
    314314
    315315    private static class Untagged extends Match {
    316316        @Override public boolean match(OsmPrimitive osm) {
    317             return !osm.tagged;
     317            return !osm.isTagged();
    318318        }
    319319        @Override public String toString() {return "untagged";}
    320320    }

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

     
    139139
    140140        Node c = new Node(selectedNode);
    141141        c.keys = null;
    142         c.tagged = false;
    143142        c.selected = false;
    144143        cmds.add(new ChangeCommand(selectedNode, c));
    145144       
    … …  
    188187            return false;
    189188       
    190189        selectedNode = (Node)n;
    191         return  selectedNode.tagged;
     190        return  selectedNode.isTagged();
    192191    }
    193192
    194193    /**

  • src/org/openstreetmap/josm/gui/layer/OsmDataLayer.java

     
    401401                    trkseg = new ArrayList<WayPoint>();
    402402                    trk.trackSegs.add(trkseg);
    403403                }
    404                 if (!n.tagged) {
     404                if (!n.isTagged()) {
    405405                    doneNodes.add(n);
    406406                }
    407407                WayPoint wpt = new WayPoint(n.coor);

  • src/org/openstreetmap/josm/gui/dialogs/RelationEditor.java

     
    409409        if (RelationEditor.this.relation == null) {
    410410            // If the user wanted to create a new relation, but hasn't added any members or
    411411            // tags, don't add an empty relation
    412             clone.checkTagged();
    413             if(clone.members.size() == 0 && !clone.tagged)
     412            if(clone.members.size() == 0 && !clone.isTagged())
    414413                return;
    415414            Main.main.undoRedo.add(new AddCommand(clone));
    416415            DataSet.fireSelectionChanged(Main.ds.getSelected());

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

     
    104104               osm.user = user;
    105105               osm.visible = visible;
    106106               osm.version = version;
    107                osm.checkTagged();
    108                osm.checkDirectionTagged();
    109107               osm.mappaintStyle = null;
    110108          }
    111109     }

  • src/org/openstreetmap/josm/data/osm/visitor/SimplePaintVisitor.java

     
    162162
    163163        //profilerN = 0;
    164164        for (final OsmPrimitive osm : data.ways)
    165             if (!osm.deleted && !osm.selected && osm.tagged)
     165            if (!osm.deleted && !osm.selected && osm.isTagged())
    166166            {
    167167                osm.visit(this);
    168168        //        profilerN++;
    … …  
    170170        displaySegments();
    171171
    172172        for (final OsmPrimitive osm : data.ways)
    173             if (!osm.deleted && !osm.selected && !osm.tagged)
     173            if (!osm.deleted && !osm.selected && !osm.isTagged())
    174174            {
    175175                osm.visit(this);
    176176        //        profilerN++;
    … …  
    254254            drawNode(n, highlightColor, selectedNodeSize, selectedNodeRadius, fillSelectedNode);
    255255        else if (n.selected)
    256256            drawNode(n, selectedColor, selectedNodeSize, selectedNodeRadius, fillSelectedNode);
    257         else if(n.tagged)
     257        else if(n.isTagged())
    258258            drawNode(n, nodeColor, taggedNodeSize, taggedNodeRadius, fillUnselectedNode);
    259259        else
    260260            drawNode(n, nodeColor, unselectedNodeSize, unselectedNodeRadius, fillUnselectedNode);
    … …  
    300300           (even if the tag is negated as in oneway=false) or the way is selected */
    301301
    302302        boolean showThisDirectionArrow = w.selected
    303         || (showDirectionArrow && (!showRelevantDirectionsOnly || w.hasDirectionKeys));
     303        || (showDirectionArrow && (!showRelevantDirectionsOnly || w.hasDirectionKeys()));
    304304        /* head only takes over control if the option is true,
    305305           the direction should be shown at all and not only because it's selected */
    306306        boolean showOnlyHeadArrowOnly = showThisDirectionArrow && !w.selected && showHeadArrowOnly;
    … …  
    312312            wayColor = highlightColor;
    313313        } else if(w.selected) {
    314314            wayColor = selectedColor;
    315         } else if (!w.tagged) {
     315        } else if (!w.isTagged()) {
    316316            wayColor = untaggedWayColor;
    317317        } else {
    318318            wayColor = dfltWayColor;

  • src/org/openstreetmap/josm/data/osm/visitor/MapPaintVisitor.java

     
    156156            drawNode(n, highlightColor, selectedNodeSize, selectedNodeRadius, fillSelectedNode);
    157157        else if (n.selected)
    158158            drawNode(n, selectedColor, selectedNodeSize, selectedNodeRadius, fillSelectedNode);
    159         else if (n.tagged)
     159        else if (n.isTagged())
    160160            drawNode(n, nodeColor, taggedNodeSize, taggedNodeRadius, fillUnselectedNode);
    161161        else
    162162            drawNode(n, nodeColor, unselectedNodeSize, unselectedNodeRadius, fillUnselectedNode);
    … …  
    244244           the way is tagged with a direction key
    245245           (even if the tag is negated as in oneway=false) or the way is selected */
    246246        boolean showDirection = w.selected || ((!useRealWidth) && (showDirectionArrow
    247         && (!showRelevantDirectionsOnly || w.hasDirectionKeys)));
     247        && (!showRelevantDirectionsOnly || w.hasDirectionKeys())));
    248248        /* head only takes over control if the option is true,
    249249           the direction should be shown at all and not only because it's selected */
    250250        boolean showOnlyHeadArrowOnly = showDirection && !w.selected && showHeadArrowOnly;

  • src/org/openstreetmap/josm/data/osm/Way.java

     
    9191        super.cloneFrom(osm);
    9292        nodes.clear();
    9393        nodes.addAll(((Way)osm).nodes);
    94         checkDirectionTagged();
    9594    }
    9695
    9796    @Override public String toString() {

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

     
    106106    public User user = null;
    107107
    108108    /**
    109      * true if this object is considered "tagged". To be "tagged", an object
    110      * must have one or more "non-standard" tags. "created_by" and "source"
    111      * are typically considered "standard" tags and do not make an object
    112      * "tagged".
    113      */
    114     public boolean tagged = false;
    115 
    116     /**
    117      * true if this object has direction dependent tags (e.g. oneway)
    118      */
    119     public boolean hasDirectionKeys = false;
    120 
    121     /**
    122109     * If set to true, this object is currently selected.
    123110     */
    124111    public volatile boolean selected = false;
    … …  
    241228                keys = new HashMap<String, String>();
    242229            keys.put(key, value);
    243230        }
    244         checkTagged();
    245         checkDirectionTagged();
    246231        mappaintStyle = null;
    247232    }
    248233    /**
    … …  
    254239            if (keys.isEmpty())
    255240                keys = null;
    256241        }
    257         checkTagged();
    258         checkDirectionTagged();
    259242        mappaintStyle = null;
    260243    }
    261244
    … …  
    291274        selected = osm.selected;
    292275        timestamp = osm.timestamp;
    293276        version = osm.version;
    294         tagged = osm.tagged;
    295277        incomplete = osm.incomplete;
    296278        clearCached();
    297279        clearErrors();
    … …  
    320302    }
    321303
    322304    /**
    323      * Updates the "tagged" flag. "keys" property should probably be made private
    324      * to make sure this gets called when keys are set.
     305     * true if this object is considered "tagged". To be "tagged", an object
     306     * must have one or more "non-standard" tags. "created_by" and "source"
     307     * are typically considered "standard" tags and do not make an object
     308     * "tagged".
    325309     */
    326     public void checkTagged() {
    327         tagged = false;
     310    public boolean isTagged() {
     311        // TODO Cache value after keys are made private
    328312        if(uninteresting == null)
    329313            uninteresting = Main.pref.getCollection("tags.uninteresting",
    330314            Arrays.asList(new String[]{"source","note","comment","converted_by","created_by"}));
    331315        if (keys != null) {
    332316            for (Entry<String,String> e : keys.entrySet()) {
    333317                if (!uninteresting.contains(e.getKey())) {
    334                     tagged = true;
    335                     break;
     318                    return true;
    336319                }
    337320            }
    338321        }
     322        return false;
    339323    }
    340324    /**
    341      * Updates the "hasDirectionKeys" flag. "keys" property should probably be made private
    342      * to make sure this gets called when keys are set.
     325     * true if this object has direction dependent tags (e.g. oneway)
    343326     */
    344     public void checkDirectionTagged() {
    345         hasDirectionKeys = false;
     327    public boolean hasDirectionKeys() {
     328        // TODO Cache value after keys are made private
    346329        if(directionKeys == null)
    347330            directionKeys = Main.pref.getCollection("tags.direction",
    348331            Arrays.asList(new String[]{"oneway","incline","incline_steep","aerialway"}));
    349332        if (keys != null) {
    350333            for (Entry<String,String> e : keys.entrySet()) {
    351334                if (directionKeys.contains(e.getKey())) {
    352                     hasDirectionKeys = true;
    353                     break;
     335                    return true;
    354336                }
    355337            }
    356338        }
     339        return false;
    357340    }
    358341}

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

     
    188188            for (OsmPrimitive osm : del) {
    189189                if (osm instanceof Way) {
    190190                    for (Node n : ((Way) osm).nodes) {
    191                         if (!n.tagged) {
     191                        if (!n.isTagged()) {
    192192                            CollectBackReferencesVisitor v = new CollectBackReferencesVisitor(Main.ds, false);
    193193                            n.visit(v);
    194194                            v.data.removeAll(del);
    … …  
    321321            Way wnew2 = new Way();
    322322            if (wnew.keys != null) {
    323323                wnew2.keys = new HashMap<String, String>(wnew.keys);
    324                 wnew2.checkTagged();
    325                 wnew2.checkDirectionTagged();
    326324            }
    327325            wnew2.nodes.addAll(n2);
    328326            cmds.add(new AddCommand(wnew2))