Changeset 16950 in josm for trunk/src/org

Timestamp:

2020-08-28T22:38:19+02:00 (4 years ago)

Author:

simon04

Message:

Revert "fix #19717 - DefaultNameFormatter: use FIRST-STRONG ISOLATE for bidirectional texts"

This reverts commit r16937

Location:

trunk/src/org/openstreetmap/josm

Files:

• data/osm/DefaultNameFormatter.java (modified) (9 diffs)
• gui/datatransfer/PrimitiveTransferable.java (modified) (1 diff)
• gui/dialogs/InspectPrimitiveDialog.java (modified) (1 diff)

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

@@ -7 +7 @@
 import static org.openstreetmap.josm.tools.I18n.trn;
 
+import java.awt.ComponentOrientation;
 import java.util.ArrayList;
 import java.util.Arrays;
@@ -15 +16 @@
 import java.util.LinkedList;
 import java.util.List;
+import java.util.Locale;
 import java.util.Map;
 import java.util.Objects;
@@ -42 +44 @@
  */
 public class DefaultNameFormatter implements NameFormatter, HistoryNameFormatter {
-    /**
-     * U+2068 FIRST STRONG ISOLATE
-     */
-    public static final char BIDI_FIRST_STRONG_ISOLATE = '\u2068';
-    /**
-     * U+2069 POP DIRECTIONAL ISOLATE
-     */
-    public static final char BIDI_POP_DIRECTIONAL_ISOLATE = '\u2069';
+
     private static DefaultNameFormatter instance;
 
@@ -147 +142 @@
                     primitive.getUniqueId() : primitive.getId();
             if (Config.getPref().getBoolean("osm-primitives.showversion") && version > 0) {
-                bdi(name, tr(" [id: {0}, v{1}]", id, version));
+                name.append(tr(" [id: {0}, v{1}]", id, version));
             } else {
-                bdi(name, tr(" [id: {0}]", id));
+                name.append(tr(" [id: {0}]", id));
             }
         } else if (Config.getPref().getBoolean("osm-primitives.showversion")) {
-            bdi(name, tr(" [v{0}]", version));
+            name.append(tr(" [v{0}]", version));
         }
     }
@@ -211 +206 @@
                     n = node.isNew() ? tr("node") : Long.toString(node.getId());
                 }
-                bdi(name, n);
+                name.append(n);
             } else {
                 preset.nameTemplate.appendText(name, (TemplateEngineDataProvider) node);
             }
             if (node.isLatLonKnown() && Config.getPref().getBoolean("osm-primitives.showcoor")) {
-                name.append(' ');
-                bdiParen(name, CoordinateFormatManager.getDefaultFormat().toString(node, ", "));
+                name.append(" \u200E(");
+                name.append(CoordinateFormatManager.getDefaultFormat().toString(node, ", "));
+                name.append(')');
             }
         }
@@ -239 +235 @@
     public String format(IWay<?> way) {
         StringBuilder name = new StringBuilder();
+
+        char mark;
+        // If current language is left-to-right (almost all languages)
+        if (ComponentOrientation.getOrientation(Locale.getDefault()).isLeftToRight()) {
+            // will insert Left-To-Right Mark to ensure proper display of text in the case when object name is right-to-left
+            mark = '\u200E';
+        } else {
+            // otherwise will insert Right-To-Left Mark to ensure proper display in the opposite case
+            mark = '\u200F';
+        }
+        // Initialize base direction of the string
+        name.append(mark);
+
         if (way.isIncomplete()) {
             name.append(tr("incomplete"));
@@ -284 +293 @@
                 }
 
-                bdi(name, n);
+                name.append(n);
             } else {
                 preset.nameTemplate.appendText(name, (TemplateEngineDataProvider) way);
@@ -294 +303 @@
             /* I18n: count of nodes as parameter */
             String nodes = trn("{0} node", "{0} nodes", nodesNo, nodesNo);
-            name.append(" ");
-            bdiParen(name, nodes);
+            name.append(mark).append(" (").append(nodes).append(')');
         }
         decorateNameWithId(name, way);
@@ -635 +643 @@
 
     /**
-     * Wraps the content in a bidirectional isolate
-     * @param builder the builder to append to
-     * @param content the text to wrap
-     */
-    private static void bdi(StringBuilder builder, String content) {
-        builder.append(BIDI_FIRST_STRONG_ISOLATE);
-        builder.append(content);
-        builder.append(BIDI_POP_DIRECTIONAL_ISOLATE);
-    }
-
-    /**
-     * Wraps the content in parenthesis and a bidirectional isolate
-     * @param builder the builder to append to
-     * @param content the text to wrap
-     */
-    private static void bdiParen(StringBuilder builder, String content) {
-        builder.append(BIDI_FIRST_STRONG_ISOLATE);
-        builder.append('(');
-        builder.append(content);
-        builder.append(')');
-        builder.append(BIDI_POP_DIRECTIONAL_ISOLATE);
-    }
-
-    /**
      * Builds a default tooltip text for an HistoryOsmPrimitive <code>primitive</code>.
      *

trunk/src/org/openstreetmap/josm/gui/datatransfer/PrimitiveTransferable.java

@@ -91 +91 @@
             sb.append(OsmPrimitiveType.from(primitive).getAPIName()).append(' ').append(primitive.getId());
         }
-        return sb.toString();
+        return sb.toString().replace("\u200E", "").replace("\u200F", "");
     }
 }

trunk/src/org/openstreetmap/josm/gui/dialogs/InspectPrimitiveDialog.java

@@ -129 +129 @@
         try {
             for (IPrimitive osm : sel) {
-                String heading = tr("Styles for \"{0}\":", osm.getDisplayName(DefaultNameFormatter.getInstance())
-                        .replace(Character.toString(DefaultNameFormatter.BIDI_FIRST_STRONG_ISOLATE), "")
-                        .replace(Character.toString(DefaultNameFormatter.BIDI_POP_DIRECTIONAL_ISOLATE), ""));
+                String heading = tr("Styles for \"{0}\":", osm.getDisplayName(DefaultNameFormatter.getInstance()));
                 txtMappaint.println(heading);
                 txtMappaint.println(repeatString("=", heading.length()));