CustomerRepository::getUniqueSecretKey() - Code Metrics - Inspection of "Merge pull request #3345 from chihiro-adachi/nanas..." - EC-CUBE/ec-cube - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Push — experimental/sf ( c0fd88...ff71e4 )

unknown

created 2018-07-17 10:04 UTC

CustomerRepository::getUniqueSecretKey() A

↳ Parent: CustomerRepository

Complexity

Conditions	2
Paths	1

Size

Total Lines

Duplication

Lines	9
Ratio	100 %

Code Coverage

Tests	5
CRAP Score	2

Importance

Changes

Metric	Value
cc	2
nc	1
nop	0
dl	9
loc	9
rs	9.9666
c	0
b	0
f	0
ccs	5
cts	5
cp	1
crap	2

<?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\Repository;

use Doctrine\ORM\EntityManagerInterface;
use Eccube\Common\EccubeConfig;
use Eccube\Doctrine\Query\Queries;
use Eccube\Entity\Customer;
use Eccube\Entity\Master\CustomerStatus;
use Eccube\Entity\Master\OrderStatus;
use Eccube\Util\StringUtil;
use Symfony\Bridge\Doctrine\RegistryInterface;
use Symfony\Component\Security\Core\Encoder\EncoderFactoryInterface;

/**
 * CustomerRepository
 *
 * This class was generated by the Doctrine ORM. Add your own custom
 * repository methods below.
 */
class CustomerRepository extends AbstractRepository
{
    /**
     * @var Queries
     */
    protected $queries;

    /**
     * @var EntityManagerInterface
     */
    protected $entityManager;

    /**
     * @var OrderRepository
     */
    protected $orderRepository;

    /**
     * @var EccubeConfig
     */
    protected $eccubeConfig;

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

    /**
     * CustomerRepository constructor.
     *
     * @param RegistryInterface $registry

     * @param Queries $queries

     * @param EntityManagerInterface $entityManager

     * @param OrderRepository $orderRepository

     * @param EncoderFactoryInterface $encoderFactory
     * @param EccubeConfig $eccubeConfig

     */
    public function __construct(
        RegistryInterface $registry,
        Queries $queries,
        EntityManagerInterface $entityManager,
        OrderRepository $orderRepository,
        EncoderFactoryInterface $encoderFactory,
        EccubeConfig $eccubeConfig
    ) {
        parent::__construct($registry, Customer::class);

        $this->queries = $queries;
        $this->entityManager = $entityManager;
        $this->orderRepository = $orderRepository;
        $this->encoderFactory = $encoderFactory;
        $this->eccubeConfig = $eccubeConfig;
    }

    public function newCustomer()
    {
        $CustomerStatus = $this->getEntityManager()
            ->find(CustomerStatus::class, CustomerStatus::PROVISIONAL);

        $Customer = new \Eccube\Entity\Customer();
        $Customer
            ->setStatus($CustomerStatus);

        return $Customer;
    }

