CWP\CWP\Extensions\CwpSiteTreeFileExtension

Complexity

Total Complexity

3

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	updateCMSFields()	0	15	1
A	BackLinkHTMLList()	0	25	2

<?php

namespace CWP\CWP\Extensions;

use SilverStripe\Core\Convert;
use SilverStripe\Forms\FieldList;
use SilverStripe\Forms\ReadonlyField;
use SilverStripe\ORM\DataExtension;
use SilverStripe\View\Requirements;

class CwpSiteTreeFileExtension extends DataExtension
{

    public function updateCMSFields(FieldList $fields)
    {
        Requirements::css('cwp/cwp:css/fieldDescriptionToggle.css');
        Requirements::javascript('cwp/cwp:javascript/fieldDescriptionToggle.js');

        $fields->insertAfter(
            'LastEdited',
            ReadonlyField::create(
                'BackLinkCount',
                _t('SilverStripe\\CMS\\Model\\SiteTreeFileExtension.BACKLINKCOUNT', 'Used on:'),
                $this->owner->BackLinkTracking()->Count() . ' '
                    . _t('SilverStripe\\CMS\\Model\\SiteTreeFileExtension.PAGES', 'page(s)')
            )
            ->addExtraClass('cms-description-toggle')
            ->setDescription($this->BackLinkHTMLList())
        );
    }

    /**
     * Generate an HTML list which provides links to where a file is used.
     *
     * @return string
     */
    public function BackLinkHTMLList()
    {
        $html = '<em>' . _t(
            __CLASS__ . '.BACKLINK_LIST_DESCRIPTION',
            'This list shows all pages where the file has been added through a WYSIWYG editor.'
        ) . '</em>';
        $html .= '<ul>';

        foreach ($this->owner->BackLinkTracking() as $backLink) {
            $listItem = '<li>';

            // Add the page link
            $listItem .= '<a href="' . $backLink->Link() . '" target="_blank">'
                . Convert::raw2xml($backLink->MenuTitle) . '</a> &ndash; ';

Are you sure SilverStripe\Core\Convert::raw2xml($backLink->MenuTitle) of type array|string can be used in concatenation?

            // Add the CMS link
            $listItem .= '<a href="' . $backLink->CMSEditLink() . '">'
                . _t(__CLASS__ . '.EDIT', 'Edit') . '</a>';

            $html .= $listItem . '</li>';
        }

        $html .= '</ul>';

        return $html;
    }
}