Merge pull request #663 from extcode/662-deliver-a-different-flash-message-for-added-but-not-useabe-coupons_v10

[TASK] Deliver a different flash message for added but not useabe coupons

Author: extcode

Classes/Controller/Cart/CouponController.php

@@ -61,39 +61,7 @@ public function addAction(): ResponseInterface
 
                     $couponWasAdded = $this->cart->addCoupon($newCartCoupon);
 
-                    if ($couponWasAdded == 1) {
-                        $this->addFlashMessage(
-                            LocalizationUtility::translate(
-                                'tx_cart.ok.coupon.added',
-                                'Cart'
-                            ),
-                            '',
-                            ContextualFeedbackSeverity::OK,
-                            true
-                        );
-                    }
-                    if ($couponWasAdded == -1) {
-                        $this->addFlashMessage(
-                            LocalizationUtility::translate(
-                                'tx_cart.error.coupon.already_added',
-                                'Cart'
-                            ),
-                            '',
-                            ContextualFeedbackSeverity::WARNING,
-                            true
-                        );
-                    }
-                    if ($couponWasAdded == -2) {
-                        $this->addFlashMessage(
-                            LocalizationUtility::translate(
-                                'tx_cart.error.coupon.not_combinable',
-                                'Cart'
-                            ),
-                            '',
-                            ContextualFeedbackSeverity::WARNING,
-                            true
-                        );
-                    }
+                    $this->addFlashMessageForAddedCoupon($couponWasAdded, $coupon);
                 } else {
                     $this->addFlashMessage(
                         LocalizationUtility::translate(
@@ -158,4 +126,61 @@ public function removeAction(): ResponseInterface
 
         return $this->redirect('show', 'Cart\Cart');
     }
+
+    private function addFlashMessageForAddedCoupon(int $couponWasAdded, Coupon $coupon): void
+    {
+        if ($couponWasAdded === 1) {
+            $messageBody = LocalizationUtility::translate(
+                'tx_cart.ok.coupon.added',
+                'Cart'
+            );
+
+            foreach ($this->cart->getCoupons() as $cartCoupon) {
+                if ($cartCoupon->getCode() !== $coupon->getCode()) {
+                    continue;
+                }
+
+                if ($cartCoupon->isUseable()) {
+                    $this->addFlashMessage(
+                        $messageBody
+                    );
+                } else {
+                    $this->addFlashMessage(
+                        LocalizationUtility::translate(
+                            'tx_cart.error.coupon.added_but_not_usable',
+                            'Cart'
+                        ),
+                        '',
+                        ContextualFeedbackSeverity::WARNING,
+                    );
+                }
+            }
+
+            return;
+        }
+
+        if ($couponWasAdded === -1) {
+            $this->addFlashMessage(
+                LocalizationUtility::translate(
+                    'tx_cart.error.coupon.already_added',
+                    'Cart'
+                ),
+                '',
+                ContextualFeedbackSeverity::WARNING,
+            );
+
+            return;
+        }
+
+        if ($couponWasAdded === -2) {
+            $this->addFlashMessage(
+                LocalizationUtility::translate(
+                    'tx_cart.error.coupon.not_combinable',
+                    'Cart'
+                ),
+                '',
+                ContextualFeedbackSeverity::WARNING,
+            );
+        }
+    }
 }

Documentation/guides.xml

@@ -11,7 +11,7 @@
                interlink-shortcode="extcode/cart"
     />
     <project title="Cart"
-             release="10.2.4"
+             release="10.2.5"
              version="10.2"
              copyright="2018 - 2024"
     />

Resources/Private/Language/locallang.xlf

@@ -491,6 +491,9 @@
             <trans-unit id="tx_cart.ok.coupon.added">
                 <source>Coupon was added.</source>
             </trans-unit>
+            <trans-unit id="tx_cart.error.coupon.added_but_not_usable">
+                <source>Coupon was added but is not usable.</source>
+            </trans-unit>
             <trans-unit id="tx_cart.error.coupon.not_accepted">
                 <source>Coupon code was not accepted.</source>
             </trans-unit>

ext_emconf.php

@@ -4,7 +4,7 @@
     'title' => 'Cart',
     'description' => 'Shopping Cart(s) for TYPO3',
     'category' => 'plugin',
-    'version' => '10.2.4',
+    'version' => '10.2.5',
     'state' => 'stable',
     'author' => 'Daniel Gohlke',
     'author_email' => 'ext@extco.de',