FileUtility::locateLocalBinaryPath() - Code Metrics - Inspection of "[TASK][BREAKING] Cleanup variables naming." - sourcebroker/deployer-extended-database - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 9a0903...3da072 )

by Krystian

created 2020-02-15 16:33 UTC

FileUtility::locateLocalBinaryPath() A

↳ Parent: FileUtility

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php

namespace SourceBroker\DeployerExtendedDatabase\Utility;

use function Deployer\runLocally;

/**
 * Class FileUtility
 * @package SourceBroker\DeployerExtendedDatabase\Utility
 */
class FileUtility
{
    /**
     * @param $filename
     * @return string
     */
    public function normalizeFilename($filename)
    {
        return preg_replace('/[^a-zA-Z0-9_]+/', '', $filename);
    }

    /**
     * @param $folder
     * @return string
     */
    public function normalizeFolder($folder)
    {
        return rtrim($folder, '/') . '/';
    }

    /**
     * @param $name
     * @return string
     */
    function locateLocalBinaryPath($name)

    {
        $nameEscaped = escapeshellarg($name);
        // Try `command`, should cover all Bourne-like shells
        // Try `which`, should cover most other cases
        // Fallback to `type` command, if the rest fails
        $path = runLocally("command -v $nameEscaped || which $nameEscaped || type -p $nameEscaped");
        if ($path) {
            // Deal with issue when `type -p` outputs something like `type -ap` in some implementations
            return trim(str_replace("$name is", "", $path));
        }
        throw new \RuntimeException("Can't locate [$nameEscaped] on instance '" . get('default_stage') . "' - neither of [command|which|type] commands are available");
    }
}


1			<?php
2
3			namespace SourceBroker\DeployerExtendedDatabase\Utility;
4
5			use function Deployer\runLocally;
6
7			/**
8			* Class FileUtility
9			* @package SourceBroker\DeployerExtendedDatabase\Utility
10			*/
11			class FileUtility
12			{
13			/**
14			* @param $filename
15			* @return string
16			*/
17			public function normalizeFilename($filename)
18			{
19			return preg_replace('/[^a-zA-Z0-9_]+/', '', $filename);
20			}
21
22			/**
23			* @param $folder
24			* @return string
25			*/
26			public function normalizeFolder($folder)
27			{
28			return rtrim($folder, '/') . '/';
29			}
30
31			/**
32			* @param $name
33			* @return string
34			*/
35			function locateLocalBinaryPath($name)
			0 ignored issues – show Best Practice introduced 2020-02-15 16:35 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
36			{
37			$nameEscaped = escapeshellarg($name);
38			// Try `command`, should cover all Bourne-like shells
39			// Try `which`, should cover most other cases
40			// Fallback to `type` command, if the rest fails
41			$path = runLocally("command -v $nameEscaped \|\| which $nameEscaped \|\| type -p $nameEscaped");
42			if ($path) {
43			// Deal with issue when `type -p` outputs something like `type -ap` in some implementations
44			return trim(str_replace("$name is", "", $path));
45			}
46			throw new \RuntimeException("Can't locate [$nameEscaped] on instance '" . get('default_stage') . "' - neither of [command\|which\|type] commands are available");
47			}
48			}
49

sourcebroker / deployer-extended-database

Push — master ( 9a0903...3da072 )

FileUtility::locateLocalBinaryPath() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like