ComposerProcessService::foreachRequires() - Code Metrics - Inspection of "ComposerServiceFactory" - kiy0taka/ec-cube - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Push — dev/store-tests ( 7e7716...b538ec )

by Kiyotaka

created 2018-09-09 08:13 UTC

ComposerProcessService::foreachRequires() A

↳ Parent: ComposerProcessService

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
nc	1
nop	5
dl	0
loc	4
rs	10
c	0
b	0
f	0

<?php

/*
 * This file is part of EC-CUBE
 *
 * Copyright(c) LOCKON CO.,LTD. All Rights Reserved.
 *
 * http://www.lockon.co.jp/
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Eccube\Service\Composer;

use Doctrine\ORM\EntityManagerInterface;
use Eccube\Common\EccubeConfig;
use Eccube\Entity\BaseInfo;
use Eccube\Exception\PluginException;
use Eccube\Service\SystemService;

/**
 * Class ComposerProcessService
 */
class ComposerProcessService implements ComposerServiceInterface
{
    /**
     * @var EccubeConfig config parameter
     */
    protected $eccubeConfig;

    /**
     * @var EntityManagerInterface
     */
    protected $entityManager;

    private $workingDir;
    private $composerFile;
    private $composerSetup;
    private $pathPHP;

    /**
     * ComposerProcessService constructor.
     *
     * @param EccubeConfig $eccubeConfig
     * @param EntityManagerInterface $entityManager
     * @param SystemService $systemService
     */
    public function __construct(EccubeConfig $eccubeConfig, EntityManagerInterface $entityManager, SystemService $systemService)
    {
        $this->eccubeConfig = $eccubeConfig;
        $this->entityManager = $entityManager;
        $this->pathPHP = $systemService->getPHP();
    }

    /**
     * This function to install a plugin by composer require
     *
     * @param string $packageName format "foo/bar foo/bar2:1.0.0"
     *
     * @throws PluginException
     */
    public function execRequire($packageName, $output = null)

    {
        set_time_limit(0);
        $this->init();

        // Build command
        $command = $this->pathPHP.' '.$this->composerFile.' require '.$packageName;
        $command .= ' --prefer-dist --no-progress --no-suggest --no-scripts --ignore-platform-reqs --update-with-dependencies --profile --no-ansi --no-interaction -d ';
        $command .= $this->workingDir.' 2>&1';
        log_info($command);
        $this->runCommand($command);
    }

    /**
     * This function to remove a plugin by composer remove
     *
     * @param string $packageName format "foo/bar foo/bar2"
     *
     * @throws PluginException
     */
    public function execRemove($packageName, $output = null)

    {
        set_time_limit(0);
        $this->init();

        // Build command
        $command = $this->pathPHP.' '.$this->composerFile.' remove '.$packageName;
        $command .= ' --no-progress --no-scripts --ignore-platform-reqs --profile --no-ansi --no-interaction -d ';
        $command .= $this->workingDir.' 2>&1';
        log_info($command);

        // Execute command
        $this->runCommand($command);
    }

    /**
     * Run command
     *
     * @throws PluginException
     *
     * @param string $command
     */
    public function runCommand($command)
    {
        $output = [];
        try {
            // Execute command
            $returnValue = -1;
            exec($command, $output, $returnValue);

            $outputString = implode(PHP_EOL, $output);
            if ($returnValue) {
0   == false // true
0   == null  // true
123 == false // false
123 == null  // false

// It is often better to use strict comparison
0 === false // false
0 === null  // false
                throw new PluginException($outputString);
            }
            log_info(PHP_EOL.$outputString.PHP_EOL);
        } catch (\Exception $exception) {
            throw new PluginException($exception->getMessage());
        }
    }

    /**
     * Set working dir
     *
     * @param string $workingDir
     */
    public function setWorkingDir($workingDir)
    {
        $this->workingDir = $workingDir;
    }

