Eccube\Controller\Install\InstallController

Complexity

Total Complexity

97

Size/Duplication

Total Lines	903
Duplicated Lines	3.88 %

Coupling/Cohesion

Components	1
Dependencies	22

Test Coverage

Coverage

32.83%

Importance

Changes

0

30 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	isValid()	0	16	4
A	getSessionData()	0	4	1
A	index()	0	6	1
A	step1()	0	19	2
A	step2()	0	21	2
C	step3()	6	70	7
B	step4()	0	41	5
B	step5()	0	59	5
B	complete()	3	26	5
A	resetNatTimer()	0	7	1
F	checkModules()	0	45	20
A	setPDO()	0	15	2
A	dropTables()	14	14	1
B	getEntityManager()	0	27	1
A	createTables()	12	12	1
A	insert()	0	59	2
A	update()	0	61	3
A	getMigration()	0	17	1
A	doMigrate()	0	19	3
A	getProtectedDirs()	0	23	3
B	createConfigYamlFile()	0	57	7
A	addInstallStatus()	0	10	1
B	createDatabaseYamlFile()	0	52	7
B	createMailYamlFile()	0	24	2
B	createPathYamlFile()	0	26	2
A	sendAppData()	0	47	3
A	migration()	0	6	1
A	migration_plugin()	0	20	2
A	migration_end()	0	13	1

<?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\Controller\Install;

use Doctrine\DBAL\Migrations\Configuration\Configuration;
use Doctrine\DBAL\Migrations\Migration;
use Doctrine\DBAL\Migrations\MigrationException;
use Doctrine\ORM\EntityManager;
use Doctrine\ORM\Tools\SchemaTool;
use Eccube\Common\Constant;
use Eccube\InstallApplication;
use Eccube\Util\Str;
use Symfony\Component\Filesystem\Filesystem;
use Symfony\Component\Finder\Finder;
use Symfony\Component\Form\Form;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\Yaml\Yaml;

class InstallController

Missing class doc comment

