Refactor ChargeRefundListener to remove single-use variables and improve readability

2025-04-23 16:00:33 +02:00
parent 5bd474a5b3
commit e70088d9a1
1 changed files with 4 additions and 6 deletions
@@ -35,12 +35,10 @@ class ChargeRefundListener
            return;
        }

-        $oldStatus = $args->getOldValue(self::FIELD_STATUS);
-        $newStatus = $args->getNewValue(self::FIELD_STATUS);
-
-        $charged = ChargeStatus::Charged->value;
-        $toRefund = ChargeStatus::ToRefund->value;
-        if ($oldStatus === $charged && $newStatus === $toRefund) {
+        if (
+            $args->getOldValue(self::FIELD_STATUS) === ChargeStatus::Charged
+            && $args->getNewValue(self::FIELD_STATUS) === ChargeStatus::ToRefund
+        ) {
            $refundStrategy = $charge->getCheckout()->getRefundStrategy();
            match ($refundStrategy) {
                RefundStrategy::ToGateway => $this->processGatewayRefund($charge),