Bavix\Wallet\Traits\CanConfirm

Complexity

Total Complexity

16

Size/Duplication

Total Lines	117
Duplicated Lines	0 %

Test Coverage

Coverage

91.11%

Importance

Changes	6
Bugs	2	Features	2

5 Methods

Rating	Name	Duplication	Size	Complexity
A	safeConfirm()	0	6	2
A	forceConfirm()	0	22	4
A	confirm()	0	18	3
A	safeResetConfirm()	0	6	2
A	resetConfirm()	0	23	5

<?php

namespace Bavix\Wallet\Traits;

use Bavix\Wallet\Exceptions\ConfirmedInvalid;
use Bavix\Wallet\Exceptions\WalletOwnerInvalid;
use Bavix\Wallet\Interfaces\Mathable;
use Bavix\Wallet\Models\Transaction;
use Bavix\Wallet\Services\CommonService;
use Bavix\Wallet\Services\DbService;
use Bavix\Wallet\Services\LockService;
use Bavix\Wallet\Services\WalletService;

trait CanConfirm
{


    /**
     * @param Transaction $transaction
     * @return bool
     */
    public function confirm(Transaction $transaction): bool
    {
        return app(LockService::class)->lock($this, __FUNCTION__, function () use ($transaction) {
            $self = $this;
            return app(DbService::class)->transaction(static function () use ($self, $transaction) {
                $wallet = app(WalletService::class)->getWallet($self);

$self of type Bavix\Wallet\Traits\CanConfirm is incompatible with the type Bavix\Wallet\Interfaces\Wallet expected by parameter $object of Bavix\Wallet\Services\WalletService::getWallet().

                if (!$wallet->refreshBalance()) {
                    return false;
                }

                if ($transaction->type === Transaction::TYPE_WITHDRAW) {
                    app(CommonService::class)->verifyWithdraw(
                        $wallet,
                        app(Mathable::class)->abs($transaction->amount)

app(Bavix\Wallet\Interfaces\Mathable::class)->abs($transaction->amount) of type string is incompatible with the type integer expected by parameter $amount of Bavix\Wallet\Services\CommonService::verifyWithdraw().

                    );
                }

                return $self->forceConfirm($transaction);
            });
        });
    }

    /**
     * @param Transaction $transaction
     * @return bool
     */
    public function safeConfirm(Transaction $transaction): bool
    {
        try {
            return $this->confirm($transaction);
        } catch (\Throwable $throwable) {
            return false;
        }
    }

    /**
     * Removal of confirmation (forced), use at your own peril and risk.
     *
     * @param Transaction $transaction
     * @return bool
     */
    public function resetConfirm(Transaction $transaction): bool
    {
        return app(LockService::class)->lock($this, __FUNCTION__, function () use ($transaction) {
            $self = $this;
            return app(DbService::class)->transaction(static function () use ($self, $transaction) {
                $wallet = app(WalletService::class)->getWallet($self);

$self of type Bavix\Wallet\Traits\CanConfirm is incompatible with the type Bavix\Wallet\Interfaces\Wallet expected by parameter $object of Bavix\Wallet\Services\WalletService::getWallet().

                if (!$wallet->refreshBalance()) {
                    return false;
                }

                if (!$transaction->confirmed) {
                    throw new ConfirmedInvalid(trans('wallet::errors.unconfirmed_invalid'));

It seems like trans('wallet::errors.unconfirmed_invalid') can also be of type array and array; however, parameter $message of Bavix\Wallet\Exceptions\ConfirmedInvalid::__construct() does only seem to accept string, maybe add an additional type check?

                }

                if ($wallet->getKey() !== $transaction->wallet_id) {

The property wallet_id does not exist on Bavix\Wallet\Models\Transaction. Did you mean wallet?

                    throw new WalletOwnerInvalid(trans('wallet::errors.owner_invalid'));

It seems like trans('wallet::errors.owner_invalid') can also be of type array and array; however, parameter $message of Bavix\Wallet\Exceptions\WalletOwnerInvalid::__construct() does only seem to accept string, maybe add an additional type check?

                }

                return $transaction->update(['confirmed' => false]) &&

                    // update balance
                    app(CommonService::class)
                        ->addBalance($wallet, -$transaction->amount);
            });
        });
    }

    /**
     * @param Transaction $transaction
     * @return bool
     */
    public function safeResetConfirm(Transaction $transaction): bool
    {
        try {
            return $this->resetConfirm($transaction);
        } catch (\Throwable $throwable) {
            return false;
        }
    }

    /**
     * @param Transaction $transaction
     * @return bool
     * @throws ConfirmedInvalid
     * @throws WalletOwnerInvalid
     */
    public function forceConfirm(Transaction $transaction): bool
    {
        return app(LockService::class)->lock($this, __FUNCTION__, function () use ($transaction) {
            $self = $this;
            return app(DbService::class)->transaction(static function () use ($self, $transaction) {

                $wallet = app(WalletService::class)
                    ->getWallet($self);

$self of type Bavix\Wallet\Traits\CanConfirm is incompatible with the type Bavix\Wallet\Interfaces\Wallet expected by parameter $object of Bavix\Wallet\Services\WalletService::getWallet().

                if ($transaction->confirmed) {
                    throw new ConfirmedInvalid(trans('wallet::errors.confirmed_invalid'));

It seems like trans('wallet::errors.confirmed_invalid') can also be of type array and array; however, parameter $message of Bavix\Wallet\Exceptions\ConfirmedInvalid::__construct() does only seem to accept string, maybe add an additional type check?

                }

                if ($wallet->getKey() !== $transaction->wallet_id) {

The property wallet_id does not exist on Bavix\Wallet\Models\Transaction. Did you mean wallet?

                    throw new WalletOwnerInvalid(trans('wallet::errors.owner_invalid'));

It seems like trans('wallet::errors.owner_invalid') can also be of type array and array; however, parameter $message of Bavix\Wallet\Exceptions\WalletOwnerInvalid::__construct() does only seem to accept string, maybe add an additional type check?

                }

                return $transaction->update(['confirmed' => true]) &&

                    // update balance
                    app(CommonService::class)
                        ->addBalance($wallet, $transaction->amount);

            });
        });
    }

}