    public function getQueryBuilderBySearchData($searchData)
    {
        $qb = $this->createQueryBuilder('c')
            ->select('c');

        if (isset($searchData['multi']) && StringUtil::isNotBlank($searchData['multi'])) {
            //スペース除去
            $clean_key_multi = preg_replace('/\s+|[　]+/u', '', $searchData['multi']);
            $id = preg_match('/^\d{0,10}$/', $clean_key_multi) ? $clean_key_multi : null;
            $qb
                ->andWhere('c.id = :customer_id OR CONCAT(c.name01, c.name02) LIKE :name OR CONCAT(c.kana01, c.kana02) LIKE :kana OR c.email LIKE :email')
                ->setParameter('customer_id', $id)
                ->setParameter('name', '%'.$clean_key_multi.'%')
                ->setParameter('kana', '%'.$clean_key_multi.'%')
                ->setParameter('email', '%'.$clean_key_multi.'%');
        }

        // Pref
        if (!empty($searchData['pref']) && $searchData['pref']) {
            $qb
                ->andWhere('c.Pref = :pref')
                ->setParameter('pref', $searchData['pref']->getId());
        }

        // sex
        if (!empty($searchData['sex']) && count($searchData['sex']) > 0) {
            $sexs = [];
            foreach ($searchData['sex'] as $sex) {
                $sexs[] = $sex->getId();
            }

            $qb
                ->andWhere($qb->expr()->in('c.Sex', ':sexs'))
                ->setParameter('sexs', $sexs);
        }

        if (!empty($searchData['birth_month']) && $searchData['birth_month']) {
            $qb
                ->andWhere('EXTRACT(MONTH FROM c.birth) = :birth_month')
                ->setParameter('birth_month', $searchData['birth_month']);
        }

        // birth
        if (!empty($searchData['birth_start']) && $searchData['birth_start']) {
            $qb
                ->andWhere('c.birth >= :birth_start')
                ->setParameter('birth_start', $searchData['birth_start']);
        }
        if (!empty($searchData['birth_end']) && $searchData['birth_end']) {
            $date = clone $searchData['birth_end'];
            $date->modify('+1 days');
            $qb
                ->andWhere('c.birth < :birth_end')
                ->setParameter('birth_end', $date);
        }

        // tel
        if (isset($searchData['phone_number']) && StringUtil::isNotBlank($searchData['phone_number'])) {
            $tel = preg_replace('/[^0-9]/ ', '', $searchData['phone_number']);
            $qb
                ->andWhere('c.phone_number LIKE :phone_number')
                ->setParameter('phone_number', '%'.$tel.'%');
        }

        // buy_total
        if (isset($searchData['buy_total_start']) && StringUtil::isNotBlank($searchData['buy_total_start'])) {
            $qb
                ->andWhere('c.buy_total >= :buy_total_start')
                ->setParameter('buy_total_start', $searchData['buy_total_start']);
        }
        if (isset($searchData['buy_total_end']) && StringUtil::isNotBlank($searchData['buy_total_end'])) {
            $qb
                ->andWhere('c.buy_total <= :buy_total_end')
                ->setParameter('buy_total_end', $searchData['buy_total_end']);
        }

        // buy_times
        if (!empty($searchData['buy_times_start']) && $searchData['buy_times_start']) {
            $qb
                ->andWhere('c.buy_times >= :buy_times_start')
                ->setParameter('buy_times_start', $searchData['buy_times_start']);
        }
        if (!empty($searchData['buy_times_end']) && $searchData['buy_times_end']) {
            $qb
                ->andWhere('c.buy_times <= :buy_times_end')
                ->setParameter('buy_times_end', $searchData['buy_times_end']);
        }

        // create_date
        if (!empty($searchData['create_date_start']) && $searchData['create_date_start']) {
            $qb
                ->andWhere('c.create_date >= :create_date_start')
                ->setParameter('create_date_start', $searchData['create_date_start']);
        }
        if (!empty($searchData['create_date_end']) && $searchData['create_date_end']) {
            $date = clone $searchData['create_date_end'];
            $date->modify('+1 days');
            $qb
                ->andWhere('c.create_date < :create_date_end')
                ->setParameter('create_date_end', $date);
        }

        // update_date
        if (!empty($searchData['update_date_start']) && $searchData['update_date_start']) {
            $qb
                ->andWhere('c.update_date >= :update_date_start')
                ->setParameter('update_date_start', $searchData['update_date_start']);
        }
        if (!empty($searchData['update_date_end']) && $searchData['update_date_end']) {
            $date = clone $searchData['update_date_end'];
            $date->modify('+1 days');
            $qb
                ->andWhere('c.update_date < :update_date_end')
                ->setParameter('update_date_end', $date);
        }

        // last_buy
        if (!empty($searchData['last_buy_start']) && $searchData['last_buy_start']) {
            $qb
                ->andWhere('c.last_buy_date >= :last_buy_start')
                ->setParameter('last_buy_start', $searchData['last_buy_start']);
        }
        if (!empty($searchData['last_buy_end']) && $searchData['last_buy_end']) {
            $date = clone $searchData['last_buy_end'];
            $date->modify('+1 days');
            $qb
                ->andWhere('c.last_buy_date < :last_buy_end')
                ->setParameter('last_buy_end', $date);
        }

        // status
        if (!empty($searchData['customer_status']) && count($searchData['customer_status']) > 0) {
            $qb
                ->andWhere($qb->expr()->in('c.Status', ':statuses'))
                ->setParameter('statuses', $searchData['customer_status']);
        }

        // buy_product_name
        if (isset($searchData['buy_product_name']) && StringUtil::isNotBlank($searchData['buy_product_name'])) {
            $qb
                ->leftJoin('c.Orders', 'o')
                ->leftJoin('o.OrderItems', 'oi')
                ->andWhere('oi.product_name LIKE :buy_product_name')
                ->setParameter('buy_product_name', '%'.$searchData['buy_product_name'].'%');
        }

        // Order By
        $qb->addOrderBy('c.update_date', 'DESC');

        return $this->queries->customize(QueryKey::CUSTOMER_SEARCH, $qb, $searchData);
    }

