From 2b93b000e4eb1901d43cc90abb2e1a7792900fa6 Mon Sep 17 00:00:00 2001
From: Susana Montes Pedreira <smontes@wirelessgalicia.com>
Date: Wed, 13 Oct 2010 22:22:43 +0200
Subject: [PATCH] It checks if the prefix of the code sequences is unique for
 each entity.

FEA :ItEr61S04NavalPlanEntities
---
 .../web/common/ConfigurationModel.java        | 24 +++++++++++++++++--
 1 file changed, 22 insertions(+), 2 deletions(-)

diff --git a/navalplanner-webapp/src/main/java/org/navalplanner/web/common/ConfigurationModel.java b/navalplanner-webapp/src/main/java/org/navalplanner/web/common/ConfigurationModel.java
index 2d9de3400..1e1af803b 100644
--- a/navalplanner-webapp/src/main/java/org/navalplanner/web/common/ConfigurationModel.java
+++ b/navalplanner-webapp/src/main/java/org/navalplanner/web/common/ConfigurationModel.java
@@ -26,8 +26,10 @@ import java.util.ArrayList;
 import java.util.Collection;
 import java.util.ConcurrentModificationException;
 import java.util.HashMap;
+import java.util.HashSet;
 import java.util.List;
 import java.util.Map;
+import java.util.Set;
 
 import org.navalplanner.business.calendars.daos.IBaseCalendarDAO;
 import org.navalplanner.business.calendars.entities.BaseCalendar;
@@ -133,13 +135,12 @@ public class ConfigurationModel implements IConfigurationModel {
     public void confirm() {
 
         checkEntitySequences();
-
         try {
             configurationDAO.save(configuration);
             storeAndRemoveEntitySequences();
         } catch (HibernateOptimisticLockingFailureException e) {
             throw new ConcurrentModificationException(
-                    _("Some order was created during the configuration process, it is impossible to update order sequence table. Please, try again later"));
+                    _("Some entity sequence was created during the configuration process, it is impossible to update entity sequence table. Please, try again later"));
         }
     }
 
@@ -157,9 +158,27 @@ public class ConfigurationModel implements IConfigurationModel {
                 throw new ValidationException(_(
                         "At least one {0} sequence must be active", entity));
             }
+            if (!checkConstraintPrefixNotRepeated(sequences)) {
+                throw new ValidationException(_(
+                        "The {0} sequence prefixes can not be repeated",
+                        entityName.getDescription()));
+            }
         }
     }
 
+    private boolean checkConstraintPrefixNotRepeated(
+            List<EntitySequence> sequences) {
+        Set<String> prefixes = new HashSet<String>();
+        for (EntitySequence sequence : sequences) {
+            String prefix = sequence.getPrefix();
+            if (prefixes.contains(prefix)) {
+                return false;
+            }
+            prefixes.add(prefix);
+        }
+        return true;
+    }
+
     private boolean isAnyActive(List<EntitySequence> sequences) {
         for (EntitySequence entitySequence : sequences) {
             if (entitySequence.isActive()) {
@@ -427,4 +446,5 @@ public class ConfigurationModel implements IConfigurationModel {
         entitySequences.get(entitySequence.getEntityName()).remove(
                 entitySequence);
     }
+
 }
\ No newline at end of file