Changeset 31555 for main/trunk/greenstone3/web/interfaces/default/js

Timestamp:

2017-03-30T15:32:31+13:00 (7 years ago)

Author:

ak19

Message:

Bugfix to commit 31547: parameter name mismatch with parameter use broke JavaScript at runtime. I've now tested that document editing works with the changes of this commit and commit 31547, by testing multiple edits of remove, set and edit metadata to 1 section and then 2 sections at a time, and checking archives and index contained the changes. Import doesn't contain the changes, since only set-archives-metadata is called by the document editor and then the save-and-rebuild button rebuilds the collection so the changes go into index too.

File:

• main/trunk/greenstone3/web/interfaces/default/js/javascript-global-functions.js (modified) (2 diffs)

main/trunk/greenstone3/web/interfaces/default/js/javascript-global-functions.js

@@ -455 +455 @@
     // For jQuery, AJAX calls ended up as GET when the payload is in URL form.
     // Default used to be payload in url form. To get the default back, 
-    // set method = "GET" (above, but also in calling functions!) and set the default here below
-    // for _use_payload_in_data_not_url_form to false.    
+    // set method = "GET" (above, but also in calling functions that specify this optional parameter!)
+    // and set the default here below for _use_payload_in_data_not_url_form to false.
     var _use_payload_in_data_not_url_form = false;
 
@@ -581 +581 @@
 // and determine which of the two forms ultimately gets used.
 // UNUSED: http://stackoverflow.com/questions/8648892/convert-url-parameters-to-a-javascript-object
-function getBasicDataForMetadataServer(metaServerCommand, collection, site, documentID, metadataName, metamode, metadataValue, prevMetadataValue, metaPosition) {
+function getBasicDataForMetadataServer(metaServerCommand, collection, site, documentID, metadataName, metamode, metadataValue, prevMetadataValue, metadataPosition) {
 
     // if we're doing set- or remove- metadata operations,