DefaultEnv::unset() - Code Metrics - chrif/cocotte - Measure and Improve Code Quality continuously with Scrutinizer

DefaultEnv::unset() A
last analyzed 2021-06-13 18:51 UTC

↳ Parent: DefaultEnv

Complexity

Conditions	1
Paths	1

Size

Total Lines	18
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	1
eloc	13
c	1
b	0
f	0
nc	1
nop	1
dl	0
loc	18
rs	9.8333

<?php declare(strict_types=1);

namespace Cocotte\Shell;

use Assert\Assertion;

final class DefaultEnv implements Env
{

    public function put(string $name, string $value)
    {
        global $_SERVER;
function myFunction($a, $b) {
    // Do something
}
        Assertion::true(putenv($name.'='.$value), "Could not put env with name '$name' and value '$value'.");
        Assertion::same(
            $value,
            $retrieved = $this->get($name),
            "Failed asserting that value '$value' for env with name '$name' has been preserved when putting it. ".
            "Retrieved value was '$retrieved'."
        );
        /**
         * @see \Symfony\Component\Process\Process::getDefaultEnv
         */
        $_SERVER[$name] = $value;
    }

    public function get(string $name, $default = null): ?string
    {
        $value = getenv($name);
        if (false !== $value) {
            return $value;
        }

        return $default;
    }

    public function unset(string $name)
    {
        global $_SERVER;
function myFunction($a, $b) {
    // Do something
}
        Assertion::true(putenv($name), "Could not unset env with name '$name'.");
        Assertion::same(
            null,
            $retrieved = $this->get($name),
            "Failed asserting that env with name '$name' has been unset. ".
            "Retrieved value was '$retrieved'."
        );
        /**
         * @see \Symfony\Component\Process\Process::getDefaultEnv
         */
        unset($_SERVER[$name]);
        Assertion::keyNotExists(
            $_SERVER,
            $name,
            "Failed asserting that env with name '$name' has been unset in global _SERVER."
        );
    }

}


DefaultEnv::unset() A
last analyzed 2021-06-13 18:51 UTC

Complexity

Size

Duplication

Importance

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1			<?php declare(strict_types=1);
2
3			namespace Cocotte\Shell;
4
5			use Assert\Assertion;
6
7			final class DefaultEnv implements Env
8			{
9
10			public function put(string $name, string $value)
11			{
12			global $_SERVER;
			0 ignored issues – show Compatibility Best Practice introduced 2018-11-24 23:17 UTC by Report Bug Copy Issue Report Use of `global` functionality is not recommended; it makes your code harder to test, and less reusable. Instead of relying on `global` state, we recommend one of these alternatives: 1. Pass all data via parameters function myFunction($a, $b) { // Do something } 2. Create a class that maintains your state class MyClass { private $a; private $b; public function __construct($a, $b) { $this->a = $a; $this->b = $b; } public function myFunction() { // Do something } } Loading history...
13			Assertion::true(putenv($name.'='.$value), "Could not put env with name '$name' and value '$value'.");
14			Assertion::same(
15			$value,
16			$retrieved = $this->get($name),
17			"Failed asserting that value '$value' for env with name '$name' has been preserved when putting it. ".
18			"Retrieved value was '$retrieved'."
19			);
20			/**
21			* @see \Symfony\Component\Process\Process::getDefaultEnv
22			*/
23			$_SERVER[$name] = $value;
24			}
25
26			public function get(string $name, $default = null): ?string
27			{
28			$value = getenv($name);
29			if (false !== $value) {
30			return $value;
31			}
32
33			return $default;
34			}
35
36			public function unset(string $name)
37			{
38			global $_SERVER;
			0 ignored issues – show Compatibility Best Practice introduced 2018-11-24 23:17 UTC by Report Bug Copy Issue Report Use of `global` functionality is not recommended; it makes your code harder to test, and less reusable. Instead of relying on `global` state, we recommend one of these alternatives: 1. Pass all data via parameters function myFunction($a, $b) { // Do something } 2. Create a class that maintains your state class MyClass { private $a; private $b; public function __construct($a, $b) { $this->a = $a; $this->b = $b; } public function myFunction() { // Do something } } Loading history...
39			Assertion::true(putenv($name), "Could not unset env with name '$name'.");
40			Assertion::same(
41			null,
42			$retrieved = $this->get($name),
43			"Failed asserting that env with name '$name' has been unset. ".
44			"Retrieved value was '$retrieved'."
45			);
46			/**
47			* @see \Symfony\Component\Process\Process::getDefaultEnv
48			*/
49			unset($_SERVER[$name]);
50			Assertion::keyNotExists(
51			$_SERVER,
52			$name,
53			"Failed asserting that env with name '$name' has been unset in global _SERVER."
54			);
55			}
56
57			}
58

chrif / cocotte

DefaultEnv::unset() A last analyzed 2021-06-13 18:51 UTC

Complexity

Size

Duplication

Importance

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

Duplication Side-by-Side

Filter issues like

DefaultEnv::unset() A
last analyzed 2021-06-13 18:51 UTC