Moo_EditableFieldMultipleOption - 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_EditableFieldMultipleOption A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	86
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	9

Test Coverage

Coverage

Importance

Changes

Metric	Value
wmc	10
lcom	1
cbo	9
dl	0
loc	86
ccs	0
cts	42
cp	0
rs	10
c	0
b	0
f	0

4 Methods

Rating	Name	Size	Complexity
A	delete()	12	3
A	duplicate()	14	3
B	getFieldConfiguration()	27	3
A	initFormField()	6	1

<?php

/**
 * Moo_EditableFieldMultipleOption is a base class for multiple option fields to extend.
 *
 * @see     Moo_EditableFieldCheckboxGroup, Moo_EditableFieldDropdown
 *
 * @package editablefield
 *
 * @author  Mohamed Alsharaf <[email protected]>
 *
 * @method  HasManyList Options()
 *
 * @property string $Name
 * @property string $Title
 */
class Moo_EditableFieldMultipleOption extends Moo_EditableField
{
    private static $has_many = [
        'Options' => 'Moo_EditableFieldOption',
    ];

    /**
     * Deletes all the options attached to this field before deleting the
     * field. Keeps stray options from floating around.
     *
     * @return void
     */
    public function delete()
    {
        $options = $this->Options();

        if ($options) {
            foreach ($options as $option) {
                $option->delete();
            }
        }

        parent::delete();
    }

    /**
     * Duplicate a pages content. We need to make sure all the fields attached
     * to that page go with it.
     *
     * @return DataObject
     */
    public function duplicate($doWrite = true)
    {
        $clonedNode = parent::duplicate();

        if ($this->Options()) {
            foreach ($this->Options() as $field) {
                $newField           = $field->duplicate();
                $newField->ParentID = $clonedNode->ID;
                $newField->write();
            }
        }

        return $clonedNode;
    }

    public function getFieldConfiguration()
    {
        if (!$this->isInDB()) {
            $field = LiteralField::create('Options', '<p class="message notice">Once you save this field you will be able to add options</p>');
        } else {
            $config = GridFieldConfig_RelationEditor::create()
                ->addComponent(new GridFieldOrderableRows('Sort'));

            $config
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
                ->getComponentByType('GridFieldDataColumns')
                ->setDisplayFields([
                    'Name'    => 'Name',
                    'Title'   => 'Title',
                    'Default' => 'Default',
                ])
                ->setFieldFormatting([
                    'Default' => function ($_, Moo_EditableFieldOption $option) {
                        return $option->Default ? 'Yes' : 'No';
                    },
                ]);
            $field = GridField::create('Options', 'Options', $this->Options(), $config);
        }

        return [
            $field,
        ];
    }

    /**
     * Return the form field for this object in the front end form view.
     *
     * @return FormField
     */
    protected function initFormField()
    {
        $options = $this->Options()->map('EscapedTitle', 'Title');

        return new OptionsetField($this->Name, $this->Title, $options);
    }
}


1			<?php
2
3			/**
4			* Moo_EditableFieldMultipleOption is a base class for multiple option fields to extend.
5			*
6			* @see Moo_EditableFieldCheckboxGroup, Moo_EditableFieldDropdown
7			*
8			* @package editablefield
9			*
10			* @author Mohamed Alsharaf <[email protected]>
11			*
12			* @method HasManyList Options()
13			*
14			* @property string $Name
15			* @property string $Title
16			*/
17			class Moo_EditableFieldMultipleOption extends Moo_EditableField
18			{
19			private static $has_many = [
20			'Options' => 'Moo_EditableFieldOption',
21			];
22
23			/**
24			* Deletes all the options attached to this field before deleting the
25			* field. Keeps stray options from floating around.
26			*
27			* @return void
28			*/
29			public function delete()
30			{
31			$options = $this->Options();
32
33			if ($options) {
34			foreach ($options as $option) {
35			$option->delete();
36			}
37			}
38
39			parent::delete();
40			}
41
42			/**
43			* Duplicate a pages content. We need to make sure all the fields attached
44			* to that page go with it.
45			*
46			* @return DataObject
47			*/
48			public function duplicate($doWrite = true)
49			{
50			$clonedNode = parent::duplicate();
51
52			if ($this->Options()) {
53			foreach ($this->Options() as $field) {
54			$newField = $field->duplicate();
55			$newField->ParentID = $clonedNode->ID;
56			$newField->write();
57			}
58			}
59
60			return $clonedNode;
61			}
62
63			public function getFieldConfiguration()
64			{
65			if (!$this->isInDB()) {
66			$field = LiteralField::create('Options', '<p class="message notice">Once you save this field you will be able to add options</p>');
67			} else {
68			$config = GridFieldConfig_RelationEditor::create()
69			->addComponent(new GridFieldOrderableRows('Sort'));
70
71			$config
			0 ignored issues – show Bug introduced 2016-10-28 09:38 UTC by Report Bug Copy Issue Report It seems like you code against a concrete implementation and not the interface `GridFieldComponent` as the method `setDisplayFields()` does only exist in the following implementations of said interface: `GridFieldDataColumns`, `GridFieldEditableColumns`, `GridFieldExternalLink`. Let’s take a look at an example: interface User { /** @return string / public function getPassword(); } class MyUser implements User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different implementation of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the interface: interface User { /** @return string / public function getPassword(); /* @return string */ public function getDisplayName(); } Loading history...
72			->getComponentByType('GridFieldDataColumns')
73			->setDisplayFields([
74			'Name' => 'Name',
75			'Title' => 'Title',
76			'Default' => 'Default',
77			])
78			->setFieldFormatting([
79			'Default' => function ($_, Moo_EditableFieldOption $option) {
80			return $option->Default ? 'Yes' : 'No';
81			},
82			]);
83			$field = GridField::create('Options', 'Options', $this->Options(), $config);
84			}
85
86			return [
87			$field,
88			];
89			}
90
91			/**
92			* Return the form field for this object in the front end form view.
93			*
94			* @return FormField
95			*/
96			protected function initFormField()
97			{
98			$options = $this->Options()->map('EscapedTitle', 'Title');
99
100			return new OptionsetField($this->Name, $this->Title, $options);
101			}
102			}
103

satrun77 / silverstripe-editablefield

Push — master ( 2bc3ae...eaf16e )

Moo_EditableFieldMultipleOption A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

4 Methods

Available Fixes

Duplication Side-by-Side

Filter issues like