    /**
     * Set init
     *
     * @throws PluginException
     */
    private function init()
    {
        if (!$this->isPhpCommandLine()) {
            throw new PluginException('Php cli not found.');
        }

        $composerMemory = $this->eccubeConfig['eccube_composer_memory_limit'];
        if (!$this->isSetCliMemoryLimit()) {
            $cliMemoryLimit = $this->getCliMemoryLimit();
            if ($cliMemoryLimit < $composerMemory && $cliMemoryLimit != -1) {
                throw new PluginException('Not enough memory limit.');
            }
        }

        /**
         * Mysql lock in transaction
         *
         * @see https://dev.mysql.com/doc/refman/5.7/en/lock-tables.html
         *
         * @var EntityManagerInterface
         */
        $em = $this->entityManager;
        if ($em->getConnection()->isTransactionActive()) {
            $em->getConnection()->commit();
            $em->getConnection()->beginTransaction();
        }

        @ini_set('memory_limit', $composerMemory.'M');
// For example instead of
@mkdir($dir);

// Better use
if (@mkdir($dir) === false) {
    throw new \RuntimeException('The directory '.$dir.' could not be created.');
}
        // Config for some environment
        putenv('COMPOSER_HOME='.$this->eccubeConfig['plugin_realdir'].'/.composer');
        $this->workingDir = $this->workingDir ? $this->workingDir : $this->eccubeConfig['root_dir'];
        $this->setupComposer();
    }

    /**
     * Check composer file and setup it
     */
    private function setupComposer()
    {
        $this->composerFile = $this->workingDir.'/composer.phar';
        $this->composerSetup = $this->workingDir.'/composer-setup.php';
        if (!file_exists($this->composerFile)) {
            if (!file_exists($this->composerSetup)) {
                $result = copy('https://getcomposer.org/installer', $this->composerSetup);
                log_info($this->composerSetup.' : '.$result);
            }
            $command = $this->pathPHP.' '.$this->composerSetup;
            $this->runCommand($command);

            unlink($this->composerSetup);
        }
    }

    /**
     * Get grep memory_limit | Megabyte
     *
     * @return int|string
     */
    private function getCliMemoryLimit()
    {
        $grepMemory = exec($this->pathPHP.' -i | grep "memory_limit"');
        if ($grepMemory) {
            $grepMemory = explode('=>', $grepMemory);

            // -1 unlimited
            if (trim($grepMemory[2]) == -1) {
                return -1;
            }

            $exp = preg_split('#(?<=\d)(?=[a-z])#i', $grepMemory[2]);
            $memo = trim($exp[0]);
            if ($exp[1] == 'M') {
                return $memo;
            } else {
                if ($exp[1] == 'GB') {
                    return $memo * 1024;
                } else {
                    return 0;
                }
            }
        }

        return 0;
    }

    /**
     * Check to set new value grep "memory_limit"
     *
     * @return bool
     */
    private function isSetCliMemoryLimit()
    {
        $oldMemory = exec($this->pathPHP.' -i | grep "memory_limit"');
        $tmpMem = '1.5GB';
        if ($oldMemory) {
            $memory = explode('=>', $oldMemory);
            $originGrepMemmory = trim($memory[2]);

            if ($originGrepMemmory == $tmpMem) {
                $tmpMem = '1.49GB';
            }

            $newMemory = exec($this->pathPHP.' -d memory_limit='.$tmpMem.' -i | grep "memory_limit"');
            if ($newMemory) {
                $newMemory = explode('=>', $newMemory);
                $grepNewMemory = trim($newMemory[2]);
                if ($grepNewMemory != $originGrepMemmory) {
                    return true;
                }
            }
        }

        return false;
    }

    /**
     * Check php command line
     *
     * @return bool
     */
    private function isPhpCommandLine()
    {
        $php = exec('which php');
        if (null != $php) {
            if (strpos(strtolower($php), 'php') !== false) {
                return true;
            }
        }

        return false;
    }

    public function configureRepository(BaseInfo $BaseInfo)
    {
        // TODO: Implement configureRepository() method.
    }

    public function foreachRequires($packageName, $version, $callback, $typeFilter = null, $level = 0)
    {
        // TODO: Implement foreachRequires() method.
    }
}


