Bavix\Wallet\Traits\CanConfirm

Complexity

Total Complexity

9

Size/Duplication

Total Lines	67
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	forceConfirm()	0	14	3
A	safeConfirm()	0	6	2
A	confirm()	0	23	4

<?php

namespace Bavix\Wallet\Traits;

use Bavix\Wallet\Exceptions\ConfirmedInvalid;
use Bavix\Wallet\Exceptions\WalletOwnerInvalid;
use Bavix\Wallet\Models\Transaction;
use Bavix\Wallet\Services\CommonService;
use Bavix\Wallet\Services\WalletService;
use Illuminate\Support\Facades\DB;

trait CanConfirm
{


    /**
     * @param Transaction $transaction
     * @return bool
     */
    public function confirm(Transaction $transaction): bool
    {
        return DB::transaction(function () use ($transaction) {
            $wallet = app(WalletService::class)
                ->getWallet($this);

$this 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,
                    \abs($transaction->amount)

It seems like abs($transaction->amount) can also be of type double; however, parameter $amount of Bavix\Wallet\Services\CommonService::verifyWithdraw() does only seem to accept integer, maybe add an additional type check?

                );
            }

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

It seems like $wallet can also be of type Bavix\Wallet\Traits\HasWallet; however, parameter $wallet of Bavix\Wallet\Services\CommonService::addBalance() does only seem to accept Bavix\Wallet\Interfaces\Wallet, maybe add an additional type check?

                
                // confirm
                $this->forceConfirm($transaction);
        });
    }

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

    /**
     * @param Transaction $transaction
     * @return bool
     * @throws ConfirmedInvalid
     * @throws WalletOwnerInvalid
     */
    public function forceConfirm(Transaction $transaction): bool
    {
        $wallet = app(WalletService::class)
            ->getWallet($this);

$this 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(); // todo
        }

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

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

            throw new WalletOwnerInvalid(); // todo
        }

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

}