ClassNameController::delete() - Code Metrics - Inspection of "Merge pull request #2643 from okazy/fix_column_ren..." - EC-CUBE/ec-cube - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — experimental/3.1 ( 878c99...05b497 )

by chihiro

created 2017-11-30 04:50 UTC

ClassNameController::delete() B

↳ Parent: ClassNameController

Complexity

Conditions	2
Paths	6

Size

Total Lines	25
Code Lines	14

Duplication

Lines	25
Ratio	100 %

Code Coverage

Tests	9
CRAP Score	2.1165

Importance

Changes

Metric	Value
cc	2
eloc	14
nc	6
nop	3
dl	25
loc	25
ccs	9
cts	13
cp	0.6923
crap	2.1165
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\Product;

use Doctrine\ORM\EntityManager;
use Eccube\Annotation\Inject;
use Eccube\Application;
use Eccube\Controller\AbstractController;
use Eccube\Entity\ClassName;
use Eccube\Event\EccubeEvents;
use Eccube\Event\EventArgs;
use Eccube\Form\Type\Admin\ClassNameType;
use Eccube\Repository\ClassNameRepository;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Method;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Template;
use Symfony\Component\EventDispatcher\EventDispatcher;
use Symfony\Component\Form\FormFactory;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;

/**
 * @Route(service=ClassNameController::class)
 */
class ClassNameController extends AbstractController
{
    /**
     * @Inject("orm.em")
     * @var EntityManager
     */
    protected $entityManager;

    /**
     * @Inject("eccube.event.dispatcher")
     * @var EventDispatcher
     */
    protected $eventDispatcher;

    /**
     * @Inject("form.factory")
     * @var FormFactory
     */
    protected $formFactory;

    /**
     * @Inject(ClassNameRepository::class)
     * @var ClassNameRepository
     */
    protected $classNameRepository;

    /**

     * @Route("/{_admin}/product/class_name", name="admin_product_class_name")
     * @Route("/{_admin}/product/class_name/{id}/edit", requirements={"id" = "\d+"}, name="admin_product_class_name_edit")
     * @Template("Product/class_name.twig")
     */

