Changeset 1930 in josm


Ignore:
Timestamp:
2009-08-08T11:59:13+02:00 (10 years ago)
Author:
jttt
Message:

Replace RelationMember.role with RelationMember.getRole() and RelationMember.hasRole()

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

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/actions/CombineWayAction.java

    r1925 r1930  
    9797                    for(Way w : selectedWays) {
    9898                        if (rm.member == w) {
    99                             Pair<Relation,String> pair = new Pair<Relation,String>(r, rm.role == null ? "" : rm.role);
     99                            Pair<Relation,String> pair = new Pair<Relation,String>(r, rm.getRole());
    100100                            HashSet<Way> waylinks = new HashSet<Way>();
    101101                            if (backlinks.containsKey(pair)) {
     
    232232                // note to re-add it later on.
    233233                if (selectedWays.contains(rm.member)) {
    234                     rolesToReAdd.add(rm.role);
     234                    rolesToReAdd.add(rm.getRole());
    235235                } else {
    236236                    newRel.members.add(rm);
  • trunk/src/org/openstreetmap/josm/actions/MergeNodesAction.java

    r1925 r1930  
    134134                    for (Node n : allNodes) {
    135135                        if (rm.member == n) {
    136                             Pair<Relation,String> pair = new Pair<Relation,String>(r, rm.role);
     136                            Pair<Relation,String> pair = new Pair<Relation,String>(r, rm.getRole());
    137137                            HashSet<Node> nodelinks = new HashSet<Node>();
    138138                            if (backlinks.containsKey(pair)) {
     
    282282                // just keep a note to re-add it later on.
    283283                if (allNodes.contains(rm.member)) {
    284                     rolesToReAdd.add(rm.role);
     284                    rolesToReAdd.add(rm.getRole());
    285285                } else {
    286286                    newRel.members.add(rm);
  • trunk/src/org/openstreetmap/josm/actions/SplitWayAction.java

    r1928 r1930  
    316316
    317317                        int j = i;
    318                         boolean backwards = "backward".equals(rm.role);
     318                        boolean backwards = "backward".equals(rm.getRole());
    319319                        for(Way wayToAdd : newWays)
    320320                        {
    321                             RelationMember em = new RelationMember(rm.role, wayToAdd);
    322                             if(em.role != null && em.role.length() > 0 && !("multipolygon".equals(type))) {
     321                            RelationMember em = new RelationMember(rm.getRole(), wayToAdd);
     322                            if(em.hasRole() && !("multipolygon".equals(type))) {
    323323                                warnmerole = true;
    324324                            }
  • trunk/src/org/openstreetmap/josm/actions/UnGlueAction.java

    r1925 r1930  
    335335                            rolesToReAdd = new HashSet<String>();
    336336                        }
    337                         rolesToReAdd.add(rm.role);
     337                        rolesToReAdd.add(rm.getRole());
    338338                    }
    339339                }
  • trunk/src/org/openstreetmap/josm/command/ChangeRelationMemberRoleCommand.java

    r1926 r1930  
    4848        }
    4949
    50         oldRole = relation.getMember(position).role;
     50        oldRole = relation.getMember(position).getRole();
    5151        relation.getMember(position).role = newRole;
    5252
  • trunk/src/org/openstreetmap/josm/command/DeleteCommand.java

    r1925 r1930  
    166166        for (RelationMember m : ref.getMembers()) {
    167167            if (m.member == osm) {
    168                 role = m.role;
     168                role = m.getRole();
    169169                break;
    170170            }
  • trunk/src/org/openstreetmap/josm/corrector/ReverseWayTagCorrector.java

    r1925 r1930  
    119119            for (RelationMember member : relation.getMembers()) {
    120120                if (!member.member.hasEqualSemanticAttributes(oldway)
    121                         || member.role.length() == 0) {
     121                        || !member.hasRole()) {
    122122                    position++;
    123123                    continue;
     
    127127                String newRole = null;
    128128                for (PrefixSuffixSwitcher prefixSuffixSwitcher : prefixSuffixSwitchers) {
    129                     newRole = prefixSuffixSwitcher.apply(member.role);
    130                     if (!newRole.equals(member.role)) {
     129                    newRole = prefixSuffixSwitcher.apply(member.getRole());
     130                    if (!newRole.equals(member.getRole())) {
    131131                        found = true;
    132132                        break;
  • trunk/src/org/openstreetmap/josm/corrector/RoleCorrectionTableModel.java

    r1814 r1930  
    4242            return NAME_FORMATTER.getName(roleCorrection.relation);
    4343        case 1:
    44             return roleCorrection.member.role;
     44            return roleCorrection.member.getRole();
    4545        case 2:
    4646            return roleCorrection.newRole;
  • trunk/src/org/openstreetmap/josm/data/osm/RelationMember.java

    r1928 r1930  
    1212    public String role;
    1313    public OsmPrimitive member;
     14
     15    /**
     16     *
     17     * @return Role name or "". Never returns null
     18     * @since 1930
     19     */
     20    public String getRole() {
     21        if (role == null) {
     22            return "";
     23        } else {
     24            return role;
     25        }
     26    }
     27
     28    /**
     29     *
     30     * @return True if role is set
     31     * @since 1930
     32     */
     33    public boolean hasRole() {
     34        return role != null && !"".equals(role);
     35    }
    1436
    1537    /**
  • trunk/src/org/openstreetmap/josm/data/osm/visitor/MapPaintVisitor.java

    r1925 r1930  
    556556                        w.getName()), true);
    557557                    }
    558                     else if("from".equals(m.role)) {
     558                    else if("from".equals(m.getRole())) {
    559559                        if(fromWay != null)
    560560                            r.putError(tr("More than one \"from\" way found."), true);
     
    562562                            fromWay = w;
    563563                        }
    564                     } else if("to".equals(m.role)) {
     564                    } else if("to".equals(m.getRole())) {
    565565                        if(toWay != null)
    566566                            r.putError(tr("More than one \"to\" way found."), true);
     
    568568                            toWay = w;
    569569                        }
    570                     } else if("via".equals(m.role)) {
     570                    } else if("via".equals(m.getRole())) {
    571571                        if(via != null)
    572572                            r.putError(tr("More than one \"via\" found."), true);
     
    575575                    }
    576576                    else
    577                         r.putError(tr("Unknown role ''{0}''.", m.role), true);
     577                        r.putError(tr("Unknown role ''{0}''.", m.getRole()), true);
    578578                }
    579579                else if(m.member instanceof Node)
    580580                {
    581581                    Node n = (Node) m.member;
    582                     if("via".equals(m.role))
     582                    if("via".equals(m.getRole()))
    583583                    {
    584584                        if(via != null)
     
    588588                    }
    589589                    else
    590                         r.putError(tr("Unknown role ''{0}''.", m.role), true);
     590                        r.putError(tr("Unknown role ''{0}''.", m.getRole()), true);
    591591                }
    592592                else
     
    801801                        w.getName()), true);
    802802                    }
    803                     else if("inner".equals(m.role))
     803                    else if("inner".equals(m.getRole()))
    804804                        inner.add(w);
    805                     else if("outer".equals(m.role))
     805                    else if("outer".equals(m.getRole()))
    806806                        outer.add(w);
    807807                    else
    808808                    {
    809809                        r.putError(tr("No useful role ''{0}'' for Way ''{1}''.",
    810                         m.role == null ? "" : m.role, w.getName()), true);
    811                         if(m.role == null || m.role.length() == 0)
     810                          m.getRole(), w.getName()), true);
     811                        if(!m.hasRole())
    812812                            outer.add(w);
    813813                        else if(r.isSelected())
  • trunk/src/org/openstreetmap/josm/gui/conflict/relation/RelationMemberListMergeModel.java

    r1928 r1930  
    2424    @Override
    2525    public boolean isEqualEntry(RelationMember e1, RelationMember e2) {
    26         boolean ret =
    27             (    (e1.role == null && e2.role == null)
    28                     || (e1.role != null && e1.role.equals(e2.role))
    29             );
     26        boolean ret = e1.getRole().equals(e2.getRole());
    3027        if (e1.member.id > 0 ) {
    3128            ret = ret && (e1.member.id == e2.member.id);
     
    10097    @Override
    10198    protected RelationMember cloneEntryForMergedList(RelationMember entry) {
    102         return new RelationMember(entry.role, entry.member);
     99        return new RelationMember(entry);
    103100    }
    104101
  • trunk/src/org/openstreetmap/josm/gui/conflict/relation/RelationMemberTableCellEditor.java

    r1631 r1930  
    1515/**
    1616 * {@see TableCellEditor} for the the role column in a table for {@see RelationMember}s.
    17  * 
     17 *
    1818 */
    1919public class RelationMemberTableCellEditor extends AbstractCellEditor implements TableCellEditor{
     
    3737        RelationMember member = (RelationMember)value;
    3838
    39         editor.setText(member.role == null ? "" : member.role);
     39        editor.setText(member.getRole());
    4040        editor.selectAll();
    4141        return editor;
  • trunk/src/org/openstreetmap/josm/gui/conflict/relation/RelationMemberTableCellRenderer.java

    r1911 r1930  
    2525/**
    2626 * This is the {@see TableCellRenderer} used in the tables of {@see RelationMemberMerger}.
    27  * 
     27 *
    2828 */
    2929public  class RelationMemberTableCellRenderer extends JLabel implements TableCellRenderer {
     
    4848    /**
    4949     * Load the image icon for an OSM primitive of type node
    50      * 
     50     *
    5151     * @return the icon; null, if not found
    5252     */
     
    153153
    154154    protected void renderRole(RelationMember member) {
    155         setText(member.role == null ? "" : member.role);
    156         setToolTipText(member.role == null ? "" : member.role);
     155        setText(member.getRole());
     156        setToolTipText(member.getRole());
    157157    }
    158158
  • trunk/src/org/openstreetmap/josm/gui/dialogs/PropertiesDialog.java

    r1928 r1930  
    535535                    for (RelationMember r : col) {
    536536                        if (text == null) {
    537                             text = r.role;
     537                            text = r.getRole();
    538538                        }
    539                         else if (!text.equals(r.role)) {
     539                        else if (!text.equals(r.getRole())) {
    540540                            text = tr("<different>");
    541541                            break;
     
    595595                                    if (rm.member == osm)
    596596                                    {
    597                                         RelationMember mem = new RelationMember(rm.role, rm.member);
    598                                         rel.members.remove(mem);
     597                                        rel.members.remove(rm);
    599598                                        break;
    600599                                    }
  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/MemberTableModel.java

    r1927 r1930  
    8787        switch (columnIndex) {
    8888            case 0:
    89                 return members.get(rowIndex).role;
     89                return members.get(rowIndex).getRole();
    9090            case 1:
    9191                return members.get(rowIndex).member;
     
    386386    /**
    387387     * Replies the set of selected referers. Never null, but may be empty.
    388      * 
     388     *
    389389     * @return the set of selected referers
    390390     */
  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/ac/AutoCompletionCache.java

    r1925 r1930  
    132132    protected void cacheRelationMemberRoles(Relation relation){
    133133        for (RelationMember m: relation.getMembers()) {
    134             if (m.role == null || m.role.trim().equals("")) {
    135                 continue;
    136             }
    137             if (!roleCache.contains(m.role)) {
    138                 roleCache.add(m.role);
     134            if (m.hasRole() && !roleCache.contains(m.getRole())) {
     135                roleCache.add(m.getRole());
    139136            }
    140137        }
  • trunk/src/org/openstreetmap/josm/io/OsmWriter.java

    r1925 r1930  
    135135                out.print(OsmPrimitiveType.from(em.member).getAPIName());
    136136                out.println("' ref='"+getUsedId(em.member)+"' role='" +
    137                         XmlWriter.encode(em.role == null ? "" : em.role) + "' />");
     137                        XmlWriter.encode(em.getRole()) + "' />");
    138138            }
    139139            addTags(e, "relation", false);
Note: See TracChangeset for help on using the changeset viewer.