fix(swordv2): guard against double-delete in ContainerManagerDSpace.removeItem

Deleting a SWORDv2 item with WorkflowManagerDefault deleted the item twice and
the second itemService.delete() threw. The fix -- deleteAll on the workspace
path plus an isItemAlreadyDeleted guard before the final delete -- was present
on dtq-dev via f79e704 and reverted by the #1031 7.6.5 upgrade merge, while
the two Swordv2IT tests that cover it were left behind. This restores the
reverted delta (identical to customer/zcu-pub 66af883).

Not taken from zcu-pub: its SwordUrlManager changes -- dtq-dev is ahead there.

Port of dataquest-dev/dspace-customers#903 (item 3).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Author: jr-rk

dspace-swordv2/src/main/java/org/dspace/sword2/ContainerManagerDSpace.java

@@ -13,6 +13,7 @@
 import java.util.List;
 import java.util.Map;
 import java.util.TreeMap;
+import java.util.UUID;
 
 import org.apache.logging.log4j.Logger;
 import org.dspace.authorize.AuthorizeException;
@@ -25,6 +26,7 @@
 import org.dspace.core.Constants;
 import org.dspace.core.Context;
 import org.dspace.core.LogHelper;
+import org.dspace.event.Event;
 import org.dspace.workflow.WorkflowItem;
 import org.dspace.workflow.WorkflowItemService;
 import org.dspace.workflow.factory.WorkflowServiceFactory;
@@ -755,14 +757,19 @@ protected void doContainerDelete(SwordContext swordContext, Item item,
             WorkflowTools wft = new WorkflowTools();
             if (wft.isItemInWorkspace(swordContext.getContext(), item)) {
                 WorkspaceItem wsi = wft.getWorkspaceItem(context, item);
-                workspaceItemService.deleteWrapper(context, wsi);
+                workspaceItemService.deleteAll(context, wsi);
+                // the item is deleted in the above call
             } else if (wft.isItemInWorkflow(context, item)) {
                 WorkflowItem wfi = wft.getWorkflowItem(context, item);
                 workflowItemService.deleteWrapper(context, wfi);
             }
 
-            // then delete the item
-            itemService.delete(context, item);
+            // then delete the item, but only if it hasn't already been deleted by the methods above.
+            // the delete method is called in `workspaceItemService.deleteAll(context, wsi);`,
+            // so it should not be called again here, as that would throw an exception.
+            if (!isItemAlreadyDeleted(context, item.getID())) {
+                itemService.delete(context, item);
+            }
         } catch (SQLException | IOException e) {
             throw new DSpaceSwordException(e);
         } catch (AuthorizeException e) {
@@ -788,4 +795,20 @@ private Item getDSpaceTarget(Context context, String editUrl,
 
         return item;
     }
+
+    /**
+     * Check if the item is already deleted in the context.
+     */
+    private boolean isItemAlreadyDeleted(Context context, UUID itemUUID) {
+        if (context.getEvents() == null) {
+            return false;
+        }
+
+        for (Event event : context.getEvents()) {
+            if (event.getEventType() == Event.DELETE && event.getSubjectID().equals(itemUUID)) {
+                return true;
+            }
+        }
+        return false;
+    }
 }