Labelable - Code Metrics - ThePHPGuys/extjs-php-wrapper - Measure and Improve Code Quality continuously with Scrutinizer

Labelable A
last analyzed 2016-12-16 19:56 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	21
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

Metric	Value
wmc	4
lcom	0
cbo	0
dl	0
loc	21
rs	10
c	0
b	0
f	0

4 Methods

Rating	Name	Size	Complexity
A	setHideLabel()	3	1
A	getHideLabel()	3	1
A	setFieldLabel()	3	1
A	getFieldLabel()	3	1

<?php
/**
 * ExtForms - Labelable.php
 *
 * Initial version by: MisterX
 * Initial version created on: 31.07.2015
 */

namespace Ext\Form;


trait Labelable
{

	public function setHideLabel($hideLabel){
		return $this->setProperty('hideLabel',$hideLabel);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
	}

	public function getHideLabel(){
		return $this->getProperty('hideLabel');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
	}

	public function setFieldLabel($fieldLabel){
		return $this->setProperty('fieldLabel',$fieldLabel);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
	}
	
	public function getFieldLabel(){
		return $this->getProperty('fieldLabel');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
	}
	
	
}

1			<?php
2			/**
3			* ExtForms - Labelable.php
4			*
5			* Initial version by: MisterX
6			* Initial version created on: 31.07.2015
7			*/
8
9			namespace Ext\Form;
10
11
12			trait Labelable
13			{
14
15			public function setHideLabel($hideLabel){
16			return $this->setProperty('hideLabel',$hideLabel);
			0 ignored issues – show Bug introduced 2016-12-16 17:56 UTC by Report Bug Copy Issue Report It seems like `setProperty()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
17			}
18
19			public function getHideLabel(){
20			return $this->getProperty('hideLabel');
			0 ignored issues – show Bug introduced 2016-12-16 17:56 UTC by Report Bug Copy Issue Report It seems like `getProperty()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
21			}
22
23			public function setFieldLabel($fieldLabel){
24			return $this->setProperty('fieldLabel',$fieldLabel);
			0 ignored issues – show Bug introduced 2016-12-16 17:56 UTC by Report Bug Copy Issue Report It seems like `setProperty()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
25			}
26
27			public function getFieldLabel(){
28			return $this->getProperty('fieldLabel');
			0 ignored issues – show Bug introduced 2016-12-16 17:56 UTC by Report Bug Copy Issue Report It seems like `getProperty()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
29			}
30
31
32			}

ThePHPGuys / extjs-php-wrapper

Labelable A last analyzed 2016-12-16 19:56 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Importance

4 Methods

Duplication Side-by-Side

Filter issues like

Labelable A
last analyzed 2016-12-16 19:56 UTC