EC-CUBE / ec-cube

src/Eccube/Di/Scanner/RepositoryDefinition.php

Doc Comments +2 added lines, -2 removed lines

@@ -30,8 +30,8 @@
 {
     /**
      * RepositoryDefinition constructor.
-     * @param $id
-     * @param $refClass
+     * @param string $id
+     * @param \ReflectionClass $refClass
      */
     public function __construct($id, $refClass)
     {

src/Eccube/Di/Scanner/RepositoryScanner.php

Doc Comments +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
 {
     /**
      * RepositoryAutoWiring constructor.
-     * @param array|string[] $scanDirs
+     * @param string[] $scanDirs
      */
     public function __construct($scanDirs)
     {

src/Eccube/Di/Di.php

Braces +3 added lines, -1 removed lines

@@ -161,7 +161,9 @@
             }
 
             if (!empty($components)) {
-                if ($snippets) $snippets .= PHP_EOL;
+                if ($snippets) {
+                    $snippets .= PHP_EOL;
+                }
                 $snippets .= $wiring->generate($twig, $components);
                 $snippets .= $wiring->generateExtend($twig, $components);
             }

src/Eccube/ServiceProvider/DiServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
 {
     public function register(Container $app)
     {
-        $app['eccube.di'] = function (Container $app) {
+        $app['eccube.di'] = function(Container $app) {
             $di = new Di(
                 $app['eccube.di.generator.dir'],
                 $app['eccube.di.generator.class'],

src/Eccube/Form/Type/Admin/ShippingType.php

Unused Use Statements -2 removed lines

@@ -27,7 +27,6 @@ 
 use Doctrine\ORM\EntityRepository;
 use Eccube\Annotation\FormType;
 use Eccube\Annotation\Inject;
-use Eccube\Application;
 use Eccube\Common\Constant;
 use Eccube\Entity\BaseInfo;
 use Eccube\Form\Type\AddressType;
@@ -35,7 +34,6 @@ 
 use Eccube\Form\Type\NameType;
 use Eccube\Form\Type\TelType;
 use Eccube\Form\Type\ZipType;
-use Eccube\Repository\BaseInfoRepository;
 use Eccube\Repository\DeliveryRepository;
 use Symfony\Bridge\Doctrine\Form\Type\EntityType;
 use Symfony\Component\Form\AbstractType;

Spacing +7 added lines, -7 removed lines

@@ -199,7 +199,7 @@ 
                 'prototype' => true,
             ))
 
-            ->addEventListener(FormEvents::PRE_SET_DATA, function (FormEvent $event) {
+            ->addEventListener(FormEvents::PRE_SET_DATA, function(FormEvent $event) {
                 if ($this->BaseInfo->getOptionMultipleShipping() == Constant::ENABLED) {
                     $form = $event->getForm();
                     $form->add('OrderItems', CollectionType::class, array(
@@ -210,7 +210,7 @@ 
                     ));
                 }
             })
-            ->addEventListener(FormEvents::POST_SET_DATA, function (FormEvent $event) {
+            ->addEventListener(FormEvents::POST_SET_DATA, function(FormEvent $event) {
                 /** @var \Eccube\Entity\Shipping $data */
                 $data = $event->getData();
                 /** @var \Symfony\Component\Form\Form $form */
@@ -229,7 +229,7 @@ 
                     'choice_label' => 'delivery_time',
                     'placeholder' => '指定なし',
                     'required' => false,
-                    'query_builder' => function (EntityRepository $er) use($Delivery) {
+                    'query_builder' => function(EntityRepository $er) use($Delivery) {
                         return $er->createQueryBuilder('dt')
                             ->where('dt.Delivery = :Delivery')
                             ->setParameter('Delivery', $Delivery);
@@ -237,7 +237,7 @@ 
                 ));
 
             })
-            ->addEventListener(FormEvents::PRE_SUBMIT, function (FormEvent $event) {
+            ->addEventListener(FormEvents::PRE_SUBMIT, function(FormEvent $event) {
                 $data = $event->getData();
                 $form = $event->getForm();
                 if (!$data) {
@@ -257,14 +257,14 @@ 
                     'choice_label' => 'delivery_time',
                     'placeholder' => '指定なし',
                     'required' => false,
-                    'query_builder' => function (EntityRepository $er) use($Delivery) {
+                    'query_builder' => function(EntityRepository $er) use($Delivery) {
                         return $er->createQueryBuilder('dt')
                             ->where('dt.Delivery = :Delivery')
                             ->setParameter('Delivery', $Delivery);
                     },
                 ));
             })
-            ->addEventListener(FormEvents::POST_SUBMIT, function (FormEvent $event) {
+            ->addEventListener(FormEvents::POST_SUBMIT, function(FormEvent $event) {
                 if ($this->BaseInfo->getOptionMultipleShipping() == Constant::ENABLED) {
                     $form = $event->getForm();
                     $OrderItems = $form['OrderItems']->getData();
@@ -275,7 +275,7 @@ 
                     }
                 }
             })
-            ->addEventListener(FormEvents::POST_SUBMIT, function (FormEvent $event) {
+            ->addEventListener(FormEvents::POST_SUBMIT, function(FormEvent $event) {
                 $Shipping = $event->getData();
                 $Delivery = $Shipping->getDelivery();
                 $Shipping->setShippingDeliveryName($Delivery ? $Delivery : null);

src/Eccube/Service/Calculator/CalculateContext.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 use Eccube\Entity\Order;
 use Eccube\Entity\PurchaseInterface;
 use Eccube\Entity\OrderItem;
-use Eccube\Service\Calculator\Strategy\CalculateStrategyInterface;
 
 class CalculateContext
 {

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
         $this->buildCalculator($this->CalculateStrategies);
 
         /** @var OrderItem $OrderItem */
-        foreach($this->OrderItems as $OrderItem) {
+        foreach ($this->OrderItems as $OrderItem) {
             if ($OrderItem instanceof OrderItem) {
                 if (!$this->Order->getItems()->contains($OrderItem)) {
                     $OrderItem->setOrder($this->Order);

src/Eccube/Service/Calculator/Strategy/CalculateChargeStrategy.php

Unused Use Statements -4 removed lines

@@ -2,13 +2,9 @@
 namespace Eccube\Service\Calculator\Strategy;
 
 use Eccube\Application;
-use Eccube\Entity\Master\OrderItemType;
-use Eccube\Entity\Master\TaxType;
-use Eccube\Entity\Master\TaxDisplayType;
 use Eccube\Entity\Order;
 use Eccube\Entity\PurchaseInterface;
 use Eccube\Entity\OrderItem;
-use Eccube\Entity\Shipping;
 use Eccube\Repository\Master\OrderItemTypeRepository;
 use Eccube\Service\Calculator\OrderItemCollection;
 

src/Eccube/Service/Calculator/Strategy/CalculateDeliveryFeeStrategy.php

Unused Use Statements -4 removed lines

@@ -2,13 +2,9 @@
 namespace Eccube\Service\Calculator\Strategy;
 
 use Eccube\Application;
-use Eccube\Entity\Master\OrderItemType;
-use Eccube\Entity\Master\TaxType;
-use Eccube\Entity\Master\TaxDisplayType;
 use Eccube\Entity\Order;
 use Eccube\Entity\PurchaseInterface;
 use Eccube\Entity\OrderItem;
-use Eccube\Entity\Shipping;
 use Eccube\Repository\Master\OrderItemTypeRepository;
 use Eccube\Service\Calculator\OrderItemCollection;
 

src/Eccube/Service/Calculator/Strategy/CalculateTotalStrategy.php

Unused Use Statements -4 removed lines

@@ -2,14 +2,10 @@
 namespace Eccube\Service\Calculator\Strategy;
 
 use Eccube\Application;
-use Eccube\Entity\Master\OrderItemType;
-use Eccube\Entity\Master\TaxType;
-use Eccube\Entity\Master\TaxDisplayType;
 use Eccube\Entity\Cart;
 use Eccube\Entity\Order;
 use Eccube\Entity\PurchaseInterface;
 use Eccube\Entity\OrderItem;
-use Eccube\Entity\Shipping;
 use Eccube\Repository\Master\OrderItemTypeRepository;
 use Eccube\Service\Calculator\OrderItemCollection;