Added a minimal description of methods and recommendations for development using the package to the code. Slightly stricter began to work internal methods. Added MaximalTaxable contract. Speed up events.

Author: rez1dent3

src/Interfaces/MaximalTaxable.php

@@ -0,0 +1,13 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Bavix\Wallet\Interfaces;
+
+interface MaximalTaxable extends Taxable
+{
+    /**
+     * @return float|int
+     */
+    public function getMaximalFee();
+}

src/Interfaces/Product.php

@@ -6,6 +6,11 @@
 
 interface Product extends Wallet
 {
+    /**
+     * The method is only needed for simple projects. For more complex projects, deprecate this method and redefine the
+     * "BasketServiceInterface" interface. Typically, in projects, this method always returns false, and the presence
+     * interface goes to the microservice and receives data on products.
+     */
     public function canBuy(Customer $customer, int $quantity = 1, bool $force = false): bool;
 
     /**

src/Internal/Service/DatabaseService.php

@@ -32,10 +32,8 @@ public function __construct(
      * @throws RecordsNotFoundException
      * @throws TransactionFailedException
      * @throws ExceptionInterface
-     *
-     * @return mixed
      */
-    public function transaction(callable $callback)
+    public function transaction(callable $callback): mixed
     {
         $level = $this->connection->transactionLevel();
         if ($level > 0 && !$this->init) {

src/Internal/Service/DatabaseServiceInterface.php

@@ -16,8 +16,6 @@ interface DatabaseServiceInterface
      * @throws TransactionStartException
      * @throws TransactionFailedException
      * @throws ExceptionInterface
-     *
-     * @return mixed
      */
-    public function transaction(callable $callback);
+    public function transaction(callable $callback): mixed;
 }

src/Internal/Service/DispatcherService.php

@@ -10,7 +10,7 @@
 final class DispatcherService implements DispatcherServiceInterface
 {
     /**
-     * @var string[]
+     * @var array<string, bool>
      */
     private array $events = [];
 
@@ -21,20 +21,20 @@ public function __construct(
 
     public function dispatch(EventInterface $event): void
     {
-        $this->events[] = $event::class;
+        $this->events[$event::class] = true;
         $this->dispatcher->push($event::class, [$event]);
     }
 
     public function flush(): void
     {
-        foreach ($this->events as $event) {
+        foreach ($this->events as $event => $value) {
             $this->dispatcher->flush($event);
         }
     }
 
     public function forgot(): void
     {
-        foreach ($this->events as $event) {
+        foreach ($this->events as $event => $value) {
             $this->dispatcher->forget($event);
         }
     }

src/Internal/Service/LockService.php

@@ -26,7 +26,7 @@ public function __construct(CacheManager $cacheManager, ConfigRepository $config
     /**
      * @throws LockProviderNotFoundException
      */
-    public function block(string $key, callable $callback)
+    public function block(string $key, callable $callback): mixed
     {
         return $this->getLockProvider()
             ->lock($key)

src/Internal/Service/LockServiceInterface.php

@@ -10,8 +10,6 @@ interface LockServiceInterface
 {
     /**
      * @throws LockProviderNotFoundException
-     *
-     * @return mixed
      */
-    public function block(string $key, callable $callback);
+    public function block(string $key, callable $callback): mixed;
 }

src/Services/AssistantService.php

@@ -16,17 +16,17 @@ public function __construct(
     }
 
     /**
-     * @param non-empty-array<TransactionDtoInterface|TransferDtoInterface> $objects
+     * @param non-empty-array<array-key, TransactionDtoInterface|TransferDtoInterface> $objects
      *
-     * @return non-empty-array<int|string, string>
+     * @return non-empty-array<array-key, string>
      */
     public function getUuids(array $objects): array
     {
         return array_map(static fn ($object): string => $object->getUuid(), $objects);
     }
 
     /**
-     * @param non-empty-array<TransactionDtoInterface> $transactions
+     * @param non-empty-array<array-key, TransactionDtoInterface> $transactions
      *
      * @return array<int, string>
      */

src/Services/AssistantServiceInterface.php

@@ -10,9 +10,9 @@
 interface AssistantServiceInterface
 {
     /**
-     * @param non-empty-array<TransactionDtoInterface|TransferDtoInterface> $objects
+     * @param non-empty-array<array-key, TransactionDtoInterface|TransferDtoInterface> $objects
      *
-     * @return non-empty-array<int|string, string>
+     * @return non-empty-array<array-key, string>
      */
     public function getUuids(array $objects): array;
 

src/Services/AtmService.php

@@ -13,7 +13,7 @@
 use Bavix\Wallet\Models\Transaction;
 use Bavix\Wallet\Models\Transfer;
 
-/** @psalm-internal */
+/** @internal */
 final class AtmService implements AtmServiceInterface
 {
     public function __construct(
@@ -26,7 +26,7 @@ public function __construct(
     }
 
     /**
-     * @param non-empty-array<int|string, TransactionDtoInterface> $objects
+     * @param non-empty-array<array-key, TransactionDtoInterface> $objects
      *
      * @return non-empty-array<string, Transaction>
      */
@@ -52,7 +52,7 @@ public function makeTransactions(array $objects): array
     }
 
     /**
-     * @param non-empty-array<int|string, TransferDtoInterface> $objects
+     * @param non-empty-array<array-key, TransferDtoInterface> $objects
      *
      * @return non-empty-array<string, Transfer>
      */