Changeset 8947 in josm for trunk/src

Timestamp:

2015-10-25T14:52:06+01:00 (9 years ago)

Author:

Don-vip

Message:

fix HeadlessException seen in unit tests

File:

• trunk/src/org/openstreetmap/josm/gui/conflict/tags/CombinePrimitiveResolverDialog.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/gui/conflict/tags/CombinePrimitiveResolverDialog.java

@@ -10 +10 @@
 import java.awt.Dimension;
 import java.awt.FlowLayout;
+import java.awt.GraphicsEnvironment;
 import java.awt.event.ActionEvent;
 import java.awt.event.HierarchyBoundsListener;
@@ -494 +495 @@
         }
 
-        // Build conflict resolution dialog
-        final CombinePrimitiveResolverDialog dialog = CombinePrimitiveResolverDialog.getInstance();
-
-        dialog.getTagConflictResolverModel().populate(tagsToEdit, completeWayTags.getKeysWithMultipleValues());
-        dialog.getRelationMemberConflictResolverModel().populate(parentRelations, primitives);
-        dialog.prepareDefaultDecisions();
-
-        // Ensure a proper title is displayed instead of a previous target (fix #7925)
-        if (targetPrimitives.size() == 1) {
-            dialog.setTargetPrimitive(targetPrimitives.iterator().next());
-        } else {
-            dialog.setTargetPrimitive(null);
-        }
-
-        // Resolve tag conflicts if necessary
-        if (!dialog.isResolvedCompletely()) {
-            dialog.setVisible(true);
-            if (dialog.isCanceled()) {
-                throw new UserCancelException();
-            }
-        }
         List<Command> cmds = new LinkedList<>();
-        for (OsmPrimitive i : targetPrimitives) {
-            dialog.setTargetPrimitive(i);
-            cmds.addAll(dialog.buildResolutionCommands());
+
+        if (!GraphicsEnvironment.isHeadless()) {
+            // Build conflict resolution dialog
+            final CombinePrimitiveResolverDialog dialog = CombinePrimitiveResolverDialog.getInstance();
+
+            dialog.getTagConflictResolverModel().populate(tagsToEdit, completeWayTags.getKeysWithMultipleValues());
+            dialog.getRelationMemberConflictResolverModel().populate(parentRelations, primitives);
+            dialog.prepareDefaultDecisions();
+
+            // Ensure a proper title is displayed instead of a previous target (fix #7925)
+            if (targetPrimitives.size() == 1) {
+                dialog.setTargetPrimitive(targetPrimitives.iterator().next());
+            } else {
+                dialog.setTargetPrimitive(null);
+            }
+
+            // Resolve tag conflicts if necessary
+            if (!dialog.isResolvedCompletely()) {
+                dialog.setVisible(true);
+                if (dialog.isCanceled()) {
+                    throw new UserCancelException();
+                }
+            }
+            for (OsmPrimitive i : targetPrimitives) {
+                dialog.setTargetPrimitive(i);
+                cmds.addAll(dialog.buildResolutionCommands());
+            }
         }
         return cmds;