Moo_EditableField - Code Metrics - Inspection of "Bug: replace GridFieldEditableManyManyExtraColumns..." - satrun77/silverstripe-editablefield - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 2bc3ae...eaf16e )

by Mohamed

created 2016-10-28 09:33 UTC

Moo_EditableField A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	237
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	11

Test Coverage

Coverage

18.29%

Importance

Changes

Metric	Value
wmc	26
lcom	1
cbo	11
dl	0
loc	237
ccs	15
cts	82
cp	0.1829
rs	10
c	0
b	0
f	0

14 Methods

Rating	Name	Size	Complexity
A	getSettings()	4	2
A	setSettings()	4	1
A	setSetting()	7	1
A	getSetting()	11	4
A	getIcon()	4	1
A	getIconTag()	4	1
B	getCMSFields()	41	4
A	getCMSValidator()	6	1
A	getTitle()	4	1
A	getSettingName()	4	1
A	onBeforeWrite()	20	3
A	getFormField()	8	2
A	initFormField()	4	1
A	getErrorMessage()	10	3

<?php

/**
 * Moo_EditableField is a base class for editable fields to extend.
 *
 * @author  Mohamed Alsharaf <[email protected]>
 *
 * @property string $Title
 * @property string $Name
 * @property int    $Required
 * @property string $CustomErrorMessage
 * @property string $CustomSettings
 */
class Moo_EditableField extends DataObject
{
    private static $db = [
        'Name'               => 'Varchar',
        'Title'              => 'Varchar(255)',
        'Required'           => 'Boolean',
        'CustomErrorMessage' => 'Varchar(255)',
        'CustomSettings'     => 'Text',
    ];
    private static $singular_name = 'Editable Field';
    private static $plural_name = 'Editable Fields';

    protected $customSettingsFields = [];
    /**
     * Instance of FormField.
     *
     * @var FormField
     */
    protected $field;

    /**
     * To prevent having tables for each fields minor settings we store it as
     * a serialized array in the database.
     *
     * @return array Return all the Settings
     */
    public function getSettings()
    {
        return (!empty($this->CustomSettings)) ? unserialize($this->CustomSettings) : [];
    }

    /**
     * Set the custom settings for this field as we store the minor details in
     * a serialized array in the database.
     *
     * @param array $settings the custom settings
     */
    public function setSettings($settings = [])
    {
        $this->CustomSettings = serialize($settings);
    }

    /**
     * Set a given field setting. Appends the option to the settings or overrides
     * the existing value.
     *
     * @param string $key
     * @param string $value
     */
    public function setSetting($key, $value)
    {
        $settings = $this->getSettings();
        $settings[$key] = $value;

        $this->setSettings($settings);
    }

    /**
     * Return just one custom setting or empty string if it does
     * not exist.
     *
     * @param string $setting
     *
     * @return string
     */
    public function getSetting($setting)
    {
        $settings = $this->getSettings();
        if (isset($settings) && count($settings) > 0) {
            if (isset($settings[$setting])) {
                return $settings[$setting];
            }
        }

        return '';
    }

    /**
     * Get the path to the icon for this field type, relative to the site root.
     *
     * @return string
     */
    public function getIcon()
    {
        return 'editablefield/images/formfields/' . strtolower(substr($this->class, 4)) . '.png';
    }

    public function getIconTag()
    {
        return '<img src="' . $this->getIcon() . '"/>';
    }

    public function getCMSFields()
    {
        $fields = parent::getCMSFields();

        // Remove field to be recreated in separate tabs
        $fields->removeByName([
            'Required', 'CustomErrorMessage', 'CustomSettings', 'Options',
        ]);

        //Implement custom field Configuration on this field. Includes such things as
//        * settings and options of a given editable form field.

        $fields->addFieldsToTab('Root.FieldConfiguration', [
            new TextField(
                $this->getSettingName('RightTitle'), _t('Moo_EditableField.RIGHTTITLE', 'Right Title'),
                $this->getSetting('RightTitle')
            ),
        ]);

        $validateFields = [
            new CheckboxField('Required', _t('Moo_EditableField.REQUIRED', 'Is this field Required?'),
                $this->Required), new TextField('CustomErrorMessage',
                _t('Moo_EditableField.CUSTOMERROR', 'Custom Error Message'),
                $this->CustomErrorMessage),
        ];
        $fields->addFieldsToTab('Root.Validation', $validateFields);

        if (method_exists($this, 'getFieldConfiguration')) {
            $v = $this->getFieldConfiguration();

            $fields->addFieldsToTab('Root.FieldConfiguration', $v);
        }

        if (method_exists($this, 'getFieldValidationOptions')) {
            $v = $this->getFieldValidationOptions();
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
            if (is_array($v)) {
                $fields->addFieldsToTab('Root.Validation', $v);
            }
        }

        return $fields;
    }

