InstallContext - Code Metrics - Inspection of "Fixes for #452" - cross-solution/YAWIK - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — develop (#453)

by ANTHONIUS

created 2018-01-21 05:12 UTC

InstallContext A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	97
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	2

Importance

Changes	1
Bugs	1	Features	0

Metric	Value
wmc	12
lcom	2
cbo	2
dl	0
loc	97
rs	10
c	1
b	1
f	0

8 Methods

Rating	Name	Size	Complexity
A	__construct()	6	1
A	iHaveInstallModuleActivated()	17	3
A	iGoToInstallPage()	6	1
A	tearDown()	4	1
A	afterScenario()	4	1
A	setUp()	4	1
A	restoreConfig()	14	3
A	iFillActiveConnectionString()	6	1

<?php

/**
 * YAWIK
 *
 * @filesource
 * @license MIT
 * @copyright  2013 - 2017 Cross Solution <http://cross-solution.de>
 */

namespace Yawik\Behat;

use Behat\Behat\Context\Context;
use Behat\Behat\Hook\Scope\AfterScenarioScope;

/**
 * Class InstallContext
 *
 * @package Yawik\Behat
 * @author Anthonius Munthi <[email protected]>
 */
class InstallContext implements Context
{
    use CommonContextTrait;

    static private $configFile;

    static private $yawikGlobalConfig;

    static private $yawikBackupConfig;

    public function __construct()
    {
        static::$configFile = getcwd() . '/config/autoload/install.module.php';
class YourClass
{
    private static $someVariable;

    public static function getSomeVariable()
    {
        return static::$someVariable;
    }
}
        static::$yawikGlobalConfig = getcwd() . '/config/autoload/yawik.config.global.php';
class YourClass
{
    private static $someVariable;

    public static function getSomeVariable()
    {
        return static::$someVariable;
    }
}
        static::$yawikBackupConfig = str_replace('yawik.config.global.php', 'yawik.backup', static::$yawikGlobalConfig);
class YourClass
{
    private static $someVariable;

    public static function getSomeVariable()
    {
        return static::$someVariable;
    }
}
    }

    /**
     * @Given I have install module activated
     */
    public function iHaveInstallModuleActivated()
    {
        $target = static::$configFile;
class YourClass
{
    private static $someVariable;

    public static function getSomeVariable()
    {
        return static::$someVariable;
    }
}
        if(!file_exists($target)){
            $source = __DIR__.'/../resources/install.module.php';
            copy($source,$target);
            chmod($target,0777);
        }

        // backup existing file
        $yawikBackupConfig = static::$yawikBackupConfig;
class YourClass
{
    private static $someVariable;

    public static function getSomeVariable()
    {
        return static::$someVariable;
    }
}
        $yawikGlobalConfig = static::$yawikGlobalConfig;
class YourClass
{
    private static $someVariable;

    public static function getSomeVariable()
    {
        return static::$someVariable;
    }
}

        if(is_file($yawikGlobalConfig)){
            rename($yawikGlobalConfig,$yawikBackupConfig);
        }
    }

    /**
     * @Given I go to the install page
     */
    public function iGoToInstallPage()
    {
        $url = $this->generateUrl('/');
        $url = str_replace('/en','',$url);
        $this->visit($url);
    }

    /**
     * @AfterSuite
     */
    static public function tearDown()

    {
        static::restoreConfig();
    }

    /**
     * @AfterScenario
     */
    public function afterScenario()
    {
        static::restoreConfig();
    }

    /**
     * @BeforeSuite
     */
    static public function setUp()

    {
        static::restoreConfig();
    }

    static public function restoreConfig()

    {
        if(is_file($file = static::$configFile)){
class YourClass
{
    private static $someVariable;

    public static function getSomeVariable()
    {
        return static::$someVariable;
    }
}
            unlink($file);
        }

        // restore backup
        $yawikBackupConfig = static::$yawikBackupConfig;
class YourClass
{
    private static $someVariable;

    public static function getSomeVariable()
    {
        return static::$someVariable;
    }
}
        $yawikGlobalConfig = static::$yawikGlobalConfig;
class YourClass
{
    private static $someVariable;

    public static function getSomeVariable()
    {
        return static::$someVariable;
    }
}

        if(is_file($yawikBackupConfig)){
            rename($yawikBackupConfig,$yawikGlobalConfig);
        }
    }

    /**
     * @Given I fill database connection with an active connection
     */
    public function iFillActiveConnectionString()
    {
        $config = $this->getService('config');
        $connection = $config['doctrine']['connection']['odm_default']['connectionString'];
        $this->minkContext->fillField('db_conn',$connection);
    }
}


1			<?php
2
3			/**
4			* YAWIK
5			*
6			* @filesource
7			* @license MIT
8			* @copyright 2013 - 2017 Cross Solution <http://cross-solution.de>
9			*/
10
11			namespace Yawik\Behat;
12
13			use Behat\Behat\Context\Context;
14			use Behat\Behat\Hook\Scope\AfterScenarioScope;
15
16			/**
17			* Class InstallContext
18			*
19			* @package Yawik\Behat
20			* @author Anthonius Munthi <[email protected]>
21			*/
22			class InstallContext implements Context
23			{
24			use CommonContextTrait;
25
26			static private $configFile;
27
28			static private $yawikGlobalConfig;
29
30			static private $yawikBackupConfig;
31
32			public function __construct()
33			{
34			static::$configFile = getcwd() . '/config/autoload/install.module.php';
			0 ignored issues – show Bug introduced 2018-01-21 05:19 UTC by Report Bug Copy Issue Report Since `$configFile` is declared private, accessing it with `static` will lead to errors in possible sub-classes; consider using `self`, or increasing the visibility of `$configFile` to at least protected. Let’s assume you have a class which uses late-static binding: class YourClass { private static $someVariable; public static function getSomeVariable() { return static::$someVariable; } } The code above will run fine in your PHP runtime. However, if you now create a sub-class and call the `getSomeVariable()` on that sub-class, you will receive a runtime error: class YourSubClass extends YourClass { } YourSubClass::getSomeVariable(); // Will cause an access error. In the case above, it makes sense to update `SomeClass` to use `self` instead: class SomeClass { private static $someVariable; public static function getSomeVariable() { return self::$someVariable; // self works fine with private. } } Loading history...
35			static::$yawikGlobalConfig = getcwd() . '/config/autoload/yawik.config.global.php';
			0 ignored issues – show Bug introduced 2018-01-21 05:19 UTC by Report Bug Copy Issue Report Since `$yawikGlobalConfig` is declared private, accessing it with `static` will lead to errors in possible sub-classes; consider using `self`, or increasing the visibility of `$yawikGlobalConfig` to at least protected. Let’s assume you have a class which uses late-static binding: class YourClass { private static $someVariable; public static function getSomeVariable() { return static::$someVariable; } } The code above will run fine in your PHP runtime. However, if you now create a sub-class and call the `getSomeVariable()` on that sub-class, you will receive a runtime error: class YourSubClass extends YourClass { } YourSubClass::getSomeVariable(); // Will cause an access error. In the case above, it makes sense to update `SomeClass` to use `self` instead: class SomeClass { private static $someVariable; public static function getSomeVariable() { return self::$someVariable; // self works fine with private. } } Loading history...
36			static::$yawikBackupConfig = str_replace('yawik.config.global.php', 'yawik.backup', static::$yawikGlobalConfig);
			0 ignored issues – show Bug introduced 2018-01-21 05:19 UTC by Report Bug Copy Issue Report Since `$yawikBackupConfig` is declared private, accessing it with `static` will lead to errors in possible sub-classes; consider using `self`, or increasing the visibility of `$yawikBackupConfig` to at least protected. Let’s assume you have a class which uses late-static binding: class YourClass { private static $someVariable; public static function getSomeVariable() { return static::$someVariable; } } The code above will run fine in your PHP runtime. However, if you now create a sub-class and call the `getSomeVariable()` on that sub-class, you will receive a runtime error: class YourSubClass extends YourClass { } YourSubClass::getSomeVariable(); // Will cause an access error. In the case above, it makes sense to update `SomeClass` to use `self` instead: class SomeClass { private static $someVariable; public static function getSomeVariable() { return self::$someVariable; // self works fine with private. } } Loading history... Bug introduced 2018-01-21 05:19 UTC by Report Bug Copy Issue Report Since `$yawikGlobalConfig` is declared private, accessing it with `static` will lead to errors in possible sub-classes; consider using `self`, or increasing the visibility of `$yawikGlobalConfig` to at least protected. Let’s assume you have a class which uses late-static binding: class YourClass { private static $someVariable; public static function getSomeVariable() { return static::$someVariable; } } The code above will run fine in your PHP runtime. However, if you now create a sub-class and call the `getSomeVariable()` on that sub-class, you will receive a runtime error: class YourSubClass extends YourClass { } YourSubClass::getSomeVariable(); // Will cause an access error. In the case above, it makes sense to update `SomeClass` to use `self` instead: class SomeClass { private static $someVariable; public static function getSomeVariable() { return self::$someVariable; // self works fine with private. } } Loading history...
37			}
38
39			/**
40			* @Given I have install module activated
41			*/
42			public function iHaveInstallModuleActivated()
43			{
44			$target = static::$configFile;
			0 ignored issues – show Bug introduced 2018-01-21 05:19 UTC by Report Bug Copy Issue Report Since `$configFile` is declared private, accessing it with `static` will lead to errors in possible sub-classes; consider using `self`, or increasing the visibility of `$configFile` to at least protected. Let’s assume you have a class which uses late-static binding: class YourClass { private static $someVariable; public static function getSomeVariable() { return static::$someVariable; } } The code above will run fine in your PHP runtime. However, if you now create a sub-class and call the `getSomeVariable()` on that sub-class, you will receive a runtime error: class YourSubClass extends YourClass { } YourSubClass::getSomeVariable(); // Will cause an access error. In the case above, it makes sense to update `SomeClass` to use `self` instead: class SomeClass { private static $someVariable; public static function getSomeVariable() { return self::$someVariable; // self works fine with private. } } Loading history...
45			if(!file_exists($target)){
46			$source = __DIR__.'/../resources/install.module.php';
47			copy($source,$target);
48			chmod($target,0777);
49			}
50
51			// backup existing file
52			$yawikBackupConfig = static::$yawikBackupConfig;
			0 ignored issues – show Bug introduced 2018-01-21 05:19 UTC by Report Bug Copy Issue Report Since `$yawikBackupConfig` is declared private, accessing it with `static` will lead to errors in possible sub-classes; consider using `self`, or increasing the visibility of `$yawikBackupConfig` to at least protected. Let’s assume you have a class which uses late-static binding: class YourClass { private static $someVariable; public static function getSomeVariable() { return static::$someVariable; } } The code above will run fine in your PHP runtime. However, if you now create a sub-class and call the `getSomeVariable()` on that sub-class, you will receive a runtime error: class YourSubClass extends YourClass { } YourSubClass::getSomeVariable(); // Will cause an access error. In the case above, it makes sense to update `SomeClass` to use `self` instead: class SomeClass { private static $someVariable; public static function getSomeVariable() { return self::$someVariable; // self works fine with private. } } Loading history...
53			$yawikGlobalConfig = static::$yawikGlobalConfig;
			0 ignored issues – show Bug introduced 2018-01-21 05:19 UTC by Report Bug Copy Issue Report Since `$yawikGlobalConfig` is declared private, accessing it with `static` will lead to errors in possible sub-classes; consider using `self`, or increasing the visibility of `$yawikGlobalConfig` to at least protected. Let’s assume you have a class which uses late-static binding: class YourClass { private static $someVariable; public static function getSomeVariable() { return static::$someVariable; } } The code above will run fine in your PHP runtime. However, if you now create a sub-class and call the `getSomeVariable()` on that sub-class, you will receive a runtime error: class YourSubClass extends YourClass { } YourSubClass::getSomeVariable(); // Will cause an access error. In the case above, it makes sense to update `SomeClass` to use `self` instead: class SomeClass { private static $someVariable; public static function getSomeVariable() { return self::$someVariable; // self works fine with private. } } Loading history...
54
55			if(is_file($yawikGlobalConfig)){
56			rename($yawikGlobalConfig,$yawikBackupConfig);
57			}
58			}
59
60			/**
61			* @Given I go to the install page
62			*/
63			public function iGoToInstallPage()
64			{
65			$url = $this->generateUrl('/');
66			$url = str_replace('/en','',$url);
67			$this->visit($url);
68			}
69
70			/**
71			* @AfterSuite
72			*/
73			static public function tearDown()
			0 ignored issues – show Coding Style introduced 2018-01-21 05:19 UTC by Report Bug Copy Issue Report As per PSR2, the `static` declaration should come after the visibility declaration. Loading history...
74			{
75			static::restoreConfig();
76			}
77
78			/**
79			* @AfterScenario
80			*/
81			public function afterScenario()
82			{
83			static::restoreConfig();
84			}
85
86			/**
87			* @BeforeSuite
88			*/
89			static public function setUp()
			0 ignored issues – show Coding Style introduced 2018-01-21 05:19 UTC by Report Bug Copy Issue Report As per PSR2, the `static` declaration should come after the visibility declaration. Loading history...
90			{
91			static::restoreConfig();
92			}
93
94			static public function restoreConfig()
			0 ignored issues – show Coding Style introduced 2018-01-21 05:19 UTC by Report Bug Copy Issue Report As per PSR2, the `static` declaration should come after the visibility declaration. Loading history...
95			{
96			if(is_file($file = static::$configFile)){
			0 ignored issues – show Bug introduced 2018-01-21 05:19 UTC by Report Bug Copy Issue Report Since `$configFile` is declared private, accessing it with `static` will lead to errors in possible sub-classes; consider using `self`, or increasing the visibility of `$configFile` to at least protected. Let’s assume you have a class which uses late-static binding: class YourClass { private static $someVariable; public static function getSomeVariable() { return static::$someVariable; } } The code above will run fine in your PHP runtime. However, if you now create a sub-class and call the `getSomeVariable()` on that sub-class, you will receive a runtime error: class YourSubClass extends YourClass { } YourSubClass::getSomeVariable(); // Will cause an access error. In the case above, it makes sense to update `SomeClass` to use `self` instead: class SomeClass { private static $someVariable; public static function getSomeVariable() { return self::$someVariable; // self works fine with private. } } Loading history...
97			unlink($file);
98			}
99
100			// restore backup
101			$yawikBackupConfig = static::$yawikBackupConfig;
			0 ignored issues – show Bug introduced 2018-01-21 05:19 UTC by Report Bug Copy Issue Report Since `$yawikBackupConfig` is declared private, accessing it with `static` will lead to errors in possible sub-classes; consider using `self`, or increasing the visibility of `$yawikBackupConfig` to at least protected. Let’s assume you have a class which uses late-static binding: class YourClass { private static $someVariable; public static function getSomeVariable() { return static::$someVariable; } } The code above will run fine in your PHP runtime. However, if you now create a sub-class and call the `getSomeVariable()` on that sub-class, you will receive a runtime error: class YourSubClass extends YourClass { } YourSubClass::getSomeVariable(); // Will cause an access error. In the case above, it makes sense to update `SomeClass` to use `self` instead: class SomeClass { private static $someVariable; public static function getSomeVariable() { return self::$someVariable; // self works fine with private. } } Loading history...
102			$yawikGlobalConfig = static::$yawikGlobalConfig;
			0 ignored issues – show Bug introduced 2018-01-21 05:19 UTC by Report Bug Copy Issue Report Since `$yawikGlobalConfig` is declared private, accessing it with `static` will lead to errors in possible sub-classes; consider using `self`, or increasing the visibility of `$yawikGlobalConfig` to at least protected. Let’s assume you have a class which uses late-static binding: class YourClass { private static $someVariable; public static function getSomeVariable() { return static::$someVariable; } } The code above will run fine in your PHP runtime. However, if you now create a sub-class and call the `getSomeVariable()` on that sub-class, you will receive a runtime error: class YourSubClass extends YourClass { } YourSubClass::getSomeVariable(); // Will cause an access error. In the case above, it makes sense to update `SomeClass` to use `self` instead: class SomeClass { private static $someVariable; public static function getSomeVariable() { return self::$someVariable; // self works fine with private. } } Loading history...
103
104			if(is_file($yawikBackupConfig)){
105			rename($yawikBackupConfig,$yawikGlobalConfig);
106			}
107			}
108
109			/**
110			* @Given I fill database connection with an active connection
111			*/
112			public function iFillActiveConnectionString()
113			{
114			$config = $this->getService('config');
115			$connection = $config['doctrine']['connection']['odm_default']['connectionString'];
116			$this->minkContext->fillField('db_conn',$connection);
117			}
118			}
119

cross-solution / YAWIK

Pull Request — develop (#453)

InstallContext A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

8 Methods

Duplication Side-by-Side

Filter issues like