BinaryPython::detectCommand() - Code Metrics - hiqdev/hidev - Measure and Improve Code Quality continuously with Scrutinizer

BinaryPython::detectCommand() A
last analyzed 2020-04-04 17:27 UTC

↳ Parent: BinaryPython

Complexity

Conditions	2
Paths	2

Size

Total Lines	5
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes	1
Bugs	0	Features	1

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

<?php
/**
 * Automation tool mixed with code generator for easier continuous development
 *
 * @link      https://github.com/hiqdev/hidev
 * @package   hidev
 * @license   BSD-3-Clause
 * @copyright Copyright (c) 2015-2018, HiQDev (http://hiqdev.com/)
 */

namespace hidev\base;

use Yii;

class BinaryPython extends Binary
{
    /**
     * Detects how to run the binary.
     * Searches in this order:
     * 1. exexcutable in project's root directory
     * 2. XXX ??? vendor directory
     * 3. /home/user/.local/bin.
     * @param string $name
     * @return string path to the binary
     */
    public function detectPath($name)
    {
        $paths = [
            Yii::getAlias("@root/$name", false),
            Yii::getAlias("@root/env/bin/$name", false),
            "$_SERVER[HOME]/.local/bin/$name",
        ];
        foreach ($paths as $path) {
            if (file_exists($path)) {
                return $path;

            }
        }

        return parent::detectPath($name);
    }

    /**
     * Detect command execution string.
     * @param mixed $path
     * @return string
     */
    public function detectCommand($path)
    {
        $path = parent::detectCommand($path);

        return is_executable($path) ? $path : '/usr/bin/env python ' . $path;
    }

    public function install()
    {
        if ($this->installer) {
            passthru('/usr/bin/env wget ' . escapeshellarg($this->installer) . ' -O- | /usr/bin/env python', $exitcode);
        } elseif ($this->download) {
            $dest = Yii::getAlias('@root/' . $this->name, false);
            passthru('/usr/bin/env wget ' . escapeshellarg($this->download) . ' -O ' . $dest, $exitcode);
        } else {
            $args = ['pip', 'install'];
            if (!$_SERVER['VIRTUAL_ENV']) {
                $args[] = '--user';
            }
            $args[] = $this->package;
            $exitcode = passthru(implode(' ', $args));
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();
        }

        return $exitcode;
    }
}


1			<?php
2			/**
3			* Automation tool mixed with code generator for easier continuous development
4			*
5			* @link https://github.com/hiqdev/hidev
6			* @package hidev
7			* @license BSD-3-Clause
8			* @copyright Copyright (c) 2015-2018, HiQDev (http://hiqdev.com/)
9			*/
10
11			namespace hidev\base;
12
13			use Yii;
14
15			class BinaryPython extends Binary
16			{
17			/**
18			* Detects how to run the binary.
19			* Searches in this order:
20			* 1. exexcutable in project's root directory
21			* 2. XXX ??? vendor directory
22			* 3. /home/user/.local/bin.
23			* @param string $name
24			* @return string path to the binary
25			*/
26			public function detectPath($name)
27			{
28			$paths = [
29			Yii::getAlias("@root/$name", false),
30			Yii::getAlias("@root/env/bin/$name", false),
31			"$_SERVER[HOME]/.local/bin/$name",
32			];
33			foreach ($paths as $path) {
34			if (file_exists($path)) {
35			return $path;
			0 ignored issues – show Bug Best Practice introduced 2018-06-17 20:04 UTC by Report Bug Copy Issue Report The expression `return $path` also could return the type `boolean` which is incompatible with the documented return type `string`. Loading history...
36			}
37			}
38
39			return parent::detectPath($name);
40			}
41
42			/**
43			* Detect command execution string.
44			* @param mixed $path
45			* @return string
46			*/
47			public function detectCommand($path)
48			{
49			$path = parent::detectCommand($path);
50
51			return is_executable($path) ? $path : '/usr/bin/env python ' . $path;
52			}
53
54			public function install()
55			{
56			if ($this->installer) {
57			passthru('/usr/bin/env wget ' . escapeshellarg($this->installer) . ' -O- \| /usr/bin/env python', $exitcode);
58			} elseif ($this->download) {
59			$dest = Yii::getAlias('@root/' . $this->name, false);
60			passthru('/usr/bin/env wget ' . escapeshellarg($this->download) . ' -O ' . $dest, $exitcode);
61			} else {
62			$args = ['pip', 'install'];
63			if (!$_SERVER['VIRTUAL_ENV']) {
64			$args[] = '--user';
65			}
66			$args[] = $this->package;
67			$exitcode = passthru(implode(' ', $args));
			0 ignored issues – show Bug introduced 2018-06-17 20:04 UTC by Report Bug Copy Issue Report Are you sure the assignment to `$exitcode` is correct as `passthru(implode(' ', $args))` seems to always return `null`. This check looks for function or method calls that always return null and whose return value is assigned to a variable. class A { function getObject() { return null; } } $a = new A(); $object = $a->getObject(); The method `getObject()` can return nothing but null, so it makes no sense to assign that value to a variable. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
68			}
69
70			return $exitcode;
71			}
72			}
73

hiqdev / hidev

BinaryPython::detectCommand() A last analyzed 2020-04-04 17:27 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

BinaryPython::detectCommand() A
last analyzed 2020-04-04 17:27 UTC