Wizard::uniques() - Code Metrics - Inspection of "index and unique migration formatting" - php-resta/resta-core - Measure and Improve Code Quality continuously with Scrutinizer

Test Setup Failed

Push — master ( 058fd4...43367b )

by Php Easy Api

created 2019-08-19 13:53 UTC

Wizard::uniques() A

↳ Parent: Wizard

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php

namespace Migratio\GrammarStructure\Mysql\Wizard;

use Migratio\Contract\TablePropertiesContract;
use Migratio\Contract\WizardContract;

class Wizard extends WizardHelper implements WizardContract
{
    /**
     * @var array
     */
    protected $alterBinds = array();

    /**
     * @var $randomInstance

     */
    public static $randomInstance;

    /**
     * @var array $error
     */
    protected $error = array();

    /**
     * @var array
     */
    protected $alterType = [];

    /**
     * @var array
     */
    protected $auto_increment=array();

    /**
     * @var $collation

     */
    protected $collation;

    /**
     * @var $comment

     */
    protected $comment;

    /**
     * @var array $primaryKey
     */
    protected $primaryKey=array();

    /**
     * @var array $references
     */
    protected $references=array();

    /**
     * @var $schemaType

     */
    protected $schemaType;

    /**
     * @var $file

     */
    protected $file;

    /**
     * @var $name array

     */
    protected $name=array();

    /**
     * @var array $nullable
     */
    protected $nullable=array();

    /**
     * @var array $types
     */
    protected $types=array();

    /**
     * @var $engine

     */
    protected $engine;

    /**
     * @var $default array

     */
    protected $default=array();

    /**
     * @var array $index
     */
    protected $index=array();

    /**
     * @var array $unique
     */
    protected $unique=array();

    /**
     * @var $table

     */
    protected $table;

    /**
     * @return mixed|void
     */
    public function auto_increment()
    {
        if(count($this->auto_increment)==0){

            if($this->getLastName()===false){
                $this->name[]='id';
                $this->setTypes('int',14);
            }
            $this->auto_increment[$this->getLastName()]=true;
            $this->primaryKey();
        }
    }

    /**
     * @param $value
     * @return $this
     */
    public function comment($value)
    {
        $this->comment[$this->getLastName()]=$value;

        return $this;
    }

    /**
     * @param $name
     * @return Types
     */
    public function name($name)
    {
        if(in_array($name,$this->name)){
           $this->setError('You have written the '.$name.' name more than 1.');
        }

        $this->name[] = $name;

        return new Types($this);

    }

    /**
     * @param $value
     * @return $this|mixed
     */
    public function default($value)
    {
        $this->default[$this->getLastName()]=$value;

        return $this;
    }

    /**
     * @param $value
     * @param bool $table
     * @return $this
     */
    public function collation($value,$table=false)
    {
        if($table===false)
        {
            $this->collation[$this->getLastName()]=$value;
        }
        else{
            $this->collation['table']=$value;
        }

        return $this;
interface HasName {
    /** @return string */
    public function getName();
}

class Name {
    public $name;
}

class User implements HasName {
    /** @return string|Name */
    public function getName() {
        return new Name('foo'); // This is a violation of the ``HasName`` interface
                                // which only allows a string value to be returned.
    }
}
    }

    /**
     * @param null $name

     * @param null $value

     * @param bool $key
     * @return $this|WizardContract
     */
    public function index($name=null,$value=null,$key=false)
    {
        $name   = ($name===null) ? $this->getLastName() : $name;

        $value  = ($value===null) ? $name : $value;


        if($key===false){
            $this->index[$this->getLastName()]=['name'=>$name,'value'=>$value];
        }

        if($key===true){
            $this->index['indexes'][]=['name'=>$name,'value'=>$value];
        }

        return $this;
    }

    /**
     * @param $index_name
     * @param array $indexes
     * @return void
     */
    public function indexes($index_name,$indexes=array())
    {
        $this->index[$index_name] = $indexes;
    }

    /**
     * @param bool $null
     * @return $this
     */
    public function nullable($null=true)
    {
        $this->nullable[$this->getLastName()]=$null;

        return $this;
    }

    /**
     * @return $this|WizardContract
     */
    public function primaryKey()
    {
        $this->primaryKey[$this->getLastName()]=true;

        return $this;
    }

    /**
     * @return TablePropertiesContract
     */
    public function table()
    {
        return new TableProperties($this);
    }

    /**
     * @param $name
     * @param null $value

     * @return $this|mixed
     */
    public function unique($name=null,$value=null)
    {
        $name   = ($name===null) ? $this->getLastName() : $name;
        $value  = ($value===null) ? $name : $value;


        $this->unique[$this->getLastName()] = ['name'=>$name,'value'=>$value];

        return $this;
    }

