Eccube\ServiceProvider\ComposerServiceProvider

Complexity

Total Complexity

4

Size/Duplication

Total Lines	21
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes	2
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	register()	0	15	4

<?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\ServiceProvider;

use Eccube\Service\Composer\ComposerApiService;
use Eccube\Service\Composer\ComposerProcessService;
use Pimple\Container;
use Pimple\ServiceProviderInterface;

/**
 * Class ComposerServiceProvider
 * @package Eccube\ServiceProvider
 */
class ComposerServiceProvider implements ServiceProviderInterface
{
    /**
     * @param Container $app
     */
    public function register(Container $app)
    {
        $app['eccube.service.composer'] = function () use ($app) {
            /**@var \Eccube\Service\SystemService $systemService */
            $systemService = $app['eccube.service.system'];
            $composerMemory = $app['config']['composer_memory_limit'];
            $memoryLimit = $systemService->getMemoryLimit();
            // Check memory and can set memory for composer api
            if ($systemService->canSetMemoryLimit($composerMemory.'M') || $memoryLimit == -1 || $memoryLimit >= $composerMemory) {
                return new ComposerApiService($app['config']);
            } else {
                return new ComposerProcessService($app['config'], $app['orm.em'], $systemService->getPHP());

It seems like $systemService->getPHP() targeting Eccube\Service\SystemService::getPHP() can also be of type false; however, Eccube\Service\Composer\ComposerProcessService::__construct() does only seem to accept string, did you maybe forget to handle an error condition?

            }
        };
    }
}