Eccube\Controller\Admin\Order\ShippingController::index()

Complexity

Conditions	21
Paths	126

Size

Total Lines

134

Duplication

Lines	6
Ratio	4.48 %

Importance

Changes

0

<?php
    public function index(Request $request, Order $Order)
    {
        $TargetShippings = $Order->getShippings();

        // 編集前の受注情報を保持
        $OriginShippings = new ArrayCollection();
        $OriginOrderItems = [];

        foreach ($TargetShippings as $key => $TargetShipping) {
            $OriginShippings->add($TargetShipping);

            // 編集前のお届け先のアイテム情報を保持
            $OriginOrderItems[$key] = new ArrayCollection();

            foreach ($TargetShipping->getOrderItems() as $OrderItem) {
                $OriginOrderItems[$key]->add($OrderItem);
            }
        }

        $builder = $this->formFactory->createBuilder();
        $builder
            ->add('shippings', CollectionType::class, [
                'entry_type' => ShippingType::class,
                'data' => $TargetShippings,
                'allow_add' => true,
                'allow_delete' => true,
                'prototype' => true,
            ]);

        // 配送先の追加フラグ
        $builder
            ->add('add_shipping', HiddenType::class, [
                'mapped' => false,
            ]);

        // 配送先の追加フラグが立っている場合は新しいお届け先を追加
        $builder->addEventListener(FormEvents::PRE_SUBMIT, function (FormEvent $event) {
            $data = $event->getData();
            if ($data['add_shipping']) {
                $Shippings = $data['shippings'];
                $newShipping = ['Delivery' => ''];
                $Shippings[] = $newShipping;
                $data['shippings'] = $Shippings;
                $data['add_shipping'] = '';
                $event->setData($data);
            }
        });

        $form = $builder->getForm();

        $form->handleRequest($request);

        if ($form->isSubmitted() && $form->isValid() && $request->get('mode') == 'register') {
            log_info('出荷登録開始', [$TargetShipping->getId()]);

The variable $TargetShipping seems to be defined by a foreach iteration on line 130. Are you sure the iterator is never empty, otherwise this variable is not defined?

            // 削除された項目の削除
            /** @var Shipping $OriginShipping */
            foreach ($OriginShippings as $key => $OriginShipping) {
                if (false === $TargetShippings->contains($OriginShipping)) {
                    // お届け先自体が削除された場合
                    // 削除されたお届け先に紐づく明細の削除
                    /** @var OrderItem $OriginOrderItem */
                    foreach ($OriginOrderItems[$key] as $OriginOrderItem) {
                        $this->entityManager->remove($OriginOrderItem);
                    }

                    // 削除されたお届け先の削除
                    $this->entityManager->remove($OriginShipping);
                } else {
                    // お届け先は削除されていない場合
                    // 削除された明細の削除
                    /** @var OrderItem $OriginOrderItem */
                    foreach ($OriginOrderItems[$key] as $OriginOrderItem) {
                        if (false === $TargetShippings[$key]->getOrderItems()->contains($OriginOrderItem)) {
                            $this->entityManager->remove($OriginOrderItem);
                        }
                    }
                }
            }

            // 追加された項目の追加
            foreach ($TargetShippings as $TargetShipping) {
                // 追加された明細の追加
                foreach ($TargetShipping->getOrderItems() as $OrderItem) {
                    $OrderItem->setShipping($TargetShipping);
                    $OrderItem->setOrder($Order);
                }

                // 追加されたお届け先の追加
                $TargetShipping->setOrder($Order);
            }

            try {
                foreach ($TargetShippings as $TargetShipping) {
                    $this->entityManager->persist($TargetShipping);
                }
                $this->entityManager->flush();

                $this->addSuccess('admin.shipping.edit.save.complete', 'admin');
                $this->addInfo('admin.shipping.edit.save.info', 'admin');
                log_info('出荷登録完了', [$Order->getId()]);

                return $this->redirectToRoute('admin_shipping_edit', ['id' => $Order->getId()]);
            } catch (\Exception $e) {
                log_error('出荷登録エラー', [$Order->getId(), $e]);
                $this->addError('admin.flash.register_failed', 'admin');
            }
        } elseif ($form->isSubmitted() && $request->get('mode') == 'register' && $form->getErrors(true)) {
            $this->addError('admin.flash.register_failed', 'admin');
        }

        // 商品検索フォーム
        $builder = $this->formFactory
            ->createBuilder(SearchProductType::class);

        $searchProductModalForm = $builder->getForm();

        // 配送業者のお届け時間
        $times = [];
        $deliveries = $this->deliveryRepository->findAll();
        foreach ($deliveries as $Delivery) {

This code seems to be duplicated across your project.

            $deliveryTiems = $Delivery->getDeliveryTimes();
            foreach ($deliveryTiems as $DeliveryTime) {
                $times[$Delivery->getId()][$DeliveryTime->getId()] = $DeliveryTime->getDeliveryTime();
            }
        }

        return [
            'form' => $form->createView(),
            'searchProductModalForm' => $searchProductModalForm->createView(),
            'Order' => $Order,
            'shippingDeliveryTimes' => $this->serializer->serialize($times, 'json'),
        ];
    }