Merge pull request #17 from SSEsmaeeli/fix/style-fix

Refactor - Fix styling

Author: HPWebdeveloper

config/pay-pocket.php

@@ -14,6 +14,7 @@
  */
 return [
     'log_reference_length' => 12,
-    'log_reference_prefix' => null,
-    'log_reference_generator' => null,
+    'log_reference_prefix' => '',
+    'log_reference_generator_class' => Illuminate\Support\Str::class,
+    'log_reference_generator_method' => 'random',
 ];

src/Exceptions/InsufficientBalanceException.php

@@ -3,16 +3,11 @@
 namespace HPWebdeveloper\LaravelPayPocket\Exceptions;
 
 use Exception;
+use Throwable;
 
 class InsufficientBalanceException extends Exception
 {
-    /**
-     * Construct the 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(string $message = 'Insufficient balance to cover the order', int $code = 0, ?Throwable $previous = null)
     {
         parent::__construct($message, $code, $previous);
     }

src/Exceptions/InvalidDepositException.php

@@ -3,16 +3,11 @@
 namespace HPWebdeveloper\LaravelPayPocket\Exceptions;
 
 use Exception;
+use Throwable;
 
 class InvalidDepositException extends Exception
 {
-    /**
-     * Construct the exception.
-     *
-     * @param  string  $message
-     * @param  int  $code
-     */
-    public function __construct($message = 'Invalid deposit operation', $code = 0, ?\Throwable $previous = null)
+    public function __construct(string $message = 'Invalid deposit operation', int $code = 0, ?Throwable $previous = null)
     {
         parent::__construct($message, $code, $previous);
     }

src/Exceptions/InvalidValueException.php

@@ -3,16 +3,11 @@
 namespace HPWebdeveloper\LaravelPayPocket\Exceptions;
 
 use Exception;
+use Throwable;
 
 class InvalidValueException extends Exception
 {
-    /**
-     * Construct the exception.
-     *
-     * @param  string  $message
-     * @param  int  $code
-     */
-    public function __construct($message = 'Invalie value to deposit', $code = 0, ?\Throwable $previous = null)
+    public function __construct(string $message = 'Invalie value to deposit', int $code = 0, ?Throwable $previous = null)
     {
         parent::__construct($message, $code, $previous);
     }

src/Exceptions/InvalidWalletTypeException.php

@@ -3,10 +3,11 @@
 namespace HPWebdeveloper\LaravelPayPocket\Exceptions;
 
 use Exception;
+use Throwable;
 
 class InvalidWalletTypeException extends Exception
 {
-    public function __construct($message = 'Invalid wallet type', $code = 0, ?\Throwable $previous = null)
+    public function __construct(string $message = 'Invalid wallet type', int $code = 0, ?Throwable $previous = null)
     {
         parent::__construct($message, $code, $previous);
     }

src/Exceptions/WalletNotFoundException.php

@@ -3,10 +3,11 @@
 namespace HPWebdeveloper\LaravelPayPocket\Exceptions;
 
 use Exception;
+use Throwable;
 
 class WalletNotFoundException extends Exception
 {
-    public function __construct($message = 'Wallet not found', $code = 0, ?\Throwable $previous = null)
+    public function __construct(string $message = 'Wallet not found', int $code = 0, ?Throwable $previous = null)
     {
         parent::__construct($message, $code, $previous);
     }

src/Facades/LaravelPayPocket.php

@@ -2,19 +2,20 @@
 
 namespace HPWebdeveloper\LaravelPayPocket\Facades;
 
+use HPWebdeveloper\LaravelPayPocket\Interfaces\WalletOperations;
 use Illuminate\Support\Facades\Facade;
 
 /**
  * @see \HPWebdeveloper\LaravelPayPocket\Services\PocketServices
  *
- * @method static void pay(\Illuminate\Database\Eloquent\Model $user, int|float $orderValue, ?string $notes = null)
- * @method static bool deposit(\Illuminate\Database\Eloquent\Model $user, string $type, int|float $amount, ?string $notes = null)
- * @method static int|float checkBalance(\Illuminate\Database\Eloquent\Model $user)
- * @method static int|float walletBalanceByType(\Illuminate\Database\Eloquent\Model $user, string $type)
+ * @method static void pay(WalletOperations $user, int|float $orderValue, ?string $notes = null)
+ * @method static bool deposit(WalletOperations $user, string $type, int|float $amount, ?string $notes = null)
+ * @method static int|float checkBalance(WalletOperations $user)
+ * @method static int|float walletBalanceByType(WalletOperations $user, string $type)
  */
 class LaravelPayPocket extends Facade
 {
-    protected static function getFacadeAccessor()
+    protected static function getFacadeAccessor(): string
     {
         return \HPWebdeveloper\LaravelPayPocket\Services\PocketServices::class;
     }

src/Interfaces/WalletOperations.php

@@ -2,6 +2,8 @@
 
 namespace HPWebdeveloper\LaravelPayPocket\Interfaces;
 
+use HPWebdeveloper\LaravelPayPocket\Exceptions\InsufficientBalanceException;
+
 interface WalletOperations
 {
     /**
@@ -12,7 +14,7 @@ public function getWalletBalanceAttribute(): int|float;
     /**
      * Get the balance of a specific wallet type.
      */
-    public function getWalletBalanceByType(string $walletType): float|int;
+    public function getWalletBalanceByType(string $walletType): int|float;
 
     /**
      *  Check if User's wallet balance is more than given value
@@ -22,16 +24,17 @@ public function hasSufficientBalance(int|float $value): bool;
     /**
      * Pay the order value from the user's wallets.
      *
-     * @param  ?string  $notes
-     *
-     * @throws \HPWebdeveloper\LaravelPayPocket\Exceptions\InsufficientBalanceException
+     * @throws InsufficientBalanceException
      */
     public function pay(int|float $orderValue, ?string $notes = null): void;
 
     /**
      * Deposit an amount to the user's wallet of a specific type.
-     *
-     * @param  ?string  $notes
      */
     public function deposit(string $type, int|float $amount, ?string $notes = null): bool;
+
+    /**
+     * Get user's wallet balance.
+     */
+    public function getWalletBalance(): int|float;
 }

src/Models/WalletsLog.php

@@ -10,6 +10,14 @@
  * HPWebdeveloper\LaravelPayPocket\Models\WalletsLog
  *
  * @property string $status
+ * @property int|float $from
+ * @property int|float $to
+ * @property string $type
+ * @property string $ip
+ * @property int|float $value
+ * @property string $wallet_name
+ * @property string $notes
+ * @property string $reference
  */
 class WalletsLog extends Model
 {

src/Services/PocketServices.php

@@ -2,44 +2,41 @@
 
 namespace HPWebdeveloper\LaravelPayPocket\Services;
 
-use Illuminate\Database\Eloquent\Model;
+use HPWebdeveloper\LaravelPayPocket\Exceptions\InsufficientBalanceException;
+use HPWebdeveloper\LaravelPayPocket\Interfaces\WalletOperations;
 
 class PocketServices
 {
     /**
      * Deposit an amount to the user's wallet of a specific type.
-     *
-     * @param  ?string  $notes
      */
-    public function deposit(Model $user, string $type, int|float $amount, ?string $notes = null): bool
+    public function deposit(WalletOperations $user, string $type, int|float $amount, ?string $notes = null): bool
     {
         return $user->deposit($type, $amount, $notes);
     }
 
     /**
      * Pay the order value from the user's wallets.
      *
-     * @param  ?string  $notes
-     *
-     * @throws \HPWebdeveloper\LaravelPayPocket\Exceptions\InsufficientBalanceException
+     * @throws InsufficientBalanceException
      */
-    public function pay(Model $user, int|float $orderValue, ?string $notes = null): void
+    public function pay(WalletOperations $user, int|float $orderValue, ?string $notes = null): void
     {
         $user->pay($orderValue, $notes);
     }
 
     /**
      * Get the balance of the user.
      */
-    public function checkBalance(Model $user): int|float
+    public function checkBalance(WalletOperations $user): int|float
     {
-        return $user->walletBalance;
+        return $user->getWalletBalance();
     }
 
     /**
      * Get the balance of a specific wallet type.
      */
-    public function walletBalanceByType(Model $user, string $type): float|int
+    public function walletBalanceByType(WalletOperations $user, string $type): int|float
     {
         return $user->getWalletBalanceByType($type);
     }