Bavix\Wallet\Services\WalletService

Complexity

Total Complexity

10

Size/Duplication

Total Lines	77
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	fee()	0	7	2
A	getWallet()	0	10	2
A	checkAmount()	0	4	2
A	refresh()	0	9	1
A	getBalance()	0	10	3

<?php

namespace Bavix\Wallet\Services;

use Bavix\Wallet\Exceptions\AmountInvalid;
use Bavix\Wallet\Interfaces\Taxing;
use Bavix\Wallet\Interfaces\Wallet;
use Bavix\Wallet\Models\Wallet as WalletModel;
use Bavix\Wallet\Traits\HasWallet;
use function app;

class WalletService
{

    /**
     * Consider the fee that the system will receive.
     *
     * @param Wallet $wallet
     * @param int $amount
     * @return int
     */
    public function fee(Wallet $wallet, int $amount): int
    {
        if ($wallet instanceof Taxing) {
            return (int)($amount * $wallet->getFeePercent() / 100);
        }

        return 0;
    }

    /**
     * The amount of checks for errors
     *
     * @param int $amount
     * @throws
     */
    public function checkAmount(int $amount): void
    {
        if ($amount < 0) {
            throw new AmountInvalid(trans('wallet::errors.price_positive'));

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

        }
    }

    /**
     * @param Wallet $object
     * @return WalletModel
     */
    public function getWallet(Wallet $object): WalletModel
    {
        if ($object instanceof WalletModel) {
            return $object;
        }

        /**
         * @var HasWallet $object
         */
        return $object->wallet;
    }

    /**
     * @param Wallet $object
     * @return int
     */
    public function getBalance(Wallet $object): int
    {
        $wallet = $this->getWallet($object);
        $wallet->exists or $wallet->save();
        $proxy = app(ProxyService::class);
        if (!$proxy->has($wallet->getKey())) {

It seems like $wallet->getKey() can also be of type boolean and null; however, parameter $key of Bavix\Wallet\Services\ProxyService::has() does only seem to accept string, maybe add an additional type check?

            $proxy->set($wallet->getKey(), (int)($this->attributes['balance'] ?? 0));

The property attributes does not exist on Bavix\Wallet\Services\WalletService. Did you maybe forget to declare it?

It seems like $wallet->getKey() can also be of type boolean and null; however, parameter $key of Bavix\Wallet\Services\ProxyService::set() does only seem to accept string, maybe add an additional type check?

        }

        return $proxy[$wallet->getKey()];
    }

    /**
     * @param WalletModel $wallet
     * @return bool
     */
    public function refresh(WalletModel $wallet): bool
    {
        $balance = $wallet->getAvailableBalance();
        $wallet->balance = $balance;

        $proxy = app(ProxyService::class);
        $proxy->set($wallet->getKey(), $balance);

It seems like $wallet->getKey() can also be of type boolean and null; however, parameter $key of Bavix\Wallet\Services\ProxyService::set() does only seem to accept string, maybe add an additional type check?

        return $wallet->save();
    }

}