PaymentController::delete() - Code Metrics - Inspection of "Merge pull request #2522 from chihiro-adachi/repos..." - EC-CUBE/ec-cube - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Push — experimental/3.1 ( 3d2ede...2919b9 )

by Yangsin

created 2017-09-27 01:42 UTC

PaymentController::delete() B

↳ Parent: PaymentController

Complexity

Conditions	3
Paths	12

Size

Total Lines	32
Code Lines	20

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	14
CRAP Score	3.0987

Importance

Changes

Metric	Value
cc	3
eloc	20
nc	12
nop	3
dl	0
loc	32
ccs	14
cts	18
cp	0.7778
crap	3.0987
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\Setting\Shop;

use Doctrine\DBAL\Exception\ForeignKeyConstraintViolationException;
use Doctrine\ORM\EntityManager;
use Eccube\Annotation\Component;
use Eccube\Annotation\Inject;
use Eccube\Application;
use Eccube\Controller\AbstractController;
use Eccube\Entity\Payment;
use Eccube\Event\EccubeEvents;
use Eccube\Event\EventArgs;
use Eccube\Form\Type\Admin\PaymentRegisterType;
use Eccube\Repository\PaymentRepository;
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\Filesystem\Filesystem;
use Symfony\Component\Form\FormFactory;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpKernel\Exception\BadRequestHttpException;
use Symfony\Component\HttpKernel\Exception\UnsupportedMediaTypeHttpException;

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

    /**
     * @Inject("config")
     * @var array
     */
    protected $appConfig;

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

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

    /**
     * @Inject(PaymentRepository::class)
     * @var PaymentRepository
     */
    protected $paymentRepository;

    /**

     * @Route("/{_admin}/setting/shop/payment", name="admin_setting_shop_payment")
     * @Template("Setting/Shop/payment.twig")
     */

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

    {
        $Payments = $this->paymentRepository
            ->findBy(
                array(),
                array('rank' => 'DESC')
            );

        $event = new EventArgs(
            array(
                'Payments' => $Payments,
            ),
            $request
        );
        $this->eventDispatcher->dispatch(EccubeEvents::ADMIN_SETTING_SHOP_PAYMENT_INDEX_COMPLETE, $event);

        return [
            'Payments' => $Payments,
        ];
    }

    /**

     * @Route("/{_admin}/setting/shop/payment/new", name="admin_setting_shop_payment_new")
     * @Route("/{_admin}/setting/shop/payment/{id}/edit", requirements={"id" = "\d+"}, name="admin_setting_shop_payment_edit")
     * @Template("Setting/Shop/payment_edit.twig")
     */

    public function edit(Application $app, Request $request, Payment $Payment = null)
    {
        if (is_null($Payment)) {
            // FIXME
            $Payment = $this->paymentRepository

                ->findOrCreate(0);
        }

        $builder = $this->formFactory
            ->createBuilder(PaymentRegisterType::class, $Payment);

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

        $form = $builder->getForm();
        $form->handleRequest($request);

        // 登録ボタン押下
        if ($form->isSubmitted() && $form->isValid()) {
            $Payment = $form->getData();

            // 手数料を設定できない場合には、手数料を0にする
            if ($Payment->getChargeFlg() == 2) {
                $Payment->setCharge(0);
            }

            // ファイルアップロード
            $file = $form['payment_image']->getData();
            $fs = new Filesystem();
            if ($file && $fs->exists($this->appConfig['image_temp_realdir'].'/'.$file)) {
                $fs->rename(
                    $this->appConfig['image_temp_realdir'].'/'.$file,
                    $this->appConfig['image_save_realdir'].'/'.$file
                );
            }

            $Payment->setVisible(true);
            $this->entityManager->persist($Payment);
            $this->entityManager->flush();

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

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

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

        return [
            'form' => $form->createView(),
            'payment_id' => $Payment->getId(),
            'Payment' => $Payment,
        ];
    }

    /**

     * @Route("/{_admin}/setting/shop/payment/image/add", name="admin_payment_image_add")
     */

    public function imageAdd(Application $app, Request $request)
    {
        if (!$request->isXmlHttpRequest()) {
            throw new BadRequestHttpException();
        }

        $images = $request->files->get('payment_register');
        $filename = null;
        if (isset($images['payment_image_file'])) {
            $image = $images['payment_image_file'];

            //ファイルフォーマット検証
            $mimeType = $image->getMimeType();
            if (0 !== strpos($mimeType, 'image')) {
                throw new UnsupportedMediaTypeHttpException();
            }

            $extension = $image->guessExtension();
            $filename = date('mdHis').uniqid('_').'.'.$extension;
            $image->move($this->appConfig['image_temp_realdir'], $filename);
        }
        $event = new EventArgs(
            array(
                'images' => $images,
                'filename' => $filename,
            ),
            $request
        );
        $this->eventDispatcher->dispatch(EccubeEvents::ADMIN_SETTING_SHOP_PAYMENT_IMAGE_ADD_COMPLETE, $event);
        $filename = $event->getArgument('filename');

        return $app->json(array('filename' => $filename), 200);
    }

    /**

     * @Method("DELETE")
     * @Route("/{_admin}/setting/shop/payment/{id}/delete", requirements={"id" = "\d+"}, name="admin_setting_shop_payment_delete")
     */

    public function delete(Application $app, Request $request, Payment $TargetPayment)
    {
        $this->isTokenValid($app);

        $rank = 1;
        $Payments = $this->paymentRepository->findBy(array(), array('rank' => 'ASC'));
        foreach ($Payments as $Payment) {
                $Payment->setRank($rank++);
        }

        try {
            $this->paymentRepository->delete($TargetPayment);
            $this->entityManager->flush();

            $event = new EventArgs(
                array(
                    'Payment' => $TargetPayment,
                ),
                $request
            );
            $this->eventDispatcher->dispatch(EccubeEvents::ADMIN_SETTING_SHOP_PAYMENT_DELETE_COMPLETE, $event);

            $app->addSuccess('admin.delete.complete', 'admin');
        } catch(ForeignKeyConstraintViolationException $e) {
            $this->entityManager->rollback();

            $message = $app->trans('admin.delete.failed.foreign_key', ['%name%' => '支払方法']);
            $app->addError($message, 'admin');
        }

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

    /**

     * @Method("PUT")
     * @Route("/{_admin}/setting/shop/payment/{id}/up", requirements={"id" = "\d+"}, name="admin_setting_shop_payment_up")
     */

    public function up(Application $app, Payment $current)

    {
        $this->isTokenValid($app);

        $currentRank = $current->getRank();
        $targetRank = $currentRank + 1;

        $target = $this->paymentRepository->findOneBy(array('rank' => $targetRank));

        $target->setRank($currentRank);
        $current->setRank($targetRank);

        $this->entityManager->flush();

        $app->addSuccess('admin.rank.move.complete', 'admin');

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

    /**

     * @Method("PUT")
     * @Route("/{_admin}/setting/shop/payment/{id}/down", requirements={"id" = "\d+"}, name="admin_setting_shop_payment_down")
     */

    public function down(Application $app, Payment $current)

    {
        $this->isTokenValid($app);

        $currentRank = $current->getRank();
        $targetRank = $currentRank - 1;

        $target = $this->paymentRepository->findOneBy(array('rank' => $targetRank));

        $target->setRank($currentRank);
        $current->setRank($targetRank);

        $this->entityManager->flush();

        $app->addSuccess('admin.rank.move.complete', 'admin');

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

    /**

     * @Method("PUT")
     * @Route("/{_admin}/setting/shop/payment/{id}/visible", requirements={"id" = "\d+"}, name="admin_setting_shop_payment_visible")
     */

    public function visible(Application $app, Payment $Payment)
    {
        $this->isTokenValid($app);

        $Payment->setVisible(!$Payment->isVisible());

        $this->entityManager->flush();

        if ($Payment->isVisible()) {
            $app->addSuccess('admin.payment.visible.complete', 'admin');
        } else {
            $app->addSuccess('admin.payment.invisible.complete', 'admin');
        }

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


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\Setting\Shop;
26
27			use Doctrine\DBAL\Exception\ForeignKeyConstraintViolationException;
28			use Doctrine\ORM\EntityManager;
29			use Eccube\Annotation\Component;
30			use Eccube\Annotation\Inject;
31			use Eccube\Application;
32			use Eccube\Controller\AbstractController;
33			use Eccube\Entity\Payment;
34			use Eccube\Event\EccubeEvents;
35			use Eccube\Event\EventArgs;
36			use Eccube\Form\Type\Admin\PaymentRegisterType;
37			use Eccube\Repository\PaymentRepository;
38			use Sensio\Bundle\FrameworkExtraBundle\Configuration\Method;
39			use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
40			use Sensio\Bundle\FrameworkExtraBundle\Configuration\Template;
41			use Symfony\Component\EventDispatcher\EventDispatcher;
42			use Symfony\Component\Filesystem\Filesystem;
43			use Symfony\Component\Form\FormFactory;
44			use Symfony\Component\HttpFoundation\Request;
45			use Symfony\Component\HttpKernel\Exception\BadRequestHttpException;
46			use Symfony\Component\HttpKernel\Exception\UnsupportedMediaTypeHttpException;
47
48			/**
49			* @Component
50			* @Route(service=PaymentController::class)
51			*/
52			class PaymentController extends AbstractController
53			{
54			/**
55			* @Inject("orm.em")
56			* @var EntityManager
57			*/
58			protected $entityManager;
59
60			/**
61			* @Inject("config")
62			* @var array
63			*/
64			protected $appConfig;
65
66			/**
67			* @Inject("form.factory")
68			* @var FormFactory
69			*/
70			protected $formFactory;
71
72			/**
73			* @Inject("eccube.event.dispatcher")
74			* @var EventDispatcher
75			*/
76			protected $eventDispatcher;
77
78			/**
79			* @Inject(PaymentRepository::class)
80			* @var PaymentRepository
81			*/
82			protected $paymentRepository;
83
84			/**
			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...
85			* @Route("/{_admin}/setting/shop/payment", name="admin_setting_shop_payment")
86			* @Template("Setting/Shop/payment.twig")
87			*/
			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...
88	1	View Code Duplication	public function index(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...
89			{
90	1		$Payments = $this->paymentRepository
91	1		->findBy(
92	1		array(),
93	1		array('rank' => 'DESC')
94			);
95
96	1		$event = new EventArgs(
97			array(
98	1		'Payments' => $Payments,
99			),
100	1		$request
101			);
102	1		$this->eventDispatcher->dispatch(EccubeEvents::ADMIN_SETTING_SHOP_PAYMENT_INDEX_COMPLETE, $event);
103
104			return [
105	1		'Payments' => $Payments,
106			];
107			}
108
109			/**
			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 "$Payment" missing Loading history...
110			* @Route("/{_admin}/setting/shop/payment/new", name="admin_setting_shop_payment_new")
111			* @Route("/{_admin}/setting/shop/payment/{id}/edit", requirements={"id" = "\d+"}, name="admin_setting_shop_payment_edit")
112			* @Template("Setting/Shop/payment_edit.twig")
113			*/
			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...
114	6		public function edit(Application $app, Request $request, Payment $Payment = null)
115			{
116	6		if (is_null($Payment)) {
117			// FIXME
118	3		$Payment = $this->paymentRepository
			0 ignored issues – show Deprecated Code introduced 2017-08-29 02:45 UTC by Report Bug Copy Issue Report The method `Eccube\Repository\Paymen...ository::findOrCreate()` has been deprecated with message: 呼び出し元で制御する This method has been deprecated. The supplier of the class has supplied an explanatory message. The explanatory message should give you some clue as to whether and when the method will be removed from the class and what other method or class to use instead. Loading history...
119	3		->findOrCreate(0);
120			}
121
122	6		$builder = $this->formFactory
123	6		->createBuilder(PaymentRegisterType::class, $Payment);
124
125	6		$event = new EventArgs(
126			array(
127	6		'builder' => $builder,
128	6		'Payment' => $Payment,
129			),
130	6		$request
131			);
132	6		$this->eventDispatcher->dispatch(EccubeEvents::ADMIN_SETTING_SHOP_PAYMENT_EDIT_INITIALIZE, $event);
133
134	6		$form = $builder->getForm();
135	6		$form->handleRequest($request);
136
137			// 登録ボタン押下
138	6		if ($form->isSubmitted() && $form->isValid()) {
139	2		$Payment = $form->getData();
140
141			// 手数料を設定できない場合には、手数料を0にする
142	2		if ($Payment->getChargeFlg() == 2) {
143			$Payment->setCharge(0);
144			}
145
146			// ファイルアップロード
147	2		$file = $form['payment_image']->getData();
148	2		$fs = new Filesystem();
149	2		if ($file && $fs->exists($this->appConfig['image_temp_realdir'].'/'.$file)) {
150			$fs->rename(
151			$this->appConfig['image_temp_realdir'].'/'.$file,
152			$this->appConfig['image_save_realdir'].'/'.$file
153			);
154			}
155
156	2		$Payment->setVisible(true);
157	2		$this->entityManager->persist($Payment);
158	2		$this->entityManager->flush();
159
160	2		$event = new EventArgs(
161			array(
162	2		'form' => $form,
163	2		'Payment' => $Payment,
164			),
165	2		$request
166			);
167	2		$this->eventDispatcher->dispatch(EccubeEvents::ADMIN_SETTING_SHOP_PAYMENT_EDIT_COMPLETE, $event);
168
169	2		$app->addSuccess('admin.register.complete', 'admin');
170
171	2		return $app->redirect($app->url('admin_setting_shop_payment'));
172			}
173
174			return [
175	4		'form' => $form->createView(),
176	4		'payment_id' => $Payment->getId(),
177	4		'Payment' => $Payment,
178			];
179			}
180
181			/**
			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...
182			* @Route("/{_admin}/setting/shop/payment/image/add", name="admin_payment_image_add")
183			*/
			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...
184	2		public function imageAdd(Application $app, Request $request)
185			{
186	2		if (!$request->isXmlHttpRequest()) {
187	1		throw new BadRequestHttpException();
188			}
189
190	1		$images = $request->files->get('payment_register');
191	1		$filename = null;
192	1		if (isset($images['payment_image_file'])) {
193			$image = $images['payment_image_file'];
194
195			//ファイルフォーマット検証
196			$mimeType = $image->getMimeType();
197			if (0 !== strpos($mimeType, 'image')) {
198			throw new UnsupportedMediaTypeHttpException();
199			}
200
201			$extension = $image->guessExtension();
202			$filename = date('mdHis').uniqid('_').'.'.$extension;
203			$image->move($this->appConfig['image_temp_realdir'], $filename);
204			}
205	1		$event = new EventArgs(
206			array(
207	1		'images' => $images,
208	1		'filename' => $filename,
209			),
210	1		$request
211			);
212	1		$this->eventDispatcher->dispatch(EccubeEvents::ADMIN_SETTING_SHOP_PAYMENT_IMAGE_ADD_COMPLETE, $event);
213	1		$filename = $event->getArgument('filename');
214
215	1		return $app->json(array('filename' => $filename), 200);
216			}
217
218			/**
			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 "$TargetPayment" missing Loading history...
219			* @Method("DELETE")
220			* @Route("/{_admin}/setting/shop/payment/{id}/delete", requirements={"id" = "\d+"}, name="admin_setting_shop_payment_delete")
221			*/
			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...
222	1		public function delete(Application $app, Request $request, Payment $TargetPayment)
223			{
224	1		$this->isTokenValid($app);
225
226	1		$rank = 1;
227	1		$Payments = $this->paymentRepository->findBy(array(), array('rank' => 'ASC'));
228	1		foreach ($Payments as $Payment) {
229	1		$Payment->setRank($rank++);
230			}
231
232			try {
233	1		$this->paymentRepository->delete($TargetPayment);
234	1		$this->entityManager->flush();
235
236	1		$event = new EventArgs(
237			array(
238	1		'Payment' => $TargetPayment,
239			),
240	1		$request
241			);
242	1		$this->eventDispatcher->dispatch(EccubeEvents::ADMIN_SETTING_SHOP_PAYMENT_DELETE_COMPLETE, $event);
243
244	1		$app->addSuccess('admin.delete.complete', 'admin');
245			} catch(ForeignKeyConstraintViolationException $e) {
246			$this->entityManager->rollback();
247
248			$message = $app->trans('admin.delete.failed.foreign_key', ['%name%' => '支払方法']);
249			$app->addError($message, 'admin');
250			}
251
252	1		return $app->redirect($app->url('admin_setting_shop_payment'));
253			}
254
255			/**
			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 "$current" missing Loading history...
256			* @Method("PUT")
257			* @Route("/{_admin}/setting/shop/payment/{id}/up", requirements={"id" = "\d+"}, name="admin_setting_shop_payment_up")
258			*/
			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...
259	1	View Code Duplication	public function up(Application $app, Payment $current)
			0 ignored issues – show Duplication introduced 2017-08-29 02:45 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...
260			{
261	1		$this->isTokenValid($app);
262
263	1		$currentRank = $current->getRank();
264	1		$targetRank = $currentRank + 1;
265
266	1		$target = $this->paymentRepository->findOneBy(array('rank' => $targetRank));
267
268	1		$target->setRank($currentRank);
269	1		$current->setRank($targetRank);
270
271	1		$this->entityManager->flush();
272
273	1		$app->addSuccess('admin.rank.move.complete', 'admin');
274
275	1		return $app->redirect($app->url('admin_setting_shop_payment'));
276			}
277
278			/**
			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 "$current" missing Loading history...
279			* @Method("PUT")
280			* @Route("/{_admin}/setting/shop/payment/{id}/down", requirements={"id" = "\d+"}, name="admin_setting_shop_payment_down")
281			*/
			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...
282	1	View Code Duplication	public function down(Application $app, Payment $current)
			0 ignored issues – show Duplication introduced 2017-08-29 02:45 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...
283			{
284	1		$this->isTokenValid($app);
285
286	1		$currentRank = $current->getRank();
287	1		$targetRank = $currentRank - 1;
288
289	1		$target = $this->paymentRepository->findOneBy(array('rank' => $targetRank));
290
291	1		$target->setRank($currentRank);
292	1		$current->setRank($targetRank);
293
294	1		$this->entityManager->flush();
295
296	1		$app->addSuccess('admin.rank.move.complete', 'admin');
297
298	1		return $app->redirect($app->url('admin_setting_shop_payment'));
299			}
300
301			/**
			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-09-19 02:32 UTC by Report Bug Copy Issue Report Doc comment for parameter "$Payment" missing Loading history...
302			* @Method("PUT")
303			* @Route("/{_admin}/setting/shop/payment/{id}/visible", requirements={"id" = "\d+"}, name="admin_setting_shop_payment_visible")
304			*/
			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...
305			public function visible(Application $app, Payment $Payment)
306			{
307			$this->isTokenValid($app);
308
309			$Payment->setVisible(!$Payment->isVisible());
310
311			$this->entityManager->flush();
312
313			if ($Payment->isVisible()) {
314			$app->addSuccess('admin.payment.visible.complete', 'admin');
315			} else {
316			$app->addSuccess('admin.payment.invisible.complete', 'admin');
317			}
318
319			return $app->redirect($app->url('admin_setting_shop_payment'));
320			}
321			}
322

EC-CUBE / ec-cube

Push — experimental/3.1 ( 3d2ede...2919b9 )

PaymentController::delete() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like