[11.2] Improvements to confirm #1651
rez1dent3Annotations
10 warnings
|
units (8.3, testing, array, redis):
src/Internal/Exceptions/TransactionRollbackException.php#L14
Escaped Mutant for Mutator "MethodCallRemoval":
@@ @@
{
public function __construct(private readonly mixed $result)
{
- parent::__construct();
+
}
public function getResult(): mixed
{
|
|
units (8.3, testing, array, redis):
src/Internal/Observers/TransferObserver.php#L46
Escaped Mutant for Mutator "ArrayItemRemoval":
@@ @@
// If the transfer does not belong to the wallet, a WalletOwnerInvalid exception will be thrown.
// If the transfer was not found, a RecordNotFoundException will be thrown.
// Block both the wallet of the user who is sending the money and the wallet of the user who is receiving the money.
- return $this->atomicService->blocks([$model->from, $model->to], function () use ($model) {
+ return $this->atomicService->blocks([$model->to], function () use ($model) {
// Reset confirmation of the transfer for the sender's wallet.
// Returns true if the confirmation was reset, false otherwise.
return $model->from->safeResetConfirm($model->withdraw) && $model->to->safeResetConfirm($model->deposit);
|
|
units (8.3, testing, array, redis):
src/Internal/Repository/WalletRepository.php#L88
Escaped Mutant for Mutator "ArrayItemRemoval":
@@ @@
*/
public function getById(int $id): Wallet
{
- return $this->getBy(['id' => $id]);
+ return $this->getBy([]);
}
/**
* @throws ModelNotFoundException
|
|
units (8.3, testing, array, redis):
src/Internal/Repository/WalletRepository.php#L98
Escaped Mutant for Mutator "ArrayItemRemoval":
@@ @@
*/
public function getByUuid(string $uuid): Wallet
{
- return $this->getBy(['uuid' => $uuid]);
+ return $this->getBy([]);
}
/**
* @throws ModelNotFoundException
|
|
units (8.3, testing, array, redis):
src/Internal/Service/LockService.php#L81
Escaped Mutant for Mutator "Continue_":
@@ @@
$lockProvider = $this->getLockProvider();
foreach ($keys as $key) {
if (!$this->isBlocked($key)) {
- continue;
+ break;
}
$lockProvider->lock(self::LOCK_KEY . $key, $this->seconds)->forceRelease();
$this->lockedKeys->delete(self::INNER_KEYS . $key);
|
|
units (8.3, testing, array, redis):
src/Objects/Cart.php#L77
Escaped Mutant for Mutator "CloneRemoval":
@@ @@
*/
public function withExtra(array $extra): self
{
- $self = clone $this;
+ $self = $this;
$self->extra = $extra;
return $self;
}
|
|
units (8.3, testing, array, redis):
src/Objects/Cart.php#L107
Escaped Mutant for Mutator "CloneRemoval":
@@ @@
*/
public function withItems(iterable $products): self
{
- $self = clone $this;
+ $self = $this;
foreach ($products as $product) {
$self = $self->withItem($product);
}
|
|
units (8.3, testing, array, redis):
src/Services/AssistantService.php#L37
Escaped Mutant for Mutator "ArrayOneItem":
@@ @@
$wallet = $this->castService->getWallet($object);
$wallets[$wallet->getKey()] = $wallet;
}
- return $wallets;
+ return count($wallets) > 1 ? array_slice($wallets, 0, 1, true) : $wallets;
}
/**
* @param non-empty-array<array-key, TransactionDtoInterface|TransferDtoInterface> $objects
|
|
units (8.3, testing, array, redis):
src/Services/BookkeeperService.php#L51
Escaped Mutant for Mutator "ArrayItem":
@@ @@
*/
public function increase(Wallet $wallet, float|int|string $value): string
{
- return current($this->multiIncrease([$wallet->uuid => $wallet], [$wallet->uuid => $value]));
+ return current($this->multiIncrease([$wallet->uuid > $wallet], [$wallet->uuid => $value]));
}
public function multiAmount(array $wallets): array
{
|
|
units (8.3, testing, array, redis):
src/Services/EagerLoaderService.php#L31
Escaped Mutant for Mutator "Continue_":
@@ @@
foreach ($basketDto->items() as $index => $item) {
// If the wallet is installed, then there is no need for lazy loading
if ($item->getReceiving() instanceof \Bavix\Wallet\Interfaces\Wallet) {
- continue;
+ break;
}
$model = $this->castService->getModel($item->getProduct());
if (!$model->relationLoaded('wallet')) {
|