Changeset 27790

Timestamp:

2013-07-09T16:01:44+12:00 (11 years ago)

Author:

sjm84

Message:

Modified the code so that no async calls we being called at the same time

File:

• main/trunk/greenstone3/web/interfaces/default/js/documentmaker_scripts_util.js (modified) (3 diffs)

main/trunk/greenstone3/web/interfaces/default/js/documentmaker_scripts_util.js

@@ -217 +217 @@
         var changedElem = changes[i];
 
+        var metadataChanges = new Array();
+        
         //Save metadata 
         if(gs.functions.hasClass(changedElem, "metaTableCell")) 
@@ -234 +236 @@
             value = escape(value);
 
-            if(changedElem.originalValue)
-            {
-                gs.functions.setArchivesMetadata(collection, gs.xsltParams.site_name, docID, name, null, value, changedElem.originalValue, "override", function(){console.log("SAVED ARCHIVES");});
-            }
-            else
-            {
-                gs.functions.setArchivesMetadata(collection, gs.xsltParams.site_name, docID, name, null, value, null, "accumulate", function(){console.log("SAVED ARCHIVES");});
-            }
+            metadataChanges.push({collection:collection, docID:docID, name:name, value:value, orig:changedElem.originalValue});
+
             changedElem.originalValue = changedElem.innerHTML;
             addCollectionToBuild(collection);
@@ -260 +256 @@
         }
     }
-
-    var request = "[";
-    for(var i = 0; i < _transactions.length; i++)
-    {
-        request += _transactions[i];
-        if(i != _transactions.length - 1)
-        {
-            request += ",";
-        }
-    }
-    request += "]";
-
-    var statusID;
-    var ajax = new gs.functions.ajaxRequest();
-    ajax.open("POST", gs.xsltParams.library_name, true);
-    ajax.setRequestHeader("Content-type", "application/x-www-form-urlencoded");
-    ajax.onreadystatechange = function()
-    {
-        if(ajax.readyState == 4 && ajax.status == 200)
-        {
-            var text = ajax.responseText;
-            var xml = validateXML(text);
-            
-            var errorElems;
-            if(!xml || checkForErrors(xml))
+    
+    var sendBuildRequest = function()
+    {
+        var request = "[";
+        for(var i = 0; i < _transactions.length; i++)
+        {
+            request += _transactions[i];
+            if(i != _transactions.length - 1)
             {
-                alert(gs.text.dse.error_saving);
-            
-                var saveButton = document.getElementById("saveButton");
-                saveButton.innerHTML = gs.text.dse.save_changes;
-                saveButton.disabled = false;
+                request += ",";
+            }
+        }
+        request += "]";
+
+        var statusID;
+        var ajax = new gs.functions.ajaxRequest();
+        ajax.open("POST", gs.xsltParams.library_name, true);
+        ajax.setRequestHeader("Content-type", "application/x-www-form-urlencoded");
+        ajax.onreadystatechange = function()
+        {
+            if(ajax.readyState == 4 && ajax.status == 200)
+            {
+                var text = ajax.responseText;
+                var xml = validateXML(text);
                 
+                var errorElems;
+                if(!xml || checkForErrors(xml))
+                {
+                    alert(gs.text.dse.error_saving);
+                
+                    var saveButton = document.getElementById("saveButton");
+                    saveButton.innerHTML = gs.text.dse.save_changes;
+                    saveButton.disabled = false;
+                    
+                    _statusBar.removeStatus(statusID);
+                    return;
+                }
+
                 _statusBar.removeStatus(statusID);
-                return;
+                buildCollections(_collectionsToBuild);
             }
-
-            _statusBar.removeStatus(statusID);
-            buildCollections(_collectionsToBuild);
-        }
-    }
-
-    if(_collectionsToBuild.length > 0)
-    {
-        var saveButton = document.getElementById("saveButton");
-        saveButton.innerHTML = gs.text.dse.saving + "...";
-        saveButton.disabled = true;
-
-        statusID = _statusBar.addStatus(gs.text.dse.modifying_archives + "...");
-        ajax.send("a=g&rt=r&s=DocumentExecuteTransaction&s1.transactions=" + request);
-    }
+        }
+
+        if(_collectionsToBuild.length > 0)
+        {
+            var saveButton = document.getElementById("saveButton");
+            saveButton.innerHTML = gs.text.dse.saving + "...";
+            saveButton.disabled = true;
+
+            statusID = _statusBar.addStatus(gs.text.dse.modifying_archives + "...");
+            ajax.send("a=g&rt=r&s=DocumentExecuteTransaction&s1.transactions=" + request);
+        }
+    }
+    
+    var setMetadataLoop = function(index)
+    {
+        var change = metadataChanges[index];
+        
+        var callbackFunction;
+        if(index + 1 == metadataChanges.length)
+        {
+            callbackFunction = sendBuildRequest;
+        }
+        else
+        {
+            callbackFunction = function(){setMetadataLoop(index + 1)};
+        }
+        
+        if(change.orig)
+        {
+            gs.functions.setArchivesMetadata(change.collection, gs.xsltParams.site_name, change.docID, change.name, null, change.value, change.orig, "override", function(){callbackFunction();});
+        }
+        else
+        {
+            gs.functions.setArchivesMetadata(change.collection, gs.xsltParams.site_name, change.docID, change.name, null, change.value, null, "accumulate", function(){callbackFunction();});
+        }
+    }
+    setMetadataLoop(0);
 }