Changeset 8873 in josm

Timestamp:

2015-10-14T22:55:08+02:00 (9 years ago)

Author:

Don-vip

Message:

fix #11967 - Regression: Object count in validator broken + add unit test

Location:

trunk

Files:

• src/org/openstreetmap/josm/data/validation/util/MultipleNameVisitor.java (modified) (1 diff)
• test/unit/org/openstreetmap/josm/data/validation/util (added)
• test/unit/org/openstreetmap/josm/data/validation/util/MultipleNameVisitorTest.java (added)

trunk/src/org/openstreetmap/josm/data/validation/util/MultipleNameVisitor.java

@@ -63 +63 @@
             displayName = name;
         } else {
-            displayName = size + ' ' + trn(multipleClassname, multiplePluralClassname, size);
+            StringBuilder sb = new StringBuilder().append(size).append(' ').append(trn(multipleClassname, multiplePluralClassname, size));
             if (multipleName.length() > 0) {
+                sb.append(": ");
                 if (multipleName.length() <= MULTIPLE_NAME_MAX_LENGTH) {
-                    displayName += ": " + multipleName;
+                    sb.append(multipleName);
                 } else {
-                    displayName += ": " + multipleName.substring(0, MULTIPLE_NAME_MAX_LENGTH) + "...";
+                    sb.append(multipleName.substring(0, MULTIPLE_NAME_MAX_LENGTH)).append("...");
                 }
             }
+            displayName = sb.toString();
         }
     }