Eventlog - Code Metrics - Inspection of "Merge pull request #53 from murrant/move-models" - librenms/librenmsv2 - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( 2606e4...9302c9 )

by Neil

created 2016-03-10 20:05 UTC

Eventlog A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
wmc	1
c	1
b	0
f	0
lcom	0
cbo	1
dl	0
loc	36
rs	10

1 Method

Rating	Name	Duplication	Size	Complexity
A	device()	0	3	1

<?php

namespace App\Models;

use Illuminate\Database\Eloquent\Model;

class Eventlog extends Model
{
    /**
     * The table associated with the model.
     *
     * @var string
     */
    protected $table = 'devices';

    /**
     * The primary key column name.
     *
     * @var string
     */
    protected $primaryKey = 'event_id';

    /**
     * Indicates if the model should be timestamped.
     *
     * @var bool
     */
    public $timestamps = false;


    // ---- Accessors/Mutators ----


    // ---- Define Reletionships ----

    /**
     * Returns the device this entry belongs to.
     */
    public function device() {
        return $this->belongsToOne('App\Models\Device', 'device_id', 'host');
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			namespace App\Models;
4
5			use Illuminate\Database\Eloquent\Model;
6
7			class Eventlog extends Model
8			{
9			/**
10			* The table associated with the model.
11			*
12			* @var string
13			*/
14			protected $table = 'devices';
15
16			/**
17			* The primary key column name.
18			*
19			* @var string
20			*/
21			protected $primaryKey = 'event_id';
22
23			/**
24			* Indicates if the model should be timestamped.
25			*
26			* @var bool
27			*/
28			public $timestamps = false;
29
30
31			// ---- Accessors/Mutators ----
32
33
34			// ---- Define Reletionships ----
35
36			/**
37			* Returns the device this entry belongs to.
38			*/
39			public function device() {
40			return $this->belongsToOne('App\Models\Device', 'device_id', 'host');
			0 ignored issues – show Documentation Bug introduced 2016-03-10 16:01 UTC by Report Bug Copy Issue Report The method `belongsToOne` does not exist on `object<App\Models\Eventlog>`? 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...
41			}
42			}
43

librenms / librenmsv2

Push — develop ( 2606e4...9302c9 )

Eventlog A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

1 Method

Duplication Side-by-Side

Filter issues like