App\Ajax\CallableSessionClass

Complexity

Total Complexity

2

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getSession()	0	4	1
A	showBalanceAmounts()	0	3	1

<?php

namespace App\Ajax;

use Siak\Tontine\Model\Session as SessionModel;
use Siak\Tontine\Service\Meeting\SessionService;

/**
 * @databag meeting
 * @before getSession
 */
class CallableSessionClass extends CallableClass
{
    /**
     * @var SessionModel|null
     */
    protected ?SessionModel $session = null;

    /**
     * @di
     * @var SessionService
     */
    protected SessionService $sessionService;

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

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

    }

    /**
     * @return void
     */
    protected function showBalanceAmounts()
    {
        $this->response->call('showBalanceAmounts');
    }
}