Make sure bulk:deprecate and bulk:keep does not point to the same res…

…ource
libris · Nov 20, 2024 · c267cc2 · c267cc2
1 parent 73c4b79
commit c267cc2
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/whelktool/src/main/resources/bulk-change-scripts/merge.groovy b/whelktool/src/main/resources/bulk-change-scripts/merge.groovy
@@ -16,6 +16,11 @@ if (!deprecateId || !keepId) return
 List<String> obsoleteThingUris = []
 
 selectByIds([deprecateId]) { obsolete ->
+    // Assert that the resource to deprecate is not the same as the one to be kept
+    if (obsolete.doc.getThingIdentifiers().first() == keepId) {
+        return
+    }
+
     obsoleteThingUris = obsolete.doc.getThingIdentifiers()
 
     selectByIds(obsolete.getDependers()) { depender ->