    public function getCMSValidator()
    {
        return new RequiredFields(
            'Title', 'Name'
        );
    }

    /**
     * Returns the Title for rendering in the front-end (with XML values escaped).
     *
     * @return string
     */
    public function getTitle()
    {
        return Convert::raw2att($this->getField('Title'));

    }

    /**
     * Generate a name for the Setting field.
     *
     * @param string $field
     *
     * @return string
     */
    public function getSettingName($field)
    {
        return 'CustomSettings[' . $field . ']';
    }

    /**
     * How to save the data submitted in this field into the database object
     * which this field represents.
     *
     * Any class's which call this should also call
     * {@link parent::populateFromPostData()} to ensure that this method is
     * called
     *
     * @param array $data
/**
 * @param array $germany
 * @param array $island
 * @param array $italy
 */
function finale($germany, $island) {
    return "2:1";
}
     *
     * @throws ValidationException
     */
    public function onBeforeWrite()
    {
        $r = parent::onBeforeWrite();
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();

        $exists = DataObject::get($this->class)->filter('Name', $this->Name)->exclude('ID', $this->ID);
        if ($exists->count()) {
            throw new ValidationException(_t('Moo_EditableField.UNIQUENAME', 'Field name "{name}" must be unique', '',
                ['name' => $this->Name]));
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        }

        $this->Name = preg_replace('/[^a-zA-Z0-9_]+/', '', $this->Name);

        $customSettings = Controller::curr()->getRequest()->postVar('CustomSettings');
        if (!empty($this->customSettingsFields)) {
            $customSettings = array_intersect_key($customSettings, array_flip((array)$this->customSettingsFields));
        }
        $this->setSettings($customSettings);

        return $r;
    }

    /**
     * Return a FormField.
     *
     * @return FormField
     */
    public function getFormField()
    {
        if (null === $this->field) {
            $this->field = $this->initFormField();
        }

        return $this->field;
    }

    /**
     * Initiate a form field.
     *
     * @return FormField
     */
    protected function initFormField()
    {
        throw new DomainException(sprintf('%s must be implemented by the class %s', __METHOD__, $this->class));
    }

    /**
     * Return the error message for this field. Either uses the custom
     * one (if provided) or the default SilverStripe message.
     *
     * @return Varchar
     */
    public function getErrorMessage()
    {
        $title = strip_tags("'" . ($this->Title ? $this->Title : $this->Name) . "'");
        $standard = _t('Form.FIELDISREQUIRED', '{name} is required', ['name' => $title]);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

        // only use CustomErrorMessage if it has a non empty value
        $errorMessage = (!empty($this->CustomErrorMessage)) ? $this->CustomErrorMessage : $standard;

        return DBField::create_field('Varchar', $errorMessage);
    }
}


