Changeset 13216 in josm for trunk/src/org/openstreetmap/josm/actions

Timestamp:

2017-12-17T22:31:30+01:00 (6 years ago)

Author:

Don-vip

Message:

see #15606 - i18n fix

File:

• trunk/src/org/openstreetmap/josm/actions/relation/ExportRelationToGpxAction.java (modified) (4 diffs)

trunk/src/org/openstreetmap/josm/actions/relation/ExportRelationToGpxAction.java

@@ -77 +77 @@
      */
     public ExportRelationToGpxAction(EnumSet<Mode> mode) {
-        super(tr("{0} starting from {1} member",
-                 mode.contains(TO_FILE) ? tr("Export GPX file") : tr("Convert to GPX layer"),
-                 mode.contains(FROM_FIRST_MEMBER) ? tr("first") : tr("last")),
-              mode.contains(TO_FILE) ? "exportgpx" : "dialogs/layerlist",
-              tr("Flatten this relation to a single gpx track "+
-                 "recursively, starting with the {0} member(s), "+
-                 "successively continuing to the {1}.",
-                 mode.contains(FROM_FIRST_MEMBER) ? tr("first") : tr("last"),
-                 mode.contains(FROM_FIRST_MEMBER) ? tr("last") : tr("first")),
-              null, false, null, false);
+        super(name(mode), mode.contains(TO_FILE) ? "exportgpx" : "dialogs/layerlist", tooltip(mode),
+                null, false, null, false);
         putValue("help", ht("/Action/ExportRelationToGpx"));
         this.mode = mode;
+    }
+
+    private static String name(EnumSet<Mode> mode) {
+        if (mode.contains(TO_FILE)) {
+            if (mode.contains(FROM_FIRST_MEMBER)) {
+                return tr("Export GPX file starting from first member");
+            } else {
+                return tr("Export GPX file starting from last member");
+            }
+        } else {
+            if (mode.contains(FROM_FIRST_MEMBER)) {
+                return tr("Convert to GPX layer starting from first member");
+            } else {
+                return tr("Convert to GPX layer starting from last member");
+            }
+        }
+    }
+
+    private static String tooltip(EnumSet<Mode> mode) {
+        if (mode.contains(FROM_FIRST_MEMBER)) {
+            return tr("Flatten this relation to a single gpx track recursively, " +
+                    "starting with the first member(s), successively continuing to the last.");
+        } else {
+            return tr("Flatten this relation to a single gpx track recursively, " +
+                    "starting with the last member(s), successively continuing to the first.");
+        }
     }
 
@@ -163 +181 @@
             trk.add(trkseg);
 
-            //RelationSorter.sortMembersByConnectivity(defaultMembers); // preserve given order
             List<WayConnectionType> wct = new WayConnectionTypeCalculator().updateLinks(flat);
             final HashMap<String, Integer> names = new HashMap<>();
@@ -176 +193 @@
                             trk.add(trkseg);
                         }
-                        if (trkAttr.isEmpty()/* && (!wct.get(i).linkNext || (i+1 == flat.size()))*/) {
+                        if (trkAttr.isEmpty()) {
                             Relation r = Way.getParentRelations(Arrays.asList(flat.get(i).getWay()))
                                     .stream().filter(relsFound::contains).findFirst().orElseGet(null);
@@ -201 +218 @@
         }
 
-        //Relation debug = new Relation(); debug.setMembers(flat);
-        //MainApplication.getLayerManager().getEditDataSet().addPrimitive(debug);
         return new GpxLayer(gpxData, layerName, true);
     }