WebHemi\Middleware\Action\Website\Directory\CategoryAction

Complexity

Total Complexity

6

Size/Duplication

Total Lines	73
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getTemplateName()	4	4	1
B	getTemplateData()	52	52	5

<?php
/**
 * WebHemi.
 *
 * PHP version 7.1
 *
 * @copyright 2012 - 2017 Gixx-web (http://www.gixx-web.com)
 * @license   https://opensource.org/licenses/MIT The MIT License (MIT)
 *
 * @link      http://www.gixx-web.com
 */
declare(strict_types = 1);

namespace WebHemi\Middleware\Action\Website\Directory;

use RuntimeException;
use WebHemi\Data\Entity;
use WebHemi\Middleware\Action\Website\IndexAction;

/**
 * Class CategoryAction.
 */
class CategoryAction extends IndexAction

This class seems to be duplicated in your project.

{
    /** @var string */
    protected $templateName = 'website-post-list';

    /**
     * Gets template map name or template file path.
     *
     * @return string
     */
    public function getTemplateName() : string
    {
        return $this->templateName;
    }

    /**
     * Gets template data.
     *
     * @return array
     */
    public function getTemplateData() : array
    {
        $blogPosts = [];
        $parameters = $this->getRoutingParameters();
        /** @var string $category */
        $category = $parameters['uri_parameter'] ?? null;

        if (!$category) {
            throw new RuntimeException('Forbidden', 403);
        }

        /** @var Entity\ApplicationEntity $applicationEntity */
        $applicationEntity = $this->getApplicationStorage()
            ->getApplicationByName($this->environmentManager->getSelectedApplication());

        $categoryEntity = $this->getFilesystemCategoryStorage()
            ->getFilesystemCategoryByApplicationAndName(
                $applicationEntity->getApplicationId(),
                $category
            );

        if (!$categoryEntity) {
            throw new RuntimeException('Not Found', 404);
        }

        /** @var Entity\Filesystem\FilesystemEntity[] $publications */
        $publications = $this->getFilesystemStorage()
            ->getPublishedDocuments(
                $applicationEntity->getApplicationId(),
                [
                    'fk_category = ?' => (int)$categoryEntity->getFilesystemCategoryId(),
                ]
            );

        if (!$publications) {

The expression $publications of type WebHemi\Data\Entity\Filesystem\FilesystemEntity[] 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.

            $this->templateName = 'website-post-list-empty';
        }

        /** @var Entity\Filesystem\FilesystemEntity $filesystemEntity */
        foreach ($publications as $filesystemEntity) {
            $blogPosts[] = $this->getBlobPostData($applicationEntity, $filesystemEntity);
        }

        return [
            'page' => [
                'title' => $categoryEntity->getTitle(),
                'type' => 'Categories',
            ],
            'activeMenu' => $category,
            'blogPosts' => $blogPosts,
        ];
    }
}