Changeset 14231 in josm for trunk/src/org/openstreetmap/josm/data

Timestamp:

2018-09-08T20:20:58+02:00 (6 years ago)

Author:

Don-vip

Message:

fix #16723 - Display changes count in changeset manager

File:

• trunk/src/org/openstreetmap/josm/data/osm/Changeset.java (modified) (4 diffs)

trunk/src/org/openstreetmap/josm/data/osm/Changeset.java

@@ -43 +43 @@
     /** the number of comments for this changeset */
     private int commentsCount;
+    /** the number of changes for this changeset */
+    private int changesCount;
     /** the map of tags */
     private Map<String, String> tags;
@@ -283 +285 @@
     }
 
+    /**
+     * Replies the number of changes for this changeset.
+     * @return the number of changes for this changeset
+     * @since 14231
+     */
+    public int getChangesCount() {
+        return changesCount;
+    }
+
+    /**
+     * Sets the number of changes for this changeset.
+     * @param changesCount the number of changes for this changeset
+     * @since 14231
+     */
+    public void setChangesCount(int changesCount) {
+        this.changesCount = changesCount;
+    }
+
     @Override
     public Map<String, String> getKeys() {
@@ -346 +366 @@
      */
     public boolean hasEqualSemanticAttributes(Changeset other) {
-        if (other == null)
-            return false;
-        if (closedAt == null) {
-            if (other.closedAt != null)
-                return false;
-        } else if (!closedAt.equals(other.closedAt))
-            return false;
-        if (createdAt == null) {
-            if (other.createdAt != null)
-                return false;
-        } else if (!createdAt.equals(other.createdAt))
-            return false;
-        if (id != other.id)
-            return false;
-        if (max == null) {
-            if (other.max != null)
-                return false;
-        } else if (!max.equals(other.max))
-            return false;
-        if (min == null) {
-            if (other.min != null)
-                return false;
-        } else if (!min.equals(other.min))
-            return false;
-        if (open != other.open)
-            return false;
-        if (!tags.equals(other.tags))
-            return false;
-        if (user == null) {
-            if (other.user != null)
-                return false;
-        } else if (!user.equals(other.user))
-            return false;
-        return commentsCount == other.commentsCount;
+        return other != null
+            && id == other.id
+            && open == other.open
+            && commentsCount == other.commentsCount
+            && changesCount == other.changesCount
+            && Objects.equals(closedAt, other.closedAt)
+            && Objects.equals(createdAt, other.createdAt)
+            && Objects.equals(min, other.min)
+            && Objects.equals(max, other.max)
+            && Objects.equals(tags, other.tags)
+            && Objects.equals(user, other.user);
     }
 
@@ -434 +431 @@
         this.max = other.max;
         this.commentsCount = other.commentsCount;
+        this.changesCount = other.changesCount;
         this.tags = new HashMap<>(other.tags);
         this.incomplete = other.incomplete;