Changeset 2291 in josm for trunk/src/org


Ignore:
Timestamp:
2009-10-15T22:20:19+02:00 (15 years ago)
Author:
Gubaer
Message:

Replaced OsmPrimtive.user by setters/getters

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

Legend:

Unmodified
Added
Removed

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

    r2273 r2291  
    230230
    231231            switch (mode) {
    232             case NONE:
    233                 return false;
    234             case MISSING_KEY:
    235                 return osm.get(key) == null;
    236             case ANY:
    237                 return true;
    238             case ANY_VALUE:
    239                 return osm.get(key) != null;
    240             case ANY_KEY:
    241                 for (String v:osm.getKeys().values()) {
    242                     if (v.equals(value))
    243                         return true;
    244                 }
    245                 return false;
    246             case EXACT:
    247                 return value.equals(osm.get(key));
    248             case ANY_KEY_REGEXP:
    249                 for (String v:osm.getKeys().values()) {
    250                     if (valuePattern.matcher(v).matches())
    251                         return true;
    252                 }
    253                 return false;
    254             case ANY_VALUE_REGEXP:
    255             case EXACT_REGEXP:
    256                 for (Entry<String, String> entry:osm.entrySet()) {
    257                     if (keyPattern.matcher(entry.getKey()).matches()) {
    258                         if (mode == Mode.ANY_VALUE_REGEXP
    259                                 || valuePattern.matcher(entry.getValue()).matches())
     232                case NONE:
     233                    return false;
     234                case MISSING_KEY:
     235                    return osm.get(key) == null;
     236                case ANY:
     237                    return true;
     238                case ANY_VALUE:
     239                    return osm.get(key) != null;
     240                case ANY_KEY:
     241                    for (String v:osm.getKeys().values()) {
     242                        if (v.equals(value))
    260243                            return true;
    261244                    }
    262                 }
    263                 return false;
    264             case MISSING_KEY_REGEXP:
    265                 for (String k:osm.keySet()) {
    266                     if (keyPattern.matcher(k).matches())
    267                         return false;
    268                 }
    269                 return true;
     245                    return false;
     246                case EXACT:
     247                    return value.equals(osm.get(key));
     248                case ANY_KEY_REGEXP:
     249                    for (String v:osm.getKeys().values()) {
     250                        if (valuePattern.matcher(v).matches())
     251                            return true;
     252                    }
     253                    return false;
     254                case ANY_VALUE_REGEXP:
     255                case EXACT_REGEXP:
     256                    for (Entry<String, String> entry:osm.entrySet()) {
     257                        if (keyPattern.matcher(entry.getKey()).matches()) {
     258                            if (mode == Mode.ANY_VALUE_REGEXP
     259                                    || valuePattern.matcher(entry.getValue()).matches())
     260                                return true;
     261                        }
     262                    }
     263                    return false;
     264                case MISSING_KEY_REGEXP:
     265                    for (String k:osm.keySet()) {
     266                        if (keyPattern.matcher(k).matches())
     267                            return false;
     268                    }
     269                    return true;
    270270            }
    271271            throw new AssertionError("Missed state");
    … …  
    331331                }
    332332            }
    333             if (osm.user != null) {
    334                 String name = osm.user.getName();
     333            if (osm.getUser() != null) {
     334                String name = osm.getUser().getName();
    335335                // is not Java 1.5
    336336                //String name = java.text.Normalizer.normalize(name, java.text.Normalizer.Form.NFC);
    … …  
    377377        }
    378378        @Override public boolean match(OsmPrimitive osm) {
    379             if (osm.user == null && user == null) return true;
    380             if (osm.user == null) return false;
    381             return osm.user.equals(user);
     379            if (osm.getUser() == null && user == null) return true;
     380            if (osm.getUser() == null) return false;
     381            return osm.getUser().equals(user);
    382382        }
    383383        @Override public String toString() {

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

    r2284 r2291  
    135135     * Never changed by JOSM.
    136136     */
    137     public User user = null;
     137    private User user = null;
    138138
    139139    /**
    … …  
    161161            this.id = id;
    162162        } else {
    163             if (id < 0) {
     163            if (id < 0)
    164164                throw new IllegalArgumentException(tr("Expected ID >= 0. Got {0}.", id));
    165             } else if (id == 0) {
     165            else if (id == 0) {
    166166                this.id = idCounter.decrementAndGet();
    167167            } else {
    … …  
    483483        setModified(deleted);
    484484        setSelected(false);
     485    }
     486
     487    /**
     488     * Replies the user who has last touched this object. May be null.
     489     *
     490     * @return the user who has last touched this object. May be null.
     491     */
     492    public User getUser() {
     493        return user;
     494    }
     495
     496    /**
     497     * Sets the user who has last touched this object.
     498     *
     499     * @param user the user
     500     */
     501    public void setUser(User user) {
     502        this.user = user;
    485503    }
    486504

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

    r2273 r2291  
    121121                        // version
    122122                        my.setVisible(other.isVisible());
    123                         my.user = other.user;
     123                        my.setUser(other.getUser());
    124124                        my.setTimestamp(other.getTimestamp());
    125125                        my.setModified(other.isModified());

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

    r2273 r2291  
    425425            }
    426426
    427             if(osm.user != null) {
    428                 text.append(" [" + tr("User:") + " " + osm.user.getName() + "]");
     427            if(osm.getUser() != null) {
     428                text.append(" [" + tr("User:") + " " + osm.getUser().getName() + "]");
    429429            }
    430430

  • trunk/src/org/openstreetmap/josm/gui/dialogs/UserListDialog.java

    r2073 r2291  
    278278            if (primitives == null || primitives.isEmpty()) return ret;
    279279            for (OsmPrimitive primitive: primitives) {
    280                 if (primitive.user == null) {
     280                if (primitive.getUser() == null) {
    281281                    continue;
    282282                }
    283                 if (ret.containsKey(primitive.user)) {
    284                     ret.put(primitive.user, ret.get(primitive.user) + 1);
     283                if (ret.containsKey(primitive.getUser())) {
     284                    ret.put(primitive.getUser(), ret.get(primitive.getUser()) + 1);
    285285                } else {
    286                     ret.put(primitive.user, 1);
     286                    ret.put(primitive.getUser(), 1);
    287287                }
    288288            }
    … …  
    335335            Collection<OsmPrimitive> byUser = new LinkedList<OsmPrimitive>();
    336336            for (OsmPrimitive p : selected) {
    337                 if (p.user == null) {
     337                if (p.getUser() == null) {
    338338                    continue;
    339339                }
    340                 if (users.contains(p.user)) {
     340                if (users.contains(p.getUser())) {
    341341                    byUser.add(p);
    342342                }

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

    r2266 r2291  
    101101            osm.setModified(modified | deleted);
    102102            osm.setTimestamp(timestamp);
    103             osm.user = user;
     103            osm.setUser(user);
    104104            osm.setVisible(visible);
    105105            osm.mappaintStyle = null;

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

    r2273 r2291  
    225225        }
    226226        // user and visible added with 0.4 API
    227         if (osm.user != null) {
    228             if(osm.user.isLocalUser()) {
    229                 out.print(" user='"+XmlWriter.encode(osm.user.getName())+"'");
    230             } else if (osm.user.isOsmUser()) {
     227        if (osm.getUser() != null) {
     228            if(osm.getUser().isLocalUser()) {
     229                out.print(" user='"+XmlWriter.encode(osm.getUser().getName())+"'");
     230            } else if (osm.getUser().isOsmUser()) {
    231231                // uid added with 0.6
    232                 out.print(" uid='"+ osm.user.getId()+"'");
    233                 out.print(" user='"+XmlWriter.encode(osm.user.getName())+"'");
     232                out.print(" uid='"+ osm.getUser().getId()+"'");
     233                out.print(" user='"+XmlWriter.encode(osm.getUser().getName())+"'");
    234234            }
    235235        }
Note: See TracChangeset for help on using the changeset viewer.