Changeset 777

Timestamp:

Feb 4, 2011, 3:16:35 AM (14 years ago)

Author:

gav

Message:

TaskProcedure? javascript improvements, childCount++ moved out of function and isNew is already set by GSP template.

File:

• branches/features/taskProcedureRework/grails-app/views/taskProcedureDetailed/_maintenanceActions.gsp (modified) (5 diffs)

branches/features/taskProcedureRework/grails-app/views/taskProcedureDetailed/_maintenanceActions.gsp

@@ -1 +1 @@
 <script type="text/javascript">
-    var childCount = ${taskProcedureInstance?.maintenanceActions.size()} + 0;
+    var ma_childCount = ${taskProcedureInstance?.maintenanceActions.size()} + 0;
     var ma_wrapperId = "ma_wrapper";
     var ma_cloneId = "maintenanceActionLazyList_clone";
@@ -8 +8 @@
 
 
-    function addChild(wrapperId, cloneId, lazyList, fields, focusField){
+    function addChild(wrapperId, cloneId, lazyList, fields, focusField, childCount){
 
         var clone = jQuery("#"+cloneId).clone();
@@ -19 +19 @@
             fieldsMap[field].attr('id',htmlId + field)
                                         .attr('name',htmlId + field);
-            if(field == 'isNew') {
-                fieldsMap[field].attr('value', 'true');
-            }
         });
 
@@ -27 +24 @@
         clone.show();
         fieldsMap[focusField].focus();
-        childCount++;
     }
 
     // Click event on add button.
     jQuery('.add-ma').live('click', function() {
-            addChild(ma_wrapperId, ma_cloneId, ma_lazyList, ma_fields, ma_focusField);
+        addChild(ma_wrapperId, ma_cloneId, ma_lazyList, ma_fields, ma_focusField, ma_childCount);
+        ma_childCount++;
     });
 
@@ -55 +52 @@
 
     jQuery(window).load(function() {
-        if(childCount == 0) {
-            addChild(ma_wrapperId, ma_cloneId, ma_lazyList, ma_fields, ma_focusField);
+        if(ma_childCount == 0) {
+            addChild(ma_wrapperId, ma_cloneId, ma_lazyList, ma_fields, ma_focusField, ma_childCount);
+            ma_childCount++;
         }
     });