Merge branch 'main' of github.com:SSEsmaeeli/laravel-pay-pocket

HPWebdeveloper · Dec 12, 2023 · 2b091e8 · 2b091e8
2 parents 013f24d + bf92ab1
commit 2b091e8
Show file tree

Hide file tree

Showing 7 changed files with 8 additions and 10 deletions.
diff --git a/.github/workflows/run-tests.yml b/.github/workflows/run-tests.yml
@@ -13,7 +13,7 @@ jobs:
       fail-fast: true
       matrix:
         os: [ubuntu-latest, windows-latest]
-        php: [8.2, 8.1]
+        php: [8.3, 8.2, 8.1]
         laravel: [10.*]
         stability: [prefer-lowest, prefer-stable]
         include:

diff --git a/src/Exceptions/InsufficientBalanceException.php b/src/Exceptions/InsufficientBalanceException.php
@@ -12,7 +12,7 @@ class InsufficientBalanceException extends Exception
      * @param  string  $message
      * @param  int  $code
      */
-    public function __construct($message = 'Insufficient balance to cover the order', $code = 0, \Throwable $previous = null)
+    public function __construct($message = 'Insufficient balance to cover the order', $code = 0, ?\Throwable $previous = null)
     {
         parent::__construct($message, $code, $previous);
     }

diff --git a/src/Exceptions/InvalidDepositException.php b/src/Exceptions/InvalidDepositException.php
@@ -12,7 +12,7 @@ class InvalidDepositException extends Exception
      * @param  string  $message
      * @param  int  $code
      */
-    public function __construct($message = 'Invalid deposit operation', $code = 0, \Throwable $previous = null)
+    public function __construct($message = 'Invalid deposit operation', $code = 0, ?\Throwable $previous = null)
     {
         parent::__construct($message, $code, $previous);
     }

diff --git a/src/Exceptions/InvalidValueException.php b/src/Exceptions/InvalidValueException.php
@@ -12,7 +12,7 @@ class InvalidValueException extends Exception
      * @param  string  $message
      * @param  int  $code
      */
-    public function __construct($message = 'Invalie value to deposit', $code = 0, \Throwable $previous = null)
+    public function __construct($message = 'Invalie value to deposit', $code = 0, ?\Throwable $previous = null)
     {
         parent::__construct($message, $code, $previous);
     }

diff --git a/src/Exceptions/InvalidWalletTypeException.php b/src/Exceptions/InvalidWalletTypeException.php
@@ -6,7 +6,7 @@
 
 class InvalidWalletTypeException extends Exception
 {
-    public function __construct($message = 'Invalid wallet type', $code = 0, \Throwable $previous = null)
+    public function __construct($message = 'Invalid wallet type', $code = 0, ?\Throwable $previous = null)
     {
         parent::__construct($message, $code, $previous);
     }

diff --git a/src/Exceptions/WalletNotFoundException.php b/src/Exceptions/WalletNotFoundException.php
@@ -6,7 +6,7 @@
 
 class WalletNotFoundException extends Exception
 {
-    public function __construct($message = 'Wallet not found', $code = 0, \Throwable $previous = null)
+    public function __construct($message = 'Wallet not found', $code = 0, ?\Throwable $previous = null)
     {
         parent::__construct($message, $code, $previous);
     }

diff --git a/src/Traits/HandlesPayment.php b/src/Traits/HandlesPayment.php
@@ -10,8 +10,6 @@ trait HandlesPayment
     /**
      * Pay the order value from the user's wallets.
      *
-     * @param int|float $orderValue
-     * @return void
      *
      * @throws InsufficientBalanceException
      */
@@ -24,9 +22,9 @@ public function pay(int|float $orderValue): void
         DB::transaction(function () use ($orderValue) {
             $remainingOrderValue = $orderValue;
 
-            $wallets = $this->wallets()->whereIn('type', $this->walletsInOrder())->get();
+            $walletsInOrder = $this->wallets()->whereIn('type', $this->walletsInOrder())->get();
 
-            foreach ($wallets as $wallet) {
+            foreach ($walletsInOrder as $wallet) {
                 if (! $wallet || ! $wallet->hasBalance()) {
                     continue;
                 }