WizardAlterGroup::dropWizardAlterInstance() - Code Metrics - Inspection of "database migration drop column fix" - php-resta/resta-core - Measure and Improve Code Quality continuously with Scrutinizer

Test Setup Failed

Push — master ( 7d63a7...ae062b )

by Php Easy Api

created 2019-07-19 12:09 UTC

WizardAlterGroup::dropWizardAlterInstance() A

↳ Parent: WizardAlterGroup

Complexity

Conditions	1
Paths	1

Size

Total Lines	5
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php

namespace Migratio\GrammarStructure\Mysql\Wizard;

use Migratio\Contract\NameContract;
use Migratio\Contract\WizardAlterContract;
use Migratio\Contract\WizardAlterGroupContract;

class WizardAlterGroup extends Wizard implements WizardAlterGroupContract
{
    /**
     * add column
     * 
     * @return WizardAlterContract
     */
    public function addColumn()
    {
        return $this->getWizardAlterInstance(__FUNCTION__);
    }

    /**
     * drop column
     * 
     * @return NameContract
     */
    public function dropColumn()
    {
        return $this->dropWizardAlterInstance(__FUNCTION__);

    }

    /**
     * change column
     * 
     * @return WizardAlterContract
     */
    public function change()
    {
        return $this->getWizardAlterInstance(__FUNCTION__);
    }

    /**
     * get wizard alter instance
     * 
     * @param $group
     * @return WizardAlter
     */
    private function getWizardAlterInstance($group)
    {
        $this->setAlterType('group',$group);

        return new WizardAlter($this);
    }

    /**
     * @param $group
     * @return $this
     */
    private function dropWizardAlterInstance($group)
    {
        $this->setAlterType('group',$group);

        return $this;
    }
}



1			<?php
2
3			namespace Migratio\GrammarStructure\Mysql\Wizard;
4
5			use Migratio\Contract\NameContract;
6			use Migratio\Contract\WizardAlterContract;
7			use Migratio\Contract\WizardAlterGroupContract;
8
9			class WizardAlterGroup extends Wizard implements WizardAlterGroupContract
10			{
11			/**
12			* add column
13			*
14			* @return WizardAlterContract
15			*/
16			public function addColumn()
17			{
18			return $this->getWizardAlterInstance(__FUNCTION__);
19			}
20
21			/**
22			* drop column
23			*
24			* @return NameContract
25			*/
26			public function dropColumn()
27			{
28			return $this->dropWizardAlterInstance(__FUNCTION__);
			0 ignored issues – show Bug Best Practice introduced 2019-07-19 12:13 UTC by Report Bug Copy Issue Report The expression `return $this->dropWizard...rInstance(__FUNCTION__)` returns the type `Migratio\GrammarStructur...Wizard\WizardAlterGroup` which is incompatible with the documented return type `Migratio\Contract\NameContract`. Loading history...
29			}
30
31			/**
32			* change column
33			*
34			* @return WizardAlterContract
35			*/
36			public function change()
37			{
38			return $this->getWizardAlterInstance(__FUNCTION__);
39			}
40
41			/**
42			* get wizard alter instance
43			*
44			* @param $group
45			* @return WizardAlter
46			*/
47			private function getWizardAlterInstance($group)
48			{
49			$this->setAlterType('group',$group);
50
51			return new WizardAlter($this);
52			}
53
54			/**
55			* @param $group
56			* @return $this
57			*/
58			private function dropWizardAlterInstance($group)
59			{
60			$this->setAlterType('group',$group);
61
62			return $this;
63			}
64			}
65
66

php-resta / resta-core

Push — master ( 7d63a7...ae062b )

WizardAlterGroup::dropWizardAlterInstance() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like