Zikula\BlocksModule\AbstractBlockHandler

Complexity

Total Complexity

15

Size/Duplication

Total Lines	131
Duplicated Lines	0 %

Importance

Changes

0

12 Methods

Rating	Name	Duplication	Size	Complexity
A	boot()	0	6	2
A	addFlash()	0	11	3
A	getFormClassName()	0	3	1
A	getFormOptions()	0	3	1
A	getFormTemplate()	0	3	1
A	__construct()	0	16	1
A	getPropertyDefaults()	0	3	1
A	getBundle()	0	3	1
A	hasPermission()	0	7	1
A	display()	0	3	1
A	renderView()	0	3	1
A	getType()	0	8	1

<?php

declare(strict_types=1);

/*
 * This file is part of the Zikula package.
 *
 * Copyright Zikula Foundation - https://ziku.la/
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Zikula\BlocksModule;

use LogicException;
use Symfony\Component\HttpFoundation\RequestStack;
use Symfony\Contracts\Translation\TranslatorInterface;
use Twig\Environment;
use Zikula\Bundle\CoreBundle\AbstractBundle;
use Zikula\Bundle\CoreBundle\Translation\TranslatorTrait;
use Zikula\ExtensionsModule\Api\ApiInterface\VariableApiInterface;
use Zikula\ExtensionsModule\ExtensionVariablesTrait;
use Zikula\PermissionsModule\Api\ApiInterface\PermissionApiInterface;

abstract class AbstractBlockHandler implements BlockHandlerInterface
{
    use TranslatorTrait;
    use ExtensionVariablesTrait;

    /**
     * @var AbstractBundle
     */
    protected $bundle;

    /**
     * @var RequestStack
     */
    protected $requestStack;

    /**
     * @var PermissionApiInterface
     */
    protected $permissionApi;

    /**
     * @var Environment
     */
    protected $twig;

    public function __construct(
        AbstractBundle $bundle,
        RequestStack $requestStack,
        TranslatorInterface $translator,
        VariableApiInterface $variableApi,
        PermissionApiInterface $permissionApi,
        Environment $twig
    ) {
        $this->bundle = $bundle;
        $this->extensionName = $bundle->getName(); // for ExtensionVariablesTrait
        $this->requestStack = $requestStack;
        $this->setTranslator($translator); // for TranslatorTrait
        $this->variableApi = $variableApi; // for ExtensionVariablesTrait

$variableApi is of type Zikula\ExtensionsModule\Api\ApiInterface\VariableApiInterface, but the property $variableApi was declared to be of type Zikula\ExtensionsModule\Api\VariableApi. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

        $this->permissionApi = $permissionApi;
        $this->twig = $twig;
        $this->boot($bundle);
    }

    /**
     * Boot the handler.
     */
    protected function boot(AbstractBundle $bundle): void
    {
        // load optional bootstrap
        $bootstrap = $bundle->getPath() . '/bootstrap.php';
        if (file_exists($bootstrap)) {
            include_once $bootstrap;
        }
    }

    public function getFormClassName(): string
    {
        return '';
    }

    public function getFormOptions(): array
    {
        return [];
    }

    public function getFormTemplate(): string
    {
        return '@ZikulaBlocksModule/Block/default_modify.html.twig';
    }

    public function getPropertyDefaults(): array
    {
        return [];
    }

    public function display(array $properties): string
    {
        return nl2br(implode("\n", $properties));
    }

    public function getType(): string
    {
        // default to the ClassName without the `Block` suffix
        // note: This string is intentionally left untranslated.
        $fqCn = get_class($this);
        $pos = mb_strrpos($fqCn, '\\');

        return mb_substr($fqCn, $pos + 1, -5);
    }

    /**
     * Adds a flash message to the current session for type.
     *
     * @throws LogicException
     */
    protected function addFlash(string $type, string $message): void
    {
        $request = $this->requestStack->getCurrentRequest();
        if (null === $request) {
            return;
        }
        if (!$request->hasSession()) {
            throw new LogicException('You can not use the addFlash method if sessions are disabled.');
        }

        $request->getSession()->getFlashBag()->add($type, $message);
    }

    /**
     * Returns a rendered view.
     */
    public function renderView(string $view, array $parameters = []): string
    {
        return $this->twig->render($view, $parameters);
    }

    /**
     * Convenience shortcut to check if user has requested permissions.
     */
    protected function hasPermission(
        string $component = null,
        string $instance = null,
        int $level = null,
        int $user = null
    ): bool {
        return $this->permissionApi->hasPermission($component, $instance, $level, $user);

It seems like $level can also be of type null; however, parameter $level of Zikula\PermissionsModule\Api\ApiInterface\PermissionApiInterface::hasPermission() does only seem to accept integer, maybe add an additional type check?

    }

    public function getBundle(): AbstractBundle
    {
        return $this->bundle;
    }
}