UserFieldOption - Code Metrics - Inspection of "changed the attendee fields to user fields and add..." - TheBnl/event-tickets - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( e38edc...acf857 )

by Bram

created 2017-06-22 16:47 UTC

UserFieldOption A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	55
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	6

Importance

Changes

Metric	Value
wmc	5
lcom	0
cbo	6
dl	0
loc	55
rs	10
c	0
b	0
f	0

5 Methods

Rating	Name	Size	Complexity
A	getCMSFields()	11	1
A	canView()	4	1
A	canEdit()	4	1
A	canDelete()	4	1
A	canCreate()	4	1

<?php
/**
 * AttendeeExtraFieldOption.php
 *
 * @author Bram de Leeuw
 * Date: 24/05/17
 */

namespace Broarm\EventTickets;

use CheckboxField;
use DataObject;
use FieldList;
use Tab;
use TabSet;
use TextField;

/**
 * Class AttendeeExtraFieldOption
 *
 * @property string Title
 * @property boolean Default
 * @method UserField Field
 */
class UserFieldOption extends DataObject
{
    private static $db = array(

        'Title' => 'Varchar(255)',
        'Default' => 'Boolean',
        'Sort' => 'Int'
    );
    
    private static $default_sort = 'Sort ASC';


    private static $has_one = array(

        'Field' => 'Broarm\EventTickets\UserField'
    );

    private static $summary_fields = array(

        'Title',
        'Default'
    );

    private static $translate = array(

        'Title'
    );

    public function getCMSFields()

    {
        $fields = new FieldList(new TabSet('Root', $mainTab = new Tab('Main')));
        $fields->addFieldsToTab('Root.Main', array(
            TextField::create('Title', _t('AttendeeExtraFieldOption.Title', 'Title')),
            CheckboxField::create('Default', _t('AttendeeExtraFieldOption.Default', 'Set as default'))
        ));

        $this->extend('updateCMSFields', $fields);
        return $fields;
    }

    public function canView($member = null)

    {
        return $this->Field()->canView($member);
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 { }
    }

    public function canEdit($member = null)

    {
        return $this->Field()->canEdit($member);
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 { }
    }

    public function canDelete($member = null)

    {
        return $this->Field()->canDelete($member);
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 { }
    }

    public function canCreate($member = null)

    {
        return $this->Field()->canCreate($member);
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 { }
    }
}


1			<?php
2			/**
3			* AttendeeExtraFieldOption.php
4			*
5			* @author Bram de Leeuw
6			* Date: 24/05/17
7			*/
8
9			namespace Broarm\EventTickets;
10
11			use CheckboxField;
12			use DataObject;
13			use FieldList;
14			use Tab;
15			use TabSet;
16			use TextField;
17
18			/**
19			* Class AttendeeExtraFieldOption
20			*
21			* @property string Title
22			* @property boolean Default
23			* @method UserField Field
24			*/
25			class UserFieldOption extends DataObject
26			{
27			private static $db = array(
			0 ignored issues – show Comprehensibility introduced 2017-05-09 14:47 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history... Unused Code introduced 2017-06-22 16:49 UTC by Report Bug Copy Issue Report The property `$db` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
28			'Title' => 'Varchar(255)',
29			'Default' => 'Boolean',
30			'Sort' => 'Int'
31			);
32
33			private static $default_sort = 'Sort ASC';
			0 ignored issues – show Comprehensibility introduced 2017-05-24 16:47 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history... Unused Code introduced 2017-05-24 16:47 UTC by Report Bug Copy Issue Report The property `$default_sort` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
34
35			private static $has_one = array(
			0 ignored issues – show Comprehensibility introduced 2017-05-24 16:47 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history... Unused Code introduced 2017-05-24 16:47 UTC by Report Bug Copy Issue Report The property `$has_one` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
36			'Field' => 'Broarm\EventTickets\UserField'
37			);
38
39			private static $summary_fields = array(
			0 ignored issues – show Comprehensibility introduced 2017-05-09 14:47 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history... Unused Code introduced 2017-04-25 13:06 UTC by Report Bug Copy Issue Report The property `$summary_fields` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
40			'Title',
41			'Default'
42			);
43
44			private static $translate = array(
			0 ignored issues – show Unused Code introduced 2017-05-24 16:47 UTC by Report Bug Copy Issue Report The property `$translate` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
45			'Title'
46			);
47
48			public function getCMSFields()
			0 ignored issues – show Documentation introduced 2017-05-09 14:47 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
49			{
50			$fields = new FieldList(new TabSet('Root', $mainTab = new Tab('Main')));
51			$fields->addFieldsToTab('Root.Main', array(
52			TextField::create('Title', _t('AttendeeExtraFieldOption.Title', 'Title')),
53			CheckboxField::create('Default', _t('AttendeeExtraFieldOption.Default', 'Set as default'))
54			));
55
56			$this->extend('updateCMSFields', $fields);
57			return $fields;
58			}
59
60			public function canView($member = null)
			0 ignored issues – show Documentation introduced 2017-04-25 13:06 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
61			{
62			return $this->Field()->canView($member);
			0 ignored issues – show Documentation Bug introduced 2017-06-22 16:49 UTC by Report Bug Copy Issue Report The method `Field` does not exist on `object<Broarm\EventTickets\UserFieldOption>`? 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...
63			}
64
65			public function canEdit($member = null)
			0 ignored issues – show Documentation introduced 2017-04-25 13:06 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
66			{
67			return $this->Field()->canEdit($member);
			0 ignored issues – show Documentation Bug introduced 2017-06-22 16:49 UTC by Report Bug Copy Issue Report The method `Field` does not exist on `object<Broarm\EventTickets\UserFieldOption>`? 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...
68			}
69
70			public function canDelete($member = null)
			0 ignored issues – show Documentation introduced 2017-05-09 14:47 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
71			{
72			return $this->Field()->canDelete($member);
			0 ignored issues – show Documentation Bug introduced 2017-06-22 16:49 UTC by Report Bug Copy Issue Report The method `Field` does not exist on `object<Broarm\EventTickets\UserFieldOption>`? 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...
73			}
74
75			public function canCreate($member = null)
			0 ignored issues – show Documentation introduced 2017-04-25 13:06 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
76			{
77			return $this->Field()->canCreate($member);
			0 ignored issues – show Documentation Bug introduced 2017-06-22 16:49 UTC by Report Bug Copy Issue Report The method `Field` does not exist on `object<Broarm\EventTickets\UserFieldOption>`? 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...
78			}
79			}
80

TheBnl / event-tickets

Push — master ( e38edc...acf857 )

UserFieldOption A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

5 Methods

Duplication Side-by-Side

Filter issues like