EWW\Dpf\ViewHelpers\Link\PreviewViewHelper::render()

Complexity

Conditions	7
Paths	9

Size

Total Lines	52
Code Lines	20

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function render(array $arguments, $pageUid, $apiPid, $class)
    {

        if ($arguments['document']) {

            // it's already a document object?
            if ($arguments['document'] instanceof \EWW\Dpf\Domain\Model\Document) {

                $document = $arguments['document'];

            } else if (MathUtility::canBeInterpretedAsInteger($arguments['document'])) {

                $document = $this->documentRepository->findByUid($arguments['document']);

            }

            // we found a valid document
            if ($document) {

The variable $document does not seem to be defined for all execution paths leading up to this point.

                $row['uid'] = $document->getUid();

$row was never initialized. Although not strictly required by PHP, it is generally a good practice to add $row = array(); before regardless.

                $row['title'] = $document->getTitle();

                $row['action'] = 'preview';

            } else {

                // ok, nothing to render. So return empty content.
                return '';

            }

        } else if ($arguments['documentObjectIdentifier']) {

            $row['action'] = 'mets';

            $row['uid'] = $arguments['documentObjectIdentifier'];

            // pass configured API secret key parameter to enable dissemination of inactive documents
            $settings = $configurationManager->getConfiguration(ConfigurationManagerInterface::CONFIGURATION_TYPE_SETTINGS);

The variable $configurationManager seems to be never defined.

The type EWW\Dpf\ViewHelpers\Link\ConfigurationManagerInterface was not found. Maybe you did not declare it correctly or list all dependencies?

            if (array_key_exists('plugin.tx_dpf.settings.deliverInactiveSecretKey', $settings)) {
                $row['deliverInactive'] = $this->settings['plugin.tx_dpf.settings.deliverInactiveSecretKey'];
            }

        }

        $insideText = $this->renderChildren();

        $content = $this->getViewIcon($row, $pageUid, $apiPid, $insideText, $class);

The variable $row does not seem to be defined for all execution paths leading up to this point.

        return $content;