Ticket #24446: 24446-2.patch

src/org/openstreetmap/josm/actions/upload/ApiPreconditionCheckerHook.java

@@ -73 +73 @@
                                         Long.toString(osmPrimitive.getId())
                                 )
                         );
-                        osmPrimitive.put(key, Utils.shortenString(value, Tagged.MAX_TAG_LENGTH));
+                        osmPrimitive.put(key, Utils.reduceCodePoints(value, Tagged.MAX_TAG_LENGTH));
                         continue;
                     }
                     JOptionPane.showMessageDialog(MainApplication.getMainFrame(),
@@ -106 +106 @@
         }
         return true;
     }
+
 }

src/org/openstreetmap/josm/tools/Utils.java

@@ -2049 +2049 @@
     public static boolean checkCodePointCount(String s, int maxLen) {
         return getCodePointCount(s) <= maxLen;
     }
+
+    /**
+     * Truncate a string to the given maximum number of code points.
+     * @param s the string
+     * @param num the maximum number of code points
+     * @return s if it is null or the number of code points is within the given limit, else a string with the first {@code num}
+     * code points
+     * @since xxx
+     */
+    public static String reduceCodePoints(String s, int num) {
+        if (checkCodePointCount(s, num))
+            return s;
+        String[] parts = s.split("\\b{g}"); // split along grapheme cluster boundaries
+        StringBuilder shorter = new StringBuilder();
+        for (int i = 0; i < num; i++) {
+            shorter.append(parts[i]);
+        }
+        return shorter.toString();
+    }
+
 }