Changeset 38232 for main/trunk/greenstone3

Timestamp:

2023-09-26T20:38:26+13:00 (8 months ago)

Author:

anupama

Message:

Got remove-import-metadata-array to work at least: metapositions needed to be provided in descending order.

File:

• main/trunk/greenstone3/web/interfaces/default/js/user_comments.js (modified) (1 diff)

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

@@ -272 +272 @@
     var failure = false;
 
-    // start deleting from end of metadata list, so we don't have to recalculate metapos each time
+    // Need to have metapositions in descending order for remove_metadata_array to work
+    // Start deleting from end of metadata list, so we don't have to recalculate metapos each time
+    /*
     for (var i = 0; i < selectedComments.length; i++) {
-
         var metapos = selectedComments[i].getAttribute("data-metapos");
-        metadataPositions.push(metapos); // add metapos to end of array
+        metadataPositions.unshift(metapos); // add metapos to start of array
+    }*/
+
+    for (var i = selectedComments.length - 1; i >= 0; i--) {
+        var metapos = selectedComments[i].getAttribute("data-metapos");
+        metadataPositions.push(metapos); // add next lowest metapos to end of array
     }
     }