HDNET\Calendarize\Form\Element\CalendarizeInfoElement

Complexity

Total Complexity

9

Size/Duplication

Total Lines	64
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	render()	0	19	2
B	getEventList()	0	31	7

<?php
declare(strict_types = 1);
namespace HDNET\Calendarize\Form\Element;


use HDNET\Calendarize\Service\IndexerService;
use HDNET\Calendarize\Utility\DateTimeUtility;
use HDNET\Calendarize\Utility\TranslateUtility;
use TYPO3\CMS\Backend\Form\Element\AbstractFormElement;
use TYPO3\CMS\Backend\Utility\BackendUtility;
use TYPO3\CMS\Core\Utility\DebugUtility;
use TYPO3\CMS\Core\Utility\GeneralUtility;

class CalendarizeInfoElement extends AbstractFormElement

There is one abstract method setLogger in this class; you could implement it, or declare this class as abstract.

{

    public function render()
    {
        $result = $this->initializeResultArray();

        $parameters = $this->data['parameterArray']['fieldConf']['config']['parameters'];

        $previewLimit = 10;
        if (isset($parameters['items'])) {
            $previewLimit = (int)$parameters['items'];
        }

        $indexService = GeneralUtility::makeInstance(IndexerService::class);
        $count = $indexService->getIndexCount($this->data['tableName'], $this->data['vanillaUid']);
        $next = $indexService->getNextEvents($this->data['tableName'], $this->data['vanillaUid'], $previewLimit);
        $content = \sprintf(TranslateUtility::get('previewLabel'), $count, $previewLimit) . $this->getEventList($next);

        $result['html'] = $content;
        return $result;
    }


    /**
     * Get event list.
     *
     * @param $events
     *
     * @return string
     */
    protected function getEventList($events)
    {
        $items = [];
        foreach ($events as $event) {
            if (!($event['start_date'] instanceof \DateTimeInterface)) {
                $event['start_date'] = new \DateTime($event['start_date']);
            }
            $startDate = \strftime(DateTimeUtility::FORMAT_DATE_BACKEND, (int)$event['start_date']->getTimestamp());

            if (!($event['end_date'] instanceof \DateTimeInterface)) {
                $event['end_date'] = new \DateTime($event['end_date']);
            }
            $endDate = \strftime(DateTimeUtility::FORMAT_DATE_BACKEND, (int)$event['end_date']->getTimestamp());
            $entry = $startDate . ' - ' . $endDate;
            if (!$event['all_day']) {
                $start = BackendUtility::time($event['start_time'] % DateTimeUtility::SECONDS_DAY, false);
                if ((bool)$event['open_end_time']) {
                    $end = '"' . TranslateUtility::get('openEndTime') . '"';
                } else {
                    $end = BackendUtility::time($event['end_time'] % DateTimeUtility::SECONDS_DAY, false);
                }
                $entry .= ' (' . $start . ' - ' . $end . ')';
            }
            $items[] = $entry;
        }
        if (!$items) {

The expression $items 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.

            $items[] = TranslateUtility::get('noEvents');
        }

        return '<ul><li>' . \implode('</li><li>', $items) . '</li></ul>';
    }

}