    public function index(Application $app, Request $request, $id = null)
    {
        if ($id) {
            $TargetClassName = $this->classNameRepository->find($id);
            if (!$TargetClassName) {
                throw new NotFoundHttpException('商品規格が存在しません');
            }
        } else {
            $TargetClassName = new \Eccube\Entity\ClassName();
        }

        $builder = $this->formFactory
            ->createBuilder(ClassNameType::class, $TargetClassName);

        $event = new EventArgs(
            array(
                'builder' => $builder,
                'TargetClassName' => $TargetClassName,
            ),
            $request
        );
        $this->eventDispatcher->dispatch(EccubeEvents::ADMIN_PRODUCT_CLASS_NAME_INDEX_INITIALIZE, $event);

        $form = $builder->getForm();

        if ($request->getMethod() === 'POST') {
            $form->handleRequest($request);
            if ($form->isValid()) {
                log_info('商品規格登録開始', array($id));

                $this->classNameRepository->save($TargetClassName);

                log_info('商品規格登録完了', array($id));

                $event = new EventArgs(
                    array(
                        'form' => $form,
                        'TargetClassName' => $TargetClassName,
                    ),
                    $request
                );
                $this->eventDispatcher->dispatch(EccubeEvents::ADMIN_PRODUCT_CLASS_NAME_INDEX_COMPLETE, $event);

                $app->addSuccess('admin.class_name.save.complete', 'admin');

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

        $ClassNames = $this->classNameRepository->getList();

        return [
            'form' => $form->createView(),
            'ClassNames' => $ClassNames,
            'TargetClassName' => $TargetClassName,
        ];
    }

    /**

     * @Method("DELETE")
     * @Route("/{_admin}/product/class_name/{id}/delete", requirements={"id" = "\d+"}, name="admin_product_class_name_delete")
     */

    public function delete(Application $app, Request $request, ClassName $ClassName)

    {
        $this->isTokenValid($app);

        log_info('商品規格削除開始', array($ClassName->getId()));

        try {
            $this->classNameRepository->delete($ClassName);

            $event = new EventArgs(['ClassName' => $ClassName,], $request);

            $this->eventDispatcher->dispatch(EccubeEvents::ADMIN_PRODUCT_CLASS_NAME_DELETE_COMPLETE, $event);

            $app->addSuccess('admin.class_name.delete.complete', 'admin');

            log_info('商品規格削除完了', array($ClassName->getId()));


        } catch (\Exception $e) {
            $message = $app->trans('admin.delete.failed.foreign_key', ['%name%' => '商品規格']);
            $app->addError($message, 'admin');

            log_error('商品企画削除エラー', [$ClassName->getId(), $e]);
        }

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

    /**

     * @Method("POST")
     * @Route("/{_admin}/product/class_name/sort_no/move", name="admin_product_class_name_sort_no_move")
     */

    public function moveSortNo(Application $app, Request $request)

    {
        if ($request->isXmlHttpRequest()) {
            $sortNos = $request->request->all();
            foreach ($sortNos as $classNameId => $sortNo) {
                $ClassName = $this->classNameRepository
                    ->find($classNameId);
                $ClassName->setSortNo($sortNo);
                $this->entityManager->persist($ClassName);
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}
            }
            $this->entityManager->flush();
        }

        return true;
    }
}


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\Product;
26
27			use Doctrine\ORM\EntityManager;
28			use Eccube\Annotation\Inject;
29			use Eccube\Application;
30			use Eccube\Controller\AbstractController;
31			use Eccube\Entity\ClassName;
32			use Eccube\Event\EccubeEvents;
33			use Eccube\Event\EventArgs;
34			use Eccube\Form\Type\Admin\ClassNameType;
35			use Eccube\Repository\ClassNameRepository;
36			use Sensio\Bundle\FrameworkExtraBundle\Configuration\Method;
37			use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
38			use Sensio\Bundle\FrameworkExtraBundle\Configuration\Template;
39			use Symfony\Component\EventDispatcher\EventDispatcher;
40			use Symfony\Component\Form\FormFactory;
41			use Symfony\Component\HttpFoundation\Request;
42			use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
43
44			/**
45			* @Route(service=ClassNameController::class)
46			*/
47			class ClassNameController extends AbstractController
48			{
49			/**
50			* @Inject("orm.em")
51			* @var EntityManager
52			*/
53			protected $entityManager;
54
55			/**
56			* @Inject("eccube.event.dispatcher")
57			* @var EventDispatcher
58			*/
59			protected $eventDispatcher;
60
61			/**
62			* @Inject("form.factory")
63			* @var FormFactory
64			*/
65			protected $formFactory;
66
67			/**
68			* @Inject(ClassNameRepository::class)
69			* @var ClassNameRepository
70			*/
71			protected $classNameRepository;
72
73			/**
			0 ignored issues – show introduced 2017-08-29 02:45 UTC by Report Bug Copy Issue Report Doc comment for parameter "$app" missing Loading history... introduced 2017-08-29 02:45 UTC by Report Bug Copy Issue Report Doc comment for parameter "$request" missing Loading history... introduced 2017-08-29 02:45 UTC by Report Bug Copy Issue Report Doc comment for parameter "$id" missing Loading history...
74			* @Route("/{_admin}/product/class_name", name="admin_product_class_name")
75			* @Route("/{_admin}/product/class_name/{id}/edit", requirements={"id" = "\d+"}, name="admin_product_class_name_edit")
76			* @Template("Product/class_name.twig")
77			*/
			0 ignored issues – show introduced 2017-08-29 02:45 UTC by Report Bug Copy Issue Report Missing @return tag in function comment Loading history...
78	3		public function index(Application $app, Request $request, $id = null)
79			{
80	3		if ($id) {
81	1		$TargetClassName = $this->classNameRepository->find($id);
82	1		if (!$TargetClassName) {
83	1		throw new NotFoundHttpException('商品規格が存在しません');
84			}
85			} else {
86	2		$TargetClassName = new \Eccube\Entity\ClassName();
87			}
88
89	3		$builder = $this->formFactory
90	3		->createBuilder(ClassNameType::class, $TargetClassName);
91
92	3		$event = new EventArgs(
93			array(
94	3		'builder' => $builder,
95	3		'TargetClassName' => $TargetClassName,
96			),
97	3		$request
98			);
99	3		$this->eventDispatcher->dispatch(EccubeEvents::ADMIN_PRODUCT_CLASS_NAME_INDEX_INITIALIZE, $event);
100
101	3		$form = $builder->getForm();
102
103	3	View Code Duplication	if ($request->getMethod() === 'POST') {
104	1		$form->handleRequest($request);
105	1		if ($form->isValid()) {
106	1		log_info('商品規格登録開始', array($id));
107
108	1		$this->classNameRepository->save($TargetClassName);
109
110	1		log_info('商品規格登録完了', array($id));
111
112	1		$event = new EventArgs(
113			array(
114	1		'form' => $form,
115	1		'TargetClassName' => $TargetClassName,
116			),
117	1		$request
118			);
119	1		$this->eventDispatcher->dispatch(EccubeEvents::ADMIN_PRODUCT_CLASS_NAME_INDEX_COMPLETE, $event);
120
121	1		$app->addSuccess('admin.class_name.save.complete', 'admin');
122
123	1		return $app->redirect($app->url('admin_product_class_name'));
124			}
125			}
126
127	2		$ClassNames = $this->classNameRepository->getList();
128
129			return [
130	2		'form' => $form->createView(),
131	2		'ClassNames' => $ClassNames,
132	2		'TargetClassName' => $TargetClassName,
133			];
134			}
135
136			/**
			0 ignored issues – show introduced 2017-08-29 02:45 UTC by Report Bug Copy Issue Report Doc comment for parameter "$app" missing Loading history... introduced 2017-08-29 02:45 UTC by Report Bug Copy Issue Report Doc comment for parameter "$request" missing Loading history... introduced 2017-09-25 09:08 UTC by Report Bug Copy Issue Report Doc comment for parameter "$ClassName" missing Loading history...
137			* @Method("DELETE")
138			* @Route("/{_admin}/product/class_name/{id}/delete", requirements={"id" = "\d+"}, name="admin_product_class_name_delete")
139			*/
			0 ignored issues – show introduced 2017-08-29 02:45 UTC by Report Bug Copy Issue Report Missing @return tag in function comment Loading history...
140	1	View Code Duplication	public function delete(Application $app, Request $request, ClassName $ClassName)
			0 ignored issues – show Duplication introduced 2017-09-25 09:08 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...
141			{
142	1		$this->isTokenValid($app);
143
144	1		log_info('商品規格削除開始', array($ClassName->getId()));
145
146			try {
147	1		$this->classNameRepository->delete($ClassName);
148
149	1		$event = new EventArgs(['ClassName' => $ClassName,], $request);
			0 ignored issues – show introduced 2017-09-25 09:08 UTC by Report Bug Copy Issue Report Add a single space after each comma delimiter Loading history...
150	1		$this->eventDispatcher->dispatch(EccubeEvents::ADMIN_PRODUCT_CLASS_NAME_DELETE_COMPLETE, $event);
151
152	1		$app->addSuccess('admin.class_name.delete.complete', 'admin');
153
154	1		log_info('商品規格削除完了', array($ClassName->getId()));
155
			0 ignored issues – show Coding Style introduced 2017-09-25 09:08 UTC by Report Bug Copy Issue Report Blank line found at end of control structure Loading history...
156			} catch (\Exception $e) {
157			$message = $app->trans('admin.delete.failed.foreign_key', ['%name%' => '商品規格']);
158			$app->addError($message, 'admin');
159
160			log_error('商品企画削除エラー', [$ClassName->getId(), $e]);
161			}
162
163	1		return $app->redirect($app->url('admin_product_class_name'));
164			}
165
166			/**
			0 ignored issues – show introduced 2017-08-29 02:45 UTC by Report Bug Copy Issue Report Doc comment for parameter "$app" missing Loading history... introduced 2017-08-29 02:45 UTC by Report Bug Copy Issue Report Doc comment for parameter "$request" missing Loading history...
167			* @Method("POST")
168			* @Route("/{_admin}/product/class_name/sort_no/move", name="admin_product_class_name_sort_no_move")
169			*/
			0 ignored issues – show introduced 2017-08-29 02:45 UTC by Report Bug Copy Issue Report Missing @return tag in function comment Loading history...
170	1	View Code Duplication	public function moveSortNo(Application $app, Request $request)
			0 ignored issues – show Unused Code introduced 2017-08-29 02:45 UTC by Report Bug Copy Issue Report The parameter `$app` 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... Duplication introduced 2017-08-02 04:20 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...
171			{
172	1		if ($request->isXmlHttpRequest()) {
173	1		$sortNos = $request->request->all();
174	1		foreach ($sortNos as $classNameId => $sortNo) {
175	1		$ClassName = $this->classNameRepository
176	1		->find($classNameId);
177	1		$ClassName->setSortNo($sortNo);
178	1		$this->entityManager->persist($ClassName);
			0 ignored issues – show Bug introduced 2017-08-29 02:45 UTC by Report Bug Copy Issue Report It seems like `$ClassName` defined by `$this->classNameRepository->find($classNameId)` on line `175` can also be of type `null`; however, `Doctrine\ORM\EntityManager::persist()` does only seem to accept `object`, maybe add an additional type check? If a method or function can return multiple different values and unless you are sure that you only can receive a single value in this context, we recommend to add an additional type check: /** * @return array\|string */ function returnsDifferentValues($x) { if ($x) { return 'foo'; } return array(); } $x = returnsDifferentValues($y); if (is_array($x)) { // $x is an array. } If this a common case that PHP Analyzer should handle natively, please let us know by opening an issue. Loading history...
179			}
180	1		$this->entityManager->flush();
181			}
182
183	1		return true;
184			}
185			}
186

EC-CUBE / ec-cube

Push — experimental/3.1 ( 878c99...05b497 )

ClassNameController::delete() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like