WaitingListRegistration::canView() - Code Metrics - Inspection of "added waiting list functionality when the event is..." - TheBnl/event-tickets - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( cc2184...cecba6 )

by Bram

created 2017-05-09 14:37 UTC

WaitingListRegistration::canView() A

↳ Parent: WaitingListRegistration

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	4
rs	10
c	0
b	0
f	0
cc	1
eloc	2
nc	1
nop	1

<?php
/**
 * WaitingListRegistration.php
 *
 * @author Bram de Leeuw
 * Date: 09/05/17
 */

namespace Broarm\EventTickets;

use DataObject;
use FieldList;
use ReadonlyField;
use Tab;
use TabSet;

/**
 * Class WaitingListRegistration
 *
 * @package Broarm\EventTickets
 *
 * @property string Title
 * @property string Email
 * @property string Telephone
 *
 * @method \CalendarEvent Event
 */
class WaitingListRegistration extends DataObject
{
    private static $db = array(

        'Title' => 'Varchar(255)',
        'Email' => 'Varchar(255)',
        'Telephone' => 'Varchar(255)'
    );

    private static $has_one = array(

        'Event' => 'CalendarEvent'
    );

    private static $summary_fields = array(

        'Title' => 'Name',
        'Email' => 'Email',
        'Telephone' => 'Telephone'
    );

    public function getCMSFields()

    {
        $fields = new FieldList(new TabSet('Root', $mainTab = new Tab('Main')));
        $fields->addFieldsToTab('Root.Main', array(
            ReadonlyField::create('Title', _t('WaitingListRegistration.Name', 'Name')),
            ReadonlyField::create('Email', _t('WaitingListRegistration.Email', 'Email')),
            ReadonlyField::create('Telephone', _t('WaitingListRegistration.Telephone', 'Telephone'))
        ));

        $fields->addFieldsToTab('Root.Main', array());
        $this->extend('updateCMSFields', $fields);
        return $fields;
    }

    public function canView($member = null)

    {
        return $this->Event()->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->Event()->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->Event()->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->Event()->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			* WaitingListRegistration.php
4			*
5			* @author Bram de Leeuw
6			* Date: 09/05/17
7			*/
8
9			namespace Broarm\EventTickets;
10
11			use DataObject;
12			use FieldList;
13			use ReadonlyField;
14			use Tab;
15			use TabSet;
16
17			/**
18			* Class WaitingListRegistration
19			*
20			* @package Broarm\EventTickets
21			*
22			* @property string Title
23			* @property string Email
24			* @property string Telephone
25			*
26			* @method \CalendarEvent Event
27			*/
28			class WaitingListRegistration extends DataObject
29			{
30			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-05-09 14:47 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...
31			'Title' => 'Varchar(255)',
32			'Email' => 'Varchar(255)',
33			'Telephone' => 'Varchar(255)'
34			);
35
36			private static $has_one = 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 `$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...
37			'Event' => 'CalendarEvent'
38			);
39
40			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...
41			'Title' => 'Name',
42			'Email' => 'Email',
43			'Telephone' => 'Telephone'
44			);
45
46			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...
47			{
48			$fields = new FieldList(new TabSet('Root', $mainTab = new Tab('Main')));
49			$fields->addFieldsToTab('Root.Main', array(
50			ReadonlyField::create('Title', _t('WaitingListRegistration.Name', 'Name')),
51			ReadonlyField::create('Email', _t('WaitingListRegistration.Email', 'Email')),
52			ReadonlyField::create('Telephone', _t('WaitingListRegistration.Telephone', 'Telephone'))
53			));
54
55			$fields->addFieldsToTab('Root.Main', array());
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->Event()->canView($member);
			0 ignored issues – show Documentation Bug introduced 2017-05-09 14:47 UTC by Report Bug Copy Issue Report The method `Event` does not exist on `object<Broarm\EventTicke...aitingListRegistration>`? 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-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...
66			{
67			return $this->Event()->canEdit($member);
			0 ignored issues – show Documentation Bug introduced 2017-05-09 14:47 UTC by Report Bug Copy Issue Report The method `Event` does not exist on `object<Broarm\EventTicke...aitingListRegistration>`? 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->Event()->canDelete($member);
			0 ignored issues – show Documentation Bug introduced 2017-05-09 14:47 UTC by Report Bug Copy Issue Report The method `Event` does not exist on `object<Broarm\EventTicke...aitingListRegistration>`? 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-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...
76			{
77			return $this->Event()->canCreate($member);
			0 ignored issues – show Documentation Bug introduced 2017-05-09 14:47 UTC by Report Bug Copy Issue Report The method `Event` does not exist on `object<Broarm\EventTicke...aitingListRegistration>`? 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 ( cc2184...cecba6 )

WaitingListRegistration::canView() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like