Changeset 33100 in osm for applications/editors/josm

Timestamp:

2016-12-09T23:03:01+01:00 (8 years ago)

Author:

donvip

Message:

fix IAE seen in JOSM integration tests

File:

• applications/editors/josm/plugins/print/src/org/openstreetmap/josm/plugins/print/PrintPlugin.java (modified) (1 diff)

applications/editors/josm/plugins/print/src/org/openstreetmap/josm/plugins/print/PrintPlugin.java

@@ -47 +47 @@
         } while (fileMenu != null && pos > 2 && fileMenu.getItem(pos) != null);
 
-        PrintAction printAction = new PrintAction();
-        printMenu = fileMenu.insert(printAction, pos);
-        printMenu.setEnabled(false);
-        printMenu.setVisible(true);
+        if (pos > 0) {
+            PrintAction printAction = new PrintAction();
+            printMenu = fileMenu.insert(printAction, pos);
+            printMenu.setEnabled(false);
+            printMenu.setVisible(true);
 
-        KeyStroke ks = printAction.getShortcut().getKeyStroke();
-        if (ks != null) {
-            printMenu.setAccelerator(ks);
+            KeyStroke ks = printAction.getShortcut().getKeyStroke();
+            if (ks != null) {
+                printMenu.setAccelerator(ks);
+            }
+
+            fileMenu.insertSeparator(pos);
         }
-
-        fileMenu.insertSeparator(pos);
 
         restorePrefs(); // Recover after crash if necessary