Changeset 1899 in josm for trunk/src


Ignore:
Timestamp:
2009-08-03T21:37:45+02:00 (15 years ago)
Author:
jttt
Message:

Add setter/getter for OsmPrimitive.selected

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

Legend:

Unmodified
Added
Removed

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

    r1862 r1899  
    153153        Node c = new Node(selectedNode);
    154154        c.keys = null;
    155         c.selected = false;
     155        c.setSelected(false);
    156156        cmds.add(new ChangeCommand(selectedNode, c));
    157157

  • trunk/src/org/openstreetmap/josm/actions/mapmode/DrawAction.java

    r1898 r1899  
    403403                    // here so /only/ the new way will be selected after this method finishes.
    404404                    if(alt) {
    405                         wnew.selected = false;
     405                        wnew.setSelected(false);
    406406                    }
    407407
    … …  
    530530                title = tr("Add node into way");
    531531                for (Way w : reuseWays) {
    532                     w.selected = false;
     532                    w.setSelected(false);
    533533                }
    534534            }

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

    r1847 r1899  
    177177                        sel.remove(osm);
    178178                    }
    179                 } else if (mode == SearchMode.add && !osm.selected && matcher.match(osm)) {
     179                } else if (mode == SearchMode.add && !osm.isSelected() && matcher.match(osm)) {
    180180                    sel.add(osm);
    181181                    ++foundMatches;
    182                 } else if (mode == SearchMode.remove && osm.selected && matcher.match(osm)) {
     182                } else if (mode == SearchMode.remove && osm.isSelected() && matcher.match(osm)) {
    183183                    sel.remove(osm);
    184184                    ++foundMatches;

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

    r1862 r1899  
    409409    private static class Selected extends Match {
    410410        @Override public boolean match(OsmPrimitive osm) {
    411             return osm.selected;
     411            return osm.isSelected();
    412412        }
    413413        @Override public String toString() {return "selected";}

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

    r1856 r1899  
    11// License: GPL. Copyright 2007 by Immanuel Scholz and others
    22package org.openstreetmap.josm.data.osm;
     3
     4import static org.openstreetmap.josm.tools.I18n.tr;
    35
    46import java.awt.geom.Area;
    … …  
    79import java.util.Collection;
    810import java.util.Comparator;
     11import java.util.HashMap;
    912import java.util.HashSet;
    10 import java.util.HashMap;
    1113import java.util.Iterator;
    1214import java.util.LinkedList;
    … …  
    1517
    1618import org.openstreetmap.josm.data.SelectionChangedListener;
    17 import static org.openstreetmap.josm.tools.I18n.tr;
    1819
    1920/**
    … …  
    108109    /**
    109110     * Adds a primitive to the dataset
    110      * 
     111     *
    111112     * @param primitive the primitive. Ignored if null.
    112113     */
    … …  
    129130     * {@see #relations}. References from other primitives to this
    130131     * primitive are left unchanged.
    131      * 
     132     *
    132133     * @param primitive the primitive. Ignored if null.
    133134     */
    … …  
    188189        clearSelection(relations);
    189190        for (OsmPrimitive osm : selection) {
    190             osm.selected = true;
     191            osm.setSelected(true);
    191192        }
    192193        fireSelectionChanged(selection);
    … …  
    203204        for (OsmPrimitive o : osm)
    204205            if (o != null) {
    205                 o.selected = true;
     206                o.setSelected(true);
    206207            }
    207208        fireSelectionChanged(Arrays.asList(osm));
    … …  
    216217            return;
    217218        for (OsmPrimitive osm : list) {
    218             osm.selected = false;
     219            osm.setSelected(false);
    219220        }
    220221    }
    … …  
    229230            return sel;
    230231        for (OsmPrimitive osm : list)
    231             if (osm.selected && !osm.deleted) {
     232            if (osm.isSelected() && !osm.deleted) {
    232233                sel.add(osm);
    233234            }
    … …  
    347348     * Replies the set of ids of all complete primitivies (i.e. those with
    348349     * ! primitive.incomplete)
    349      * 
     350     *
    350351     * @return the set of ids of all complete primitivies
    351352     */
    … …  
    377378    /**
    378379     * removes all references from ways in this dataset to a particular node
    379      * 
     380     *
    380381     * @param node the node
    381382     */
    … …  
    393394    /**
    394395     * removes all references from relations in this dataset  to this primitive
    395      * 
     396     *
    396397     * @param primitive the primitive
    397398     */
    … …  
    411412     * removes all references from from other primitives  to the
    412413     * referenced primitive
    413      * 
     414     *
    414415     * @param referencedPrimitive the referenced primitive
    415416     */
    … …  
    426427     * Replies a list of parent relations which refer to the relation
    427428     * <code>child</code>. Replies an empty list if child is null.
    428      * 
     429     *
    429430     * @param child the child relation
    430431     * @return a list of parent relations which refer to the relation

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

    r1843 r1899  
    100100     * If set to true, this object is currently selected.
    101101     */
     102    @Deprecated
    102103    public volatile boolean selected = false;
     104
     105    /**
     106     *
     107     * @since 1899
     108     */
     109    public void setSelected(boolean selected) {
     110        this.selected = selected;
     111    }
     112    /**
     113     *
     114     * @since 1899
     115     */
     116    public boolean isSelected() {
     117        return selected;
     118    }
    103119
    104120    /**
    … …  
    179195    public final void delete(boolean deleted) {
    180196        this.deleted = deleted;
    181         selected = false;
     197        setSelected(false);
    182198        modified = true;
    183199    }
    … …  
    303319        modified = osm.modified;
    304320        deleted = osm.deleted;
    305         selected = osm.selected;
     321        setSelected(osm.isSelected());
    306322        timestamp = osm.timestamp;
    307323        version = osm.version;

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

    r1898 r1899  
    147147
    148148        if (nodeStyle != null && isZoomOk(nodeStyle) && showIcons > dist)
    149             drawNode(n, nodeStyle.icon, nodeStyle.annotate, n.selected);
     149            drawNode(n, nodeStyle.icon, nodeStyle.annotate, n.isSelected());
    150150        else if (n.highlighted)
    151151            drawNode(n, highlightColor, selectedNodeSize, selectedNodeRadius, fillSelectedNode);
    152         else if (n.selected)
     152        else if (n.isSelected())
    153153            drawNode(n, selectedColor, selectedNodeSize, selectedNodeRadius, fillSelectedNode);
    154154        else if (n.isTagged())
    … …  
    209209            //profilerVisibleWays++;
    210210            //if(!profilerOmitDraw)
    211             drawWay(w, null, untaggedColor, w.selected);
     211            drawWay(w, null, untaggedColor, w.isSelected());
    212212        }
    213213        else if(wayStyle instanceof LineElemStyle)
    … …  
    216216            //profilerVisibleWays++;
    217217            //if(!profilerOmitDraw)
    218             drawWay(w, (LineElemStyle)wayStyle, untaggedColor, w.selected);
     218            drawWay(w, (LineElemStyle)wayStyle, untaggedColor, w.isSelected());
    219219        }
    220220        else if (wayStyle instanceof AreaElemStyle)
    … …  
    227227            {
    228228            //    profilerVisibleAreas++;
    229                 drawArea(w, w.selected ? selectedColor : areaStyle.color);
     229                drawArea(w, w.isSelected() ? selectedColor : areaStyle.color);
    230230                if(!w.isClosed())
    231231                    w.putError(tr("Area style way is not closed."), true);
    232232            }
    233             drawWay(w, areaStyle.line, areaStyle.color, w.selected);
     233            drawWay(w, areaStyle.line, areaStyle.color, w.isSelected());
    234234            //}
    235235        }
    … …  
    240240           the way is tagged with a direction key
    241241           (even if the tag is negated as in oneway=false) or the way is selected */
    242         boolean showDirection = w.selected || ((!useRealWidth) && (showDirectionArrow
     242        boolean showDirection = w.isSelected() || ((!useRealWidth) && (showDirectionArrow
    243243        && (!showRelevantDirectionsOnly || w.hasDirectionKeys())));
    244244        /* head only takes over control if the option is true,
    245245           the direction should be shown at all and not only because it's selected */
    246         boolean showOnlyHeadArrowOnly = showDirection && !w.selected && showHeadArrowOnly;
     246        boolean showOnlyHeadArrowOnly = showDirection && !w.isSelected() && showHeadArrowOnly;
    247247        int width = defaultSegmentWidth;
    248248        int realWidth = 0; /* the real width of the element in meters */
    … …  
    283283        if(w.highlighted)
    284284            color = highlightColor;
    285         else if(w.selected)
     285        else if(w.isSelected())
    286286            color = selectedColor;
    287287
    … …  
    298298                        if(lastN != null)
    299299                        {
    300                             drawSeg(lastN, n, s.color != null  && !w.selected ? s.color : color,
     300                            drawSeg(lastN, n, s.color != null  && !w.isSelected() ? s.color : color,
    301301                            false, s.getWidth(width), s.dashed, s.dashedColor);
    302302                        }
    … …  
    331331                        if(lastN != null)
    332332                        {
    333                             drawSeg(lastN, n, s.color != null && !w.selected ? s.color : color,
     333                            drawSeg(lastN, n, s.color != null && !w.isSelected() ? s.color : color,
    334334                            false, s.getWidth(width), s.dashed, s.dashedColor);
    335335                        }
    … …  
    377377                        Way c = (Way)joinArray[i];
    378378                        if(w == null)
    379                         { w = c; selected = w.selected; joinArray[i] = null; --left; }
     379                        { w = c; selected = w.isSelected(); joinArray[i] = null; --left; }
    380380                        else
    381381                        {
    … …  
    403403                                joinArray[i] = null;
    404404                                joined = true;
    405                                 if(c.selected) selected = true;
     405                                if(c.isSelected()) selected = true;
    406406                                --left;
    407407                                if(n == null) n = w.getNodes();
    … …  
    430430                w = new Way(w);
    431431                w.setNodes(n);
    432                 w.selected = selected;
     432                w.setSelected(selected);
    433433            }
    434434            if(!w.isClosed())
    … …  
    505505        }
    506506
    507         if(r.selected) /* draw ways*/
     507        if(r.isSelected()) /* draw ways*/
    508508        {
    509509            for (RelationMember m : r.members)
    … …  
    765765        smallIcon.paintIcon ( Main.map.mapView, g, (int)(pVia.x+vx+vx2)-w/2, (int)(pVia.y+vy+vy2)-h/2 );
    766766
    767         if (r.selected)
     767        if (r.isSelected())
    768768        {
    769769            g.setColor (  selectedColor );
    … …  
    811811                        if(m.role == null || m.role.length() == 0)
    812812                            outer.add(w);
    813                         else if(r.selected)
     813                        else if(r.isSelected())
    814814                            drawSelectedMember(m.member, styles != null
    815815                            ? getPrimitiveStyle(m.member) : null, true, true);
    … …  
    977977                    if(isPolygonVisible(p))
    978978                    {
    979                         drawAreaPolygon(p, (pd.way.selected || r.selected) ? selectedColor
     979                        drawAreaPolygon(p, (pd.way.isSelected() || r.isSelected()) ? selectedColor
    980980                        : areaStyle.color);
    981981                        visible = true;
    … …  
    10011001                    {
    10021002                        drawWay(wInner, ((AreaElemStyle)wayStyle).line,
    1003                         ((AreaElemStyle)wayStyle).color, wInner.selected
    1004                         || r.selected);
     1003                        ((AreaElemStyle)wayStyle).color, wInner.isSelected()
     1004                        || r.isSelected());
    10051005                    }
    10061006                    wInner.mappaintDrawnCode = paintid;
    … …  
    10081008                else
    10091009                {
    1010                     if(r.selected)
     1010                    if(r.isSelected())
    10111011                    {
    10121012                        drawSelectedMember(wInner, innerStyle,
    1013                         !wayStyle.equals(innerStyle), wInner.selected);
     1013                        !wayStyle.equals(innerStyle), wInner.isSelected());
    10141014                    }
    10151015                    if(wayStyle.equals(innerStyle))
    … …  
    10171017                        r.putError(tr("Style for inner way ''{0}'' equals multipolygon.",
    10181018                        wInner.getName()), false);
    1019                         if(!r.selected)
     1019                        if(!r.isSelected())
    10201020                            wInner.mappaintDrawnAreaCode = paintid;
    10211021                    }
    … …  
    10301030                    {
    10311031                        drawWay(wOuter, ((AreaElemStyle)wayStyle).line,
    1032                         ((AreaElemStyle)wayStyle).color, wOuter.selected
    1033                         || r.selected);
     1032                        ((AreaElemStyle)wayStyle).color, wOuter.isSelected()
     1033                        || r.isSelected());
    10341034                    }
    10351035                    wOuter.mappaintDrawnCode = paintid;
    … …  
    10431043                        wOuter.getName()), true);
    10441044                    }
    1045                     if(r.selected)
     1045                    if(r.isSelected())
    10461046                    {
    10471047                        drawSelectedMember(wOuter, outerStyle, false, false);
    … …  
    13631363        //    profilerN = 0;
    13641364            for (final OsmPrimitive osm : data.ways)
    1365                 if (!osm.incomplete && !osm.deleted && !osm.selected
     1365                if (!osm.incomplete && !osm.deleted && !osm.isSelected()
    13661366                && osm.mappaintVisibleCode != viewid )
    13671367                {

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

    r1898 r1899  
    149149        //profilerN = 0;
    150150        for (final OsmPrimitive osm : data.relations)
    151             if (!osm.deleted && !osm.selected)
     151            if (!osm.deleted && !osm.isSelected())
    152152            {
    153153                osm.visit(this);
    … …  
    163163        //profilerN = 0;
    164164        for (final OsmPrimitive osm : data.ways)
    165             if (!osm.deleted && !osm.selected && osm.isTagged())
     165            if (!osm.deleted && !osm.isSelected() && osm.isTagged())
    166166            {
    167167                osm.visit(this);
    … …  
    171171
    172172        for (final OsmPrimitive osm : data.ways)
    173             if (!osm.deleted && !osm.selected && !osm.isTagged())
     173            if (!osm.deleted && !osm.isSelected() && !osm.isTagged())
    174174            {
    175175                osm.visit(this);
    … …  
    202202        //profilerN = 0;
    203203        for (final OsmPrimitive osm : data.nodes)
    204             if (!osm.deleted && !osm.selected)
     204            if (!osm.deleted && !osm.isSelected())
    205205            {
    206206                osm.visit(this);
    … …  
    253253        else if (n.highlighted)
    254254            drawNode(n, highlightColor, selectedNodeSize, selectedNodeRadius, fillSelectedNode);
    255         else if (n.selected)
     255        else if (n.isSelected())
    256256            drawNode(n, selectedColor, selectedNodeSize, selectedNodeRadius, fillSelectedNode);
    257257        else if(n.isTagged())
    … …  
    300300           (even if the tag is negated as in oneway=false) or the way is selected */
    301301
    302         boolean showThisDirectionArrow = w.selected
     302        boolean showThisDirectionArrow = w.isSelected()
    303303        || (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 */
    306         boolean showOnlyHeadArrowOnly = showThisDirectionArrow && !w.selected && showHeadArrowOnly;
     306        boolean showOnlyHeadArrowOnly = showThisDirectionArrow && !w.isSelected() && showHeadArrowOnly;
    307307        Color wayColor;
    308308
    … …  
    311311        } else if(w.highlighted) {
    312312            wayColor = highlightColor;
    313         } else if(w.selected) {
     313        } else if(w.isSelected()) {
    314314            wayColor = selectedColor;
    315315        } else if (!w.isTagged()) {
    … …  
    341341        if (inactive) {
    342342            col = inactiveColor;
    343         } else if (r.selected) {
     343        } else if (r.isSelected()) {
    344344            col = selectedColor;
    345345        } else {

  • trunk/src/org/openstreetmap/josm/gui/NavigatableComponent.java

    r1898 r1899  
    297297            // when multiple nodes on one point, prefer new or selected nodes
    298298            else if(dist == minDistanceSq && minPrimitive != null
    299                     && ((n.id == 0 && n.selected)
    300                             || (!minPrimitive.selected && (n.selected || n.id == 0)))) {
     299                    && ((n.id == 0 && n.isSelected())
     300                            || (!minPrimitive.isSelected() && (n.isSelected() || n.id == 0)))) {
    301301                minPrimitive = n;
    302302            }
    … …  
    336336                double perDist = a-(a-b+c)*(a-b+c)/4/c; // perpendicular distance squared
    337337                if (perDist < snapDistance && a < c+snapDistance && b < c+snapDistance) {
    338                     if(w.selected) {
     338                    if(w.isSelected()) {
    339339                        perDist -= 0.00001;
    340340                    }

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

    r1881 r1899  
    8686            osm.keys = keys;
    8787            osm.modified = modified;
    88             osm.selected = selected;
     88            osm.setSelected(selected);
    8989            osm.deleted = deleted;
    9090            osm.setTimestamp(timestamp);
Note: See TracChangeset for help on using the changeset viewer.