HistoryController::initializeView() - Code Metrics - Inspection of "Merge pull request #18 from aertmann/ElementareTei..." - aertmann/history - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 571d43...4ce82d )

by Aske

created 2019-05-06 09:19 UTC

HistoryController::initializeView() A

↳ Parent: HistoryController

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	2
nc	1
nop	1
dl	0
loc	4
rs	10
c	0
b	0
f	0

<?php
namespace AE\History\Controller;

use AE\History\Domain\Repository\NodeEventRepository;
use Neos\Flow\Annotations as Flow;
use Neos\Flow\Mvc\View\ViewInterface;
use Neos\Flow\Security\Account;
use Neos\Flow\Security\AccountRepository;
use Neos\Flow\Security\Context;
use Neos\Fusion\View\FusionView;
use Neos\Neos\Controller\CreateContentContextTrait;
use Neos\Neos\Controller\Module\AbstractModuleController;
use Neos\Neos\Domain\Repository\DomainRepository;
use Neos\Neos\Domain\Repository\SiteRepository;
use Neos\Neos\EventLog\Domain\Model\EventsOnDate;
use Neos\Neos\EventLog\Domain\Model\NodeEvent;

/**
 * Controller for the history module of Neos, displaying the timeline of changes.
 */
class HistoryController extends AbstractModuleController
{
    use CreateContentContextTrait;

    /**
     * @Flow\Inject
     * @var AccountRepository
     */
    protected $accountRepository;

    /**
     * @var string
     */
    protected $defaultViewObjectName = FusionView::class;

    /**
     * @Flow\Inject
     * @var DomainRepository
     */
    protected $domainRepository;

    /**
     * @Flow\Inject
     * @var NodeEventRepository
     */
    protected $nodeEventRepository;

    /**
     * @Flow\Inject
     * @var Context
     */
    protected $securityContext;

    /**
     * @Flow\Inject
     * @var SiteRepository
     */
    protected $siteRepository;

    /**
     * Show event overview.
     *
     * @param int $offset
     * @param int $limit
     * @param string|null $siteIdentifier
     * @param string|null $nodeIdentifier
     * @param string|null $accountIdentifier
     *
     * @return void
     */
    public function indexAction(
        int $offset = 0,
        int $limit = 25,
        string $siteIdentifier = null,
        string $nodeIdentifier = null,
        string $accountIdentifier = null
    ) {
        if ($accountIdentifier === '') {
            $accountIdentifier = null;
        }
        if ($nodeIdentifier === '') {
            $nodeIdentifier = null;
        }
        if ($siteIdentifier === '') {
            $siteIdentifier = null;
        }

        $numberOfSites = 0;
        // In case a user can only access a single site, but more sites exists
        $this->securityContext->withoutAuthorizationChecks(function () use (&$numberOfSites) {
            $numberOfSites = $this->siteRepository->countAll();
        });
        $sites = $this->siteRepository->findOnline();
        if ($numberOfSites > 1 && $siteIdentifier === null) {
            $domain = $this->domainRepository->findOneByActiveRequest();
            if ($domain !== null) {
                $siteIdentifier = $this->persistenceManager->getIdentifierByObject($domain->getSite());
            }
        }

        /** @var Account[] $accounts */
        $accounts = $this->accountRepository->findByAuthenticationProviderName('Neos.Neos:Backend')->toArray();

        /** @var NodeEvent[] $events */
        $events = $this->nodeEventRepository
            ->findRelevantEventsByWorkspace(
                $offset,
                $limit + 1,
                'live',
                $siteIdentifier,
                $nodeIdentifier,
                $accountIdentifier
            )
            ->toArray()
        ;

        $nextPage = null;
        if (count($events) > $limit) {
            $events = array_slice($events, 0, $limit);

            $nextPage = $this->controllerContext
                ->getUriBuilder()
                ->setCreateAbsoluteUri(true)
                ->uriFor(
                    'Index',
                    ['offset' => $offset + $limit, 'siteIdentifier' => $siteIdentifier],
                    'History',
                    'Neos.Neos'
                )
            ;
        }

        /** @var EventsOnDate[] $eventsByDate */
        $eventsByDate = [];
        foreach ($events as $event) {
            if ($event->getChildEvents()->count() === 0) {
                continue;
            }
            $timestamp = $event->getTimestamp();
            $day = $timestamp->format('Y-m-d');
            if (!isset($eventsByDate[$day])) {
                $eventsByDate[$day] = new EventsOnDate($timestamp);
            }

            $eventsOnThisDay = $eventsByDate[$day];
            $eventsOnThisDay->add($event);
        }

        $firstEvent = current($events);
        if ($firstEvent === false) {
            $node = $this->createContentContext('live')->getNodeByIdentifier($nodeIdentifier);
            if ($node !== null) {
                $firstEvent = [
                    'data' => [
                        'documentNodeLabel' => $node->getLabel(),
                        'documentNodeType' => $node->getNodeType()->getName(),
                    ],
                    'node' => $node,
                    'nodeIdentifier' => $nodeIdentifier,
                ];
            }
        }

        $this->view->assignMultiple([
            'accountIdentifier' => $accountIdentifier,
            'accounts' => $accounts,
            'eventsByDate' => $eventsByDate,
            'firstEvent' => $firstEvent,
            'nextPage' => $nextPage,
            'nodeIdentifier' => $nodeIdentifier,
            'siteIdentifier' => $siteIdentifier,
            'sites' => $sites,
        ]);
    }

    /**
     * Simply sets the Fusion path pattern on the view.
     *
     * @param ViewInterface $view
     *
     * @return void
     */
    protected function initializeView(ViewInterface $view)
    {
        parent::initializeView($view);
        $view->setFusionPathPattern('resource://AE.History/Private/Fusion');
               setFusionPathPattern('resource://AE.History/Private/Fusion');
    }
}


aertmann / history

Push — master ( 571d43...4ce82d )

HistoryController::initializeView() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like