Merge pull request #634 from sp-apertus/develop

check wether ID is unique when adding new placements, throw error if not unique
openpnp · Sep 1, 2017 · be1f85b · be1f85b
2 parents 63b567b + b4ad212
commit be1f85b
Showing 1 changed file with 10 additions and 1 deletion.
diff --git a/src/main/java/org/openpnp/gui/JobPlacementsPanel.java b/src/main/java/org/openpnp/gui/JobPlacementsPanel.java
@@ -302,7 +302,16 @@ public void actionPerformed(ActionEvent arg0) {
             if (id == null) {
                 return;
             }
-            // TODO: Make sure it's unique.
+
+            // Check if the new placement ID is unique
+            for(Placement compareplacement : boardLocation.getBoard().getPlacements()) {
+            	if (compareplacement.getId().equals(id)) {
+            		MessageBoxes.errorBox(getTopLevelAncestor(), "Error",
+                            "The ID for the new placement already exists");
+                    return;
+            	}
+            }
+
             Placement placement = new Placement(id);
 
             placement.setPart(Configuration.get().getParts().get(0));