AbstractRow - Code Metrics - Inspection of "test fix" - oscarotero/simple-crud - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 0eeb04...4002ae )

by Oscar

created 2016-02-17 00:37 UTC

AbstractRow A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	128
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	5
Bugs	0	Features	0

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

10 Methods

Rating	Name	Size	Complexity
A	__construct()	4	1
A	getTable()	4	1
A	getDatabase()	4	1
A	jsonSerialize()	4	1
A	__toString()	4	1
	toArray()	1	?
	__get()	1	?
	__set()	1	?
A	delete()	14	2
A	__call()	20	3

<?php

namespace SimpleCrud;

use JsonSerializable;
use SimpleCrud\Scheme\Scheme;

/**
 * Base class used by Row and RowCollection.
 *
 * @property mixed $id
 */
abstract class AbstractRow implements JsonSerializable
{
    protected $table;

    /**
     * Constructor.
     *
     * @param Table $table
     */
    public function __construct(Table $table)
    {
        $this->table = $table;
    }

    /**
     * Returns the table associated with this row.
     *
     * @return Table
     */
    public function getTable()
    {
        return $this->table;
    }

    /**
     * Returns the database associated with this row.
     *
     * @return SimpleCrud
     */
    public function getDatabase()
    {
        return $this->getTable()->getDatabase();
    }

    /**
     * @see JsonSerializable
     *
     * @return array
     */
    public function jsonSerialize()
    {
        return $this->toArray();
    }

    /**
     * Magic method to stringify the values.
     *
     * @return string
     */
    public function __toString()
    {
        return json_encode($this, JSON_NUMERIC_CHECK);
    }

    /**
     * Converts this object into an array.
     * 
     * @param array $bannedEntities
     *
     * @return array
     */
    abstract public function toArray(array $bannedEntities = []);

    /**
     * Magic method to return properties.
     * 
     * @param string $name
     *
     * @return mixed
     */
    abstract public function __get($name);

    /**
     * Magic method to edit a property.
     * 
     * @param string $name
     * @param mixed  $value
     *
     * @return mixed
     */
    abstract public function __set($name, $value);

    /**
     * Deletes the row(s) in the database.
     *
     * @return self
     */
    public function delete()
    {
        $id = $this->id;

        if (!empty($id)) {
            $this->getTable()->delete()
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 { }
                ->byId($id)
                ->run();

            $this->id = null;
        }

        return $this;
    }

    /**
     * Magic method to execute queries.
     *
     * @param string $name
     */
    public function __call($name, $arguments)
    {
        $scheme = $this->getTable()->getScheme();

        if (!isset($scheme['relations'][$name])) {
            throw new \BadMethodCallException(sprintf('Call to undefined method %s', $name));
        }

        $table = $this->getTable();
        $relation = $table->getScheme()['relations'][$name];
        $related = $table->getDatabase()->$name;

        switch ($relation[0]) {
            case Scheme::HAS_ONE:
                return $related->select()->one()->relatedWith($this);

            default:
                return $related->select()->relatedWith($this);
        }
    }
}


1			<?php
2
3			namespace SimpleCrud;
4
5			use JsonSerializable;
6			use SimpleCrud\Scheme\Scheme;
7
8			/**
9			* Base class used by Row and RowCollection.
10			*
11			* @property mixed $id
12			*/
13			abstract class AbstractRow implements JsonSerializable
14			{
15			protected $table;
16
17			/**
18			* Constructor.
19			*
20			* @param Table $table
21			*/
22			public function __construct(Table $table)
23			{
24			$this->table = $table;
25			}
26
27			/**
28			* Returns the table associated with this row.
29			*
30			* @return Table
31			*/
32			public function getTable()
33			{
34			return $this->table;
35			}
36
37			/**
38			* Returns the database associated with this row.
39			*
40			* @return SimpleCrud
41			*/
42			public function getDatabase()
43			{
44			return $this->getTable()->getDatabase();
45			}
46
47			/**
48			* @see JsonSerializable
49			*
50			* @return array
51			*/
52			public function jsonSerialize()
53			{
54			return $this->toArray();
55			}
56
57			/**
58			* Magic method to stringify the values.
59			*
60			* @return string
61			*/
62			public function __toString()
63			{
64			return json_encode($this, JSON_NUMERIC_CHECK);
65			}
66
67			/**
68			* Converts this object into an array.
69			*
70			* @param array $bannedEntities
71			*
72			* @return array
73			*/
74			abstract public function toArray(array $bannedEntities = []);
75
76			/**
77			* Magic method to return properties.
78			*
79			* @param string $name
80			*
81			* @return mixed
82			*/
83			abstract public function __get($name);
84
85			/**
86			* Magic method to edit a property.
87			*
88			* @param string $name
89			* @param mixed $value
90			*
91			* @return mixed
92			*/
93			abstract public function __set($name, $value);
94
95			/**
96			* Deletes the row(s) in the database.
97			*
98			* @return self
99			*/
100			public function delete()
101			{
102			$id = $this->id;
103
104			if (!empty($id)) {
105			$this->getTable()->delete()
			0 ignored issues – show Documentation Bug introduced 2016-02-17 00:40 UTC by Report Bug Copy Issue Report The method `delete` does not exist on `object<SimpleCrud\Table>`? 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...
106			->byId($id)
107			->run();
108
109			$this->id = null;
110			}
111
112			return $this;
113			}
114
115			/**
116			* Magic method to execute queries.
117			*
118			* @param string $name
119			*/
120			public function __call($name, $arguments)
121			{
122			$scheme = $this->getTable()->getScheme();
123
124			if (!isset($scheme['relations'][$name])) {
125			throw new \BadMethodCallException(sprintf('Call to undefined method %s', $name));
126			}
127
128			$table = $this->getTable();
129			$relation = $table->getScheme()['relations'][$name];
130			$related = $table->getDatabase()->$name;
131
132			switch ($relation[0]) {
133			case Scheme::HAS_ONE:
134			return $related->select()->one()->relatedWith($this);
135
136			default:
137			return $related->select()->relatedWith($this);
138			}
139			}
140			}
141

oscarotero / simple-crud

Push — master ( 0eeb04...4002ae )

AbstractRow A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

10 Methods

Duplication Side-by-Side

Filter issues like