EC-CUBE / ec-cube

src/Eccube/Twig/Extension/EccubeExtension.php

Spacing +3 added lines, -3 removed lines

@@ -66,7 +66,7 @@ 
                     if (is_callable($function)) {
                         return call_user_func_array($function, $arg_list);
                     }
-                    trigger_error('Called to an undefined function : php_'. $function, E_USER_WARNING);
+                    trigger_error('Called to an undefined function : php_'.$function, E_USER_WARNING);
 
             }, ['pre_escape' => 'html', 'is_safe' => ['html']]),
 
@@ -154,7 +154,7 @@ 
     public function getCsrfTokenForAnchor()
     {
         $token = $this->app['csrf.token_manager']->getToken(Constant::TOKEN_NAME)->getValue();
-        return 'token-for-anchor=\'' . $token . '\'';
+        return 'token-for-anchor=\''.$token.'\'';
     }
 
     /**
@@ -189,7 +189,7 @@ 
     public function getPriceFilter($number, $decimals = 0, $decPoint = '.', $thousandsSep = ',')
     {
         $price = number_format($number, $decimals, $decPoint, $thousandsSep);
-        $price = '¥ ' . $price;
+        $price = '¥ '.$price;
 
         return $price;
     }

src/Eccube/Doctrine/ORM/Mapping/Driver/AnnotationDriver.php

Doc Comments +3 added lines

@@ -32,6 +32,9 @@
 {
     protected $trait_proxies_directory;
 
+    /**
+     * @param string $dir
+     */
     public function setTraitProxiesDirectory($dir)
     {
         $this->trait_proxies_directory = $dir;

Spacing +4 added lines, -4 removed lines

@@ -54,7 +54,7 @@ 
         $includedFiles = [];
 
         foreach ($this->paths as $path) {
-            if ( ! is_dir($path)) {
+            if (!is_dir($path)) {
                 throw MappingException::fileMappingDriversRequireConfiguredDirectoryPath($path);
             }
 
@@ -63,14 +63,14 @@ 
                     new \RecursiveDirectoryIterator($path, \FilesystemIterator::SKIP_DOTS),
                     \RecursiveIteratorIterator::LEAVES_ONLY
                 ),
-                '/^.+' . preg_quote($this->fileExtension) . '$/i',
+                '/^.+'.preg_quote($this->fileExtension).'$/i',
                 \RecursiveRegexIterator::GET_MATCH
             );
 
             foreach ($iterator as $file) {
                 $sourceFile = $file[0];
 
-                if ( ! preg_match('(^phar:)i', $sourceFile)) {
+                if (!preg_match('(^phar:)i', $sourceFile)) {
                     $sourceFile = realpath($sourceFile);
                 }
 
@@ -101,7 +101,7 @@ 
         foreach ($declared as $className) {
             $rc = new \ReflectionClass($className);
             $sourceFile = $rc->getFileName();
-            if (in_array($sourceFile, $includedFiles) && ! $this->isTransient($className)) {
+            if (in_array($sourceFile, $includedFiles) && !$this->isTransient($className)) {
                 $classes[] = $className;
             }
         }

src/Eccube/Form/Extension/DoctrineOrmExtension.php

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
     {
         $builder->addEventListener(
             FormEvents::PRE_SET_DATA,
-            function (FormEvent $event) {
+            function(FormEvent $event) {
                 $form = $event->getForm();
                 $config = $form->getConfig();
                 // data_classオプションが必要

src/Eccube/ServiceProvider/MobileDetectServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
             return new \Mobile_Detect();
         };
 
-        $app['mobile_detect.device_type'] = function ($app) {
+        $app['mobile_detect.device_type'] = function($app) {
             if ($app['mobile_detect']->isMobile()) {
                 return \Eccube\Entity\Master\DeviceType::DEVICE_TYPE_SP;
             } else {

src/Eccube/Doctrine/Query/JoinClause.php

Doc Comments +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
 
     /**
      * JoinClause constructor.
-     * @param $leftJoin
+     * @param boolean $leftJoin
      * @param $join
      * @param $alias
      * @param $conditionType

src/Eccube/Doctrine/Query/OrderByClause.php

Doc Comments +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
 
     /**
      * OrderByClause constructor.
-     * @param $sort
+     * @param string $sort
      * @param string $order
      */
     function __construct($sort, $order = 'asc')

src/Eccube/Doctrine/Query/WhereClause.php

Doc Comments +3 added lines

@@ -53,6 +53,9 @@
         $this->params = $params;
     }
 
+    /**
+     * @param Expr\Comparison $expr
+     */
     private static function newWhereClause($expr, $x, $y)
     {
         if ($y) {

src/Eccube/Repository/QueryKey.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
 {
     const PRODUCT_SEARCH = 'Product.getQueryBuilderBySearchData';
     const PRODUCT_SEARCH_ADMIN = 'Product.getQueryBuilderBySearchDataForAdmin';
-    const PRODUCT_GET_FAVORITE  = 'Product.getFavoriteProductQueryBuilderByCustomer';
+    const PRODUCT_GET_FAVORITE = 'Product.getFavoriteProductQueryBuilderByCustomer';
 
     const CUSTOMER_SEARCH = 'Customer.getQueryBuilderBySearchData';
 

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

Doc Comments +3 added lines

@@ -53,6 +53,9 @@
 
     protected $app;
 
+    /**
+     * @param \Pimple\Container $app
+     */
     public function __construct($app)
     {
         $this->app = $app;

Unused Use Statements -1 removed lines

@@ -35,7 +35,6 @@
 use Symfony\Component\Form\AbstractType;
 use Symfony\Component\Form\Extension\Core\Type\CollectionType;
 use Symfony\Component\Form\Extension\Core\Type\EmailType;
-use Symfony\Component\Form\Extension\Core\Type\FormType;
 use Symfony\Component\Form\Extension\Core\Type\HiddenType;
 use Symfony\Component\Form\Extension\Core\Type\MoneyType;
 use Symfony\Component\Form\Extension\Core\Type\TextareaType;

Spacing +9 added lines, -11 removed lines

@@ -256,7 +256,7 @@ 
         /**
          * 複数配送オプション有効時の画面制御を行う.
          */
-        $builder->addEventListener(FormEvents::PRE_SUBMIT, function (FormEvent $event) use ($BaseInfo) {
+        $builder->addEventListener(FormEvents::PRE_SUBMIT, function(FormEvent $event) use ($BaseInfo) {
 
             if ($BaseInfo->getOptionMultipleShipping() != Constant::ENABLED) {
                 return;
@@ -266,17 +266,16 @@ 
             $orderDetails = &$data['OrderDetails'];
 
             // 数量0フィルター
-            $quantityFilter = function ($v) {
+            $quantityFilter = function($v) {
                 return !(isset($v['quantity']) && preg_match('/^0+$/', trim($v['quantity'])));
             };
 
             // $shippings = &$data['Shippings'];
 
             // 数量を抽出
-            $getQuantity = function ($v) {
+            $getQuantity = function($v) {
                 return (isset($v['quantity']) && preg_match('/^\d+$/', trim($v['quantity']))) ?
-                    trim($v['quantity']) :
-                    0;
+                    trim($v['quantity']) : 0;
             };
 
             // foreach ($shippings as &$shipping) {
@@ -293,7 +292,7 @@ 
                     $orderDetail['quantity'] = 0;
 
                     // 受注詳細と同じ商品規格のみ抽出
-                    $productClassFilter = function ($v) use ($orderDetail) {
+                    $productClassFilter = function($v) use ($orderDetail) {
                         return $orderDetail['ProductClass'] === $v['ProductClass'];
                     };
 
@@ -302,10 +301,9 @@ 
                         if (!empty($shipping['ShipmentItems'])) {
 
                             // 同じ商品規格の受注詳細の価格を適用
-                            $applyPrice = function (&$v) use ($orderDetail) {
+                            $applyPrice = function(&$v) use ($orderDetail) {
                                 $v['price'] = ($v['ProductClass'] === $orderDetail['ProductClass']) ?
-                                    $orderDetail['price'] :
-                                    $v['price'];
+                                    $orderDetail['price'] : $v['price'];
                             };
                             array_walk($shipping['ShipmentItems'], $applyPrice);
 
@@ -334,7 +332,7 @@ 
         //     }
         // });
         // 選択された支払い方法の名称をエンティティにコピーする
-        $builder->addEventListener(FormEvents::POST_SUBMIT, function (FormEvent $event) {
+        $builder->addEventListener(FormEvents::POST_SUBMIT, function(FormEvent $event) {
             $Order = $event->getData();
             $Payment = $Order->getPayment();
             if (!is_null($Payment)) {
@@ -342,7 +340,7 @@ 
             }
         });
         // 会員受注の場合、会員の性別/職業/誕生日をエンティティにコピーする
-        $builder->addEventListener(FormEvents::POST_SUBMIT, function (FormEvent $event) {
+        $builder->addEventListener(FormEvents::POST_SUBMIT, function(FormEvent $event) {
             $Order = $event->getData();
             $Customer = $Order->getCustomer();
             if (!is_null($Customer)) {