draft config

rez1dent3 · rez1dent3 · commit 7772d6a9a203 · 2022-09-03T21:09:56.000+03:00
diff --git a/config/config.php b/config/config.php
@@ -19,6 +19,7 @@
 use Bavix\Wallet\Internal\Repository\TransferRepository;
 use Bavix\Wallet\Internal\Repository\WalletRepository;
 use Bavix\Wallet\Internal\Service\ClockService;
+use Bavix\Wallet\Internal\Service\ConfigService;
 use Bavix\Wallet\Internal\Service\DatabaseService;
 use Bavix\Wallet\Internal\Service\DispatcherService;
 use Bavix\Wallet\Internal\Service\JsonService;
@@ -80,6 +81,7 @@
      */
     'internal' => [
         'clock' => ClockService::class,
+        'config' => ConfigService::class,
         'database' => DatabaseService::class,
         'dispatcher' => DispatcherService::class,
         'json' => JsonService::class,
diff --git a/deptrac.yaml b/deptrac.yaml
@@ -201,7 +201,7 @@ parameters:
       - RepositoryInterface
       - TransformInterface
       - InternalException
-      - ServiceInterface # json service only
+      - ServiceInterface # json&config service only
       - QueryInterface
       - DtoInterface
       - UIException
diff --git a/src/Internal/Repository/WalletRepository.php b/src/Internal/Repository/WalletRepository.php
@@ -6,12 +6,14 @@
 
 use Bavix\Wallet\Internal\Exceptions\ExceptionInterface;
 use Bavix\Wallet\Internal\Exceptions\ModelNotFoundException;
+use Bavix\Wallet\Internal\Service\ConfigServiceInterface;
 use Bavix\Wallet\Models\Wallet;
 use Illuminate\Database\Eloquent\ModelNotFoundException as EloquentModelNotFoundException;
 
 final class WalletRepository implements WalletRepositoryInterface
 {
     public function __construct(
+        private ConfigServiceInterface $configService,
         private Wallet $wallet
     ) {
     }
@@ -91,7 +93,7 @@ public function getBySlug(string $holderType, int|string $holderId, string $slug
     public function findDefaultAll(string $holderType, array $holderIds): array
     {
         return $this->wallet->newQuery()
-            ->where('slug', config('wallet.wallet.default.slug', 'default'))
+            ->where('slug', $this->configService->getString('wallet.wallet.default.slug', 'default'))
             ->where('holder_type', $holderType)
             ->whereIn('holder_id', $holderIds)
             ->get()
diff --git a/src/Internal/Service/ConfigService.php b/src/Internal/Service/ConfigService.php
@@ -0,0 +1,35 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Bavix\Wallet\Internal\Service;
+
+final class ConfigService implements ConfigServiceInterface
+{
+    public function getClass(string $name, string $default): string
+    {
+        $value = $this->getString($name, $default);
+        assert(class_exists($value));
+
+        return $value;
+    }
+
+    public function getString(string $name, string $default): string
+    {
+        $value = config($name, $default);
+        assert(is_string($value));
+
+        return $value;
+    }
+
+    /**
+     * @return array<mixed>
+     */
+    public function getArray(string $name): array
+    {
+        $value = config($name, []);
+        assert(is_array($value));
+
+        return $value;
+    }
+}
diff --git a/src/Internal/Service/ConfigServiceInterface.php b/src/Internal/Service/ConfigServiceInterface.php
@@ -0,0 +1,21 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Bavix\Wallet\Internal\Service;
+
+interface ConfigServiceInterface
+{
+    /**
+     * @param class-string $default
+     * @return class-string
+     */
+    public function getClass(string $name, string $default): string;
+
+    public function getString(string $name, string $default): string;
+
+    /**
+     * @return array<mixed>
+     */
+    public function getArray(string $name): array;
+}
diff --git a/src/Models/Transaction.php b/src/Models/Transaction.php
@@ -5,10 +5,10 @@
 namespace Bavix\Wallet\Models;
 
 use Bavix\Wallet\Interfaces\Wallet;
+use Bavix\Wallet\Internal\Service\ConfigServiceInterface;
 use Bavix\Wallet\Internal\Service\MathServiceInterface;
 use Bavix\Wallet\Models\Wallet as WalletModel;
 use Bavix\Wallet\Services\CastServiceInterface;
-use function config;
 use Illuminate\Database\Eloquent\Model;
 use Illuminate\Database\Eloquent\Relations\BelongsTo;
 use Illuminate\Database\Eloquent\Relations\MorphTo;
@@ -61,7 +61,7 @@ class Transaction extends Model
     public function getTable(): string
     {
         if ((string) $this->table === '') {
-            $this->table = config('wallet.transaction.table', 'transactions');
+            $this->table = app(ConfigServiceInterface::class)->getString('wallet.transaction.table', 'transactions');
         }
 
         return parent::getTable();
@@ -74,7 +74,9 @@ public function payable(): MorphTo
 
     public function wallet(): BelongsTo
     {
-        return $this->belongsTo(config('wallet.wallet.model', WalletModel::class));
+        return $this->belongsTo(
+            app(ConfigServiceInterface::class)->getClass('wallet.wallet.model', WalletModel::class)
+        );
     }
 
     public function getAmountIntAttribute(): int
diff --git a/src/Models/Transfer.php b/src/Models/Transfer.php
@@ -4,7 +4,7 @@
 
 namespace Bavix\Wallet\Models;
 
-use function config;
+use Bavix\Wallet\Internal\Service\ConfigServiceInterface;
 use Illuminate\Database\Eloquent\Model;
 use Illuminate\Database\Eloquent\Relations\BelongsTo;
 use Illuminate\Database\Eloquent\Relations\MorphTo;
@@ -67,7 +67,7 @@ class Transfer extends Model
     public function getTable(): string
     {
         if ((string) $this->table === '') {
-            $this->table = config('wallet.transfer.table', 'transfers');
+            $this->table = app(ConfigServiceInterface::class)->getString('wallet.transfer.table', 'transfers');
         }
 
         return parent::getTable();
diff --git a/src/Models/Wallet.php b/src/Models/Wallet.php
@@ -13,6 +13,7 @@
 use Bavix\Wallet\Internal\Exceptions\ExceptionInterface;
 use Bavix\Wallet\Internal\Exceptions\LockProviderNotFoundException;
 use Bavix\Wallet\Internal\Exceptions\TransactionFailedException;
+use Bavix\Wallet\Internal\Service\ConfigServiceInterface;
 use Bavix\Wallet\Internal\Service\MathServiceInterface;
 use Bavix\Wallet\Internal\Service\UuidFactoryServiceInterface;
 use Bavix\Wallet\Services\AtomicServiceInterface;
@@ -21,7 +22,6 @@
 use Bavix\Wallet\Traits\CanExchange;
 use Bavix\Wallet\Traits\CanPayFloat;
 use Bavix\Wallet\Traits\HasGift;
-use function config;
 use Illuminate\Database\Eloquent\Model;
 use Illuminate\Database\Eloquent\Relations\MorphTo;
 use Illuminate\Database\RecordsNotFoundException;
@@ -83,7 +83,7 @@ class Wallet extends Model implements Customer, WalletFloat, Confirmable, Exchan
     public function getTable(): string
     {
         if ((string) $this->table === '') {
-            $this->table = config('wallet.wallet.table', 'wallets');
+            $this->table = app(ConfigServiceInterface::class)->getString('wallet.wallet.table', 'wallets');
         }
 
         return parent::getTable();
diff --git a/src/Services/AtomicServiceInterface.php b/src/Services/AtomicServiceInterface.php
@@ -14,7 +14,6 @@ interface AtomicServiceInterface
 {
     /**
      * @template T
-     * @param Wallet $object
      * @param callable(): T $callback
      * @return T
      *
diff --git a/src/Services/WalletService.php b/src/Services/WalletService.php
@@ -7,6 +7,7 @@
 use Bavix\Wallet\Internal\Assembler\WalletCreatedEventAssemblerInterface;
 use Bavix\Wallet\Internal\Exceptions\ModelNotFoundException;
 use Bavix\Wallet\Internal\Repository\WalletRepositoryInterface;
+use Bavix\Wallet\Internal\Service\ConfigServiceInterface;
 use Bavix\Wallet\Internal\Service\DispatcherServiceInterface;
 use Bavix\Wallet\Internal\Service\UuidFactoryServiceInterface;
 use Bavix\Wallet\Models\Wallet;
@@ -21,14 +22,15 @@ public function __construct(
         private WalletCreatedEventAssemblerInterface $walletCreatedEventAssembler,
         private UuidFactoryServiceInterface $uuidFactoryService,
         private DispatcherServiceInterface $dispatcherService,
-        private WalletRepositoryInterface $walletRepository
+        private WalletRepositoryInterface $walletRepository,
+        private ConfigServiceInterface $configService
     ) {
     }
 
     public function create(Model $model, array $data): Wallet
     {
         $wallet = $this->walletRepository->create(array_merge(
-            config('wallet.wallet.creating', []),
+            $this->configService->getArray('wallet.wallet.creating'),
             [
                 'uuid' => $this->uuidFactoryService->uuid4(),
             ],
diff --git a/src/Traits/HasWallet.php b/src/Traits/HasWallet.php
@@ -13,6 +13,7 @@
 use Bavix\Wallet\Internal\Exceptions\ExceptionInterface;
 use Bavix\Wallet\Internal\Exceptions\LockProviderNotFoundException;
 use Bavix\Wallet\Internal\Exceptions\TransactionFailedException;
+use Bavix\Wallet\Internal\Service\ConfigServiceInterface;
 use Bavix\Wallet\Internal\Service\MathServiceInterface;
 use Bavix\Wallet\Models\Transaction;
 use Bavix\Wallet\Models\Transfer;
@@ -24,7 +25,6 @@
 use Bavix\Wallet\Services\RegulatorServiceInterface;
 use Bavix\Wallet\Services\TransactionServiceInterface;
 use Bavix\Wallet\Services\TransferServiceInterface;
-use function config;
 use Illuminate\Database\Eloquent\Relations\HasMany;
 use Illuminate\Database\Eloquent\Relations\MorphMany;
 use Illuminate\Database\RecordsNotFoundException;
@@ -82,7 +82,10 @@ public function walletTransactions(): HasMany
     {
         return app(CastServiceInterface::class)
             ->getWallet($this, false)
-            ->hasMany(config('wallet.transaction.model', Transaction::class), 'wallet_id')
+            ->hasMany(
+                app(ConfigServiceInterface::class)->getClass('wallet.transaction.model', Transaction::class),
+                'wallet_id'
+            )
         ;
     }
 
@@ -93,7 +96,10 @@ public function transactions(): MorphMany
     {
         return app(CastServiceInterface::class)
             ->getHolder($this)
-            ->morphMany(config('wallet.transaction.model', Transaction::class), 'payable')
+            ->morphMany(
+                app(ConfigServiceInterface::class)->getClass('wallet.transaction.model', Transaction::class),
+                'payable'
+            )
         ;
     }
 
@@ -217,7 +223,7 @@ public function transfers(): MorphMany
         /** @var Wallet $this */
         return app(CastServiceInterface::class)
             ->getWallet($this, false)
-            ->morphMany(config('wallet.transfer.model', Transfer::class), 'from')
+            ->morphMany(app(ConfigServiceInterface::class)->getClass('wallet.transfer.model', Transfer::class), 'from')
         ;
     }
 }
diff --git a/src/Traits/HasWallets.php b/src/Traits/HasWallets.php
@@ -6,6 +6,7 @@
 
 use function array_key_exists;
 use Bavix\Wallet\Internal\Exceptions\ModelNotFoundException;
+use Bavix\Wallet\Internal\Service\ConfigServiceInterface;
 use Bavix\Wallet\Models\Wallet as WalletModel;
 use Bavix\Wallet\Services\WalletServiceInterface;
 use function config;
@@ -82,7 +83,10 @@ public function getWalletOrFail(string $slug): WalletModel
      */
     public function wallets(): MorphMany
     {
-        return $this->morphMany(config('wallet.wallet.model', WalletModel::class), 'holder');
+        return $this->morphMany(
+            app(ConfigServiceInterface::class)->getClass('wallet.wallet.model', WalletModel::class),
+            'holder'
+        );
     }
 
     public function createWallet(array $data): WalletModel
diff --git a/src/Traits/MorphOneWallet.php b/src/Traits/MorphOneWallet.php
@@ -4,6 +4,7 @@
 
 namespace Bavix\Wallet\Traits;
 
+use Bavix\Wallet\Internal\Service\ConfigServiceInterface;
 use Bavix\Wallet\Models\Wallet as WalletModel;
 use Bavix\Wallet\Services\CastServiceInterface;
 use Illuminate\Database\Eloquent\Relations\MorphOne;
@@ -21,24 +22,21 @@ trait MorphOneWallet
      */
     public function wallet(): MorphOne
     {
+        $configService = app(ConfigServiceInterface::class);
         $castService = app(CastServiceInterface::class);
-        /** @var class-string $related */
-        $related = config('wallet.wallet.model', WalletModel::class);
-        assert(is_string($related));
-
-        $slug = config('wallet.wallet.default.slug', 'default');
-        assert(is_string($slug));
+        $related = $configService->getClass('wallet.wallet.model', WalletModel::class);
+        $slug = $configService->getString('wallet.wallet.default.slug', 'default');
 
         return $castService
             ->getHolder($this)
             ->morphOne($related, 'holder')
             ->where('slug', $slug)
-            ->withDefault(static function (WalletModel $wallet, object $holder) use ($castService) {
+            ->withDefault(static function (WalletModel $wallet, object $holder) use ($configService, $castService) {
                 $model = $castService->getModel($holder);
-                $wallet->forceFill(array_merge((array) config('wallet.wallet.creating', []), [
-                    'name' => config('wallet.wallet.default.name', 'Default Wallet'),
-                    'slug' => config('wallet.wallet.default.slug', 'default'),
-                    'meta' => config('wallet.wallet.default.meta', []),
+                $wallet->forceFill(array_merge($configService->getArray('wallet.wallet.creating'), [
+                    'name' => $configService->getString('wallet.wallet.default.name', 'Default Wallet'),
+                    'slug' => $configService->getString('wallet.wallet.default.slug', 'default'),
+                    'meta' => $configService->getArray('wallet.wallet.default.meta'),
                     'balance' => 0,
                 ]));
 
diff --git a/src/WalletServiceProvider.php b/src/WalletServiceProvider.php
@@ -41,6 +41,8 @@
 use Bavix\Wallet\Internal\Repository\WalletRepositoryInterface;
 use Bavix\Wallet\Internal\Service\ClockService;
 use Bavix\Wallet\Internal\Service\ClockServiceInterface;
+use Bavix\Wallet\Internal\Service\ConfigService;
+use Bavix\Wallet\Internal\Service\ConfigServiceInterface;
 use Bavix\Wallet\Internal\Service\DatabaseService;
 use Bavix\Wallet\Internal\Service\DatabaseServiceInterface;
 use Bavix\Wallet\Internal\Service\DispatcherService;
@@ -203,6 +205,7 @@ private function internal(array $configure): void
         $this->app->bind(StorageServiceInterface::class, $configure['storage'] ?? StorageService::class);
 
         $this->app->singleton(ClockServiceInterface::class, $configure['clock'] ?? ClockService::class);
+        $this->app->singleton(ConfigServiceInterface::class, $configure['config'] ?? ConfigService::class);
         $this->app->singleton(DatabaseServiceInterface::class, $configure['database'] ?? DatabaseService::class);
         $this->app->singleton(DispatcherServiceInterface::class, $configure['dispatcher'] ?? DispatcherService::class);
         $this->app->singleton(JsonServiceInterface::class, $configure['json'] ?? JsonService::class);

Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@`
`4`	`4`
`5`	`5`	`namespace Bavix\Wallet\Models;`
`6`	`6`
`7`		`-use function config;`
	`7`	`+use Bavix\Wallet\Internal\Service\ConfigServiceInterface;`
`8`	`8`	`use Illuminate\Database\Eloquent\Model;`
`9`	`9`	`use Illuminate\Database\Eloquent\Relations\BelongsTo;`
`10`	`10`	`use Illuminate\Database\Eloquent\Relations\MorphTo;`
`@@ -67,7 +67,7 @@ class Transfer extends Model`
`67`	`67`	`public function getTable(): string`
`68`	`68`	`{`
`69`	`69`	`if ((string) $this->table === '') {`
`70`		`- $this->table = config('wallet.transfer.table', 'transfers');`
	`70`	`+ $this->table = app(ConfigServiceInterface::class)->getString('wallet.transfer.table', 'transfers');`
`71`	`71`	`}`
`72`	`72`
`73`	`73`	`return parent::getTable();`
Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,6 @@ interface AtomicServiceInterface`
`14`	`14`	`{`
`15`	`15`	`/**`
`16`	`16`	`* @template T`
`17`		`- * @param Wallet $object`
`18`	`17`	`* @param callable(): T $callback`
`19`	`18`	`* @return T`
`20`	`19`	`*`