{

    const MCRYPT = 'mcrypt';

    private $app;
    private $PDO;
    private $config_path;
    private $dist_path;
    private $cache_path;
    private $session_data;
    private $required_modules = array('pdo', 'phar', 'mbstring', 'zlib', 'ctype', 'session', 'JSON', 'xml', 'libxml', 'OpenSSL', 'zip', 'cURL', 'fileinfo');
    private $recommended_module = array('hash', self::MCRYPT);

    const SESSION_KEY = 'eccube.session.install';

    public function __construct()

Missing function doc comment

    {
        $this->config_path = __DIR__ . '/../../../../app/config/eccube';

Concat operator must not be surrounded by spaces

        $this->dist_path = __DIR__ . '/../../Resource/config';

Concat operator must not be surrounded by spaces

        $this->cache_path = __DIR__ . '/../../../../app/cache';

Concat operator must not be surrounded by spaces

    }

    private function isValid(Request $request, Form $form)
    {
        $session = $request->getSession();
        if ('POST' === $request->getMethod()) {
            $form->handleRequest($request);
            if ($form->isValid()) {
                $sessionData = $session->get(self::SESSION_KEY) ?: array();
                $formData = array_replace_recursive($sessionData, $form->getData());
                $session->set(self::SESSION_KEY, $formData);

                return true;
            }
        }

        return false;
    }

    private function getSessionData(Request $request)
    {
        return $this->session_data = $request->getSession()->get(self::SESSION_KEY);
    }

    // 最初からやり直す場合、SESSION情報をクリア
    public function index(InstallApplication $app, Request $request)

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

You must use "/**" style comments for a function comment

    {
        $request->getSession()->remove(self::SESSION_KEY);

        return $app->redirect($app->path('install_step1'));
    }

    // ようこそ
    public function step1(InstallApplication $app, Request $request)

You must use "/**" style comments for a function comment

    {
        $form = $app['form.factory']
            ->createBuilder('install_step1')
            ->getForm();
        $sessionData = $this->getSessionData($request);
        $form->setData($sessionData);

        if ($this->isValid($request, $form)) {
            return $app->redirect($app->path('install_step2'));
        }

        $this->checkModules($app);

        return $app['twig']->render('step1.twig', array(
                'form' => $form->createView(),
                'publicPath' => '..' . RELATIVE_PUBLIC_DIR_PATH . '/',

Concat operator must not be surrounded by spaces

        ));
    }

    // 権限チェック
    public function step2(InstallApplication $app, Request $request)

You must use "/**" style comments for a function comment

    {
        $this->getSessionData($request);

        $protectedDirs = $this->getProtectedDirs();

        // 権限がある場合, キャッシュディレクトリをクリア
        if (empty($protectedDirs)) {
            $finder = Finder::create()
                ->in($this->cache_path)
                ->directories()
                ->depth(0);
            $fs = new Filesystem();
            $fs->remove($finder);
        }

        return $app['twig']->render('step2.twig', array(
                'protectedDirs' => $protectedDirs,
                'publicPath' => '..' . RELATIVE_PUBLIC_DIR_PATH . '/',

Concat operator must not be surrounded by spaces

        ));
    }

    //    サイトの設定
    public function step3(InstallApplication $app, Request $request)

You must use "/**" style comments for a function comment

    {
        $form = $app['form.factory']
            ->createBuilder('install_step3')
            ->getForm();
        $sessionData = $this->getSessionData($request);

        if (empty($sessionData['shop_name'])) {

Blank line found at start of control structure

            $config_file = $this->config_path . '/config.yml';

Concat operator must not be surrounded by spaces

            $fs = new Filesystem();

            if ($fs->exists($config_file)) {
                // すでに登録されていた場合、登録データを表示
                $this->setPDO();
                $stmt = $this->PDO->query("SELECT shop_name, email01 FROM dtb_base_info WHERE id = 1;");

                foreach ($stmt as $row) {
                    $sessionData['shop_name'] = $row['shop_name'];
                    $sessionData['email'] = $row['email01'];
                }

                // セキュリティの設定
                $config_file = $this->config_path . '/path.yml';

Concat operator must not be surrounded by spaces

                $config = Yaml::parse(file_get_contents($config_file));
                $sessionData['admin_dir'] = $config['admin_route'];

                $config_file = $this->config_path . '/config.yml';

Concat operator must not be surrounded by spaces

                $config = Yaml::parse(file_get_contents($config_file));

                $allowHost = $config['admin_allow_host'];
                if (count($allowHost) > 0) {

This code seems to be duplicated across your project.

                    $sessionData['admin_allow_hosts'] = Str::convertLineFeed(implode("\n", $allowHost));
                }
                $sessionData['admin_force_ssl'] = (bool) $config['force_ssl'];

                // ロードバランサー、プロキシサーバ設定
                $sessionData['trusted_proxies_connection_only'] = (bool)$config['trusted_proxies_connection_only'];

As per coding-style, a cast statement should be followed by a single space.

                $trustedProxies = $config['trusted_proxies'];
                if (count($trustedProxies) > 0) {

This code seems to be duplicated across your project.

                    $sessionData['trusted_proxies'] = Str::convertLineFeed(implode("\n", $trustedProxies));
                }

                // メール設定
                $config_file = $this->config_path . '/mail.yml';

Concat operator must not be surrounded by spaces

                $config = Yaml::parse(file_get_contents($config_file));
                $mail = $config['mail'];
                $sessionData['mail_backend'] = $mail['transport'];
                $sessionData['smtp_host'] = $mail['host'];
                $sessionData['smtp_port'] = $mail['port'];
                $sessionData['smtp_username'] = $mail['username'];
                $sessionData['smtp_password'] = $mail['password'];
            } else {
                // 初期値にmailを設定
                $sessionData['mail_backend'] = 'mail';
            }
        }

        $form->setData($sessionData);
        if ($this->isValid($request, $form)) {
            $data = $form->getData();

$data is not used, you could remove the assignment.

            return $app->redirect($app->path('install_step4'));
        }

        return $app['twig']->render('step3.twig', array(
                'form' => $form->createView(),
                'publicPath' => '..' . RELATIVE_PUBLIC_DIR_PATH . '/',

Concat operator must not be surrounded by spaces

        ));
    }

    //    データベースの設定
    public function step4(InstallApplication $app, Request $request)

You must use "/**" style comments for a function comment

    {
        $form = $app['form.factory']
            ->createBuilder('install_step4')
            ->getForm();

        $sessionData = $this->getSessionData($request);

        if (empty($sessionData['database'])) {

Blank line found at start of control structure

            $config_file = $this->config_path . '/database.yml';

Concat operator must not be surrounded by spaces

            $fs = new Filesystem();

            if ($fs->exists($config_file)) {
                // すでに登録されていた場合、登録データを表示
                // データベース設定
                $config = Yaml::parse(file_get_contents($config_file));
                $database = $config['database'];
                $sessionData['database'] = $database['driver'];
                if ($database['driver'] != 'pdo_sqlite') {
                    $sessionData['database_host'] = $database['host'];
                    $sessionData['database_port'] = $database['port'];
                    $sessionData['database_name'] = $database['dbname'];
                    $sessionData['database_user'] = $database['user'];
                    $sessionData['database_password'] = $database['password'];
                }
            }
        }

        $form->setData($sessionData);

        if ($this->isValid($request, $form)) {

Blank line found at start of control structure

            return $app->redirect($app->path('install_step5'));
        }

        return $app['twig']->render('step4.twig', array(
                'form' => $form->createView(),
                'publicPath' => '..' . RELATIVE_PUBLIC_DIR_PATH . '/',

Concat operator must not be surrounded by spaces

        ));
    }

    //    データベースの初期化
    public function step5(InstallApplication $app, Request $request)

You must use "/**" style comments for a function comment

    {
        set_time_limit(0);
        $this->app = $app;
        $form = $app['form.factory']
            ->createBuilder('install_step5')
            ->getForm();
        $sessionData = $this->getSessionData($request);
        $form->setData($sessionData);

        if ($this->isValid($request, $form)) {

Blank line found at start of control structure

            $this
                ->createDatabaseYamlFile($sessionData)
                ->createMailYamlFile($sessionData)
                ->createPathYamlFile($sessionData, $request);

            if (!$form['no_update']->getData()) {
                set_time_limit(0);
                $this->createConfigYamlFile($sessionData);

                $this
                    ->setPDO()
                    ->dropTables()
                    ->createTables()
                    ->doMigrate()
                    ->insert();
            } else {
                // データベースを初期化しない場合、auth_magicは初期化しない
                $this->createConfigYamlFile($sessionData, false);

                $this
                    ->setPDO()
                    ->update();
            }


            if (isset($sessionData['agree']) && $sessionData['agree'] == '1') {
                $host = $request->getSchemeAndHttpHost();
                $basePath = $request->getBasePath();
                $params = array(
                    'http_url' => $host . $basePath,

Concat operator must not be surrounded by spaces

                    'shop_name' => $sessionData['shop_name'],
                );

                $this->sendAppData($params);
            }
            $this->addInstallStatus();

            $request->getSession()->remove(self::SESSION_KEY);

            return $app->redirect($app->path('install_complete'));
        }

        return $app['twig']->render('step5.twig', array(
                'form' => $form->createView(),
                'publicPath' => '..' . RELATIVE_PUBLIC_DIR_PATH . '/',

Concat operator must not be surrounded by spaces

        ));
    }

    //    インストール完了
    public function complete(InstallApplication $app, Request $request)

You must use "/**" style comments for a function comment

    {
        $config_yml = $this->config_path . '/config.yml';

Concat operator must not be surrounded by spaces

        $config = Yaml::parse(file_get_contents($config_yml));
        $config_path = $this->config_path . '/path.yml';

Concat operator must not be surrounded by spaces

        $path_yml = Yaml::parse(file_get_contents($config_path));

        $config = array_replace_recursive($path_yml, $config);


        if (isset($config['trusted_proxies_connection_only']) && !empty($config['trusted_proxies_connection_only'])) {
            Request::setTrustedProxies(array_merge(array($request->server->get('REMOTE_ADDR')), $config['trusted_proxies']));
        } elseif (isset($config['trusted_proxies']) && !empty($config['trusted_proxies'])) {

This code seems to be duplicated across your project.

            Request::setTrustedProxies($config['trusted_proxies']);
        }

        $host = $request->getSchemeAndHttpHost();
        $basePath = $request->getBasePath();

        $adminUrl = $host . $basePath . '/' . $config['admin_dir'];

Concat operator must not be surrounded by spaces

        return $app['twig']->render('complete.twig', array(
                'admin_url' => $adminUrl,
                'publicPath' => '..' . RELATIVE_PUBLIC_DIR_PATH . '/',

Concat operator must not be surrounded by spaces

        ));
    }

    private function resetNatTimer()
    {
        // NATの無通信タイマ対策（仮）
        echo str_repeat(' ', 4 * 1024);
        ob_flush();
        flush();
    }

    private function checkModules($app)

checkModules uses the super-global variable $_SERVER which is generally not recommended.

    {
        foreach ($this->required_modules as $module) {
            if (!extension_loaded($module)) {
                $app->addDanger('[必須] ' . $module . ' 拡張モジュールが有効になっていません。', 'install');

Concat operator must not be surrounded by spaces

            }
        }

        if (!extension_loaded('pdo_mysql') && !extension_loaded('pdo_pgsql')) {
            $app->addDanger('[必須] ' . 'pdo_pgsql又はpdo_mysql 拡張モジュールを有効にしてください。', 'install');

Concat operator must not be surrounded by spaces

        }

        foreach ($this->recommended_module as $module) {
            if (!extension_loaded($module)) {
                if ($module == self::MCRYPT && PHP_VERSION_ID >= 70100) {
                    //The mcrypt extension has been deprecated in PHP 7.1.x 
                    //http://php.net/manual/en/migration71.deprecated.php
                    continue;
                }
                $app->addInfo('[推奨] '.$module.' 拡張モジュールが有効になっていません。', 'install');
            }
        }

        if ('\\' === DIRECTORY_SEPARATOR) { // for Windows
            if (!extension_loaded('wincache')) {
                $app->addInfo('[推奨] WinCache 拡張モジュールが有効になっていません。', 'install');
            }
        } else {
            if (!extension_loaded('apc')) {
                $app->addInfo('[推奨] APC 拡張モジュールが有効になっていません。', 'install');
            }
        }

        if (isset($_SERVER['SERVER_SOFTWARE']) && strpos('Apache', $_SERVER['SERVER_SOFTWARE']) !== false) {
            if (!function_exists('apache_get_modules')) {
                $app->addWarning('mod_rewrite が有効になっているか不明です。', 'install');
            } elseif (!in_array('mod_rewrite', apache_get_modules())) {
                $app->addDanger('[必須] ' . 'mod_rewriteを有効にしてください。', 'install');

Concat operator must not be surrounded by spaces

            }
        } elseif (isset($_SERVER['SERVER_SOFTWARE']) && strpos('Microsoft-IIS', $_SERVER['SERVER_SOFTWARE']) !== false) {

This elseif statement is empty, and could be removed.

            // iis
        } elseif (isset($_SERVER['SERVER_SOFTWARE']) && strpos('nginx', $_SERVER['SERVER_SOFTWARE']) !== false) {

This elseif statement is empty, and could be removed.

            // nginx
        }
    }

    private function setPDO()
    {
        $config_file = $this->config_path . '/database.yml';

Concat operator must not be surrounded by spaces

        $config = Yaml::parse(file_get_contents($config_file));

        try {
            $this->PDO = \Doctrine\DBAL\DriverManager::getConnection($config['database'], new \Doctrine\DBAL\Configuration());
            $this->PDO->connect();
        } catch (\Exception $e) {
            $this->PDO->close();
            throw $e;
        }

        return $this;
    }

    private function dropTables()

This method seems to be duplicated in your project.

    {
        $this->resetNatTimer();

        $em = $this->getEntityManager();
        $metadatas = $em->getMetadataFactory()->getAllMetadata();
        $schemaTool = new SchemaTool($em);

        $schemaTool->dropSchema($metadatas);

        $em->getConnection()->executeQuery('DROP TABLE IF EXISTS doctrine_migration_versions');

        return $this;
    }

    /**
     * @return EntityManager
     */
    private function getEntityManager()
    {
        $config_file = $this->config_path . '/database.yml';

Concat operator must not be surrounded by spaces

        $database = Yaml::parse(file_get_contents($config_file));

        $this->app->register(new \Silex\Provider\DoctrineServiceProvider(), array(

Add a comma after each item in a multi-line array

            'db.options' => $database['database']
        ));

        $this->app->register(new \Dflydev\Silex\Provider\DoctrineOrm\DoctrineOrmServiceProvider(), array(

Add a comma after each item in a multi-line array

            'orm.proxies_dir' => __DIR__ . '/../../app/cache/doctrine',

Concat operator must not be surrounded by spaces

            'orm.em.options' => array(
                'mappings' => array(
                    array(
                        'type' => 'yml',
                        'namespace' => 'Eccube\Entity',
                        'path' => array(
                            __DIR__ . '/../../Resource/doctrine',

Concat operator must not be surrounded by spaces

                            __DIR__ . '/../../Resource/doctrine/master',

Concat operator must not be surrounded by spaces

                        ),
                    ),
                ),
            )
        ));

        return $em = $this->app['orm.em'];

$em is not used, you could remove the assignment.

    }

    private function createTables()

This method seems to be duplicated in your project.

    {
        $this->resetNatTimer();

        $em = $this->getEntityManager();
        $metadatas = $em->getMetadataFactory()->getAllMetadata();
        $schemaTool = new SchemaTool($em);

        $schemaTool->createSchema($metadatas);

        return $this;
    }

    private function insert()
    {
        $this->resetNatTimer();

        $config_file = $this->config_path . '/database.yml';

Concat operator must not be surrounded by spaces

        $database = Yaml::parse(file_get_contents($config_file));
        $config['database'] = $database['database'];

$config was never initialized. Although not strictly required by PHP, it is generally a good practice to add $config = array(); before regardless.

        $config_file = $this->config_path . '/config.yml';

Concat operator must not be surrounded by spaces

        $baseConfig = Yaml::parse(file_get_contents($config_file));
        $config['config'] = $baseConfig;

        $this->PDO->beginTransaction();

        try {

Blank line found at start of control structure

            $config = array(
                'auth_type' => '',
                'auth_magic' => $config['config']['auth_magic'],
                'password_hash_algos' => 'sha256',
            );
            $passwordEncoder = new \Eccube\Security\Core\Encoder\PasswordEncoder($config);
            $salt = \Eccube\Util\Str::random(32);

            $encodedPassword = $passwordEncoder->encodePassword($this->session_data['login_pass'], $salt);
            $sth = $this->PDO->prepare('INSERT INTO dtb_base_info (
                id,
                shop_name,
                email01,
                email02,
                email03,
                email04,
                update_date,
                option_product_tax_rule
            ) VALUES (
                1,
                :shop_name,
                :admin_mail,
                :admin_mail,
                :admin_mail,
                :admin_mail,
                current_timestamp,
                0);');
            $sth->execute(array(

Add a comma after each item in a multi-line array

                ':shop_name' => $this->session_data['shop_name'],
                ':admin_mail' => $this->session_data['email']
            ));

            $sth = $this->PDO->prepare("INSERT INTO dtb_member (member_id, login_id, password, salt, work, del_flg, authority, creator_id, rank, update_date, create_date,name,department) VALUES (2, :login_id, :admin_pass , :salt , '1', '0', '0', '1', '1', current_timestamp, current_timestamp,'管理者','EC-CUBE SHOP');");
            $sth->execute(array(':login_id' => $this->session_data['login_id'], ':admin_pass' => $encodedPassword, ':salt' => $salt));

            $this->PDO->commit();
        } catch (\Exception $e) {
            $this->PDO->rollback();
            throw $e;
        }

        return $this;
    }

    private function update()
    {
        $this->resetNatTimer();

        $config_file = $this->config_path . '/database.yml';

Concat operator must not be surrounded by spaces

        $database = Yaml::parse(file_get_contents($config_file));
        $config['database'] = $database['database'];

$config was never initialized. Although not strictly required by PHP, it is generally a good practice to add $config = array(); before regardless.

        $config_file = $this->config_path . '/config.yml';

Concat operator must not be surrounded by spaces

        $baseConfig = Yaml::parse(file_get_contents($config_file));
        $config['config'] = $baseConfig;

        $this->PDO->beginTransaction();

        try {

Blank line found at start of control structure

            $config = array(
                'auth_type' => '',
                'auth_magic' => $config['config']['auth_magic'],
                'password_hash_algos' => 'sha256',
            );
            $passwordEncoder = new \Eccube\Security\Core\Encoder\PasswordEncoder($config);
            $salt = \Eccube\Util\Str::random(32);

            $stmt = $this->PDO->prepare("SELECT member_id FROM dtb_member WHERE login_id = :login_id;");
            $stmt->execute(array(':login_id' => $this->session_data['login_id']));
            $rs = $stmt->fetch();

            $encodedPassword = $passwordEncoder->encodePassword($this->session_data['login_pass'], $salt);

            if ($rs) {
                // 同一の管理者IDであればパスワードのみ更新
                $sth = $this->PDO->prepare("UPDATE dtb_member set password = :admin_pass, salt = :salt, update_date = current_timestamp WHERE login_id = :login_id;");
                $sth->execute(array(':admin_pass' => $encodedPassword, ':salt' => $salt, ':login_id' => $this->session_data['login_id']));
            } else {
                // 新しい管理者IDが入力されたらinsert
                $sth = $this->PDO->prepare("INSERT INTO dtb_member (login_id, password, salt, work, del_flg, authority, creator_id, rank, update_date, create_date,name,department) VALUES (:login_id, :admin_pass , :salt , '1', '0', '0', '1', '1', current_timestamp, current_timestamp,'管理者','EC-CUBE SHOP');");
                $sth->execute(array(':login_id' => $this->session_data['login_id'], ':admin_pass' => $encodedPassword, ':salt' => $salt));
            }

            $sth = $this->PDO->prepare('UPDATE dtb_base_info set
                shop_name = :shop_name,
                email01 = :admin_mail,
                email02 = :admin_mail,
                email03 = :admin_mail,
                email04 = :admin_mail,
                update_date = current_timestamp
            WHERE id = 1;');
            $sth->execute(array(

Add a comma after each item in a multi-line array

                ':shop_name' => $this->session_data['shop_name'],
                ':admin_mail' => $this->session_data['email']
            ));

            $this->PDO->commit();
        } catch (\Exception $e) {
            $this->PDO->rollback();
            throw $e;
        }

        return $this;
    }

    private function getMigration()
    {
        $app = \Eccube\Application::getInstance();
        $app->initialize();
        $app->boot();

        $config = new Configuration($app['db']);
        $config->setMigrationsNamespace('DoctrineMigrations');

        $migrationDir = __DIR__ . '/../../Resource/doctrine/migration';

Concat operator must not be surrounded by spaces

        $config->setMigrationsDirectory($migrationDir);
        $config->registerMigrationsFromDirectory($migrationDir);

        $migration = new Migration($config);

        return $migration;
    }

    private function doMigrate()
    {
        try {
            $migration = $this->getMigration();

            // DBとのコネクションを維持するためpingさせる
            if (is_null($this->PDO)) {
                $this->setPDO();
            }
            $this->PDO->ping();

            // nullを渡すと最新バージョンまでマイグレートする
            $migration->migrate(null, false);
        } catch (MigrationException $e) {

Consider adding a comment why this CATCH block is empty.

Blank line found at start of control structure

Please trim any trailing whitespace

        }

        return $this;
    }

    private function getProtectedDirs()
    {
        $protectedDirs = array();
        $base = __DIR__ . '/../../../..';

Concat operator must not be surrounded by spaces

        $dirs = array(
            '/html',
            '/app',
            '/app/template',
            '/app/cache',
            '/app/config',
            '/app/config/eccube',
            '/app/log',
            '/app/Plugin',
        );

        foreach ($dirs as $dir) {
            if (!is_writable($base . $dir)) {

Concat operator must not be surrounded by spaces

                $protectedDirs[] = $dir;
            }
        }

        return $protectedDirs;
    }

    private function createConfigYamlFile($data, $auth = true)
    {
        $fs = new Filesystem();
        $config_file = $this->config_path . '/config.yml';

Concat operator must not be surrounded by spaces

        if ($fs->exists($config_file)) {
            $config = Yaml::parse(file_get_contents($config_file));
            $fs->remove($config_file);
        }

        if ($auth) {
            $auth_magic = Str::random(32);
        } else {
            if (isset($config['auth_magic'])) {
                $auth_magic = $config['auth_magic'];
            } else {
                $auth_magic = Str::random(32);
            }
        }

        $allowHost = Str::convertLineFeed($data['admin_allow_hosts']);
        if (empty($allowHost)) {
            $adminAllowHosts = array();
        } else {
            $adminAllowHosts = explode("\n", $allowHost);
        }
        $trustedProxies = Str::convertLineFeed($data['trusted_proxies']);
        if (empty($trustedProxies)) {
            $adminTrustedProxies = array();
        } else {
            $adminTrustedProxies = explode("\n", $trustedProxies);
            // ループバックアドレスを含める
            $adminTrustedProxies = array_merge($adminTrustedProxies, array('127.0.0.1/8', '::1'));
        }
        if ($data['trusted_proxies_connection_only']) {
            // ループバックアドレスを含める
            $adminTrustedProxies = array('127.0.0.1/8', '::1');
        }

        $target = array('${AUTH_MAGIC}', '${SHOP_NAME}', '${ECCUBE_INSTALL}', '${FORCE_SSL}');
        $replace = array($auth_magic, $data['shop_name'], '0', $data['admin_force_ssl']);

        $fs = new Filesystem();
        $content = str_replace(
            $target, $replace, file_get_contents($this->dist_path . '/config.yml.dist')

Concat operator must not be surrounded by spaces

        );
        $fs->dumpFile($config_file, $content);

        $config = Yaml::parse(file_get_contents($config_file));
        $config['admin_allow_host'] = $adminAllowHosts;
        $config['trusted_proxies_connection_only'] = $data['trusted_proxies_connection_only'];
        $config['trusted_proxies'] = $adminTrustedProxies;
        $yml = Yaml::dump($config);
        file_put_contents($config_file, $yml);

        return $this;
    }

    private function addInstallStatus()
    {
        $config_file = $this->config_path . '/config.yml';

Concat operator must not be surrounded by spaces

        $config = Yaml::parse(file_get_contents($config_file));
        $config['eccube_install'] = 1;
        $yml = Yaml::dump($config);
        file_put_contents($config_file, $yml);

        return $this;
    }

    private function createDatabaseYamlFile($data)
    {
        $fs = new Filesystem();
        $config_file = $this->config_path . '/database.yml';

Concat operator must not be surrounded by spaces

        if ($fs->exists($config_file)) {
            $fs->remove($config_file);
        }

        if ($data['database'] != 'pdo_sqlite') {
            switch ($data['database'])
            {
                case 'pdo_pgsql':
                    if (empty($data['db_port'])) {
                        $data['db_port'] = '5432';
                    }
                    $data['db_driver'] = 'pdo_pgsql';
                    break;
                case 'pdo_mysql':
                    if (empty($data['db_port'])) {
                        $data['db_port'] = '3306';
                    }
                    $data['db_driver'] = 'pdo_mysql';
                    break;
            }
            $target = array('${DBDRIVER}', '${DBSERVER}', '${DBNAME}', '${DBPORT}', '${DBUSER}', '${DBPASS}');
            $replace = array(

Add a comma after each item in a multi-line array

                $data['db_driver'],
                $data['database_host'],
                $data['database_name'],
                $data['database_port'],
                $data['database_user'],
                $data['database_password']
            );

            $fs = new Filesystem();
            $content = str_replace(
                $target, $replace, file_get_contents($this->dist_path . '/database.yml.dist')

Concat operator must not be surrounded by spaces

            );
        } else {
            $content = Yaml::dump(
                    array(

Add a comma after each item in a multi-line array

                        'database' => array(

Add a comma after each item in a multi-line array

                            'driver' => 'pdo_sqlite',
                            'path' => realpath($this->config_path . '/eccube.db')

Concat operator must not be surrounded by spaces

                        )
                    )
            );
        }
        $fs->dumpFile($config_file, $content);

        return $this;
    }

    private function createMailYamlFile($data)
    {
        $fs = new Filesystem();
        $config_file = $this->config_path . '/mail.yml';

Concat operator must not be surrounded by spaces

        if ($fs->exists($config_file)) {
            $fs->remove($config_file);
        }
        $target = array('${MAIL_BACKEND}', '${MAIL_HOST}', '${MAIL_PORT}', '${MAIL_USER}', '${MAIL_PASS}');
        $replace = array(

Add a comma after each item in a multi-line array

            $data['mail_backend'],
            $data['smtp_host'],
            $data['smtp_port'],
            $data['smtp_username'],
            $data['smtp_password']
        );

        $fs = new Filesystem();
        $content = str_replace(
            $target, $replace, file_get_contents($this->dist_path . '/mail.yml.dist')

Concat operator must not be surrounded by spaces

        );
        $fs->dumpFile($config_file, $content);

        return $this;
    }

    private function createPathYamlFile($data, Request $request)
    {
        $fs = new Filesystem();
        $config_file = $this->config_path . '/path.yml';

Concat operator must not be surrounded by spaces

        if ($fs->exists($config_file)) {
            $fs->remove($config_file);
        }

        $ADMIN_ROUTE = $data['admin_dir'];
        $TEMPLATE_CODE = 'default';
        $USER_DATA_ROUTE = 'user_data';
        $ROOT_DIR = realpath(__DIR__ . '/../../../../');

Concat operator must not be surrounded by spaces

        $ROOT_URLPATH = $request->getBasePath();
        $ROOT_PUBLIC_URLPATH = $ROOT_URLPATH . RELATIVE_PUBLIC_DIR_PATH;

Concat operator must not be surrounded by spaces

        $target = array('${ADMIN_ROUTE}', '${TEMPLATE_CODE}', '${USER_DATA_ROUTE}', '${ROOT_DIR}', '${ROOT_URLPATH}', '${ROOT_PUBLIC_URLPATH}');
        $replace = array($ADMIN_ROUTE, $TEMPLATE_CODE, $USER_DATA_ROUTE, $ROOT_DIR, $ROOT_URLPATH, $ROOT_PUBLIC_URLPATH);

        $fs = new Filesystem();
        $content = str_replace(
            $target, $replace, file_get_contents($this->dist_path . '/path.yml.dist')

Concat operator must not be surrounded by spaces

        );
        $fs->dumpFile($config_file, $content);

        return $this;
    }

    private function sendAppData($params)
    {
        $config_file = $this->config_path . '/database.yml';

Concat operator must not be surrounded by spaces

        $db_config = Yaml::parse(file_get_contents($config_file));

        $this->setPDO();
        $stmt = $this->PDO->query('select version() as v');

        $version = '';
        foreach ($stmt as $row) {
            $version = $row['v'];
        }

        if ($db_config['database']['driver'] === 'pdo_mysql') {
            $db_ver = 'MySQL:' . $version;

Concat operator must not be surrounded by spaces

        } else {
            $db_ver = $version;
        }

        $data = http_build_query(
            array(
                'site_url' => $params['http_url'],
                'shop_name' => $params['shop_name'],
                'cube_ver' => Constant::VERSION,
                'php_ver' => phpversion(),
                'db_ver' => $db_ver,
                'os_type' => php_uname(),
            )
        );

        $header = array(
            'Content-Type: application/x-www-form-urlencoded',
            'Content-Length: ' . strlen($data),

Concat operator must not be surrounded by spaces

        );
        $context = stream_context_create(
            array(

Add a comma after each item in a multi-line array

                'http' => array(
                    'method' => 'POST',
                    'header' => $header,
                    'content' => $data,
                )
            )
        );
        file_get_contents('http://www.ec-cube.net/mall/use_site.php', false, $context);

        return $this;
    }

    /**
     * マイグレーション画面を表示する.
     *
     * @param InstallApplication $app
     * @param Request $request

Expected 12 spaces after parameter type; 1 found

     *
     * @return \Symfony\Component\HttpFoundation\Response
     */
    public function migration(InstallApplication $app, Request $request)

The parameter $request is not used and could be removed.

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

    {
        return $app['twig']->render('migration.twig', array(
                'publicPath' => '..' . RELATIVE_PUBLIC_DIR_PATH . '/',

Concat operator must not be surrounded by spaces

        ));
    }

    /**
     * インストール済プラグインの一覧を表示する.
     * プラグインがインストールされていない場合は, マイグレーション実行画面へリダイレクトする.
     *
     * @param InstallApplication $app
     * @param Request $request

Expected 12 spaces after parameter type; 1 found

     *
     * @return \Symfony\Component\HttpFoundation\Response
     */
    public function migration_plugin(InstallApplication $app, Request $request)

The parameter $request is not used and could be removed.

Method name "InstallController::migration_plugin" is not in camel caps format

    {
        $eccube = \Eccube\Application::getInstance();
        $eccube->initialize();
        $eccube->boot();

        $pluginRepository = $eccube['orm.em']->getRepository('Eccube\Entity\Plugin');
        $Plugins = $pluginRepository->findBy(array('del_flg' => Constant::DISABLED));

        if (empty($Plugins)) {
            // インストール済プラグインがない場合はマイグレーション実行画面へリダイレクト.
            return $app->redirect($app->path('migration_end'));
        } else {
            return $app['twig']->render('migration_plugin.twig', array(
                    'Plugins' => $Plugins,
                    'version' => Constant::VERSION,
                    'publicPath' => '..' . RELATIVE_PUBLIC_DIR_PATH . '/',

Concat operator must not be surrounded by spaces

            ));
        }
    }

    /**
     * マイグレーションを実行し, 完了画面を表示させる
     *
     * @param InstallApplication $app
     * @param Request $request

Expected 12 spaces after parameter type; 1 found

     *
     * @return \Symfony\Component\HttpFoundation\Response
     */
    public function migration_end(InstallApplication $app, Request $request)

The parameter $request is not used and could be removed.

Method name "InstallController::migration_end" is not in camel caps format

    {
        $this->doMigrate();

        $config_app = new \Eccube\Application(); // install用のappだとconfigが取れないので
        $config_app->initialize();
        $config_app->boot();
        \Eccube\Util\Cache::clear($config_app, true);

        return $app['twig']->render('migration_end.twig', array(
                'publicPath' => '..' . RELATIVE_PUBLIC_DIR_PATH . '/',

Concat operator must not be surrounded by spaces

        ));
    }
}