    /**
     * @param $name
     * @param null $value

     * @return $this|mixed
     */
    public function uniques($unique_name,$uniques=array())
    {
        $this->unique[$unique_name] = $uniques;
    }
}



1			<?php
2
3			namespace Migratio\GrammarStructure\Mysql\Wizard;
4
5			use Migratio\Contract\TablePropertiesContract;
6			use Migratio\Contract\WizardContract;
7
8			class Wizard extends WizardHelper implements WizardContract
9			{
10			/**
11			* @var array
12			*/
13			protected $alterBinds = array();
14
15			/**
16			* @var $randomInstance
			0 ignored issues – show Documentation Bug introduced 2019-07-19 15:27 UTC by Report Bug Copy Issue Report The doc comment `$randomInstance` at position `0` could not be parsed: Unknown type name '$randomInstance' at position 0 in $randomInstance. Loading history...
17			*/
18			public static $randomInstance;
19
20			/**
21			* @var array $error
22			*/
23			protected $error = array();
24
25			/**
26			* @var array
27			*/
28			protected $alterType = [];
29
30			/**
31			* @var array
32			*/
33			protected $auto_increment=array();
34
35			/**
36			* @var $collation
			0 ignored issues – show Documentation Bug introduced 2019-06-27 08:32 UTC by Report Bug Copy Issue Report The doc comment `$collation` at position `0` could not be parsed: Unknown type name '$collation' at position 0 in $collation. Loading history...
37			*/
38			protected $collation;
39
40			/**
41			* @var $comment
			0 ignored issues – show Documentation Bug introduced 2019-06-27 08:32 UTC by Report Bug Copy Issue Report The doc comment `$comment` at position `0` could not be parsed: Unknown type name '$comment' at position 0 in $comment. Loading history...
42			*/
43			protected $comment;
44
45			/**
46			* @var array $primaryKey
47			*/
48			protected $primaryKey=array();
49
50			/**
51			* @var array $references
52			*/
53			protected $references=array();
54
55			/**
56			* @var $schemaType
			0 ignored issues – show Documentation Bug introduced 2019-06-27 08:32 UTC by Report Bug Copy Issue Report The doc comment `$schemaType` at position `0` could not be parsed: Unknown type name '$schemaType' at position 0 in $schemaType. Loading history...
57			*/
58			protected $schemaType;
59
60			/**
61			* @var $file
			0 ignored issues – show Documentation Bug introduced 2019-06-27 08:32 UTC by Report Bug Copy Issue Report The doc comment `$file` at position `0` could not be parsed: Unknown type name '$file' at position 0 in $file. Loading history...
62			*/
63			protected $file;
64
65			/**
66			* @var $name array
			0 ignored issues – show Documentation Bug introduced 2019-06-27 08:32 UTC by Report Bug Copy Issue Report The doc comment `$name` at position `0` could not be parsed: Unknown type name '$name' at position 0 in $name. Loading history...
67			*/
68			protected $name=array();
69
70			/**
71			* @var array $nullable
72			*/
73			protected $nullable=array();
74
75			/**
76			* @var array $types
77			*/
78			protected $types=array();
79
80			/**
81			* @var $engine
			0 ignored issues – show Documentation Bug introduced 2019-06-27 08:32 UTC by Report Bug Copy Issue Report The doc comment `$engine` at position `0` could not be parsed: Unknown type name '$engine' at position 0 in $engine. Loading history...
82			*/
83			protected $engine;
84
85			/**
86			* @var $default array
			0 ignored issues – show Documentation Bug introduced 2019-06-27 08:32 UTC by Report Bug Copy Issue Report The doc comment `$default` at position `0` could not be parsed: Unknown type name '$default' at position 0 in $default. Loading history...
87			*/
88			protected $default=array();
89
90			/**
91			* @var array $index
92			*/
93			protected $index=array();
94
95			/**
96			* @var array $unique
97			*/
98			protected $unique=array();
99
100			/**
101			* @var $table
			0 ignored issues – show Documentation Bug introduced 2019-06-27 08:32 UTC by Report Bug Copy Issue Report The doc comment `$table` at position `0` could not be parsed: Unknown type name '$table' at position 0 in $table. Loading history...
102			*/
103			protected $table;
104
105			/**
106			* @return mixed\|void
107			*/
108			public function auto_increment()
109			{
110			if(count($this->auto_increment)==0){
111
112			if($this->getLastName()===false){
113			$this->name[]='id';
114			$this->setTypes('int',14);
115			}
116			$this->auto_increment[$this->getLastName()]=true;
117			$this->primaryKey();
118			}
119			}
120
121			/**
122			* @param $value
123			* @return $this
124			*/
125			public function comment($value)
126			{
127			$this->comment[$this->getLastName()]=$value;
128
129			return $this;
130			}
131
132			/**
133			* @param $name
134			* @return Types
135			*/
136			public function name($name)
137			{
138			if(in_array($name,$this->name)){
139			$this->setError('You have written the '.$name.' name more than 1.');
140			}
141
142			$this->name[] = $name;
143
144			return new Types($this);
145
146			}
147
148			/**
149			* @param $value
150			* @return $this\|mixed
151			*/
152			public function default($value)
153			{
154			$this->default[$this->getLastName()]=$value;
155
156			return $this;
157			}
158
159			/**
160			* @param $value
161			* @param bool $table
162			* @return $this
163			*/
164			public function collation($value,$table=false)
165			{
166			if($table===false)
167			{
168			$this->collation[$this->getLastName()]=$value;
169			}
170			else{
171			$this->collation['table']=$value;
172			}
173
174			return $this;
			0 ignored issues – show Bug Best Practice introduced 2019-06-27 08:32 UTC by Report Bug Copy Issue Report The expression `return $this` returns the type `Migratio\GrammarStructure\Mysql\Wizard\Wizard` which is incompatible with the return type mandated by `Migratio\Contract\WizardContract::collation()` of `Migratio\Contract\TablePropertiesContract`. In the issue above, the returned value is violating the contract defined by the mentioned interface. Let's take a look at an example: interface HasName { /** @return string / public function getName(); } class Name { public $name; } class User implements HasName { /* @return string\|Name */ public function getName() { return new Name('foo'); // This is a violation of the ``HasName`` interface // which only allows a string value to be returned. } } Loading history...
175			}
176
177			/**
178			* @param null $name
			0 ignored issues – show Documentation Bug introduced 2019-05-05 19:56 UTC by Report Bug Copy Issue Report Are you sure the doc-type for parameter `$name` is correct as it would always require `null` to be passed? Loading history...
179			* @param null $value
			0 ignored issues – show Documentation Bug introduced 2019-06-27 08:32 UTC by Report Bug Copy Issue Report Are you sure the doc-type for parameter `$value` is correct as it would always require `null` to be passed? Loading history...
180			* @param bool $key
181			* @return $this\|WizardContract
182			*/
183			public function index($name=null,$value=null,$key=false)
184			{
185			$name = ($name===null) ? $this->getLastName() : $name;
			0 ignored issues – show introduced 2019-06-27 08:32 UTC by Report Bug Copy Issue Report The condition `$name === null` is always `true`. Loading history...
186			$value = ($value===null) ? $name : $value;
			0 ignored issues – show introduced 2019-06-27 08:32 UTC by Report Bug Copy Issue Report The condition `$value === null` is always `true`. Loading history...
187
188			if($key===false){
189			$this->index[$this->getLastName()]=['name'=>$name,'value'=>$value];
190			}
191
192			if($key===true){
193			$this->index['indexes'][]=['name'=>$name,'value'=>$value];
194			}
195
196			return $this;
197			}
198
199			/**
200			* @param $index_name
201			* @param array $indexes
202			* @return void
203			*/
204			public function indexes($index_name,$indexes=array())
205			{
206			$this->index[$index_name] = $indexes;
207			}
208
209			/**
210			* @param bool $null
211			* @return $this
212			*/
213			public function nullable($null=true)
214			{
215			$this->nullable[$this->getLastName()]=$null;
216
217			return $this;
218			}
219
220			/**
221			* @return $this\|WizardContract
222			*/
223			public function primaryKey()
224			{
225			$this->primaryKey[$this->getLastName()]=true;
226
227			return $this;
228			}
229
230			/**
231			* @return TablePropertiesContract
232			*/
233			public function table()
234			{
235			return new TableProperties($this);
236			}
237
238			/**
239			* @param $name
240			* @param null $value
			0 ignored issues – show Documentation Bug introduced 2019-06-27 08:32 UTC by Report Bug Copy Issue Report Are you sure the doc-type for parameter `$value` is correct as it would always require `null` to be passed? Loading history...
241			* @return $this\|mixed
242			*/
243			public function unique($name=null,$value=null)
244			{
245			$name = ($name===null) ? $this->getLastName() : $name;
246			$value = ($value===null) ? $name : $value;
			0 ignored issues – show introduced 2019-06-27 08:32 UTC by Report Bug Copy Issue Report The condition `$value === null` is always `true`. Loading history...
247
248			$this->unique[$this->getLastName()] = ['name'=>$name,'value'=>$value];
249
250			return $this;
251			}
252
253			/**
254			* @param $name
255			* @param null $value
			0 ignored issues – show Documentation Bug introduced 2019-08-19 13:58 UTC by Report Bug Copy Issue Report Are you sure the doc-type for parameter `$value` is correct as it would always require `null` to be passed? Loading history...
256			* @return $this\|mixed
257			*/
258			public function uniques($unique_name,$uniques=array())
259			{
260			$this->unique[$unique_name] = $uniques;
261			}
262			}
263
264

php-resta / resta-core

Push — master ( 058fd4...43367b )

Wizard::uniques() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like