OwnerStoreController::doConfirm() - Code Metrics - Inspection of "Merge pull request #2567 from chihiro-adachi/fix-c..." - EC-CUBE/ec-cube - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — experimental/3.1 ( 24445a...d4772e )

by Yangsin

created 2017-10-18 01:53 UTC

OwnerStoreController::doConfirm() B

↳ Parent: OwnerStoreController

Complexity

Conditions	3
Paths	3

Size

Total Lines	27
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	12

Importance

Changes

Metric	Value
cc	3
eloc	14
nc	3
nop	3
dl	0
loc	27
ccs	0
cts	17
cp	0
crap	12
rs	8.8571
c	0
b	0
f	0

<?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\Admin\Store;

use Eccube\Annotation\Inject;
use Eccube\Application;
use Eccube\Common\Constant;
use Eccube\Controller\AbstractController;
use Eccube\Entity\Plugin;
use Eccube\Repository\PluginRepository;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Template;
use Symfony\Component\Config\Definition\Exception\Exception;
use Symfony\Component\HttpFoundation\RedirectResponse;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
use Symfony\Component\Process\Process;

/**
 * @Route(service=OwnerStoreController::class)
 */
class OwnerStoreController extends AbstractController
{
    /**
     * @Inject("config")
     * @var array
     */
    protected $appConfig;

    /**
     * @Inject(PluginRepository::class)
     * @var PluginRepository
     */
    protected $pluginRepository;

    /**
     * Owner's Store Plugin Installation Screen - Search function
     *
     * @Route("/{_admin}/store/plugin/search", name="admin_store_plugin_owners_search")
     * @Template("Store/plugin_search.twig")
     * @param Application $app
     * @param Request     $request
     * @return array
     */
    public function search(Application $app, Request $request)

    {
        // Acquire downloadable plug-in information from owners store
        $success = 0;
        $items = array();
        $promotionItems = array();
        $message = '';
        // Owner's store communication
        $url = $this->appConfig['owners_store_url'].'?method=list';
        list($json, $info) = $this->getRequestApi($url, $app);
        if ($json === false) {
            $message = $this->getResponseErrorMessage($info);
        } else {
            $data = json_decode($json, true);
            if (isset($data['success'])) {
                $success = $data['success'];
                if ($success == '1') {
                    $items = array();
                    // Check plugin installed
                    $arrPluginInstalled = $this->pluginRepository->findAll();
                    // Update_status 1 : not install/purchased 、2 : Installed、 3 : Update、4 : paid purchase
                    foreach ($data['item'] as $item) {
                        // Not install/purchased
                        $item['update_status'] = 1;
                        /** @var Plugin $plugin */
                        foreach ($arrPluginInstalled as $plugin) {
                            if ($plugin->getSource() == $item['product_id']) {
                                // Need update
                                $item['update_status'] = 3;
                                if ($plugin->getVersion() == $item['version']) {
                                    // Installed
                                    $item['update_status'] = 2;
                                }
                            }
                        }
                        $items[] = $item;
                    }

                    // EC-CUBE version check
                    foreach ($items as &$item) {
                        // Not applicable version
                        $item['version_check'] = 0;
                        if (in_array(Constant::VERSION, $item['eccube_version'])) {
                            // Match version
                            $item['version_check'] = 1;
                        }
                        if ($item['price'] != '0' && $item['purchased'] == '0') {
                            // Not purchased with paid items
                            $item['update_status'] = 4;
                        }
                    }
                    unset($item);

                    // Promotion item
                    $i = 0;
                    foreach ($items as $item) {
                        if ($item['promotion'] == 1) {
                            $promotionItems[] = $item;
                            unset($items[$i]);
                        }
                        $i++;
                    }
                } else {
                    $message = $data['error_code'] . ' : ' . $data['error_message'];

                }
            } else {
                $success = 0;
                $message = "EC-CUBEオーナーズストアにエラーが発生しています。";
            }
        }

        return [
            'success' => $success,
            'items' => $items,
            'promotionItems' => $promotionItems,
            'message' => $message,
        ];
    }

