RM-473 ("Rename" allows you to duplicate list name)

git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/modules/recordsmanagement/HEAD@53995 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261

rm-server/config/alfresco/templates/webscripts/org/alfresco/rma/admin/rmconstraint/rmconstraint-utils.js

@@ -0,0 +1,18 @@
+function existsTitle(caveatConfig, title)
+{
+   var constraints = caveatConfig.allConstraints;
+
+   // Check for existing constraint...
+   var alreadyExists = false;
+   for (var i = 0; i < constraints.length; i++)
+   {
+      var currTitle = constraints[i].title;
+      if (currTitle + "" == title)
+      {
+         alreadyExists = true;
+         break;
+      }
+   }
+
+   return alreadyExists;
+}

rm-server/config/alfresco/templates/webscripts/org/alfresco/rma/admin/rmconstraint/rmconstraint.put.json.ftl

@@ -2,6 +2,10 @@
 
 <#escape x as jsonUtils.encodeJSONString(x)>
 {
-	"data": <@rmconstraintLib.constraintJSON constraint=constraint />       
+   <#if !errorMessage??>
+      "data": <@rmconstraintLib.constraintJSON constraint=constraint />
+   <#else>
+      "message" : "${msg(errorMessage, title)}"
+   </#if>
 }
 </#escape>

rm-server/config/alfresco/templates/webscripts/org/alfresco/rma/admin/rmconstraint/rmconstraint.put.json.js

@@ -1,42 +1,51 @@
+<import resource="classpath:/alfresco/templates/webscripts/org/alfresco/rma/admin/rmconstraint/rmconstraint-utils.js">
+
 /**
  * Update an rm constraint
- */ 
+ */
 function main()
 {
    // Get the shortname
    var shortName = url.extension;
-   
+
    // Get the constraint
    var constraint = caveatConfig.getConstraint(shortName);
-   
+
    if (constraint != null)
    {
-      var allowedValues
+      var allowedValues,
-      var title = null;
+         title = null;
-      
+
       if (json.has("constraintTitle"))
       {
          title = json.get("constraintTitle");
+         if (existsTitle(caveatConfig, title))
+         {
+            status.code = 400;
+            model.errorMessage = "rm.admin.list-already-exists";
+            model.title = title;
+            return;
+         }
          constraint.updateTitle(title);
       }
-      
+
       if (json.has("allowedValues"))
       {
          values = json.getJSONArray("allowedValues");
-         
+
          var i = 0;
          allowedValues = new Array();
-         
+
          if (values != null)
          {
             for (var x = 0; x < values.length(); x++)
-            {  
+            {
-               allowedValues[i++] = values.get(x);            
+               allowedValues[i++] = values.get(x);
             }
          }
          constraint.updateAllowedValues(allowedValues);
       }
-      
+
       // Pass the constraint detail to the template
       model.constraint = constraint;
    }

rm-server/config/alfresco/templates/webscripts/org/alfresco/rma/admin/rmconstraint/rmconstraint.put.properties

@@ -0,0 +1 @@
+rm.admin.list-already-exists=A list with the name "{0}" already exists.

rm-server/config/alfresco/templates/webscripts/org/alfresco/rma/admin/rmconstraint/rmconstraints.post.json.js

@@ -1,70 +1,54 @@
+<import resource="classpath:/alfresco/templates/webscripts/org/alfresco/rma/admin/rmconstraint/rmconstraint-utils.js">
+
 /**
  * Create a new RM Constraint List
- */ 
+ */
 function main()
 {
    // Parse the passed in details
-   var title = null;
+   var title = null,
-   var name = null;
+      name = null,
-   var allowedValues = {};
+      allowedValues = {};
-   
+
-   if (json.has("allowedValues"))
-   {
-      values = json.getJSONArray("allowedValues");
-      
-      var i = 0;
-      allowedValues = new Array();
-      
-      if (values != null)
-      {
-         for (var x = 0; x < values.length(); x++)
-         {  
-            allowedValues[i++] = values.get(x);
-         }
-      }
-   }
-   
    if (json.has("constraintName"))
    {
-      name = json.get("constraintName"); 
+      name = json.get("constraintName");
    }
-   
+
    if (json.has("constraintTitle"))
    {
-      title = json.get("constraintTitle"); 
+      title = json.get("constraintTitle");
    }
    else
    {
       title = name;
    }
-   
 
-   var constraints = caveatConfig.allConstraints;
+   if (existsTitle(caveatConfig, title))
-   
-   // Check for existing constraint...
-   var alreadyExists = false;
-   for (var i=0; i<constraints.length; i++)
-   {
-      var currTitle = constraints[i].title;
-      if (currTitle + "" == title)
-      {
-         alreadyExists = true;
-         break;
-      }
-   }
-   
-   var existingConstraint = caveatConfig.getConstraint(title);
-   if (!alreadyExists)
-   {
-      var constraint = caveatConfig.createConstraint(name, title, allowedValues);
-      model.constraint = constraint;
-   }
-   else
    {
       status.code = 400;
       model.errorMessage = "rm.admin.list-already-exists";
       model.title = title;
+      return;
    }
+
+   if (json.has("allowedValues"))
+   {
+      values = json.getJSONArray("allowedValues");
+
+      var i = 0;
+      allowedValues = new Array();
+
+      if (values != null)
+      {
+         for (var x = 0; x < values.length(); x++)
+         {
+            allowedValues[i++] = values.get(x);
+         }
+      }
+   }
+
+   model.constraint = caveatConfig.createConstraint(name, title, allowedValues);
 }
 
 main();

rm-server/config/alfresco/templates/webscripts/org/alfresco/rma/admin/rmconstraint/values/rmconstraint.post.json.ftl

@@ -2,6 +2,6 @@
 
 <#escape x as jsonUtils.encodeJSONString(x)>
 {
-	"data":  <@rmconstraintLib.constraintWithValuesJSON constraint=constraint />
+	"data": <@rmconstraintLib.constraintWithValuesJSON constraint=constraint />
 }
 </#escape>