    /**
     * ユニークなシークレットキーを返す.
     *
     * @return string
     */
    public function getUniqueSecretKey()

    {
        do {
            $key = StringUtil::random(32);
            $Customer = $this->findOneBy(['secret_key' => $key]);
        } while ($Customer);

        return $key;
    }

    /**
     * ユニークなパスワードリセットキーを返す
     *
     * @return string
     */
    public function getUniqueResetKey()

    {
        do {
            $key = StringUtil::random(32);
            $Customer = $this->findOneBy(['reset_key' => $key]);
        } while ($Customer);

        return $key;
    }

    /**

     * 仮会員をシークレットキーで検索する.
     *
     * @param $secretKey

     *
     * @return null|Customer 見つからない場合はnullを返す.
     */
    public function getProvisionalCustomerBySecretKey($secretKey)
    {
        return $this->findOneBy([
            'secret_key' => $secretKey,
            'Status' => CustomerStatus::PROVISIONAL,
        ]);
    }

    /**

     * 本会員をemailで検索する.
     *
     * @param $email

     *
     * @return null|Customer 見つからない場合はnullを返す.
     */
    public function getRegularCustomerByEmail($email)
    {
        return $this->findOneBy([
            'email' => $email,
            'Status' => CustomerStatus::REGULAR,
        ]);
    }

    /**

     * 本会員をリセットキーで検索する.
     *
     * @param $resetKey

     *
     * @return null|Customer 見つからない場合はnullを返す.
     */
    public function getRegularCustomerByResetKey($resetKey)
    {
        return $this->createQueryBuilder('c')
            ->where('c.reset_key = :reset_key AND c.Status = :status AND c.reset_expire >= :reset_expire')
            ->setParameter('reset_key', $resetKey)
            ->setParameter('status', CustomerStatus::REGULAR)
            ->setParameter('reset_expire', new \DateTime())
            ->setMaxResults(1)
            ->getQuery()
            ->getOneOrNullResult();
    }

    /**
     * リセット用パスワードを生成する.
     *
     * @return string
     */
    public function getResetPassword()
    {
        return StringUtil::random(8);
    }

