CustomerRepository::getRegularCustomerByEmail() - Code Metrics - Inspection of "管理画面の検索機能をDateTimeTypeに対応" - EC-CUBE/ec-cube - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Retried Inspection

Pull Request — 4.0 (#4619)

by Hideki

created 2020-07-13 05:00 UTC

CustomerRepository::getRegularCustomerByEmail() A

↳ Parent: CustomerRepository

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	1

Importance

Changes

Metric	Value
cc	1
nc	1
nop	1
dl	0
loc	7
ccs	2
cts	2
cp	1
crap	1
rs	10
c	0
b	0
f	0

<?php

/*
 * This file is part of EC-CUBE
 *
 * Copyright(c) EC-CUBE CO.,LTD. All Rights Reserved.
 *
 * http://www.ec-cube.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\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 array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}

        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 (isset($searchData['buy_times_start']) && StringUtil::isNotBlank($searchData['buy_times_start'])) {
            $qb
                ->andWhere('c.buy_times >= :buy_times_start')
                ->setParameter('buy_times_start', $searchData['buy_times_start']);
        }
        if (isset($searchData['buy_times_end']) && StringUtil::isNotBlank($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_datetime_start']) && $searchData['create_datetime_start']) {
            $date = $searchData['create_datetime_start'];
            $qb
                ->andWhere('c.create_date >= :create_date_start')
                ->setParameter('create_date_start', $date);
        } elseif (!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_datetime_end']) && $searchData['create_datetime_end']) {
            $date = $searchData['create_datetime_end'];
            $qb
                ->andWhere('c.create_date < :create_date_end')
                ->setParameter('create_date_end', $date);
        } elseif (!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_datetime_start']) && $searchData['update_datetime_start']) {
            $date = $searchData['update_datetime_start'];
            $qb
                ->andWhere('c.update_date >= :update_date_start')
                ->setParameter('update_date_start', $date);
        } elseif (!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_datetime_end']) && $searchData['update_datetime_end']) {
            $date = $searchData['update_datetime_end'];
            $qb
                ->andWhere('c.update_date < :update_date_end')
                ->setParameter('update_date_end', $date);
        } elseif (!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_datetime_start']) && $searchData['last_buy_datetime_start']) {
            $date = $searchData['last_buy_datetime_start'];
            $qb
                ->andWhere('c.last_buy_date >= :last_buy_start')
                ->setParameter('last_buy_start', $date);
        } elseif (!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_datetime_end']) && $searchData['last_buy_datetime_end']) {
            $date = $searchData['last_buy_datetime_end'];
            $qb
                ->andWhere('c.last_buy_date < :last_buy_end')
                ->setParameter('last_buy_end', $date);
        } elseif (!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
     * @param $email
     *
     * @return null|Customer 見つからない場合はnullを返す.
     */
    public function getRegularCustomerByResetKey($resetKey, $email = null)
    {
        $qb = $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());

        if ($email) {
            $qb
                ->andWhere('c.email = :email')
                ->setParameter('email', $email);
        }

        return $qb->setMaxResults(1)
            ->getQuery()
            ->getOneOrNullResult();
    }

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

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

EC-CUBE / ec-cube

Pull Request — 4.0 (#4619)

CustomerRepository::getRegularCustomerByEmail() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like