Improve UX of batch update

BernhardWebstudio · Feb 24, 2024 · 8e04da6 · 8e04da6
1 parent 9c428a1
commit 8e04da6
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/main/java/edu/harvard/mcz/imagecapture/ui/dialog/BatchUpdateDialog.java b/src/main/java/edu/harvard/mcz/imagecapture/ui/dialog/BatchUpdateDialog.java
@@ -139,8 +139,9 @@ private void doApplyChange() {
             // ask for confirmation
             int dialogButton = JOptionPane.YES_NO_OPTION;
             int dialogResult = JOptionPane.showConfirmDialog(this, nrOfChanges + " Specimen would be affected. Continue?", "Warning", dialogButton);
-            if (dialogResult == JOptionPane.NO_OPTION) {
+            if (!(dialogResult == JOptionPane.YES_OPTION)) {
                 log.debug("Will not do batch update. Confirmation denied.");
+                session.getTransaction().rollback();
                 return;
             }
         } catch (Exception e) {