    /**
     * 会員の初回購入時間、購入時間、購入回数、購入金額を更新する
     *
     * @param Customer $Customer
     * @param null $isNewOrder

     */
    public function updateBuyData(Customer $Customer, $isNewOrder = null)
    {
        // 会員の場合、初回購入時間・購入時間・購入回数・購入金額を更新

        $arr = [
            OrderStatus::NEW,
            OrderStatus::PAY_WAIT,
            OrderStatus::BACK_ORDER,
            OrderStatus::DELIVERED,
            OrderStatus::PAID,
        ];

        $result = $this->orderRepository->getCustomerCount($Customer, $arr);

        if (!empty($result)) {
            $data = $result[0];

            $now = new \DateTime();

            $firstBuyDate = $Customer->getFirstBuyDate();
            if (empty($firstBuyDate)) {
                $Customer->setFirstBuyDate($now);
            }

            if ($isNewOrder) {
                $Customer->setLastBuyDate($now);
            } else {
                $Order = $this->orderRepository->find($data['order_id']);
                if ($Order) {
                    $Customer->setLastBuyDate($Order->getOrderDate());
                }
            }

            $Customer->setBuyTimes($data['buy_times']);
            $Customer->setBuyTotal($data['buy_total']);
        } else {
            // 受注データが存在しなければ初期化
            $Customer->setFirstBuyDate(null);
            $Customer->setLastBuyDate(null);
            $Customer->setBuyTimes(0);
            $Customer->setBuyTotal(0);
        }

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

    /**
     * 仮会員, 本会員の会員を返す.
     * Eccube\Entity\CustomerのUniqueEntityバリデーションで使用しています.
     *
     * @param array $criteria
     *
     * @return Customer[]
     */
    public function getNonWithdrawingCustomers(array $criteria = [])
    {
        $criteria['Status'] = [
            CustomerStatus::PROVISIONAL,
            CustomerStatus::REGULAR,
        ];

        return $this->findBy($criteria);
    }
}


1			<?php
2
3			/*
4			* This file is part of EC-CUBE
5			*
6			* Copyright(c) LOCKON CO.,LTD. All Rights Reserved.
7			*
8			* http://www.lockon.co.jp/
9			*
10			* For the full copyright and license information, please view the LICENSE
11			* file that was distributed with this source code.
12			*/
13
14			namespace Eccube\Repository;
15
16			use Doctrine\ORM\EntityManagerInterface;
17			use Eccube\Common\EccubeConfig;
18			use Eccube\Doctrine\Query\Queries;
19			use Eccube\Entity\Customer;
20			use Eccube\Entity\Master\CustomerStatus;
21			use Eccube\Entity\Master\OrderStatus;
22			use Eccube\Util\StringUtil;
23			use Symfony\Bridge\Doctrine\RegistryInterface;
24			use Symfony\Component\Security\Core\Encoder\EncoderFactoryInterface;
25
26			/**
27			* CustomerRepository
28			*
29			* This class was generated by the Doctrine ORM. Add your own custom
30			* repository methods below.
31			*/
32			class CustomerRepository extends AbstractRepository
33			{
34			/**
35			* @var Queries
36			*/
37			protected $queries;
38
39			/**
40			* @var EntityManagerInterface
41			*/
42			protected $entityManager;
43
44			/**
45			* @var OrderRepository
46			*/
47			protected $orderRepository;
48
49			/**
50			* @var EccubeConfig
51			*/
52			protected $eccubeConfig;
53
54			/**
55			* @var EncoderFactoryInterface
56			*/
57			protected $encoderFactory;
58
59			/**
60			* CustomerRepository constructor.
61			*
62			* @param RegistryInterface $registry
			0 ignored issues – show introduced 2018-06-08 04:37 UTC by Report Bug Copy Issue Report Expected 7 spaces after parameter type; 1 found Loading history...
63			* @param Queries $queries
			0 ignored issues – show introduced 2018-06-08 04:37 UTC by Report Bug Copy Issue Report Expected 17 spaces after parameter type; 1 found Loading history...
64			* @param EntityManagerInterface $entityManager
			0 ignored issues – show introduced 2018-06-08 04:37 UTC by Report Bug Copy Issue Report Expected 2 spaces after parameter type; 1 found Loading history...
65			* @param OrderRepository $orderRepository
			0 ignored issues – show introduced 2018-06-08 04:37 UTC by Report Bug Copy Issue Report Expected 9 spaces after parameter type; 1 found Loading history...
66			* @param EncoderFactoryInterface $encoderFactory
67			* @param EccubeConfig $eccubeConfig
			0 ignored issues – show introduced 2018-06-08 04:37 UTC by Report Bug Copy Issue Report Expected 12 spaces after parameter type; 1 found Loading history...
68			*/
69	746		public function __construct(
70			RegistryInterface $registry,
71			Queries $queries,
72			EntityManagerInterface $entityManager,
73			OrderRepository $orderRepository,
74			EncoderFactoryInterface $encoderFactory,
75			EccubeConfig $eccubeConfig
76			) {
77	746		parent::__construct($registry, Customer::class);
78
79	746		$this->queries = $queries;
80	746		$this->entityManager = $entityManager;
81	746		$this->orderRepository = $orderRepository;
82	746		$this->encoderFactory = $encoderFactory;
83	746		$this->eccubeConfig = $eccubeConfig;
84			}
85
86	8		public function newCustomer()
87			{
88	8		$CustomerStatus = $this->getEntityManager()
89	8		->find(CustomerStatus::class, CustomerStatus::PROVISIONAL);
90
91	8		$Customer = new \Eccube\Entity\Customer();
92			$Customer
93	8		->setStatus($CustomerStatus);
94
95	8		return $Customer;
96			}
97
98	41		public function getQueryBuilderBySearchData($searchData)
99			{
100	41		$qb = $this->createQueryBuilder('c')
101	41		->select('c');
102
103	41		if (isset($searchData['multi']) && StringUtil::isNotBlank($searchData['multi'])) {
104			//スペース除去
105	14		$clean_key_multi = preg_replace('/\s+\|[　]+/u', '', $searchData['multi']);
106	14		$id = preg_match('/^\d{0,10}$/', $clean_key_multi) ? $clean_key_multi : null;
107			$qb
108	14		->andWhere('c.id = :customer_id OR CONCAT(c.name01, c.name02) LIKE :name OR CONCAT(c.kana01, c.kana02) LIKE :kana OR c.email LIKE :email')
109	14		->setParameter('customer_id', $id)
110	14		->setParameter('name', '%'.$clean_key_multi.'%')
111	14		->setParameter('kana', '%'.$clean_key_multi.'%')
112	14		->setParameter('email', '%'.$clean_key_multi.'%');
113			}
114
115			// Pref
116	41		if (!empty($searchData['pref']) && $searchData['pref']) {
117			$qb
118	1		->andWhere('c.Pref = :pref')
119	1		->setParameter('pref', $searchData['pref']->getId());
120			}
121
122			// sex
123	41		if (!empty($searchData['sex']) && count($searchData['sex']) > 0) {
124	2		$sexs = [];
125	2		foreach ($searchData['sex'] as $sex) {
126	2		$sexs[] = $sex->getId();
127			}
128
129			$qb
130	2		->andWhere($qb->expr()->in('c.Sex', ':sexs'))
131	2		->setParameter('sexs', $sexs);
132			}
133
134	41		if (!empty($searchData['birth_month']) && $searchData['birth_month']) {
135			$qb
136	1		->andWhere('EXTRACT(MONTH FROM c.birth) = :birth_month')
137	1		->setParameter('birth_month', $searchData['birth_month']);
138			}
139
140			// birth
141	41		if (!empty($searchData['birth_start']) && $searchData['birth_start']) {
142			$qb
143	2		->andWhere('c.birth >= :birth_start')
144	2		->setParameter('birth_start', $searchData['birth_start']);
145			}
146	41	View Code Duplication	if (!empty($searchData['birth_end']) && $searchData['birth_end']) {
147	2		$date = clone $searchData['birth_end'];
148	2		$date->modify('+1 days');
149			$qb
150	2		->andWhere('c.birth < :birth_end')
151	2		->setParameter('birth_end', $date);
152			}
153
154			// tel
155	41	View Code Duplication	if (isset($searchData['phone_number']) && StringUtil::isNotBlank($searchData['phone_number'])) {
156	1		$tel = preg_replace('/[^0-9]/ ', '', $searchData['phone_number']);
157			$qb
158	1		->andWhere('c.phone_number LIKE :phone_number')
159	1		->setParameter('phone_number', '%'.$tel.'%');
160			}
161
162			// buy_total
163	41	View Code Duplication	if (isset($searchData['buy_total_start']) && StringUtil::isNotBlank($searchData['buy_total_start'])) {
164			$qb
165	1		->andWhere('c.buy_total >= :buy_total_start')
166	1		->setParameter('buy_total_start', $searchData['buy_total_start']);
167			}
168	41	View Code Duplication	if (isset($searchData['buy_total_end']) && StringUtil::isNotBlank($searchData['buy_total_end'])) {
169			$qb
170	1		->andWhere('c.buy_total <= :buy_total_end')
171	1		->setParameter('buy_total_end', $searchData['buy_total_end']);
172			}
173
174			// buy_times
175	41		if (!empty($searchData['buy_times_start']) && $searchData['buy_times_start']) {
176			$qb
177	1		->andWhere('c.buy_times >= :buy_times_start')
178	1		->setParameter('buy_times_start', $searchData['buy_times_start']);
179			}
180	41		if (!empty($searchData['buy_times_end']) && $searchData['buy_times_end']) {
181			$qb
182	1		->andWhere('c.buy_times <= :buy_times_end')
183	1		->setParameter('buy_times_end', $searchData['buy_times_end']);
184			}
185
186			// create_date
187	41		if (!empty($searchData['create_date_start']) && $searchData['create_date_start']) {
188			$qb
189	1		->andWhere('c.create_date >= :create_date_start')
190	1		->setParameter('create_date_start', $searchData['create_date_start']);
191			}
192	41	View Code Duplication	if (!empty($searchData['create_date_end']) && $searchData['create_date_end']) {
193	1		$date = clone $searchData['create_date_end'];
194	1		$date->modify('+1 days');
195			$qb
196	1		->andWhere('c.create_date < :create_date_end')
197	1		->setParameter('create_date_end', $date);
198			}
199
200			// update_date
201	41		if (!empty($searchData['update_date_start']) && $searchData['update_date_start']) {
202			$qb
203	1		->andWhere('c.update_date >= :update_date_start')
204	1		->setParameter('update_date_start', $searchData['update_date_start']);
205			}
206	41	View Code Duplication	if (!empty($searchData['update_date_end']) && $searchData['update_date_end']) {
207	1		$date = clone $searchData['update_date_end'];
208	1		$date->modify('+1 days');
209			$qb
210	1		->andWhere('c.update_date < :update_date_end')
211	1		->setParameter('update_date_end', $date);
212			}
213
214			// last_buy
215	41		if (!empty($searchData['last_buy_start']) && $searchData['last_buy_start']) {
216			$qb
217	1		->andWhere('c.last_buy_date >= :last_buy_start')
218	1		->setParameter('last_buy_start', $searchData['last_buy_start']);
219			}
220	41	View Code Duplication	if (!empty($searchData['last_buy_end']) && $searchData['last_buy_end']) {
221	1		$date = clone $searchData['last_buy_end'];
222	1		$date->modify('+1 days');
223			$qb
224	1		->andWhere('c.last_buy_date < :last_buy_end')
225	1		->setParameter('last_buy_end', $date);
226			}
227
228			// status
229	41		if (!empty($searchData['customer_status']) && count($searchData['customer_status']) > 0) {
230			$qb
231	3		->andWhere($qb->expr()->in('c.Status', ':statuses'))
232	3		->setParameter('statuses', $searchData['customer_status']);
233			}
234
235			// buy_product_name
236	41	View Code Duplication	if (isset($searchData['buy_product_name']) && StringUtil::isNotBlank($searchData['buy_product_name'])) {
237			$qb
238	2		->leftJoin('c.Orders', 'o')
239	2		->leftJoin('o.OrderItems', 'oi')
240	2		->andWhere('oi.product_name LIKE :buy_product_name')
241	2		->setParameter('buy_product_name', '%'.$searchData['buy_product_name'].'%');
242			}
243
244			// Order By
245	41		$qb->addOrderBy('c.update_date', 'DESC');
246
247	41		return $this->queries->customize(QueryKey::CUSTOMER_SEARCH, $qb, $searchData);
248			}
249
250			/**
251			* ユニークなシークレットキーを返す.
252			*
253			* @return string
254			*/
255	302	View Code Duplication	public function getUniqueSecretKey()
			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...
256			{
257			do {
258	302		$key = StringUtil::random(32);
259	302		$Customer = $this->findOneBy(['secret_key' => $key]);
260	302		} while ($Customer);
261
262	302		return $key;
263			}
264
265			/**
266			* ユニークなパスワードリセットキーを返す
267			*
268			* @return string
269			*/
270		View Code Duplication	public function getUniqueResetKey()
			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...
271			{
272			do {
273			$key = StringUtil::random(32);
274			$Customer = $this->findOneBy(['reset_key' => $key]);
275			} while ($Customer);
276
277			return $key;
278			}
279
280			/**
			0 ignored issues – show introduced 2017-09-25 09:08 UTC by Report Bug Copy Issue Report Doc comment for parameter "$secretKey" missing Loading history...
281			* 仮会員をシークレットキーで検索する.
282			*
283			* @param $secretKey
			0 ignored issues – show introduced 2018-06-08 04:37 UTC by Report Bug Copy Issue Report Missing parameter name Loading history...
284			*
285			* @return null\|Customer 見つからない場合はnullを返す.
286			*/
287	4		public function getProvisionalCustomerBySecretKey($secretKey)
288			{
289	4		return $this->findOneBy([
290	4		'secret_key' => $secretKey,
291	4		'Status' => CustomerStatus::PROVISIONAL,
292			]);
293			}
294
295			/**
			0 ignored issues – show introduced 2017-09-25 09:08 UTC by Report Bug Copy Issue Report Doc comment for parameter "$email" missing Loading history...
296			* 本会員をemailで検索する.
297			*
298			* @param $email
			0 ignored issues – show introduced 2018-06-08 04:37 UTC by Report Bug Copy Issue Report Missing parameter name Loading history...
299			*
300			* @return null\|Customer 見つからない場合はnullを返す.
301			*/
302	1		public function getRegularCustomerByEmail($email)
303			{
304	1		return $this->findOneBy([
305	1		'email' => $email,
306	1		'Status' => CustomerStatus::REGULAR,
307			]);
308			}
309
310			/**
			0 ignored issues – show introduced 2017-09-25 09:08 UTC by Report Bug Copy Issue Report Doc comment for parameter "$resetKey" missing Loading history...
311			* 本会員をリセットキーで検索する.
312			*
313			* @param $resetKey
			0 ignored issues – show introduced 2018-06-08 04:37 UTC by Report Bug Copy Issue Report Missing parameter name Loading history...
314			*
315			* @return null\|Customer 見つからない場合はnullを返す.
316			*/
317	3		public function getRegularCustomerByResetKey($resetKey)
318			{
319	3		return $this->createQueryBuilder('c')
320	3		->where('c.reset_key = :reset_key AND c.Status = :status AND c.reset_expire >= :reset_expire')
321	3		->setParameter('reset_key', $resetKey)
322	3		->setParameter('status', CustomerStatus::REGULAR)
323	3		->setParameter('reset_expire', new \DateTime())
324	3		->setMaxResults(1)
325	3		->getQuery()
326	3		->getOneOrNullResult();
327			}
328
329			/**
330			* リセット用パスワードを生成する.
331			*
332			* @return string
333			*/
334	2		public function getResetPassword()
335			{
336	2		return StringUtil::random(8);
337			}
338
339			/**
340			* 会員の初回購入時間、購入時間、購入回数、購入金額を更新する
341			*
342			* @param Customer $Customer
343			* @param null $isNewOrder
			0 ignored issues – show introduced 2018-07-11 15:02 UTC by Report Bug Copy Issue Report Expected 5 spaces after parameter type; 1 found Loading history...
344			*/
345	1		public function updateBuyData(Customer $Customer, $isNewOrder = null)
346			{
347			// 会員の場合、初回購入時間・購入時間・購入回数・購入金額を更新
348
349			$arr = [
350	1		OrderStatus::NEW,
351			OrderStatus::PAY_WAIT,
352			OrderStatus::BACK_ORDER,
353			OrderStatus::DELIVERED,
354			OrderStatus::PAID,
355			];
356
357	1		$result = $this->orderRepository->getCustomerCount($Customer, $arr);
358
359	1		if (!empty($result)) {
360	1		$data = $result[0];
361
362	1		$now = new \DateTime();
363
364	1		$firstBuyDate = $Customer->getFirstBuyDate();
365	1		if (empty($firstBuyDate)) {
366	1		$Customer->setFirstBuyDate($now);
367			}
368
369	1		if ($isNewOrder) {
370	1		$Customer->setLastBuyDate($now);
371			} else {
372			$Order = $this->orderRepository->find($data['order_id']);
373			if ($Order) {
374			$Customer->setLastBuyDate($Order->getOrderDate());
375			}
376			}
377
378	1		$Customer->setBuyTimes($data['buy_times']);
379	1		$Customer->setBuyTotal($data['buy_total']);
380			} else {
381			// 受注データが存在しなければ初期化
382	1		$Customer->setFirstBuyDate(null);
383	1		$Customer->setLastBuyDate(null);
384	1		$Customer->setBuyTimes(0);
385	1		$Customer->setBuyTotal(0);
386			}
387
388	1		$this->entityManager->persist($Customer);
389	1		$this->entityManager->flush();
390			}
391
392			/**
393			* 仮会員, 本会員の会員を返す.
394			* Eccube\Entity\CustomerのUniqueEntityバリデーションで使用しています.
395			*
396			* @param array $criteria
397			*
398			* @return Customer[]
399			*/
400	42		public function getNonWithdrawingCustomers(array $criteria = [])
401			{
402	42		$criteria['Status'] = [
403	42		CustomerStatus::PROVISIONAL,
404	42		CustomerStatus::REGULAR,
405			];
406
407	42		return $this->findBy($criteria);
408			}
409			}
410

EC-CUBE / ec-cube

Push — experimental/sf ( c0fd88...ff71e4 )

CustomerRepository::getUniqueSecretKey() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like