    /**
     * Do confirm page
     *
     * @Route("/{_admin}/store/plugin/confirm/{pluginId}" , name="admin_store_plugin_install_confirm")
     * @Template("Store/plugin_confirm.twig")
     * @param Application $app
     * @param Request     $request
     * @param string      $pluginId
     * @return array
     */
    public function doConfirm(Application $app, Request $request, $pluginId)

    {
        // Owner's store communication
        $url = $this->appConfig['owners_store_url'].'?method=list';
        list($json, $info) = $this->getRequestApi($url, $app);
<?php

function returnThreeValues() {
    return array('a', 'b', 'c');
}

list($a, $b, $c) = returnThreeValues();

print $a . " - " . $c;
        $data = json_decode($json, true);
        $items = $data['item'];

        // Find plugin in api
        $index = array_search($pluginId, array_column($items, 'product_id'));
        if ($index === false) {
            throw new NotFoundHttpException();
        }
        // Get target plugin in return of api
        $plugin = $items[$index];

        // Check the eccube version that the plugin supports.
        $plugin['is_supported_eccube_version'] = 0;
        if (in_array(Constant::VERSION, $plugin['eccube_version'])) {
            // Match version
            $plugin['is_supported_eccube_version'] = 1;
        }

        return [
            'item' => $plugin,
        ];
    }

    /**
     * Api Install plugin by composer connect with package repo
     *
     * @Route("/{_admin}/store/plugin/api/{pluginCode}/{eccubeVersion}/{version}" , name="admin_store_plugin_api_install")
     *
     * @param Application $app
     * @param Request     $request
     * @param string      $pluginCode
     * @param string      $eccubeVersion
     * @param string      $version
     * @return RedirectResponse
     */
    public function apiInstall(Application $app, Request $request, $pluginCode, $eccubeVersion, $version)

    {
        // Check plugin code
        $url = $this->appConfig['owners_store_url'].'?eccube_version='.$eccubeVersion.'&plugin_code='.$pluginCode.'&version='.$version;
        list($json, $info) = $this->getRequestApi($url, $app);
<?php

function returnThreeValues() {
    return array('a', 'b', 'c');
}

list($a, $b, $c) = returnThreeValues();

print $a . " - " . $c;
        $existFlg = false;
        $data = json_decode($json, true);
        if ($data && isset($data['success'])) {
            $success = $data['success'];
            if ($success == '1') {
                foreach ($data['item'] as $item) {
                    if ($item['product_code'] == $pluginCode) {
                        $existFlg = true;
                        break;
                    }
                }
            }
        }
        if ($existFlg === false) {
            $app->log(sprintf('%s plugin not found!', $pluginCode));
            $app->addError('admin.plugin.not.found', 'admin');

            return $app->redirect($app->url('admin_store_plugin_owners_search'));
        }

        try {
            $execute = sprintf('cd %s &&', $this->appConfig['root_dir']);
            $execute .= sprintf(' composer require ec-cube/%s', $pluginCode);

            $install = new Process($execute);
            $install->setTimeout(null);
            $install->run();
            if ($install->isSuccessful()) {
                $app->addSuccess('admin.plugin.install.complete', 'admin');
                $app->log(sprintf('Install %s plugin successful!', $pluginCode));

                return $app->redirect($app->url('admin_store_plugin'));
            }
            $app->addError('admin.plugin.install.fail', 'admin');
        } catch (Exception $exception) {
            $app->addError($exception->getMessage(), 'admin');
            $app->log($exception->getCode().' : '.$exception->getMessage());
        }
        $app->log(sprintf('Install %s plugin fail!', $pluginCode));

        return $app->redirect($app->url('admin_store_plugin_owners_search'));
    }

    /**
     * API request processing
     *
     * @param string  $url
     * @param Application $app
     * @return array
     */
    private function getRequestApi($url, $app)
    {
        $curl = curl_init($url);

        // Option array
        $options = array(
            // HEADER
            CURLOPT_HTTPGET => true,
            CURLOPT_SSL_VERIFYPEER => false,
            CURLOPT_RETURNTRANSFER => true,
            CURLOPT_FAILONERROR => true,
            CURLOPT_CAINFO => \Composer\CaBundle\CaBundle::getSystemCaRootBundlePath(),
        );

        // Set option value
        curl_setopt_array($curl, $options);
        $result = curl_exec($curl);
        $info = curl_getinfo($curl);
        $message = curl_error($curl);
        $info['message'] = $message;
        curl_close($curl);

        $app->log('http get_info', $info);

        return array($result, $info);
    }

