0053769: Gift Card is consumed although it is removed with the Undo button

diff --git a/web/org.openbravo.retail.giftcards/app/model/business-object/ticket/actions/AddProductActionPreparations.js b/web/org.openbravo.retail.giftcards/app/model/business-object/ticket/actions/AddProductActionPreparations.js
index 3ab4754..51c6231 100644
--- a/web/org.openbravo.retail.giftcards/app/model/business-object/ticket/actions/AddProductActionPreparations.js
+++ b/web/org.openbravo.retail.giftcards/app/model/business-object/ticket/actions/AddProductActionPreparations.js
@@ -1,6 +1,6 @@
 /*
  ************************************************************************************
- * Copyright (C) 2020 Openbravo S.L.U.
+ * Copyright (C) 2020-2023 Openbravo S.L.U.
  * Licensed under the Openbravo Commercial License version 1.0
  * You may obtain a copy of the License at http://www.openbravo.com/legal/obcl.html
  * or in the legal folder of this module distribution.
@@ -72,3 +72,43 @@ OB.App.StateAPI.Ticket.addProduct.addActionPreparation(
   async (ticket, payload) => payload,
   150
 );
+
+// cancelling giftcard when undo action removes the giftcard consumed line
+OB.App.StateAPI.Ticket.addProduct.addActionPreparation(
+  async (ticket, payload) => {
+    return payload;
+  },
+  async (ticket, payload) => {
+    const giftCardTransactions = [];
+    payload.products.forEach(
+      tline =>
+        tline.product.giftCardTransaction &&
+        giftCardTransactions.push(tline.product.giftCardTransaction)
+    );
+    if (giftCardTransactions.length > 0) {
+      await Promise.all(
+        giftCardTransactions.map(async transaction => {
+          let data;
+          try {
+            data = await OB.App.Request.mobileServiceRequest(
+              'org.openbravo.retail.giftcards.CancelGiftCardTransaction',
+              {
+                transaction
+              }
+            );
+          } catch (error) {
+            const { message } = data.response.error;
+            throw new OB.App.Class.ActionCanceled({
+              errorConfirmation:
+                message.substr(0, message.indexOf(':')) ||
+                'GCNV_ErrorGenericMessage',
+              messageParams: [message.substr(message.indexOf(':'))]
+            });
+          }
+        })
+      );
+    }
+    return payload;
+  },
+  150
+);
diff --git a/web/org.openbravo.retail.giftcards/app/model/business-object/ticket/actions/DeleteLineActionPreparations.js b/web/org.openbravo.retail.giftcards/app/model/business-object/ticket/actions/DeleteLineActionPreparations.js
index 875ee8c..d9b4d2d 100644
--- a/web/org.openbravo.retail.giftcards/app/model/business-object/ticket/actions/DeleteLineActionPreparations.js
+++ b/web/org.openbravo.retail.giftcards/app/model/business-object/ticket/actions/DeleteLineActionPreparations.js
@@ -1,6 +1,6 @@
 /*
  ************************************************************************************
- * Copyright (C) 2020 Openbravo S.L.U.
+ * Copyright (C) 2020-2023 Openbravo S.L.U.
  * Licensed under the Openbravo Commercial License version 1.0
  * You may obtain a copy of the License at http://www.openbravo.com/legal/obcl.html
  * or in the legal folder of this module distribution.
@@ -33,26 +33,29 @@ OB.App.StateAPI.Ticket.deleteLine.addActionPreparation(
         });
       }
     }
-
-    new Set(giftCardTransactions).forEach(async transaction => {
-      let data;
-      try {
-        data = await OB.App.Request.mobileServiceRequest(
-          'org.openbravo.retail.giftcards.CancelGiftCardTransaction',
-          {
-            transaction
+    if (giftCardTransactions.length > 0) {
+      await Promise.all(
+        giftCardTransactions.map(async transaction => {
+          let data;
+          try {
+            data = await OB.App.Request.mobileServiceRequest(
+              'org.openbravo.retail.giftcards.CancelGiftCardTransaction',
+              {
+                transaction
+              }
+            );
+          } catch (error) {
+            const { message } = data.response.error;
+            throw new OB.App.Class.ActionCanceled({
+              errorConfirmation:
+                message.substr(0, message.indexOf(':')) ||
+                'GCNV_ErrorGenericMessage',
+              messageParams: [message.substr(message.indexOf(':'))]
+            });
           }
-        );
-      } catch (error) {
-        const { message } = data.response.error;
-        throw new OB.App.Class.ActionCanceled({
-          errorConfirmation:
-            message.substr(0, message.indexOf(':')) ||
-            'GCNV_ErrorGenericMessage',
-          messageParams: [message.substr(message.indexOf(':'))]
-        });
-      }
-    });
+        })
+      );
+    }
 
     return newPayload;
   },