AbterPhp\Admin\Http\Controllers\Admin\Execute\ApiClient

Complexity

Total Complexity

3

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getPostData()	0	9	2
A	__construct()	0	15	1

<?php

declare(strict_types=1);

namespace AbterPhp\Admin\Http\Controllers\Admin\Execute;

use AbterPhp\Admin\Service\Execute\ApiClient as RepoService;
use AbterPhp\Framework\Constant\Session;
use AbterPhp\Framework\Http\Controllers\Admin\ExecuteAbstract;
use AbterPhp\Framework\I18n\ITranslator;
use AbterPhp\Framework\Session\FlashService;
use Opulence\Routing\Urls\UrlGenerator;
use Opulence\Sessions\ISession;
use Psr\Log\LoggerInterface;

class ApiClient extends ExecuteAbstract
{
    const ENTITY_SINGULAR = 'apiClient';
    const ENTITY_PLURAL   = 'apiClients';

    const ENTITY_TITLE_SINGULAR = 'admin:apiClient';
    const ENTITY_TITLE_PLURAL   = 'admin:apiClients';

    /**
     * ApiClient constructor.
     *
     * @param FlashService    $flashService
     * @param ITranslator     $translator
     * @param UrlGenerator    $urlGenerator
     * @param LoggerInterface $logger
     * @param RepoService     $repoService
     * @param ISession        $session
     */
    public function __construct(
        FlashService $flashService,
        ITranslator $translator,
        UrlGenerator $urlGenerator,
        LoggerInterface $logger,
        RepoService $repoService,
        ISession $session
    ) {
        parent::__construct(
            $flashService,
            $translator,
            $urlGenerator,
            $logger,

$logger of type Psr\Log\LoggerInterface is incompatible with the type AbterPhp\Framework\Http\Service\Execute\IRepoService expected by parameter $repoService of AbterPhp\Framework\Http\Controllers\Admin\ExecuteAbstract::__construct().

            $repoService,

$repoService of type AbterPhp\Admin\Service\Execute\ApiClient is incompatible with the type Opulence\Sessions\ISession expected by parameter $session of AbterPhp\Framework\Http\Controllers\Admin\ExecuteAbstract::__construct().

            $session

$session of type Opulence\Sessions\ISession is incompatible with the type Psr\Log\LoggerInterface expected by parameter $logger of AbterPhp\Framework\Http\Controllers\Admin\ExecuteAbstract::__construct().

        );
    }

    /**
     * @return array
     */
    protected function getPostData(): array
    {
        $postData = $this->request->getPost()->getAll();

        if ($postData) {

The expression $postData of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            $postData['user_id'] = $this->session->get(Session::USER_ID);
        }

        return $postData;
    }
}