From b4ad212e97622c38a99d9500dd72663f2b29ed56 Mon Sep 17 00:00:00 2001
From: Sebastian Pichelhofer <sp@apertus.org>
Date: Fri, 1 Sep 2017 16:13:34 +0200
Subject: [PATCH] check wether ID is unique when adding new placements, throw
 error if not unique

---
 src/main/java/org/openpnp/gui/JobPlacementsPanel.java | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/src/main/java/org/openpnp/gui/JobPlacementsPanel.java b/src/main/java/org/openpnp/gui/JobPlacementsPanel.java
index 75ddbd23aa7..b5064b857c4 100644
--- 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));