Eccube\Controller\Admin\Customer\CustomerEditController

Complexity

Total Complexity

10

Size/Duplication

Total Lines	118
Duplicated Lines	7.63 %

Coupling/Cohesion

Components	1
Dependencies	12

Test Coverage

Coverage

92.31%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
C	index()	9	92	9

<?php

/*
 * This file is part of EC-CUBE
 *
 * Copyright(c) LOCKON CO.,LTD. All Rights Reserved.
 *
 * http://www.lockon.co.jp/
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Eccube\Controller\Admin\Customer;

use Eccube\Controller\AbstractController;
use Eccube\Event\EccubeEvents;
use Eccube\Event\EventArgs;
use Eccube\Entity\Master\CustomerStatus;
use Eccube\Form\Type\Admin\CustomerType;
use Eccube\Repository\CustomerRepository;
use Eccube\Util\StringUtil;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Template;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
use Symfony\Component\Security\Core\Encoder\EncoderFactoryInterface;

class CustomerEditController extends AbstractController

Missing class doc comment

{
    /**
     * @var CustomerRepository
     */
    protected $customerRepository;

    /**
     * @var EncoderFactoryInterface
     */
    protected $encoderFactory;

    public function __construct(
        CustomerRepository $customerRepository,
        EncoderFactoryInterface $encoderFactory
    ) {
        $this->customerRepository = $customerRepository;
        $this->encoderFactory = $encoderFactory;
    }

    /**

Doc comment for parameter "$request" missing

Doc comment for parameter "$id" missing

     * @Route("/%eccube_admin_route%/customer/new", name="admin_customer_new")
     * @Route("/%eccube_admin_route%/customer/{id}/edit", requirements={"id" = "\d+"}, name="admin_customer_edit")
     * @Template("@admin/Customer/edit.twig")
     */

Missing @return tag in function comment

    public function index(Request $request, $id = null)
    {
        //$this->entityManager->getFilters()->enable('incomplete_order_status_hidden');
        // 編集
        if ($id) {
            $Customer = $this->customerRepository
                ->find($id);

            if (is_null($Customer)) {
                throw new NotFoundHttpException();
            }

            $oldStatusId = $Customer->getStatus()->getId();
            // 編集用にデフォルトパスワードをセット
            $previous_password = $Customer->getPassword();
            $Customer->setPassword($this->eccubeConfig['eccube_default_password']);
        // 新規登録
        } else {
            $Customer = $this->customerRepository->newCustomer();
            $Customer->setBuyTimes(0);
            $Customer->setBuyTotal(0);
            $oldStatusId = null;
        }

        // 会員登録フォーム
        $builder = $this->formFactory
            ->createBuilder(CustomerType::class, $Customer);

        $event = new EventArgs(
            [
                'builder' => $builder,
                'Customer' => $Customer,
            ],
            $request
        );
        $this->eventDispatcher->dispatch(EccubeEvents::ADMIN_CUSTOMER_EDIT_INDEX_INITIALIZE, $event);

        $form = $builder->getForm();

        $form->handleRequest($request);

        if ($form->isSubmitted()) {
            if ($form->isValid()) {
                log_info('会員登録開始', [$Customer->getId()]);

                $encoder = $this->encoderFactory->getEncoder($Customer);

                if ($Customer->getPassword() === $this->eccubeConfig['eccube_default_password']) {
                    $Customer->setPassword($previous_password);

The variable $previous_password does not seem to be defined for all execution paths leading up to this point.

                } else {
                    if ($Customer->getSalt() === null) {
                        $Customer->setSalt($encoder->createSalt());
                        $Customer->setSecretKey($this->customerRepository->getUniqueSecretKey());
                    }
                    $Customer->setPassword($encoder->encodePassword($Customer->getPassword(), $Customer->getSalt()));
                }

                // 退会ステータスに更新の場合、ダミーのアドレスに更新
                $newStatusId = $Customer->getStatus()->getId();
                if ($oldStatusId != $newStatusId && $newStatusId == CustomerStatus::WITHDRAWING) {
                    $Customer->setEmail(StringUtil::random(60).'@dummy.dummy');
                }

                $this->entityManager->persist($Customer);
                $this->entityManager->flush();

                log_info('会員登録完了', [$Customer->getId()]);

                $event = new EventArgs(
                    [
                        'form' => $form,
                        'Customer' => $Customer,
                    ],
                    $request
                );
                $this->eventDispatcher->dispatch(EccubeEvents::ADMIN_CUSTOMER_EDIT_INDEX_COMPLETE, $event);

                $this->addSuccess('admin.customer.save.complete', 'admin');

                return $this->redirectToRoute('admin_customer_edit', [
                    'id' => $Customer->getId(),
                ]);
            } else {
                $this->addError('admin.customer.save.failed', 'admin');
            }
        }

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