App\Ajax\Web\Meeting\Charge\FixedFee

Complexity

Total Complexity

5

Size/Duplication

Total Lines	81
Duplicated Lines	0 %

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	page()	0	24	1
A	home()	0	8	1
A	getSession()	0	4	1
A	show()	0	5	1
A	__construct()	0	3	1

<?php

namespace App\Ajax\Web\Meeting\Charge;

use App\Ajax\CallableClass;
use Siak\Tontine\Service\LocaleService;
use Siak\Tontine\Service\Meeting\Charge\FixedFeeService;
use Siak\Tontine\Model\Session as SessionModel;

use function Jaxon\jq;

/**
 * @databag meeting
 * @before getSession
 */
class FixedFee extends CallableClass
{
    /**
     * @di
     * @var LocaleService
     */
    protected LocaleService $localeService;

    /**
     * @var FixedFeeService
     */
    protected FixedFeeService $feeService;

    /**
     * @var SessionModel|null
     */
    protected ?SessionModel $session;

    /**
     * The constructor
     *
     * @param FixedFeeService $feeService
     */
    public function __construct(FixedFeeService $feeService)
    {
        $this->feeService = $feeService;
    }

    /**
     * @return void
     */
    protected function getSession()
    {
        $sessionId = $this->bag('meeting')->get('session.id');
        $this->session = $this->feeService->getSession($sessionId);

It seems like $sessionId can also be of type null; however, parameter $sessionId of Siak\Tontine\Service\Meeting\Charge\FixedFeeService::getSession() does only seem to accept integer, maybe add an additional type check?

    }

    /**
     * @exclude
     */
    public function show(SessionModel $session)
    {
        $this->session = $session;

        return $this->home();
    }

    public function home()
    {
        $html = $this->view()->render('tontine.pages.meeting.charge.fixed.home')
            ->with('session', $this->session);
        $this->response->html('meeting-fees-fixed', $html);

It seems like $html can also be of type null; however, parameter $sData of Jaxon\Response\Response::html() does only seem to accept string, maybe add an additional type check?

        $this->jq('#btn-fees-fixed-refresh')->click($this->rq()->home());

        return $this->page(1);
    }

    public function page(int $pageNumber)
    {
        $chargeCount = $this->feeService->getFeeCount();
        [$pageNumber, $perPage] = $this->pageNumber($pageNumber, $chargeCount,
            'meeting', 'fee.fixed.page');
        $charges = $this->feeService->getFees($pageNumber);
        $pagination = $this->rq()->page()->paginate($pageNumber, $perPage, $chargeCount);
        // Bill and settlement counts and amounts
        $bills = $this->feeService->getBills($this->session);

It seems like $this->session can also be of type null; however, parameter $session of Siak\Tontine\Service\Meeting\Charge\FixedFeeService::getBills() does only seem to accept Siak\Tontine\Model\Session, maybe add an additional type check?

        $settlements = $this->feeService->getSettlements($this->session);

It seems like $this->session can also be of type null; however, parameter $session of Siak\Tontine\Service\Meeting\Charge\FixedFeeService::getSettlements() does only seem to accept Siak\Tontine\Model\Session, maybe add an additional type check?

        $html = $this->view()->render('tontine.pages.meeting.charge.fixed.page')
            ->with('session', $this->session)
            ->with('charges', $charges)
            ->with('bills', $bills)
            ->with('settlements', $settlements)
            ->with('pagination', $pagination);
        $this->response->html('meeting-fees-fixed-page', $html);

It seems like $html can also be of type null; however, parameter $sData of Jaxon\Response\Response::html() does only seem to accept string, maybe add an additional type check?

        $chargeId = jq()->parent()->attr('data-charge-id')->toInt();
        $this->jq('.btn-fee-fixed-settlements')
            ->click($this->cl(Fixed\Settlement::class)->rq()->home($chargeId));

        return $this->response;
    }
}