From 78fa2cf4e8d01735bfe8491a3c0a0c010903f862 Mon Sep 17 00:00:00 2001
From: rjy7 <rjy7@43c2fc9b-1942-4403-87b2-35b4218f7473>
Date: Thu, 15 Jul 2010 12:14:29 +0000
Subject: [PATCH] NIHVIVO-797 Javascript support for edit mode in roles forms

---
 .../edit/forms/addRoleToPersonTwoStage.jsp    |  3 +-
 .../js/customFormWithAdvanceTypeSelection.js  | 38 ++++++++++++-------
 2 files changed, 26 insertions(+), 15 deletions(-)

diff --git a/productMods/edit/forms/addRoleToPersonTwoStage.jsp b/productMods/edit/forms/addRoleToPersonTwoStage.jsp
index 1fd8e2dd..91ecf068 100644
--- a/productMods/edit/forms/addRoleToPersonTwoStage.jsp
+++ b/productMods/edit/forms/addRoleToPersonTwoStage.jsp
@@ -315,6 +315,7 @@
         <c:set var="editMode" value="add" />
     </c:otherwise>
 </c:choose>
+
 <jsp:include page="${preForm}" />
 
 <h2>${titleText}&nbsp;${roleActivityTypeLabel} entry for <%= subjectName %></h2>
@@ -326,7 +327,7 @@
     
     <div class="fullViewOnly">
         
-	    <p><v:input type="text" id="label" name="activityLabel" label="Title" cssClass="acSelector" size="50" /></p>
+	    <p><v:input type="text" id="label" name="activityLabel" label="Name ${requiredHint}" cssClass="acSelector" size="50" /></p>
 
 	    <div class="acSelection">
 	        <%-- RY maybe make this a label and input field. See what looks best. --%>
diff --git a/productMods/edit/forms/js/customFormWithAdvanceTypeSelection.js b/productMods/edit/forms/js/customFormWithAdvanceTypeSelection.js
index f1497f40..621599df 100644
--- a/productMods/edit/forms/js/customFormWithAdvanceTypeSelection.js
+++ b/productMods/edit/forms/js/customFormWithAdvanceTypeSelection.js
@@ -39,8 +39,9 @@ var customForm = {
         this.verifyMatchBaseHref = this.verifyMatch.attr('href');    
         this.acSelectorWrapper = this.acSelector.parent();
         
+        this.existingIndividualLabel = $('#label');
         // This is the label element for the field with name 'label'
-        this.labelFieldLabel = $('label[for=' + $('#label').attr('id') + ']');       
+        this.labelFieldLabel = $('label[for=' + this.existingIndividualLabel.attr('id') + ']');       
         // Get this on page load, so we can prepend to it. We can't just prepend to the current label text,
         // because it may have already been modified for a previous selection.
         this.baseLabelText = this.labelFieldLabel.html();
@@ -58,10 +59,10 @@ var customForm = {
     initPage: function() {
 
         if (!this.editMode) {
-            this.editMode = "add"; // edit vs add: default to add
+            this.editMode = 'add'; // edit vs add: default to add
         }
         
-        if (!this.typeSelector.length || this.editMode == "edit") {
+        if (!this.typeSelector.length || this.editMode == 'edit') {
             this.formSteps = 1;
         // there's also going to be a 3-step form - look for this.subTypeSelector
         } else {
@@ -97,7 +98,7 @@ var customForm = {
     },
     
     initFormTypeView: function() {
-  
+        
         this.hideFields(this.fullViewOnly);
         this.button.hide();
         this.requiredLegend.hide();
@@ -107,16 +108,19 @@ var customForm = {
     },
     
     initFormFullView: function() {
-
+        
+        if (this.editMode == 'edit') {
+            this.initFormEditView();
+        }
+        
         this.fullViewOnly.show();
         this.or.show();
         this.requiredLegend.show();
         this.button.show();
-        this.toggleButtonText("new");
+        this.toggleButtonText('new');
         this.cancel.unbind('click');     
            
         if( this.formSteps > 1 ){
-
             this.cancel.click(function() {
                 customForm.clearFormData(); // clear any input and validation errors
                 customForm.initFormTypeView();
@@ -125,6 +129,12 @@ var customForm = {
         }
     },
     
+    initFormEditView: function() {
+        // These should not be editable: only properties of the role are editable.
+        this.typeSelector.attr('disabled', 'disabled');
+        this.existingIndividualLabel.attr('disabled', 'disabled');
+    },
+    
     // Bind event listeners that persist over the life of the page.
     bindEventListeners: function() {
         
@@ -231,11 +241,11 @@ var customForm = {
         filteredResults = [];
         $.each(results, function() {
             if ($.inArray(this.uri, customForm.acFilter) == -1) {
-                // console.log("adding " + this.label + " to filtered results");
+                // console.log('adding ' + this.label + ' to filtered results');
                 filteredResults.push(this);
             }
             else {
-                // console.log("filtering out " + this.label);
+                // console.log('filtering out ' + this.label);
             }
         });
         return filteredResults;
@@ -316,17 +326,17 @@ var customForm = {
     },
     
     toggleButtonText: function(newOrExisting) {
-        if (this.editMode == "edit") {
-            this.button.val("Edit " + this.baseButtonText);
+        if (this.editMode == 'edit') {
+            this.button.val('Edit ' + this.baseButtonText);
         }
         // RY Make this better for roles and grants forms later. The verbs
         // don't quite work. It should be "Create X and <role>" vs.
         // "Create <role>" or "Add <role>"
-        else if (newOrExisting == "new") {
-            this.button.val("Create " + this.baseButtonText);                
+        else if (newOrExisting == 'new') {
+            this.button.val('Create ' + this.baseButtonText);                
         }
         else {
-            this.button.val("Add " + this.baseButtonText);
+            this.button.val('Add ' + this.baseButtonText);
         } 
     }