Binary::locateBinary() - Code Metrics - teqneers/PHP-Stream-Wrapper-for-Git - Measure and Improve Code Quality continuously with Scrutinizer

Binary::locateBinary() A
last analyzed 2018-03-19 14:46 UTC

↳ Parent: Binary

Complexity

Conditions	2
Paths	2

Size

Total Lines	7
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes

Metric	Value
dl	0
loc	7
ccs	0
cts	5
cp	0
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	5
nc	2
nop	0
crap	6

<?php
/*
 * Copyright (C) 2017 by TEQneers GmbH & Co. KG
 *
 * Permission is hereby granted, free of charge, to any person obtaining a copy
 * of this software and associated documentation files (the "Software"), to deal
 * in the Software without restriction, including without limitation the rights
 * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
 * copies of the Software, and to permit persons to whom the Software is
 * furnished to do so, subject to the following conditions:
 *
 * The above copyright notice and this permission notice shall be included in
 * all copies or substantial portions of the Software.
 *
 * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
 * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
 * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
 * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
 * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
 * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 * THE SOFTWARE.
 */

/**
 * Git Stream Wrapper for PHP
 *
 * @category   TQ
 * @package    TQ_VCS
 * @subpackage Git
 * @copyright  Copyright (C) 2018 by TEQneers GmbH & Co. KG
 */

namespace TQ\Git\Cli;

use TQ\Vcs\Cli\Binary as VcsBinary;
use TQ\Vcs\Cli\Call;

/**
 * Encapsulates access to th Git command line binary
 *
 * @author     Stefan Gehrig <gehrigteqneers.de>
 * @category   TQ
 * @package    TQ_VCS
 * @subpackage Git
 * @copyright  Copyright (C) 2018 by TEQneers GmbH & Co. KG
 */
class Binary extends VcsBinary {
    /**
     * Try to find the Git binary on the system
     *
     * @return  string
     */
    public static function locateBinary() {
        if (!self::isWindows()) {
            $result = Call::create('which git')->execute();
            return $result->getStdOut();
        }
        return '';
    }

    /**
     * Creates a Git binary interface
     *
     * If no path is given the class tries to find the correct
     * binary {@see locateBinary()}
     *
     * @param   string|null $path The path to the Git binary or NULL to auto-detect
     * @throws  \InvalidArgumentException   If no binary is found
     */
    public function __construct($path = null) {
        if (!$path) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
            $path = self::locateBinary();
        }
        parent::__construct($path);
    }
}



1		<?php
2		/*
3		* Copyright (C) 2017 by TEQneers GmbH & Co. KG
4		*
5		* Permission is hereby granted, free of charge, to any person obtaining a copy
6		* of this software and associated documentation files (the "Software"), to deal
7		* in the Software without restriction, including without limitation the rights
8		* to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
9		* copies of the Software, and to permit persons to whom the Software is
10		* furnished to do so, subject to the following conditions:
11		*
12		* The above copyright notice and this permission notice shall be included in
13		* all copies or substantial portions of the Software.
14		*
15		* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
16		* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
17		* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
18		* AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
19		* LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
20		* OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
21		* THE SOFTWARE.
22		*/
23
24		/**
25		* Git Stream Wrapper for PHP
26		*
27		* @category TQ
28		* @package TQ_VCS
29		* @subpackage Git
30		* @copyright Copyright (C) 2018 by TEQneers GmbH & Co. KG
31		*/
32
33		namespace TQ\Git\Cli;
34
35		use TQ\Vcs\Cli\Binary as VcsBinary;
36		use TQ\Vcs\Cli\Call;
37
38		/**
39		* Encapsulates access to th Git command line binary
40		*
41		* @author Stefan Gehrig <gehrigteqneers.de>
42		* @category TQ
43		* @package TQ_VCS
44		* @subpackage Git
45		* @copyright Copyright (C) 2018 by TEQneers GmbH & Co. KG
46		*/
47		class Binary extends VcsBinary {
48		/**
49		* Try to find the Git binary on the system
50		*
51		* @return string
52		*/
53		public static function locateBinary() {
54		if (!self::isWindows()) {
55		$result = Call::create('which git')->execute();
56		return $result->getStdOut();
57		}
58		return '';
59		}
60
61		/**
62		* Creates a Git binary interface
63		*
64		* If no path is given the class tries to find the correct
65		* binary {@see locateBinary()}
66		*
67		* @param string\|null $path The path to the Git binary or NULL to auto-detect
68		* @throws \InvalidArgumentException If no binary is found
69		*/
70	100	public function __construct($path = null) {
71	100	if (!$path) {
		0 ignored issues – show Bug Best Practice introduced 2015-11-26 15:24 UTC by Report Bug Copy Issue Report The expression `$path` of type `string\|null` is loosely compared to `false`; this is ambiguous if the string can be empty. You might want to explicitly use `=== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
72		$path = self::locateBinary();
73		}
74	100	parent::__construct($path);
75	100	}
76		}
77
78

teqneers / PHP-Stream-Wrapper-for-Git

Binary::locateBinary() A last analyzed 2018-03-19 14:46 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

Binary::locateBinary() A
last analyzed 2018-03-19 14:46 UTC