Changeset 28 in josm for src/org/openstreetmap/josm/data

Timestamp:

2005-11-30T21:40:09+01:00 (18 years ago)

Author:

imi

Message:

• fixed bug: second layer does not get the pending line segments of the first
• fixed bug: property page edit action "cancel" effect the display of the table
• data entries with same id now same data

Location:

src/org/openstreetmap/josm/data/osm

Files:

• DataSet.java (modified) (4 diffs)
• OsmPrimitive.java (modified) (1 diff)

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

@@ -218 +218 @@
         public void clearSelection() {
                 clearSelection(nodes);
+                clearSelection(pendingLineSegments);
                 clearSelection(tracks);
                 for (Track t : tracks)
@@ -242 +243 @@
          * to both datasets. So use mergeFrom only if you are about to abandon the
          * other dataset.
-         * 
+         *
          * Elements are tried to merged. 
          * Nodes are merged first, if their lat/lon are equal.
          * Line segments are merged, if they have the same nodes.
-         * Tracs are merged, if they consist of the same line segments.
-         * 
+         * Tracks are merged, if they consist of the same line segments.
+         *
          * Additional to that, every two objects with the same id are merged.
-         * 
+         *
          * @param ds    The DataSet to merge into this one.
          * @return A list of all primitives that were used in the conjunction. That
@@ -304 +305 @@
                                         lsMap.put(otherLS, myLS);
                 // add pendings (ls from track are added later
+                data.addAll(new HashSet<LineSegment>(lsMap.values()));
                 for (LineSegment ls : ds.pendingLineSegments) {
                         if (!lsMap.containsKey(ls)) {
@@ -318 +320 @@
                         }
                 }
-                
-                
+
+
                 // merge tracks
                 LinkedList<Track> trackToAdd = new LinkedList<Track>();

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

@@ -84 +84 @@
                 return selected;
         }
+
+
+        /**
+         * Equal, if the id is equal. If both ids are 0, use the super classes equal
+         * instead.
+         */
+        @Override
+        public boolean equals(Object obj) {
+                if (!(obj instanceof OsmPrimitive))
+                        return false;
+                OsmPrimitive osm = (OsmPrimitive)obj;
+                if (id == 0 && osm.id == 0)
+                        return super.equals(obj);
+                return id == osm.id;
+        }
+
+        /**
+         * Return the id as hashcode or supers hashcode if 0.
+         */
+        @Override
+        public int hashCode() {
+                return id == 0 ? super.hashCode() : (int)id;
+        }
 }