Failed Conditions
Pull Request — experimental/3.1 (#2633)
by
unknown
44:13
created

ProductController::getPageTitle()   B

Complexity

Conditions 5
Paths 3

Size

Total Lines 10
Code Lines 7

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 0
CRAP Score 30

Importance

Changes 0
Metric Value
dl 0
loc 10
rs 8.8571
c 0
b 0
f 0
ccs 0
cts 0
cp 0
cc 5
eloc 7
nc 3
nop 1
crap 30
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;
26
27
use Doctrine\ORM\EntityManager;
28
use Eccube\Annotation\Inject;
29
use Eccube\Application;
30
use Eccube\Common\Constant;
31
use Eccube\Entity\BaseInfo;
32
use Eccube\Entity\Master\ProductStatus;
33
use Eccube\Entity\Product;
34
use Eccube\Event\EccubeEvents;
35
use Eccube\Event\EventArgs;
36
use Eccube\Exception\CartException;
37
use Eccube\Form\Type\AddCartType;
38
use Eccube\Form\Type\Master\ProductListMaxType;
39
use Eccube\Form\Type\Master\ProductListOrderByType;
40
use Eccube\Form\Type\SearchProductType;
41
use Eccube\Repository\CustomerFavoriteProductRepository;
42
use Eccube\Repository\ProductRepository;
43
use Eccube\Service\CartService;
44
use Eccube\Service\PurchaseFlow\PurchaseFlow;
45
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Method;
46
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
47
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Template;
48
use Symfony\Component\EventDispatcher\EventDispatcher;
49
use Symfony\Component\Form\FormFactory;
50
use Symfony\Component\HttpFoundation\JsonResponse;
51
use Symfony\Component\HttpFoundation\Request;
52
use Symfony\Component\HttpFoundation\Session\Session;
53
use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
54
55
/**
56
 * @Route(service=ProductController::class)
57
 */
58
class ProductController
59
{
60
    /**
61
     * @Inject("eccube.purchase.flow.cart")
62
     * @var PurchaseFlow
63
     */
64
    protected $purchaseFlow;
65
66
    /**
67
     * @Inject("session")
68
     * @var Session
69
     */
70
    protected $session;
71
72
    /**
73
     * @Inject(CustomerFavoriteProductRepository::class)
74
     * @var CustomerFavoriteProductRepository
75
     */
76
    protected $customerFavoriteProductRepository;
77
78
    /**
79
     * @Inject(CartService::class)
80
     * @var CartService
81
     */
82
    protected $cartService;
83
84
    /**
85
     * @Inject(ProductRepository::class)
86
     * @var ProductRepository
87
     */
88
    protected $productRepository;
89
90
    /**
91
     * @Inject("eccube.event.dispatcher")
92
     * @var EventDispatcher
93
     */
94
    protected $eventDispatcher;
95
96
    /**
97
     * @Inject("form.factory")
98
     * @var FormFactory
99
     */
100
    protected $formFactory;
101
102
    /**
103
     * @Inject("orm.em")
104
     * @var EntityManager
105
     */
106
    protected $entityManager;
107
108
    /**
109
     * @Inject(BaseInfo::class)
110
     * @var BaseInfo
111
     */
112
    protected $BaseInfo;
113
114
    private $title;
115
116
    public function __construct()
0 ignored issues
show
introduced by
Missing function doc comment
Loading history...
117
    {
118
        $this->title = '';
119
    }
120
121
    /**
0 ignored issues
show
introduced by
Doc comment for parameter "$app" missing
Loading history...
introduced by
Doc comment for parameter "$request" missing
Loading history...
122
     * 商品一覧画面.
123
     *
124
     * @Route("/products/list", name="product_list")
125
     * @Template("Product/list.twig")
126
     */
0 ignored issues
show
introduced by
Missing @return tag in function comment
Loading history...
127 3
    public function index(Application $app, Request $request)
128
    {
129
        // Doctrine SQLFilter
130 3
        if ($this->BaseInfo->getNostockHidden() === Constant::ENABLED) {
131
            $this->entityManager->getFilters()->enable('nostock_hidden');
132
        }
133
134
        // handleRequestは空のqueryの場合は無視するため
135 3
        if ($request->getMethod() === 'GET') {
136 3
            $request->query->set('pageno', $request->query->get('pageno', ''));
137
        }
138
139
        // searchForm
140
        /* @var $builder \Symfony\Component\Form\FormBuilderInterface */
141 3
        $builder = $this->formFactory->createNamedBuilder('', SearchProductType::class);
142 3
        $builder->setAttribute('freeze', true);
143 3
        $builder->setAttribute('freeze_display_text', false);
144 3
        if ($request->getMethod() === 'GET') {
145 3
            $builder->setMethod('GET');
146
        }
147
148 3
        $event = new EventArgs(
149
            array(
150 3
                'builder' => $builder,
151
            ),
152 3
            $request
153
        );
154 3
        $this->eventDispatcher->dispatch(EccubeEvents::FRONT_PRODUCT_INDEX_INITIALIZE, $event);
155
156
        /* @var $searchForm \Symfony\Component\Form\FormInterface */
157 3
        $searchForm = $builder->getForm();
158
159 3
        $searchForm->handleRequest($request);
160
161
        // paginator
162 3
        $searchData = $searchForm->getData();
163 3
        $qb = $this->productRepository->getQueryBuilderBySearchData($searchData);
164
165 3
        $event = new EventArgs(
166
            array(
167 3
                'searchData' => $searchData,
168 3
                'qb' => $qb,
169
            ),
170 3
            $request
171
        );
172 3
        $this->eventDispatcher->dispatch(EccubeEvents::FRONT_PRODUCT_INDEX_SEARCH, $event);
173 3
        $searchData = $event->getArgument('searchData');
174
175 3
        $pagination = $app['paginator']()->paginate(
176 3
            $qb,
177 3
            !empty($searchData['pageno']) ? $searchData['pageno'] : 1,
178 3
            $searchData['disp_number']->getId()
179
        );
180
181
        // addCart form
182 3
        $forms = array();
183 3
        foreach ($pagination as $Product) {
184
            /* @var $builder \Symfony\Component\Form\FormBuilderInterface */
185 3
            $builder = $this->formFactory->createNamedBuilder(
186 3
                '',
187 3
                AddCartType::class,
188 3
                null,
189
                array(
190 3
                    'product' => $Product,
191
                    'allow_extra_fields' => true,
192
                )
193
            );
194 3
            $addCartForm = $builder->getForm();
195
196 3
            if ($request->getMethod() === 'POST' && (string)$Product->getId() === $request->get('product_id')) {
0 ignored issues
show
Coding Style introduced by
As per coding-style, a cast statement should be followed by a single space.
Loading history...
197
                $addCartForm->handleRequest($request);
198
199
                if ($addCartForm->isValid()) {
200
                    $addCartData = $addCartForm->getData();
201
202
                    try {
203
                        $this->cartService->addProduct(
0 ignored issues
show
Bug introduced by
The method save cannot be called on $this->cartService->addP...ddCartData['quantity']) (of type boolean).

Methods can only be called on objects. This check looks for methods being called on variables that have been inferred to never be objects.

Loading history...
204
                            $addCartData['product_class_id'],
205
                            $addCartData['quantity']
206
                        )->save();
207
                    } catch (CartException $e) {
208
                        $app->addRequestError($e->getMessage());
209
                    }
210
211
                    $event = new EventArgs(
212
                        array(
213
                            'form' => $addCartForm,
214
                            'Product' => $Product,
215
                        ),
216
                        $request
217
                    );
218
                    $this->eventDispatcher->dispatch(EccubeEvents::FRONT_PRODUCT_INDEX_COMPLETE, $event);
219
220
                    if ($event->getResponse() !== null) {
221
                        return $event->getResponse();
222
                    }
223
224
                    return $app->redirect($app->url('cart'));
225
                }
226
            }
227
228 3
            $forms[$Product->getId()] = $addCartForm->createView();
229
        }
230
231
        // 表示件数
232 3
        $builder = $this->formFactory->createNamedBuilder(
233 3
            'disp_number',
234 3
            ProductListMaxType::class,
235 3
            null,
236
            array(
237 3
                'required' => false,
238
                'label' => '表示件数',
239
                'allow_extra_fields' => true,
240
            )
241
        );
242 3
        if ($request->getMethod() === 'GET') {
243 3
            $builder->setMethod('GET');
244
        }
245
246 3
        $event = new EventArgs(
247
            array(
248 3
                'builder' => $builder,
249
            ),
250 3
            $request
251
        );
252 3
        $this->eventDispatcher->dispatch(EccubeEvents::FRONT_PRODUCT_INDEX_DISP, $event);
253
254 3
        $dispNumberForm = $builder->getForm();
255
256 3
        $dispNumberForm->handleRequest($request);
257
258
        // ソート順
259 3
        $builder = $this->formFactory->createNamedBuilder(
260 3
            'orderby',
261 3
            ProductListOrderByType::class,
262 3
            null,
263
            array(
264 3
                'required' => false,
265
                'label' => '表示順',
266
                'allow_extra_fields' => true,
267
            )
268
        );
269 3
        if ($request->getMethod() === 'GET') {
270 3
            $builder->setMethod('GET');
271
        }
272
273 3
        $event = new EventArgs(
274
            array(
275 3
                'builder' => $builder,
276
            ),
277 3
            $request
278
        );
279 3
        $this->eventDispatcher->dispatch(EccubeEvents::FRONT_PRODUCT_INDEX_ORDER, $event);
280
281 3
        $orderByForm = $builder->getForm();
282
283 3
        $orderByForm->handleRequest($request);
284
285 3
        $Category = $searchForm->get('category_id')->getData();
286
287
        return [
288 3
            'subtitle' => $this->getPageTitle($searchData),
289 3
            'pagination' => $pagination,
290 3
            'search_form' => $searchForm->createView(),
291 3
            'disp_number_form' => $dispNumberForm->createView(),
292 3
            'order_by_form' => $orderByForm->createView(),
293 3
            'forms' => $forms,
294 3
            'Category' => $Category,
295
        ];
296
    }
297
298
    /**
0 ignored issues
show
introduced by
Doc comment for parameter "$app" missing
Loading history...
introduced by
Doc comment for parameter "$request" missing
Loading history...
introduced by
Doc comment for parameter "$Product" missing
Loading history...
299
     * 商品詳細画面.
300
     *
301
     * @Method("GET")
302
     * @Route("/products/detail/{id}", name="product_detail", requirements={"id" = "\d+"})
303
     * @Template("Product/detail.twig")
304 4
     */
0 ignored issues
show
introduced by
Missing @return tag in function comment
Loading history...
305
    public function detail(Application $app, Request $request, Product $Product)
306 4
    {
307
        if (!$this->checkVisibility($Product)) {
308
            throw new NotFoundHttpException();
309 4
        }
310
311 4
        $builder = $this->formFactory->createNamedBuilder(
312
            '',
313
            AddCartType::class,
314
            null,
315
            array(
316
                'product' => $Product,
317 4
                'id_add_product_id' => false,
318
            )
319
        );
320
321
        $event = new EventArgs(
322 4
            array(
323 4
                'builder' => $builder,
324 4
                'Product' => $Product,
325 4
            ),
326
            $request
327 4
        );
328
        $this->eventDispatcher->dispatch(EccubeEvents::FRONT_PRODUCT_DETAIL_INITIALIZE, $event);
329
330
        $is_favorite = false;
331
        if ($app->isGranted('ROLE_USER')) {
332 4
            $Customer = $app->user();
333
            $is_favorite = $this->customerFavoriteProductRepository->isFavorite($Customer, $Product);
334 4
        }
335 4
336
        return [
337 4
            'title' => $this->title,
338
            'subtitle' => $Product->getName(),
339 4
            'form' => $builder->getForm()->createView(),
340
            'Product' => $Product,
341
            'is_favorite' => $is_favorite,
342 4
        ];
343
    }
344 4
345 3
    /**
0 ignored issues
show
introduced by
Doc comment for parameter "$app" missing
Loading history...
introduced by
Doc comment for parameter "$Product" missing
Loading history...
346
     * お気に入り追加.
347 3
     *
348 2
     * @Route("/products/add_favorite/{id}", name="product_add_favorite", requirements={"id" = "\d+"})
349 2
     */
0 ignored issues
show
introduced by
Missing @return tag in function comment
Loading history...
350 2
    public function addFavorite(Application $app, Product $Product)
351 2
    {
352 2
        $this->checkVisibility($Product);
353 2
354
        // TODO イベント発火
355 2
356
        if ($app->isGranted('ROLE_USER')) {
357 2
            $Customer = $app->user();
358 2
            $this->customerFavoriteProductRepository->addFavorite($Customer, $Product);
359
            $this->session->getFlashBag()->set('product_detail.just_added_favorite', $Product->getId());
360 2
361
            // TODO イベント発火
362 2
363
            return $app->redirect($app->url('product_detail', array('id' => $Product->getId())));
364 2
        } else {
365
            // 非会員の場合、ログイン画面を表示
366
            //  ログイン後の画面遷移先を設定
367
            $app->setLoginTargetPath($app->url('product_add_favorite', array('id' => $Product->getId())));
368 2
            $this->session->getFlashBag()->set('eccube.add.favorite', true);
0 ignored issues
show
Documentation introduced by
true is of type boolean, but the function expects a string|array.

It seems like the type of the argument is not accepted by the function/method which you are calling.

In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug.

We suggest to add an explicit type cast like in the following example:

function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
Loading history...
369
370
            return $app->redirect($app->url('mypage_login'));
371
        }
372
    }
373
374
    /**
0 ignored issues
show
introduced by
Doc comment for parameter "$app" missing
Loading history...
introduced by
Doc comment for parameter "$request" missing
Loading history...
introduced by
Doc comment for parameter "$Product" missing
Loading history...
375
     * カートに追加.
376
     *
377
     * @Method("POST")
378
     * @Route("/products/add_cart/{id}", name="product_add_cart", requirements={"id" = "\d+"})
379
     */
0 ignored issues
show
introduced by
Missing @return tag in function comment
Loading history...
380
    public function addCart(Application $app, Request $request, Product $Product)
381
    {
382
        // エラーメッセージの配列
383
        $errorMessages = array();
384
385
        if (!$this->checkVisibility($Product)) {
386
            throw new NotFoundHttpException();
387
        }
388
389
        $builder = $this->formFactory->createNamedBuilder(
390
            '',
391
            AddCartType::class,
392
            null,
393
            array(
394
                'product' => $Product,
395
                'id_add_product_id' => false,
396
            )
397
        );
398
399
        // TODO イベント発火
400
//        $event = new EventArgs(
401
//            array(
402
//                'builder' => $builder,
403
//                'Product' => $Product,
404
//            ),
405
//            $request
406
//        );
407
//        $this->eventDispatcher->dispatch(EccubeEvents::FRONT_PRODUCT_DETAIL_INITIALIZE, $event);
408
409
        /* @var $form \Symfony\Component\Form\FormInterface */
410
        $form = $builder->getForm();
411
        $form->handleRequest($request);
412
413
        if (!$form->isValid()) {
414
            throw new NotFoundHttpException();
415
        }
416
417
        $addCartData = $form->getData();
418
419
        log_info(
420
            'カート追加処理開始',
421
            array(
422
                'product_id' => $Product->getId(),
423
                'product_class_id' => $addCartData['product_class_id'],
424
                'quantity' => $addCartData['quantity'],
425
            )
426
        );
427
428
        // カートへ追加
429
        $this->cartService->addProduct($addCartData['product_class_id'], $addCartData['quantity']);
430
431
        // 明細の正規化
432
        $flow = $this->purchaseFlow;
433
        $Cart = $this->cartService->getCart();
434 1
        $result = $flow->calculate($Cart, $app['eccube.purchase.context']());
435
436
        // 復旧不可のエラーが発生した場合は追加した明細を削除.
437
        if ($result->hasError()) {
438 3
            $this->cartService->removeProduct($addCartData['product_class_id']);
439 3
            foreach ($result->getErrors() as $error) {
440
                array_push($errorMessages, $error->getMessage());
441
            }
442
        }
443
444
        foreach ($result->getWarning() as $warning) {
445
            array_push($errorMessages, $warning->getMessage());
446
        }
447
448
        $this->cartService->save();
449 4
450 4
        log_info(
451 2
            'カート追加処理完了',
452 2
            array(
453
                'product_id' => $Product->getId(),
454
                'product_class_id' => $addCartData['product_class_id'],
455
                'quantity' => $addCartData['quantity'],
456 4
            )
457 4
        );
458 4
459 4
        // TODO イベント発火
460 4
//        $event = new EventArgs(
461
//            array(
462
//                'form' => $form,
463
//                'Product' => $Product,
464
//            ),
465
//            $request
466
//        );
467
//        $this->eventDispatcher->dispatch(EccubeEvents::FRONT_PRODUCT_DETAIL_COMPLETE, $event);
468
//
469
//        if ($event->getResponse() !== null) {
470 3
//            return $event->getResponse();
471
//        }
472 3
473
        if ($request->isXmlHttpRequest()) {
474 3
            // ajaxでのリクエストの場合は結果をjson形式で返す。
475 1
476
            // 初期化
477 2
            $done = null;
0 ignored issues
show
Unused Code introduced by
$done is not used, you could remove the assignment.

This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently.

$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

Both the $myVar assignment in line 1 and the $higher assignment in line 2 are dead. The first because $myVar is never used and the second because $higher is always overwritten for every possible time line.

Loading history...
478
            $messages = array();
479
480
            if (empty($errorMessages)) {
481
                // エラーが発生していない場合
482
                $done = true;
483
                array_push($messages, 'カートに追加しました。');
484
            } else {
485
                // エラーが発生している場合
486
                $done = false;
487
                $messages = $errorMessages;
488
            }
489
490
            return new JsonResponse(array('done' => $done, 'messages' => $messages));
491
0 ignored issues
show
Coding Style introduced by
Blank line found at end of control structure
Loading history...
492
        } else {
493
            // ajax以外でのリクエストの場合はカート画面へリダイレクト
494
            foreach ($errorMessages as $errorMessage) {
495
                $app->addRequestError($errorMessage);
496
            }
497
498
            return $app->redirect($app->url('cart'));
499
        }
500
    }
501
502
    /**
503
     * ページタイトルの設定
504
     *
505
     * @param  null|array $searchData
506
     * @return str
507
     */
508
    private function getPageTitle($searchData)
509
    {
510
        if (isset($searchData['name']) && !empty($searchData['name'])) {
511
            return '検索結果';
512
        } elseif (isset($searchData['category_id']) && $searchData['category_id']) {
513
            return $searchData['category_id']->getName();
514
        } else {
515
            return '全商品';
516
        }
517
    }
518
519
    /**
520
     * 閲覧可能な商品かどうかを判定
521
     * @param Product $Product
522
     * @return boolean 閲覧可能な場合はtrue
523
     */
524
    private function checkVisibility(Product $Product)
525
    {
526
        $is_admin = $this->session->has('_security_admin');
527
528
        // 管理ユーザの場合はステータスやオプションにかかわらず閲覧可能.
529
        if (!$is_admin) {
530
            // 在庫なし商品の非表示オプションが有効な場合.
531
            if ($this->BaseInfo->getNostockHidden()) {
532
                if (!$Product->getStockFind()) {
533
                    return false;
534
                }
535
            }
536
            // 公開ステータスでない商品は表示しない.
537
            if ($Product->getStatus()->getId() !== ProductStatus::DISPLAY_SHOW) {
538
                return false;
539
            }
540
        }
541
        return true;
0 ignored issues
show
introduced by
Missing blank line before return statement
Loading history...
542
    }
543
}
544