1		<?php
2
3		/*
4		* This file is part of EC-CUBE
5		*
6		* Copyright(c) LOCKON CO.,LTD. All Rights Reserved.
7		*
8		* http://www.lockon.co.jp/
9		*
10		* For the full copyright and license information, please view the LICENSE
11		* file that was distributed with this source code.
12		*/
13
14		namespace Eccube\Service\Composer;
15
16		use Doctrine\ORM\EntityManagerInterface;
17		use Eccube\Common\EccubeConfig;
18		use Eccube\Entity\BaseInfo;
19		use Eccube\Exception\PluginException;
20		use Eccube\Service\SystemService;
21
22		/**
23		* Class ComposerProcessService
24		*/
25		class ComposerProcessService implements ComposerServiceInterface
26		{
27		/**
28		* @var EccubeConfig config parameter
29		*/
30		protected $eccubeConfig;
31
32		/**
33		* @var EntityManagerInterface
34		*/
35		protected $entityManager;
36
37		private $workingDir;
38		private $composerFile;
39		private $composerSetup;
40		private $pathPHP;
41
42		/**
43		* ComposerProcessService constructor.
44		*
45		* @param EccubeConfig $eccubeConfig
46		* @param EntityManagerInterface $entityManager
47		* @param SystemService $systemService
48		*/
49		public function __construct(EccubeConfig $eccubeConfig, EntityManagerInterface $entityManager, SystemService $systemService)
50		{
51		$this->eccubeConfig = $eccubeConfig;
52		$this->entityManager = $entityManager;
53		$this->pathPHP = $systemService->getPHP();
54		}
55
56		/**
57		* This function to install a plugin by composer require
58		*
59		* @param string $packageName format "foo/bar foo/bar2:1.0.0"
60		*
61		* @throws PluginException
62		*/
63	View Code Duplication	public function execRequire($packageName, $output = null)
		0 ignored issues – show Duplication introduced 2018-09-09 07:16 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
64		{
65		set_time_limit(0);
66		$this->init();
67
68		// Build command
69		$command = $this->pathPHP.' '.$this->composerFile.' require '.$packageName;
70		$command .= ' --prefer-dist --no-progress --no-suggest --no-scripts --ignore-platform-reqs --update-with-dependencies --profile --no-ansi --no-interaction -d ';
71		$command .= $this->workingDir.' 2>&1';
72		log_info($command);
73		$this->runCommand($command);
74		}
75
76		/**
77		* This function to remove a plugin by composer remove
78		*
79		* @param string $packageName format "foo/bar foo/bar2"
80		*
81		* @throws PluginException
82		*/
83	View Code Duplication	public function execRemove($packageName, $output = null)
		0 ignored issues – show Duplication introduced 2018-09-09 07:16 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
84		{
85		set_time_limit(0);
86		$this->init();
87
88		// Build command
89		$command = $this->pathPHP.' '.$this->composerFile.' remove '.$packageName;
90		$command .= ' --no-progress --no-scripts --ignore-platform-reqs --profile --no-ansi --no-interaction -d ';
91		$command .= $this->workingDir.' 2>&1';
92		log_info($command);
93
94		// Execute command
95		$this->runCommand($command);
96		}
97
98		/**
99		* Run command
100		*
101		* @throws PluginException
102		*
103		* @param string $command
104		*/
105		public function runCommand($command)
106		{
107		$output = [];
108		try {
109		// Execute command
110		$returnValue = -1;
111		exec($command, $output, $returnValue);
112
113		$outputString = implode(PHP_EOL, $output);
114		if ($returnValue) {
		0 ignored issues – show Bug Best Practice introduced 2018-08-02 03:20 UTC by Report Bug Copy Issue Report The expression `$returnValue` of type `integer\|null` is loosely compared to `true`; this is ambiguous if the integer can be zero. 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 `integer` values, zero is a special case, in particular the following results might be unexpected: 0 == false // true 0 == null // true 123 == false // false 123 == null // false // It is often better to use strict comparison 0 === false // false 0 === null // false Loading history...
115		throw new PluginException($outputString);
116		}
117		log_info(PHP_EOL.$outputString.PHP_EOL);
118		} catch (\Exception $exception) {
119		throw new PluginException($exception->getMessage());
120		}
121		}
122
123		/**
124		* Set working dir
125		*
126		* @param string $workingDir
127		*/
128		public function setWorkingDir($workingDir)
129		{
130		$this->workingDir = $workingDir;
131		}
132
133		/**
134		* Set init
135		*
136		* @throws PluginException
137		*/
138		private function init()
139		{
140		if (!$this->isPhpCommandLine()) {
141		throw new PluginException('Php cli not found.');
142		}
143
144		$composerMemory = $this->eccubeConfig['eccube_composer_memory_limit'];
145		if (!$this->isSetCliMemoryLimit()) {
146		$cliMemoryLimit = $this->getCliMemoryLimit();
147		if ($cliMemoryLimit < $composerMemory && $cliMemoryLimit != -1) {
148		throw new PluginException('Not enough memory limit.');
149		}
150		}
151
152		/**
153		* Mysql lock in transaction
154		*
155		* @see https://dev.mysql.com/doc/refman/5.7/en/lock-tables.html
156		*
157		* @var EntityManagerInterface
158		*/
159		$em = $this->entityManager;
160		if ($em->getConnection()->isTransactionActive()) {
161		$em->getConnection()->commit();
162		$em->getConnection()->beginTransaction();
163		}
164
165		@ini_set('memory_limit', $composerMemory.'M');
		0 ignored issues – show Security Best Practice introduced 2018-08-02 03:20 UTC by Report Bug Copy Issue Report It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended. If you suppress an error, we recommend checking for the error condition explicitly: // For example instead of @mkdir($dir); // Better use if (@mkdir($dir) === false) { throw new \RuntimeException('The directory '.$dir.' could not be created.'); } Loading history...
166		// Config for some environment
167		putenv('COMPOSER_HOME='.$this->eccubeConfig['plugin_realdir'].'/.composer');
168		$this->workingDir = $this->workingDir ? $this->workingDir : $this->eccubeConfig['root_dir'];
169		$this->setupComposer();
170		}
171
172		/**
173		* Check composer file and setup it
174		*/
175		private function setupComposer()
176		{
177		$this->composerFile = $this->workingDir.'/composer.phar';
178		$this->composerSetup = $this->workingDir.'/composer-setup.php';
179		if (!file_exists($this->composerFile)) {
180		if (!file_exists($this->composerSetup)) {
181		$result = copy('https://getcomposer.org/installer', $this->composerSetup);
182		log_info($this->composerSetup.' : '.$result);
183		}
184		$command = $this->pathPHP.' '.$this->composerSetup;
185		$this->runCommand($command);
186
187		unlink($this->composerSetup);
188		}
189		}
190
191		/**
192		* Get grep memory_limit \| Megabyte
193		*
194		* @return int\|string
195		*/
196		private function getCliMemoryLimit()
197		{
198		$grepMemory = exec($this->pathPHP.' -i \| grep "memory_limit"');
199		if ($grepMemory) {
200		$grepMemory = explode('=>', $grepMemory);
201
202		// -1 unlimited
203		if (trim($grepMemory[2]) == -1) {
204		return -1;
205		}
206
207		$exp = preg_split('#(?<=\d)(?=[a-z])#i', $grepMemory[2]);
208		$memo = trim($exp[0]);
209		if ($exp[1] == 'M') {
210		return $memo;
211		} else {
212		if ($exp[1] == 'GB') {
213		return $memo * 1024;
214		} else {
215		return 0;
216		}
217		}
218		}
219
220		return 0;
221		}
222
223		/**
224		* Check to set new value grep "memory_limit"
225		*
226		* @return bool
227		*/
228		private function isSetCliMemoryLimit()
229		{
230		$oldMemory = exec($this->pathPHP.' -i \| grep "memory_limit"');
231		$tmpMem = '1.5GB';
232		if ($oldMemory) {
233		$memory = explode('=>', $oldMemory);
234		$originGrepMemmory = trim($memory[2]);
235
236		if ($originGrepMemmory == $tmpMem) {
237		$tmpMem = '1.49GB';
238		}
239
240		$newMemory = exec($this->pathPHP.' -d memory_limit='.$tmpMem.' -i \| grep "memory_limit"');
241		if ($newMemory) {
242		$newMemory = explode('=>', $newMemory);
243		$grepNewMemory = trim($newMemory[2]);
244		if ($grepNewMemory != $originGrepMemmory) {
245		return true;
246		}
247		}
248		}
249
250		return false;
251		}
252
253		/**
254		* Check php command line
255		*
256		* @return bool
257		*/
258		private function isPhpCommandLine()
259		{
260		$php = exec('which php');
261		if (null != $php) {
262		if (strpos(strtolower($php), 'php') !== false) {
263		return true;
264		}
265		}
266
267		return false;
268		}
269
270		public function configureRepository(BaseInfo $BaseInfo)
271		{
272		// TODO: Implement configureRepository() method.
273		}
274
275		public function foreachRequires($packageName, $version, $callback, $typeFilter = null, $level = 0)
276		{
277		// TODO: Implement foreachRequires() method.
278		}
279		}
280

kiy0taka / ec-cube

Push — dev/store-tests ( 7e7716...b538ec )

ComposerProcessService::foreachRequires() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like