CwpSiteTreeExtension::updateSettingsFields() - Code Metrics - Inspection of "Merge branch '1.7' into 1.8" - silverstripe/cwp - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — 1.8 ( 8b2854...6b63e4 )

by Robbie

created 2018-01-09 03:39 UTC

CwpSiteTreeExtension::updateSettingsFields() A

↳ Parent: CwpSiteTreeExtension

Complexity

Conditions	1
Paths	1

Size

Total Lines	12
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	12
rs	9.4285
cc	1
eloc	8
nc	1
nop	1

<?php

class CwpSiteTreeExtension extends DataExtension
{
	private static $db = array(

		'ShowPageUtilities' => 'Boolean(1)'
	);

	private static $defaults = array(

		'ShowPageUtilities' => true
	);

	/**
	 * Modify the settings for a SiteTree
	 *
	 * {@inheritDoc}
	 *
	 * @param FieldList $fields
	 */
	public function updateSettingsFields(FieldList $fields)
	{
		$helpText = _t(
			'SiteTree.SHOW_PAGE_UTILITIES_HELP',
			'You can disable page utilities (print, share, etc) for this page'
		);

		$fields->addFieldsToTab(
			'Root.Settings',
			array(
				LiteralField::create('PageUtilitiesHelp', $helpText),
				CheckboxField::create('ShowPageUtilities', $this->owner->fieldLabel('ShowPageUtilities'))
			)
		);
	}

	public function updateFieldLabels(&$labels)
	{
		$labels['ShowPageUtilities'] = _t('SiteTree.SHOW_PAGE_UTILITIES', 'Show page utilities?');
	}
}


1			<?php
2
3			class CwpSiteTreeExtension extends DataExtension
4			{
5			private static $db = array(
			0 ignored issues – show introduced 2017-12-14 22:02 UTC by Report Bug Copy Issue Report The private property `$db` is not used, and could be removed. Loading history...
6			'ShowPageUtilities' => 'Boolean(1)'
7			);
8
9			private static $defaults = array(
			0 ignored issues – show introduced 2017-12-14 22:02 UTC by Report Bug Copy Issue Report The private property `$defaults` is not used, and could be removed. Loading history...
10			'ShowPageUtilities' => true
11			);
12
13			/**
14			* Modify the settings for a SiteTree
15			*
16			* {@inheritDoc}
17			*
18			* @param FieldList $fields
19			*/
20			public function updateSettingsFields(FieldList $fields)
21			{
22			$helpText = _t(
23			'SiteTree.SHOW_PAGE_UTILITIES_HELP',
24			'You can disable page utilities (print, share, etc) for this page'
25			);
26
27			$fields->addFieldsToTab(
28			'Root.Settings',
29			array(
30			LiteralField::create('PageUtilitiesHelp', $helpText),
31			CheckboxField::create('ShowPageUtilities', $this->owner->fieldLabel('ShowPageUtilities'))
32			)
33			);
34			}
35
36			public function updateFieldLabels(&$labels)
37			{
38			$labels['ShowPageUtilities'] = _t('SiteTree.SHOW_PAGE_UTILITIES', 'Show page utilities?');
39			}
40			}
41

silverstripe / cwp

Push — 1.8 ( 8b2854...6b63e4 )

CwpSiteTreeExtension::updateSettingsFields() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like