CwpSiteTreeFileExtension

Complexity

Total Complexity

3

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	BackLinkHTMLList()	0	17	2
A	updateCMSFields()	0	12	1

<?php
class CwpSiteTreeFileExtension extends DataExtension {

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

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

'LastEdited' of type string is incompatible with the type FormField expected by parameter $item of FieldList::insertAfter().

		);
	}

	/**
	 * Generate an HTML list which provides links to where a file is used.
	 *
	 * @return String
	 */
	public function BackLinkHTMLList() {
		$html = '<em>' . _t('SiteTreeFileExtension.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 Convert::raw2xml($backLink->MenuTitle) of type string|array can be used in concatenation?

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

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

		return $html .= '</ul>';
	}

}