    /**
     * Get message
     *
     * @param $info
     * @return string
     */
    private function getResponseErrorMessage($info)

    {
        if (!empty($info)) {
            $statusCode = $info['http_code'];
            $message = $info['message'];

            $message = $statusCode.' : '.$message;
        } else {
            $message = "タイムアウトエラーまたはURLの指定に誤りがあります。";
        }

        return $message;
    }
}


1		<?php
2		/*
3		* This file is part of EC-CUBE
4		*
5		* Copyright(c) 2000-2015 LOCKON CO.,LTD. All Rights Reserved.
6		*
7		* http://www.lockon.co.jp/
8		*
9		* This program is free software; you can redistribute it and/or
10		* modify it under the terms of the GNU General Public License
11		* as published by the Free Software Foundation; either version 2
12		* of the License, or (at your option) any later version.
13		*
14		* This program is distributed in the hope that it will be useful,
15		* but WITHOUT ANY WARRANTY; without even the implied warranty of
16		* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
17		* GNU General Public License for more details.
18		*
19		* You should have received a copy of the GNU General Public License
20		* along with this program; if not, write to the Free Software
21		* Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
22		*/
23
24
25		namespace Eccube\Controller\Admin\Store;
26
27		use Eccube\Annotation\Inject;
28		use Eccube\Application;
29		use Eccube\Common\Constant;
30		use Eccube\Controller\AbstractController;
31		use Eccube\Entity\Plugin;
32		use Eccube\Repository\PluginRepository;
33		use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
34		use Sensio\Bundle\FrameworkExtraBundle\Configuration\Template;
35		use Symfony\Component\Config\Definition\Exception\Exception;
36		use Symfony\Component\HttpFoundation\RedirectResponse;
37		use Symfony\Component\HttpFoundation\Request;
38		use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
39		use Symfony\Component\Process\Process;
40
41		/**
42		* @Route(service=OwnerStoreController::class)
43		*/
44		class OwnerStoreController extends AbstractController
45		{
46		/**
47		* @Inject("config")
48		* @var array
49		*/
50		protected $appConfig;
51
52		/**
53		* @Inject(PluginRepository::class)
54		* @var PluginRepository
55		*/
56		protected $pluginRepository;
57
58		/**
59		* Owner's Store Plugin Installation Screen - Search function
60		*
61		* @Route("/{_admin}/store/plugin/search", name="admin_store_plugin_owners_search")
62		* @Template("Store/plugin_search.twig")
63		* @param Application $app
64		* @param Request $request
65		* @return array
66		*/
67		public function search(Application $app, Request $request)
		0 ignored issues – show Unused Code introduced 2017-09-25 09:40 UTC by Report Bug Copy Issue Report The parameter `$request` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
68		{
69		// Acquire downloadable plug-in information from owners store
70		$success = 0;
71		$items = array();
72		$promotionItems = array();
73		$message = '';
74		// Owner's store communication
75		$url = $this->appConfig['owners_store_url'].'?method=list';
76		list($json, $info) = $this->getRequestApi($url, $app);
77		if ($json === false) {
78		$message = $this->getResponseErrorMessage($info);
79		} else {
80		$data = json_decode($json, true);
81		if (isset($data['success'])) {
82		$success = $data['success'];
83		if ($success == '1') {
84		$items = array();
85		// Check plugin installed
86		$arrPluginInstalled = $this->pluginRepository->findAll();
87		// Update_status 1 : not install/purchased 、2 : Installed、 3 : Update、4 : paid purchase
88		foreach ($data['item'] as $item) {
89		// Not install/purchased
90		$item['update_status'] = 1;
91		/** @var Plugin $plugin */
92		foreach ($arrPluginInstalled as $plugin) {
93		if ($plugin->getSource() == $item['product_id']) {
94		// Need update
95		$item['update_status'] = 3;
96		if ($plugin->getVersion() == $item['version']) {
97		// Installed
98		$item['update_status'] = 2;
99		}
100		}
101		}
102		$items[] = $item;
103		}
104
105		// EC-CUBE version check
106	View Code Duplication	foreach ($items as &$item) {
107		// Not applicable version
108		$item['version_check'] = 0;
109		if (in_array(Constant::VERSION, $item['eccube_version'])) {
110		// Match version
111		$item['version_check'] = 1;
112		}
113		if ($item['price'] != '0' && $item['purchased'] == '0') {
114		// Not purchased with paid items
115		$item['update_status'] = 4;
116		}
117		}
118		unset($item);
119
120		// Promotion item
121		$i = 0;
122	View Code Duplication	foreach ($items as $item) {
123		if ($item['promotion'] == 1) {
124		$promotionItems[] = $item;
125		unset($items[$i]);
126		}
127		$i++;
128		}
129		} else {
130		$message = $data['error_code'] . ' : ' . $data['error_message'];
		0 ignored issues – show Coding Style introduced 2016-02-09 04:40 UTC by Report Bug Copy Issue Report Concat operator must not be surrounded by spaces Loading history...
131		}
132		} else {
133		$success = 0;
134		$message = "EC-CUBEオーナーズストアにエラーが発生しています。";
135		}
136		}
137
138		return [
139		'success' => $success,
140		'items' => $items,
141		'promotionItems' => $promotionItems,
142		'message' => $message,
143		];
144		}
145
146		/**
147		* Do confirm page
148		*
149		* @Route("/{_admin}/store/plugin/confirm/{pluginId}" , name="admin_store_plugin_install_confirm")
150		* @Template("Store/plugin_confirm.twig")
151		* @param Application $app
152		* @param Request $request
153		* @param string $pluginId
154		* @return array
155		*/
156		public function doConfirm(Application $app, Request $request, $pluginId)
		0 ignored issues – show Unused Code introduced 2017-10-17 10:24 UTC by Report Bug Copy Issue Report The parameter `$request` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
157		{
158		// Owner's store communication
159		$url = $this->appConfig['owners_store_url'].'?method=list';
160		list($json, $info) = $this->getRequestApi($url, $app);
		0 ignored issues – show Unused Code introduced 2017-10-16 11:33 UTC by Report Bug Copy Issue Report The assignment to `$info` is unused. Consider omitting it like so `list($first,,$third)`. This checks looks for assignemnts to variables using the `list(...)` function, where not all assigned variables are subsequently used. Consider the following code example. <?php function returnThreeValues() { return array('a', 'b', 'c'); } list($a, $b, $c) = returnThreeValues(); print $a . " - " . $c; Only the variables `$a` and `$c` are used. There was no need to assign `$b`. Instead, the list call could have been. list($a,, $c) = returnThreeValues(); Loading history...
161		$data = json_decode($json, true);
162		$items = $data['item'];
163
164		// Find plugin in api
165		$index = array_search($pluginId, array_column($items, 'product_id'));
166		if ($index === false) {
167		throw new NotFoundHttpException();
168		}
169		// Get target plugin in return of api
170		$plugin = $items[$index];
171
172		// Check the eccube version that the plugin supports.
173		$plugin['is_supported_eccube_version'] = 0;
174		if (in_array(Constant::VERSION, $plugin['eccube_version'])) {
175		// Match version
176		$plugin['is_supported_eccube_version'] = 1;
177		}
178
179		return [
180		'item' => $plugin,
181		];
182		}
183
184		/**
185		* Api Install plugin by composer connect with package repo
186		*
187		* @Route("/{_admin}/store/plugin/api/{pluginCode}/{eccubeVersion}/{version}" , name="admin_store_plugin_api_install")
188		*
189		* @param Application $app
190		* @param Request $request
191		* @param string $pluginCode
192		* @param string $eccubeVersion
193		* @param string $version
194		* @return RedirectResponse
195		*/
196		public function apiInstall(Application $app, Request $request, $pluginCode, $eccubeVersion, $version)
		0 ignored issues – show Unused Code introduced 2017-10-05 09:12 UTC by Report Bug Copy Issue Report The parameter `$request` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
197		{
198		// Check plugin code
199		$url = $this->appConfig['owners_store_url'].'?eccube_version='.$eccubeVersion.'&plugin_code='.$pluginCode.'&version='.$version;
200		list($json, $info) = $this->getRequestApi($url, $app);
		0 ignored issues – show Unused Code introduced 2017-10-17 10:24 UTC by Report Bug Copy Issue Report The assignment to `$info` is unused. Consider omitting it like so `list($first,,$third)`. This checks looks for assignemnts to variables using the `list(...)` function, where not all assigned variables are subsequently used. Consider the following code example. <?php function returnThreeValues() { return array('a', 'b', 'c'); } list($a, $b, $c) = returnThreeValues(); print $a . " - " . $c; Only the variables `$a` and `$c` are used. There was no need to assign `$b`. Instead, the list call could have been. list($a,, $c) = returnThreeValues(); Loading history...
201		$existFlg = false;
202		$data = json_decode($json, true);
203		if ($data && isset($data['success'])) {
204		$success = $data['success'];
205		if ($success == '1') {
206		foreach ($data['item'] as $item) {
207		if ($item['product_code'] == $pluginCode) {
208		$existFlg = true;
209		break;
210		}
211		}
212		}
213		}
214		if ($existFlg === false) {
215		$app->log(sprintf('%s plugin not found!', $pluginCode));
216		$app->addError('admin.plugin.not.found', 'admin');
217
218		return $app->redirect($app->url('admin_store_plugin_owners_search'));
219		}
220
221		try {
222		$execute = sprintf('cd %s &&', $this->appConfig['root_dir']);
223		$execute .= sprintf(' composer require ec-cube/%s', $pluginCode);
224
225		$install = new Process($execute);
226		$install->setTimeout(null);
227		$install->run();
228		if ($install->isSuccessful()) {
229		$app->addSuccess('admin.plugin.install.complete', 'admin');
230		$app->log(sprintf('Install %s plugin successful!', $pluginCode));
231
232		return $app->redirect($app->url('admin_store_plugin'));
233		}
234		$app->addError('admin.plugin.install.fail', 'admin');
235		} catch (Exception $exception) {
236		$app->addError($exception->getMessage(), 'admin');
237		$app->log($exception->getCode().' : '.$exception->getMessage());
238		}
239		$app->log(sprintf('Install %s plugin fail!', $pluginCode));
240
241		return $app->redirect($app->url('admin_store_plugin_owners_search'));
242		}
243
244		/**
245		* API request processing
246		*
247		* @param string $url
248		* @param Application $app
249		* @return array
250		*/
251		private function getRequestApi($url, $app)
252		{
253		$curl = curl_init($url);
254
255		// Option array
256		$options = array(
257		// HEADER
258		CURLOPT_HTTPGET => true,
259		CURLOPT_SSL_VERIFYPEER => false,
260		CURLOPT_RETURNTRANSFER => true,
261		CURLOPT_FAILONERROR => true,
262		CURLOPT_CAINFO => \Composer\CaBundle\CaBundle::getSystemCaRootBundlePath(),
263		);
264
265		// Set option value
266		curl_setopt_array($curl, $options);
267		$result = curl_exec($curl);
268		$info = curl_getinfo($curl);
269		$message = curl_error($curl);
270		$info['message'] = $message;
271		curl_close($curl);
272
273		$app->log('http get_info', $info);
274
275		return array($result, $info);
276		}
277
278		/**
279		* Get message
280		*
281		* @param $info
282		* @return string
283		*/
284	View Code Duplication	private function getResponseErrorMessage($info)
		0 ignored issues – show Duplication introduced 2017-09-25 09:40 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...
285		{
286		if (!empty($info)) {
287		$statusCode = $info['http_code'];
288		$message = $info['message'];
289
290		$message = $statusCode.' : '.$message;
291		} else {
292		$message = "タイムアウトエラーまたはURLの指定に誤りがあります。";
293		}
294
295		return $message;
296		}
297		}
298

EC-CUBE / ec-cube

Push — experimental/3.1 ( 24445a...d4772e )

OwnerStoreController::doConfirm() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like