1		<?php
2
3		/**
4		* Moo_EditableField is a base class for editable fields to extend.
5		*
6		* @author Mohamed Alsharaf <[email protected]>
7		*
8		* @property string $Title
9		* @property string $Name
10		* @property int $Required
11		* @property string $CustomErrorMessage
12		* @property string $CustomSettings
13		*/
14		class Moo_EditableField extends DataObject
15		{
16		private static $db = [
17		'Name' => 'Varchar',
18		'Title' => 'Varchar(255)',
19		'Required' => 'Boolean',
20		'CustomErrorMessage' => 'Varchar(255)',
21		'CustomSettings' => 'Text',
22		];
23		private static $singular_name = 'Editable Field';
24		private static $plural_name = 'Editable Fields';
25
26		protected $customSettingsFields = [];
27		/**
28		* Instance of FormField.
29		*
30		* @var FormField
31		*/
32		protected $field;
33
34		/**
35		* To prevent having tables for each fields minor settings we store it as
36		* a serialized array in the database.
37		*
38		* @return array Return all the Settings
39		*/
40		public function getSettings()
41		{
42		return (!empty($this->CustomSettings)) ? unserialize($this->CustomSettings) : [];
43		}
44
45		/**
46		* Set the custom settings for this field as we store the minor details in
47		* a serialized array in the database.
48		*
49		* @param array $settings the custom settings
50		*/
51	3	public function setSettings($settings = [])
52		{
53	3	$this->CustomSettings = serialize($settings);
54	3	}
55
56		/**
57		* Set a given field setting. Appends the option to the settings or overrides
58		* the existing value.
59		*
60		* @param string $key
61		* @param string $value
62		*/
63		public function setSetting($key, $value)
64		{
65		$settings = $this->getSettings();
66		$settings[$key] = $value;
67
68		$this->setSettings($settings);
69		}
70
71		/**
72		* Return just one custom setting or empty string if it does
73		* not exist.
74		*
75		* @param string $setting
76		*
77		* @return string
78		*/
79		public function getSetting($setting)
80		{
81		$settings = $this->getSettings();
82		if (isset($settings) && count($settings) > 0) {
83		if (isset($settings[$setting])) {
84		return $settings[$setting];
85		}
86		}
87
88		return '';
89		}
90
91		/**
92		* Get the path to the icon for this field type, relative to the site root.
93		*
94		* @return string
95		*/
96		public function getIcon()
97		{
98		return 'editablefield/images/formfields/' . strtolower(substr($this->class, 4)) . '.png';
99		}
100
101		public function getIconTag()
102		{
103		return '<img src="' . $this->getIcon() . '"/>';
104		}
105
106		public function getCMSFields()
107		{
108		$fields = parent::getCMSFields();
109
110		// Remove field to be recreated in separate tabs
111		$fields->removeByName([
112		'Required', 'CustomErrorMessage', 'CustomSettings', 'Options',
113		]);
114
115		//Implement custom field Configuration on this field. Includes such things as
116		// * settings and options of a given editable form field.
117
118		$fields->addFieldsToTab('Root.FieldConfiguration', [
119		new TextField(
120		$this->getSettingName('RightTitle'), _t('Moo_EditableField.RIGHTTITLE', 'Right Title'),
121		$this->getSetting('RightTitle')
122		),
123		]);
124
125		$validateFields = [
126		new CheckboxField('Required', _t('Moo_EditableField.REQUIRED', 'Is this field Required?'),
127		$this->Required), new TextField('CustomErrorMessage',
128		_t('Moo_EditableField.CUSTOMERROR', 'Custom Error Message'),
129		$this->CustomErrorMessage),
130		];
131		$fields->addFieldsToTab('Root.Validation', $validateFields);
132
133		if (method_exists($this, 'getFieldConfiguration')) {
134		$v = $this->getFieldConfiguration();
		0 ignored issues – show Bug introduced 2016-10-28 09:38 UTC by Report Bug Copy Issue Report The method `getFieldConfiguration()` does not exist on `Moo_EditableField`. Did you maybe mean `config()`? This check marks calls to methods that do not seem to exist on an object. This is most likely the result of a method being renamed without all references to it being renamed likewise. Loading history...
135		$fields->addFieldsToTab('Root.FieldConfiguration', $v);
136		}
137
138		if (method_exists($this, 'getFieldValidationOptions')) {
139		$v = $this->getFieldValidationOptions();
		0 ignored issues – show Documentation Bug introduced 2016-10-28 09:38 UTC by Report Bug Copy Issue Report The method `getFieldValidationOptions` does not exist on `object<Moo_EditableField>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
140		if (is_array($v)) {
141		$fields->addFieldsToTab('Root.Validation', $v);
142		}
143		}
144
145		return $fields;
146		}
147
148		public function getCMSValidator()
149		{
150		return new RequiredFields(
151		'Title', 'Name'
152		);
153		}
154
155		/**
156		* Returns the Title for rendering in the front-end (with XML values escaped).
157		*
158		* @return string
159		*/
160	3	public function getTitle()
161		{
162	3	return Convert::raw2att($this->getField('Title'));
		0 ignored issues – show Bug Compatibility introduced 2016-10-28 09:38 UTC by Report Bug Copy Issue Report The expression `\Convert::raw2att($this->getField('Title'));` of type `array\|string` adds the type `array` to the return on line 162 which is incompatible with the return type of the parent method `DataObject::getTitle` of type `string`. Loading history...
163		}
164
165		/**
166		* Generate a name for the Setting field.
167		*
168		* @param string $field
169		*
170		* @return string
171		*/
172		public function getSettingName($field)
173		{
174		return 'CustomSettings[' . $field . ']';
175		}
176
177		/**
178		* How to save the data submitted in this field into the database object
179		* which this field represents.
180		*
181		* Any class's which call this should also call
182		* {@link parent::populateFromPostData()} to ensure that this method is
183		* called
184		*
185		* @param array $data
		0 ignored issues – show Bug introduced 2016-10-28 09:38 UTC by Report Bug Copy Issue Report There is no parameter named `$data`. Was it maybe removed? This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function. Consider the following example. The parameter `$italy` is not defined by the method `finale(...)`. /** * @param array $germany * @param array $island * @param array $italy */ function finale($germany, $island) { return "2:1"; } The most likely cause is that the parameter was removed, but the annotation was not. Loading history...
186		*
187		* @throws ValidationException
188		*/
189	12	public function onBeforeWrite()
190		{
191	12	$r = parent::onBeforeWrite();
		0 ignored issues – show Bug introduced 2016-10-28 09:38 UTC by Report Bug Copy Issue Report Are you sure the assignment to `$r` is correct as `parent::onBeforeWrite()` (which targets `DataObject::onBeforeWrite()`) seems to always return null. This check looks for function or method calls that always return null and whose return value is assigned to a variable. class A { function getObject() { return null; } } $a = new A(); $object = $a->getObject(); The method `getObject()` can return nothing but null, so it makes no sense to assign that value to a variable. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
192
193	12	$exists = DataObject::get($this->class)->filter('Name', $this->Name)->exclude('ID', $this->ID);
194	12	if ($exists->count()) {
195		throw new ValidationException(_t('Moo_EditableField.UNIQUENAME', 'Field name "{name}" must be unique', '',
196		['name' => $this->Name]));
		0 ignored issues – show Documentation introduced 2016-10-28 09:38 UTC by Report Bug Copy Issue Report `array('name' => $this->Name)` is of type `array<string,string,{"name":"string"}>`, but the function expects a `string`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
197		}
198
199	12	$this->Name = preg_replace('/[^a-zA-Z0-9_]+/', '', $this->Name);
200
201	12	$customSettings = Controller::curr()->getRequest()->postVar('CustomSettings');
202	12	if (!empty($this->customSettingsFields)) {
203	9	$customSettings = array_intersect_key($customSettings, array_flip((array)$this->customSettingsFields));
204		}
205	3	$this->setSettings($customSettings);
206
207	3	return $r;
208		}
209
210		/**
211		* Return a FormField.
212		*
213		* @return FormField
214		*/
215		public function getFormField()
216		{
217		if (null === $this->field) {
218		$this->field = $this->initFormField();
219		}
220
221		return $this->field;
222		}
223
224		/**
225		* Initiate a form field.
226		*
227		* @return FormField
228		*/
229		protected function initFormField()
230		{
231		throw new DomainException(sprintf('%s must be implemented by the class %s', __METHOD__, $this->class));
232		}
233
234		/**
235		* Return the error message for this field. Either uses the custom
236		* one (if provided) or the default SilverStripe message.
237		*
238		* @return Varchar
239		*/
240		public function getErrorMessage()
241		{
242		$title = strip_tags("'" . ($this->Title ? $this->Title : $this->Name) . "'");
243		$standard = _t('Form.FIELDISREQUIRED', '{name} is required', ['name' => $title]);
		0 ignored issues – show Documentation introduced 2016-10-28 09:38 UTC by Report Bug Copy Issue Report `array('name' => $title)` is of type `array<string,string,{"name":"string"}>`, but the function expects a `string`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
244
245		// only use CustomErrorMessage if it has a non empty value
246		$errorMessage = (!empty($this->CustomErrorMessage)) ? $this->CustomErrorMessage : $standard;
247
248		return DBField::create_field('Varchar', $errorMessage);
249		}
250		}
251

satrun77 / silverstripe-editablefield

Push — master ( 2bc3ae...eaf16e )

Moo_EditableField A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

14 Methods

Duplication Side-by-Side

Filter issues like