Eccube\Service\Composer\ComposerProcessService

Complexity

Total Complexity

31

Size/Duplication

Total Lines	222
Duplicated Lines	14.41 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	2
Bugs	0	Features	0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	execRequire()	15	15	2
A	execRemove()	17	17	2
A	runCommand()	0	7	1
A	setWorkingDir()	0	4	1
C	init()	0	25	7
A	setupComposer()	0	15	3
B	getCliMemoryLimit()	0	28	5
A	isPhpCommandLine()	0	11	4
A	__construct()	0	6	1
B	isSetCliMemoryLimit()	0	26	5

<?php
/*
 * This file is part of EC-CUBE
 *
 * Copyright(c) 2000-2015 LOCKON CO.,LTD. All Rights Reserved.
 *
 * http://www.lockon.co.jp/
 *
 * This program is free software; you can redistribute it and/or
 * modify it under the terms of the GNU General Public License
 * as published by the Free Software Foundation; either version 2
 * of the License, or (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License
 * along with this program; if not, write to the Free Software
 * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111-1307, USA.
 */
namespace Eccube\Service\Composer;

use Doctrine\ORM\EntityManagerInterface;
use Eccube\Annotation\Service;

/**
 * Class ComposerProcessService
 * @package Eccube\Service\Composer
 * @Service
 */
class ComposerProcessService implements ComposerServiceInterface
{
    /**
     * @var array
     */
    protected $appConfig;

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

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

    public function __construct($appConfig, $entityManager, $pathPHP)

Missing function doc comment

    {
        $this->appConfig = $appConfig;
        $this->entityManager = $entityManager;
        $this->pathPHP = $pathPHP;
    }

    /**
     * This function to install a plugin by composer require
     *
     * @param string $packageName format foo/bar or foo/bar:1.0.0 or "foo/bar 1.0.0"
     * @return bool
     */
    public function execRequire($packageName)

This method seems to be duplicated in your project.

    {
        set_time_limit(0);
        if(false === $this->init()){
            return false;
        }
        // Build command
        $command = $this->pathPHP.' '.$this->composerFile.' require '.$packageName;
        $command .= ' --prefer-dist --no-progress --no-suggest --no-scripts --ignore-platform-reqs --profile --no-ansi --no-interaction -d ';
        $command .= $this->workingDir.' 2>&1';
        log_info($command);
        $this->runCommand($command);

        return true;
    }

    /**
     * This function to remove a plugin by composer remove
     * Note: Remove with dependency, if not, please add " --no-update-with-dependencies"
     *
     * @param string $packageName format foo/bar or foo/bar:1.0.0 or "foo/bar 1.0.0"
     * @return bool
     */
    public function execRemove($packageName)

This method seems to be duplicated in your project.

    {
        set_time_limit(0);
        if(false === $this->init()){
            return false;
        }
        // Build command
        $command = $this->pathPHP.' '.$this->composerFile.' remove '.$packageName;
        $command .= ' --no-progress --no-scripts --ignore-platform-reqs --profile --no-ansi --no-interaction --no-update-with-dependencies -d ';
        $command .= $this->workingDir.' 2>&1';
        log_info($command);

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

        return true;
    }

    /**
     * Run command
     *
     * @param string $command
     * @return void
     */
    public function runCommand($command)
    {
        // Execute command
        $output = array();
        exec($command, $output);
        log_info(PHP_EOL.implode(PHP_EOL, $output).PHP_EOL);
    }

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

    /**
     * Set init
     */
    private function init()
    {
        if (!$this->isPhpCommandLine()) {
            return false;
        }

        if (!$this->isSetCliMemoryLimit()) {
            $composerMemory = $this->appConfig['composer_memory_limit'];
            if ($this->getCliMemoryLimit() < $composerMemory && $this->getCliMemoryLimit() != -1) {
                return false;
            }
        }

        $em = $this->entityManager;
        if ($em->getConnection()->isTransactionActive()) {
            $em->getConnection()->commit();
            $em->getConnection()->beginTransaction();
        }

        @ini_set('memory_limit', '1536M');

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

        // Config for some environment
        putenv('COMPOSER_HOME='.$this->appConfig['plugin_realdir'].'/.composer');
        $this->workingDir = $this->workingDir ? $this->workingDir : $this->appConfig['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
     */
    public function getCliMemoryLimit(){

Declare public methods first, then protected ones and finally private ones

        $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') {

Blank line found at start of control structure

                return $memo;
            } else {
                if ($exp[1] == 'GB') {

Blank line found at start of control structure

                    return $memo * 1024;
                } else {

Blank line found at start of control structure

                    return 0;
                }
            }
        }

        return 0;
    }

    /**
     * Check to set new value grep "memory_limit"
     * @return bool
     */
    public 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) {

Blank line found at start of control structure

                    return true;
                }
            }
        }

        return false;
    }

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

        return false;
    }
}