EC-CUBE / ec-cube

src/Eccube/Controller/Admin/Product/ProductController.php

Doc Comments +1 added lines

@@ -792,6 +792,7 @@
      * ProductCategory作成
      * @param \Eccube\Entity\Product $Product
      * @param \Eccube\Entity\Category $Category
+     * @param integer $count
      * @return \Eccube\Entity\ProductCategory
      */
     private function createProductCategory($Product, $Category, $count)

Spacing +14 added lines, -14 removed lines

@@ -319,7 +319,7 @@ 
                     }
 
                     $extension = $image->getClientOriginalExtension();
-                    $filename = date('mdHis') . uniqid('_') . '.' . $extension;
+                    $filename = date('mdHis').uniqid('_').'.'.$extension;
                     $image->move($this->appConfig['image_temp_realdir'], $filename);
                     $files[] = $filename;
                 }
@@ -400,7 +400,7 @@ 
         $form = $builder->getForm();
 
         if (!$has_class) {
-            $ProductClass->setStockUnlimited((boolean)$ProductClass->getStockUnlimited());
+            $ProductClass->setStockUnlimited((boolean) $ProductClass->getStockUnlimited());
             $form['class']->setData($ProductClass);
         }
 
@@ -486,8 +486,8 @@ 
                 $Categories = $form->get('Category')->getData();
                 $categoriesIdList = array();
                 foreach ($Categories as $Category) {
-                    foreach($Category->getPath() as $ParentCategory){
-                        if (!isset($categoriesIdList[$ParentCategory->getId()])){
+                    foreach ($Category->getPath() as $ParentCategory) {
+                        if (!isset($categoriesIdList[$ParentCategory->getId()])) {
                             $ProductCategory = $this->createProductCategory($Product, $ParentCategory, $count);
                             $this->entityManager->persist($ProductCategory);
                             $count++;
@@ -496,7 +496,7 @@ 
                             $categoriesIdList[$ParentCategory->getId()] = true;
                         }
                     }
-                    if (!isset($categoriesIdList[$Category->getId()])){
+                    if (!isset($categoriesIdList[$Category->getId()])) {
                         $ProductCategory = $this->createProductCategory($Product, $Category, $count);
                         $this->entityManager->persist($ProductCategory);
                         $count++;
@@ -518,7 +518,7 @@ 
                     $this->entityManager->persist($ProductImage);
 
                     // 移動
-                    $file = new File($this->appConfig['image_temp_realdir'] . '/' . $add_image);
+                    $file = new File($this->appConfig['image_temp_realdir'].'/'.$add_image);
                     $file->move($this->appConfig['image_save_realdir']);
                 }
 
@@ -538,7 +538,7 @@ 
 
                     // 削除
                     $fs = new Filesystem();
-                    $fs->remove($this->appConfig['image_save_realdir'] . '/' . $delete_image);
+                    $fs->remove($this->appConfig['image_save_realdir'].'/'.$delete_image);
                 }
                 $this->entityManager->persist($Product);
                 $this->entityManager->flush();
@@ -702,7 +702,7 @@ 
                 foreach ($deleteImages as $deleteImage) {
                     try {
                         $fs = new Filesystem();
-                        $fs->remove($this->appConfig['image_save_realdir'] . '/' . $deleteImage);
+                        $fs->remove($this->appConfig['image_save_realdir'].'/'.$deleteImage);
                     } catch (\Exception $e) {
                         // エラーが発生しても無視する
                     }
@@ -776,10 +776,10 @@ 
 
                     // 画像ファイルを新規作成
                     $extension = pathinfo($Image->getFileName(), PATHINFO_EXTENSION);
-                    $filename = date('mdHis') . uniqid('_') . '.' . $extension;
+                    $filename = date('mdHis').uniqid('_').'.'.$extension;
                     try {
                         $fs = new Filesystem();
-                        $fs->copy($this->appConfig['image_save_realdir'] . '/' . $Image->getFileName(), $this->appConfig['image_save_realdir'] . '/' . $filename);
+                        $fs->copy($this->appConfig['image_save_realdir'].'/'.$Image->getFileName(), $this->appConfig['image_save_realdir'].'/'.$filename);
                     } catch (\Exception $e) {
                         // エラーが発生しても無視する
                     }
@@ -859,7 +859,7 @@ 
         $em->getConfiguration()->setSQLLogger(null);
 
         $response = new StreamedResponse();
-        $response->setCallback(function () use ($app, $request) {
+        $response->setCallback(function() use ($app, $request) {
 
             // CSV種別を元に初期化.
             $this->csvExportService->initCsvType(CsvType::CSV_TYPE_PRODUCT);
@@ -899,7 +899,7 @@ 
             // データ行の出力.
             $this->csvExportService->setExportQueryBuilder($qb);
 
-            $this->csvExportService->exportData(function ($entity, CsvExportService $csvService) use ($app, $request) {
+            $this->csvExportService->exportData(function($entity, CsvExportService $csvService) use ($app, $request) {
                 $Csvs = $csvService->getCsvs();
 
                 /** @var $Product \Eccube\Entity\Product */
@@ -942,9 +942,9 @@ 
         });
 
         $now = new \DateTime();
-        $filename = 'product_' . $now->format('YmdHis') . '.csv';
+        $filename = 'product_'.$now->format('YmdHis').'.csv';
         $response->headers->set('Content-Type', 'application/octet-stream');
-        $response->headers->set('Content-Disposition', 'attachment; filename=' . $filename);
+        $response->headers->set('Content-Disposition', 'attachment; filename='.$filename);
         $response->send();
 
         log_info('商品CSV出力ファイル名', array($filename));

src/Eccube/Controller/Admin/Product/CsvImportController.php

Indentation +5 added lines, -5 removed lines

@@ -1155,11 +1155,11 @@
     }
     
         /**
-     * ProductCategory作成
-     * @param \Eccube\Entity\Product $Product
-     * @param \Eccube\Entity\Category $Category
-     * @return ProductCategory
-     */
+         * ProductCategory作成
+         * @param \Eccube\Entity\Product $Product
+         * @param \Eccube\Entity\Category $Category
+         * @return ProductCategory
+         */
     private function makeProductCategory($Product, $Category, $rank)
     {
         $ProductCategory = new ProductCategory();

Doc Comments +11 added lines

@@ -679,6 +679,8 @@ 
     /**
      * 登録、更新時のエラー画面表示
      *
+     * @param Application $app
+     * @param \Symfony\Component\Form\FormInterface $form
      */
     protected function render($app, $form, $headers)
     {
@@ -710,6 +712,7 @@ 
      * アップロードされたCSVファイルの行ごとの処理
      *
      * @param $formFile
+     * @param Application $app
      * @return CsvImportService
      */
     protected function getImportData($app, $formFile)
@@ -787,6 +790,8 @@ 
 
     /**
      * 商品カテゴリの削除、登録
+     * @param Application $app
+     * @param CsvImportService $data
      */
     protected function createProductCategory($row, Product $Product, $app, $data)
     {
@@ -886,6 +891,8 @@ 
 
     /**
      * 商品規格分類1、商品規格分類2がnullとなる商品規格情報を作成
+     * @param Application $app
+     * @param CsvImportService $data
      */
     protected function createProductClass($row, Product $Product, $app, $data, $ClassCategory1 = null, $ClassCategory2 = null)
     {
@@ -1027,6 +1034,8 @@ 
 
     /**
      * 商品規格情報を更新
+     * @param Application $app
+     * @param CsvImportService $data
      */
     protected function updateProductClass($row, Product $Product, ProductClass $ProductClass, $app, $data)
     {
@@ -1173,6 +1182,7 @@ 
     /**
      * 登録、更新時のエラー画面表示
      *
+     * @param string $message
      */
     protected function addErrors($message)
     {
@@ -1247,6 +1257,7 @@ 
      * ProductCategory作成
      * @param \Eccube\Entity\Product $Product
      * @param \Eccube\Entity\Category $Category
+     * @param integer $rank
      * @return ProductCategory
      */
     private function makeProductCategory($Product, $Category, $rank)

Spacing +78 added lines, -78 removed lines

@@ -192,7 +192,7 @@ 
                     foreach ($data as $row) {
 
                         if ($headerSize != count($row)) {
-                            $this->addErrors(($data->key() + 1) . '行目のCSVフォーマットが一致しません。');
+                            $this->addErrors(($data->key() + 1).'行目のCSVフォーマットが一致しません。');
                             return $this->render($app, $form, $headers);
                         }
 
@@ -203,33 +203,33 @@ 
                             if (preg_match('/^\d+$/', $row['商品ID'])) {
                                 $Product = $this->productRepository->find($row['商品ID']);
                                 if (!$Product) {
-                                    $this->addErrors(($data->key() + 1) . '行目の商品IDが存在しません。');
+                                    $this->addErrors(($data->key() + 1).'行目の商品IDが存在しません。');
                                     return $this->render($app, $form, $headers);
                                 }
                             } else {
-                                $this->addErrors(($data->key() + 1) . '行目の商品IDが存在しません。');
+                                $this->addErrors(($data->key() + 1).'行目の商品IDが存在しません。');
                                 return $this->render($app, $form, $headers);
                             }
 
                         }
 
                         if ($row['公開ステータス(ID)'] == '') {
-                            $this->addErrors(($data->key() + 1) . '行目の公開ステータス(ID)が設定されていません。');
+                            $this->addErrors(($data->key() + 1).'行目の公開ステータス(ID)が設定されていません。');
                         } else {
                             if (preg_match('/^\d+$/', $row['公開ステータス(ID)'])) {
                                 $Disp = $this->dispRepository->find($row['公開ステータス(ID)']);
                                 if (!$Disp) {
-                                    $this->addErrors(($data->key() + 1) . '行目の公開ステータス(ID)が存在しません。');
+                                    $this->addErrors(($data->key() + 1).'行目の公開ステータス(ID)が存在しません。');
                                 } else {
                                     $Product->setStatus($Disp);
                                 }
                             } else {
-                                $this->addErrors(($data->key() + 1) . '行目の公開ステータス(ID)が存在しません。');
+                                $this->addErrors(($data->key() + 1).'行目の公開ステータス(ID)が存在しません。');
                             }
                         }
 
                         if (Str::isBlank($row['商品名'])) {
-                            $this->addErrors(($data->key() + 1) . '行目の商品名が設定されていません。');
+                            $this->addErrors(($data->key() + 1).'行目の商品名が設定されていません。');
                             return $this->render($app, $form, $headers);
                         } else {
                             $Product->setName(Str::trimAll($row['商品名']));
@@ -268,10 +268,10 @@ 
                         if ($row['商品削除フラグ'] == '') {
                             $Product->setDelFlg(Constant::DISABLED);
                         } else {
-                            if ($row['商品削除フラグ'] == (string)Constant::DISABLED || $row['商品削除フラグ'] == (string)Constant::ENABLED) {
+                            if ($row['商品削除フラグ'] == (string) Constant::DISABLED || $row['商品削除フラグ'] == (string) Constant::ENABLED) {
                                 $Product->setDelFlg($row['商品削除フラグ']);
                             } else {
-                                $this->addErrors(($data->key() + 1) . '行目の商品削除フラグが設定されていません。');
+                                $this->addErrors(($data->key() + 1).'行目の商品削除フラグが設定されていません。');
                                 return $this->render($app, $form, $headers);
                             }
                         }
@@ -298,7 +298,7 @@ 
                                     if (preg_match('/^\d+$/', $deliveryFee) && $deliveryFee >= 0) {
                                         $ProductClassOrg->setDeliveryFee($deliveryFee);
                                     } else {
-                                        $this->addErrors(($data->key() + 1) . '行目の送料は0以上の数値を設定してください。');
+                                        $this->addErrors(($data->key() + 1).'行目の送料は0以上の数値を設定してください。');
                                     }
                                 }
                             }
@@ -306,7 +306,7 @@ 
                             if ($row['規格分類1(ID)'] != '') {
 
                                 if ($row['規格分類1(ID)'] == $row['規格分類2(ID)']) {
-                                    $this->addErrors(($data->key() + 1) . '行目の規格分類1(ID)と規格分類2(ID)には同じ値を使用できません。');
+                                    $this->addErrors(($data->key() + 1).'行目の規格分類1(ID)と規格分類2(ID)には同じ値を使用できません。');
                                 } else {
                                     // 商品規格あり
                                     // 企画分類あり商品を作成
@@ -321,30 +321,30 @@ 
                                     if (preg_match('/^\d+$/', $row['規格分類1(ID)'])) {
                                         $ClassCategory1 = $this->classCategoryRepository->find($row['規格分類1(ID)']);
                                         if (!$ClassCategory1) {
-                                            $this->addErrors(($data->key() + 1) . '行目の規格分類1(ID)が存在しません。');
+                                            $this->addErrors(($data->key() + 1).'行目の規格分類1(ID)が存在しません。');
                                         } else {
                                             $ProductClass->setClassCategory1($ClassCategory1);
                                         }
                                     } else {
-                                        $this->addErrors(($data->key() + 1) . '行目の規格分類1(ID)が存在しません。');
+                                        $this->addErrors(($data->key() + 1).'行目の規格分類1(ID)が存在しません。');
                                     }
 
                                     if ($row['規格分類2(ID)'] != '') {
                                         if (preg_match('/^\d+$/', $row['規格分類2(ID)'])) {
                                             $ClassCategory2 = $this->classCategoryRepository->find($row['規格分類2(ID)']);
                                             if (!$ClassCategory2) {
-                                                $this->addErrors(($data->key() + 1) . '行目の規格分類2(ID)が存在しません。');
+                                                $this->addErrors(($data->key() + 1).'行目の規格分類2(ID)が存在しません。');
                                             } else {
                                                 if ($ClassCategory1 &&
                                                     ($ClassCategory1->getClassName()->getId() == $ClassCategory2->getClassName()->getId())
                                                 ) {
-                                                    $this->addErrors(($data->key() + 1) . '行目の規格分類1(ID)と規格分類2(ID)の規格名が同じです。');
+                                                    $this->addErrors(($data->key() + 1).'行目の規格分類1(ID)と規格分類2(ID)の規格名が同じです。');
                                                 } else {
                                                     $ProductClass->setClassCategory2($ClassCategory2);
                                                 }
                                             }
                                         } else {
-                                            $this->addErrors(($data->key() + 1) . '行目の規格分類2(ID)が存在しません。');
+                                            $this->addErrors(($data->key() + 1).'行目の規格分類2(ID)が存在しません。');
                                         }
                                     }
                                     $ProductClass->setProductStock($ProductStock);
@@ -356,7 +356,7 @@ 
 
                             } else {
                                 if ($row['規格分類2(ID)'] != '') {
-                                    $this->addErrors(($data->key() + 1) . '行目の規格分類1(ID)が存在しません。');
+                                    $this->addErrors(($data->key() + 1).'行目の規格分類1(ID)が存在しません。');
                                 }
                             }
 
@@ -384,7 +384,7 @@ 
                                             if (preg_match('/^\d+$/', $deliveryFee) && $deliveryFee >= 0) {
                                                 $pc->setDeliveryFee($deliveryFee);
                                             } else {
-                                                $this->addErrors(($data->key() + 1) . '行目の送料は0以上の数値を設定してください。');
+                                                $this->addErrors(($data->key() + 1).'行目の送料は0以上の数値を設定してください。');
                                             }
                                         }
                                     }
@@ -406,7 +406,7 @@ 
                                 }
 
                                 if ($row['規格分類1(ID)'] == $row['規格分類2(ID)']) {
-                                    $this->addErrors(($data->key() + 1) . '行目の規格分類1(ID)と規格分類2(ID)には同じ値を使用できません。');
+                                    $this->addErrors(($data->key() + 1).'行目の規格分類1(ID)と規格分類2(ID)には同じ値を使用できません。');
                                 } else {
 
                                     // 必ず規格分類1がセットされている
@@ -415,36 +415,36 @@ 
                                     if (preg_match('/^\d+$/', $classCategoryId1)) {
                                         $ClassCategory1 = $this->classCategoryRepository->find($classCategoryId1);
                                         if (!$ClassCategory1) {
-                                            $this->addErrors(($data->key() + 1) . '行目の規格分類1(ID)が存在しません。');
+                                            $this->addErrors(($data->key() + 1).'行目の規格分類1(ID)が存在しません。');
                                         }
                                     } else {
-                                        $this->addErrors(($data->key() + 1) . '行目の規格分類1(ID)が存在しません。');
+                                        $this->addErrors(($data->key() + 1).'行目の規格分類1(ID)が存在しません。');
                                     }
 
                                     $ClassCategory2 = null;
                                     if ($row['規格分類2(ID)'] != '') {
                                         if ($pc->getClassCategory1() != null && $pc->getClassCategory2() == null) {
-                                            $this->addErrors(($data->key() + 1) . '行目の規格分類2(ID)は設定できません。');
+                                            $this->addErrors(($data->key() + 1).'行目の規格分類2(ID)は設定できません。');
                                         } else {
                                             if (preg_match('/^\d+$/', $classCategoryId2)) {
                                                 $ClassCategory2 = $this->classCategoryRepository->find($classCategoryId2);
                                                 if (!$ClassCategory2) {
-                                                    $this->addErrors(($data->key() + 1) . '行目の規格分類2(ID)が存在しません。');
+                                                    $this->addErrors(($data->key() + 1).'行目の規格分類2(ID)が存在しません。');
                                                 } else {
                                                     if ($ClassCategory1 &&
                                                         ($ClassCategory1->getClassName()->getId() == $ClassCategory2->getClassName()->getId())
                                                     ) {
-                                                        $this->addErrors(($data->key() + 1) . '行目の規格分類1(ID)と規格分類2(ID)の規格名が同じです。');
+                                                        $this->addErrors(($data->key() + 1).'行目の規格分類1(ID)と規格分類2(ID)の規格名が同じです。');
                                                     }
                                                 }
                                             } else {
-                                                $this->addErrors(($data->key() + 1) . '行目の規格分類2(ID)が存在しません。');
+                                                $this->addErrors(($data->key() + 1).'行目の規格分類2(ID)が存在しません。');
                                             }
 
                                         }
                                     } else {
                                         if ($pc->getClassCategory1() != null && $pc->getClassCategory2() != null) {
-                                            $this->addErrors(($data->key() + 1) . '行目の規格分類2(ID)に値を設定してください。');
+                                            $this->addErrors(($data->key() + 1).'行目の規格分類2(ID)に値を設定してください。');
                                         }
                                     }
                                     $ProductClass = $this->createProductClass($row, $Product, $app, $data, $ClassCategory1, $ClassCategory2);
@@ -455,7 +455,7 @@ 
                                             if (preg_match('/^\d+$/', $deliveryFee) && $deliveryFee >= 0) {
                                                 $ProductClass->setDeliveryFee($deliveryFee);
                                             } else {
-                                                $this->addErrors(($data->key() + 1) . '行目の送料は0以上の数値を設定してください。');
+                                                $this->addErrors(($data->key() + 1).'行目の送料は0以上の数値を設定してください。');
                                             }
                                         }
                                     }
@@ -545,7 +545,7 @@ 
                     foreach ($data as $row) {
 
                         if ($headerSize != count($row)) {
-                            $this->addErrors(($data->key() + 1) . '行目のCSVフォーマットが一致しません。');
+                            $this->addErrors(($data->key() + 1).'行目のCSVフォーマットが一致しません。');
                             return $this->render($app, $form, $headers);
                         }
 
@@ -553,23 +553,23 @@ 
                             $Category = new Category();
                         } else {
                             if (!preg_match('/^\d+$/', $row['カテゴリID'])) {
-                                $this->addErrors(($data->key() + 1) . '行目のカテゴリIDが存在しません。');
+                                $this->addErrors(($data->key() + 1).'行目のカテゴリIDが存在しません。');
                                 return $this->render($app, $form, $headers);
                             }
                             $Category = $this->categoryRepository->find($row['カテゴリID']);
                             if (!$Category) {
-                                $this->addErrors(($data->key() + 1) . '行目のカテゴリIDが存在しません。');
+                                $this->addErrors(($data->key() + 1).'行目のカテゴリIDが存在しません。');
                                 return $this->render($app, $form, $headers);
                             }
                             if ($row['カテゴリID'] == $row['親カテゴリID']) {
-                                $this->addErrors(($data->key() + 1) . '行目のカテゴリIDと親カテゴリIDが同じです。');
+                                $this->addErrors(($data->key() + 1).'行目のカテゴリIDと親カテゴリIDが同じです。');
                                 return $this->render($app, $form, $headers);
                             }
 
                         }
 
                         if (Str::isBlank($row['カテゴリ名'])) {
-                            $this->addErrors(($data->key() + 1) . '行目のカテゴリ名が設定されていません。');
+                            $this->addErrors(($data->key() + 1).'行目のカテゴリ名が設定されていません。');
                             return $this->render($app, $form, $headers);
                         } else {
                             $Category->setName(Str::trimAll($row['カテゴリ名']));
@@ -578,13 +578,13 @@ 
                         if ($row['親カテゴリID'] != '') {
 
                             if (!preg_match('/^\d+$/', $row['親カテゴリID'])) {
-                                $this->addErrors(($data->key() + 1) . '行目の親カテゴリIDが存在しません。');
+                                $this->addErrors(($data->key() + 1).'行目の親カテゴリIDが存在しません。');
                                 return $this->render($app, $form, $headers);
                             }
 
                             $ParentCategory = $this->categoryRepository->find($row['親カテゴリID']);
                             if (!$ParentCategory) {
-                                $this->addErrors(($data->key() + 1) . '行目の親カテゴリIDが存在しません。');
+                                $this->addErrors(($data->key() + 1).'行目の親カテゴリIDが存在しません。');
                                 return $this->render($app, $form, $headers);
                             }
 
@@ -600,14 +600,14 @@ 
                         }
 
                         if ($this->appConfig['category_nest_level'] < $Category->getLevel()) {
-                            $this->addErrors(($data->key() + 1) . '行目のカテゴリが最大レベルを超えているため設定できません。');
+                            $this->addErrors(($data->key() + 1).'行目のカテゴリが最大レベルを超えているため設定できません。');
                             return $this->render($app, $form, $headers);
                         }
 
                         $status = $this->categoryRepository->save($Category);
 
                         if (!$status) {
-                            $this->addErrors(($data->key() + 1) . '行目のカテゴリが設定できません。');
+                            $this->addErrors(($data->key() + 1).'行目のカテゴリが設定できません。');
                         }
 
                         if ($this->hasErrors()) {
@@ -654,7 +654,7 @@ 
             throw new NotFoundHttpException();
         }
 
-        $response->setCallback(function () use ($app, $request, $headers) {
+        $response->setCallback(function() use ($app, $request, $headers) {
 
             // ヘッダ行の出力
             $row = array();
@@ -669,7 +669,7 @@ 
         });
 
         $response->headers->set('Content-Type', 'application/octet-stream');
-        $response->headers->set('Content-Disposition', 'attachment; filename=' . $filename);
+        $response->headers->set('Content-Disposition', 'attachment; filename='.$filename);
         $response->send();
 
         return $response;
@@ -692,7 +692,7 @@ 
         if (!empty($this->fileName)) {
             try {
                 $fs = new Filesystem();
-                $fs->remove($this->appConfig['csv_temp_realdir'] . '/' . $this->fileName);
+                $fs->remove($this->appConfig['csv_temp_realdir'].'/'.$this->fileName);
             } catch (\Exception $e) {
                 // エラーが発生しても無視する
             }
@@ -715,10 +715,10 @@ 
     protected function getImportData($app, $formFile)
     {
         // アップロードされたCSVファイルを一時ディレクトリに保存
-        $this->fileName = 'upload_' . Str::random() . '.' . $formFile->getClientOriginalExtension();
+        $this->fileName = 'upload_'.Str::random().'.'.$formFile->getClientOriginalExtension();
         $formFile->move($this->appConfig['csv_temp_realdir'], $this->fileName);
 
-        $file = file_get_contents($this->appConfig['csv_temp_realdir'] . '/' . $this->fileName);
+        $file = file_get_contents($this->appConfig['csv_temp_realdir'].'/'.$this->fileName);
 
         if ('\\' === DIRECTORY_SEPARATOR && PHP_VERSION_ID >= 70000) {
             // Windows 環境の PHP7 の場合はファイルエンコーディングを CP932 に合わせる
@@ -814,8 +814,8 @@ 
                 if (!$Category) {
                     $this->addErrors(($data->key() + 1).'行目の商品カテゴリ(ID)「'.$category.'」が存在しません。');
                 } else {
-                    foreach($Category->getPath() as $ParentCategory){
-                        if (!isset($categoriesIdList[$ParentCategory->getId()])){
+                    foreach ($Category->getPath() as $ParentCategory) {
+                        if (!isset($categoriesIdList[$ParentCategory->getId()])) {
                             $ProductCategory = $this->makeProductCategory($Product, $ParentCategory, $rank);
                             $this->entityManager->persist($ProductCategory);
                             $rank++;
@@ -823,7 +823,7 @@ 
                             $categoriesIdList[$ParentCategory->getId()] = true;
                         }
                     }
-                    if (!isset($categoriesIdList[$Category->getId()])){
+                    if (!isset($categoriesIdList[$Category->getId()])) {
                         $ProductCategory = $this->makeProductCategory($Product, $Category, $rank);
                         $rank++;
                         $this->em->persist($ProductCategory);
@@ -878,7 +878,7 @@ 
                 }
             }
             if (!$Tag) {
-                $this->addErrors(($data->key() + 1) . '行目のタグ(ID)「' . $tag_id . '」が存在しません。');
+                $this->addErrors(($data->key() + 1).'行目のタグ(ID)「'.$tag_id.'」が存在しません。');
             }
         }
     }
@@ -896,17 +896,17 @@ 
 
 
         if ($row['商品種別(ID)'] == '') {
-            $this->addErrors(($data->key() + 1) . '行目の商品種別(ID)が設定されていません。');
+            $this->addErrors(($data->key() + 1).'行目の商品種別(ID)が設定されていません。');
         } else {
             if (preg_match('/^\d+$/', $row['商品種別(ID)'])) {
                 $ProductType = $this->productTypeRepository->find($row['商品種別(ID)']);
                 if (!$ProductType) {
-                    $this->addErrors(($data->key() + 1) . '行目の商品種別(ID)が存在しません。');
+                    $this->addErrors(($data->key() + 1).'行目の商品種別(ID)が存在しません。');
                 } else {
                     $ProductClass->setProductType($ProductType);
                 }
             } else {
-                $this->addErrors(($data->key() + 1) . '行目の商品種別(ID)が存在しません。');
+                $this->addErrors(($data->key() + 1).'行目の商品種別(ID)が存在しません。');
             }
         }
 
@@ -917,12 +917,12 @@ 
             if (preg_match('/^\d+$/', $row['発送日目安(ID)'])) {
                 $DeliveryDate = $this->deliveryDateRepository->find($row['発送日目安(ID)']);
                 if (!$DeliveryDate) {
-                    $this->addErrors(($data->key() + 1) . '行目の発送日目安(ID)が存在しません。');
+                    $this->addErrors(($data->key() + 1).'行目の発送日目安(ID)が存在しません。');
                 } else {
                     $ProductClass->setDeliveryDate($DeliveryDate);
                 }
             } else {
-                $this->addErrors(($data->key() + 1) . '行目の発送日目安(ID)が存在しません。');
+                $this->addErrors(($data->key() + 1).'行目の発送日目安(ID)が存在しません。');
             }
         }
 
@@ -933,19 +933,19 @@ 
         }
 
         if ($row['在庫数無制限フラグ'] == '') {
-            $this->addErrors(($data->key() + 1) . '行目の在庫数無制限フラグが設定されていません。');
+            $this->addErrors(($data->key() + 1).'行目の在庫数無制限フラグが設定されていません。');
         } else {
             if ($row['在庫数無制限フラグ'] == (string) Constant::DISABLED) {
                 $ProductClass->setStockUnlimited(Constant::DISABLED);
                 // 在庫数が設定されていなければエラー
                 if ($row['在庫数'] == '') {
-                    $this->addErrors(($data->key() + 1) . '行目の在庫数が設定されていません。');
+                    $this->addErrors(($data->key() + 1).'行目の在庫数が設定されていません。');
                 } else {
                     $stock = str_replace(',', '', $row['在庫数']);
                     if (preg_match('/^\d+$/', $stock) && $stock >= 0) {
                         $ProductClass->setStock($stock);
                     } else {
-                        $this->addErrors(($data->key() + 1) . '行目の在庫数は0以上の数値を設定してください。');
+                        $this->addErrors(($data->key() + 1).'行目の在庫数は0以上の数値を設定してください。');
                     }
                 }
 
@@ -953,7 +953,7 @@ 
                 $ProductClass->setStockUnlimited(Constant::ENABLED);
                 $ProductClass->setStock(null);
             } else {
-                $this->addErrors(($data->key() + 1) . '行目の在庫数無制限フラグが設定されていません。');
+                $this->addErrors(($data->key() + 1).'行目の在庫数無制限フラグが設定されていません。');
             }
         }
 
@@ -962,7 +962,7 @@ 
             if (preg_match('/^\d+$/', $saleLimit) && $saleLimit >= 0) {
                 $ProductClass->setSaleLimit($saleLimit);
             } else {
-                $this->addErrors(($data->key() + 1) . '行目の販売制限数は0以上の数値を設定してください。');
+                $this->addErrors(($data->key() + 1).'行目の販売制限数は0以上の数値を設定してください。');
             }
         }
 
@@ -971,18 +971,18 @@ 
             if (preg_match('/^\d+$/', $price01) && $price01 >= 0) {
                 $ProductClass->setPrice01($price01);
             } else {
-                $this->addErrors(($data->key() + 1) . '行目の通常価格は0以上の数値を設定してください。');
+                $this->addErrors(($data->key() + 1).'行目の通常価格は0以上の数値を設定してください。');
             }
         }
 
         if ($row['販売価格'] == '') {
-            $this->addErrors(($data->key() + 1) . '行目の販売価格が設定されていません。');
+            $this->addErrors(($data->key() + 1).'行目の販売価格が設定されていません。');
         } else {
             $price02 = str_replace(',', '', $row['販売価格']);
             if (preg_match('/^\d+$/', $price02) && $price02 >= 0) {
                 $ProductClass->setPrice02($price02);
             } else {
-                $this->addErrors(($data->key() + 1) . '行目の販売価格は0以上の数値を設定してください。');
+                $this->addErrors(($data->key() + 1).'行目の販売価格は0以上の数値を設定してください。');
             }
         }
 
@@ -991,7 +991,7 @@ 
             if (preg_match('/^\d+$/', $delivery_fee) && $delivery_fee >= 0) {
                 $ProductClass->setDeliveryFee($delivery_fee);
             } else {
-                $this->addErrors(($data->key() + 1) . '行目の送料は0以上の数値を設定してください。');
+                $this->addErrors(($data->key() + 1).'行目の送料は0以上の数値を設定してください。');
             }
         }
 
@@ -1001,7 +1001,7 @@ 
             if ($row['商品規格削除フラグ'] == (string) Constant::DISABLED || $row['商品規格削除フラグ'] == (string) Constant::ENABLED) {
                 $ProductClass->setDelFlg($row['商品規格削除フラグ']);
             } else {
-                $this->addErrors(($data->key() + 1) . '行目の商品規格削除フラグが設定されていません。');
+                $this->addErrors(($data->key() + 1).'行目の商品規格削除フラグが設定されていません。');
             }
         }
 
@@ -1034,17 +1034,17 @@ 
         $ProductClass->setProduct($Product);
 
         if ($row['商品種別(ID)'] == '') {
-            $this->addErrors(($data->key() + 1) . '行目の商品種別(ID)が設定されていません。');
+            $this->addErrors(($data->key() + 1).'行目の商品種別(ID)が設定されていません。');
         } else {
             if (preg_match('/^\d+$/', $row['商品種別(ID)'])) {
                 $ProductType = $this->productTypeRepository->find($row['商品種別(ID)']);
                 if (!$ProductType) {
-                    $this->addErrors(($data->key() + 1) . '行目の商品種別(ID)が存在しません。');
+                    $this->addErrors(($data->key() + 1).'行目の商品種別(ID)が存在しません。');
                 } else {
                     $ProductClass->setProductType($ProductType);
                 }
             } else {
-                $this->addErrors(($data->key() + 1) . '行目の商品種別(ID)が存在しません。');
+                $this->addErrors(($data->key() + 1).'行目の商品種別(ID)が存在しません。');
             }
         }
 
@@ -1053,12 +1053,12 @@ 
             if (preg_match('/^\d+$/', $row['規格分類1(ID)'])) {
                 $ClassCategory = $this->classCategoryRepository->find($row['規格分類1(ID)']);
                 if (!$ClassCategory) {
-                    $this->addErrors(($data->key() + 1) . '行目の規格分類1(ID)が存在しません。');
+                    $this->addErrors(($data->key() + 1).'行目の規格分類1(ID)が存在しません。');
                 } else {
                     $ProductClass->setClassCategory1($ClassCategory);
                 }
             } else {
-                $this->addErrors(($data->key() + 1) . '行目の規格分類1(ID)が存在しません。');
+                $this->addErrors(($data->key() + 1).'行目の規格分類1(ID)が存在しません。');
             }
         }
 
@@ -1066,12 +1066,12 @@ 
             if (preg_match('/^\d+$/', $row['規格分類2(ID)'])) {
                 $ClassCategory = $this->classCategoryRepository->find($row['規格分類2(ID)']);
                 if (!$ClassCategory) {
-                    $this->addErrors(($data->key() + 1) . '行目の規格分類2(ID)が存在しません。');
+                    $this->addErrors(($data->key() + 1).'行目の規格分類2(ID)が存在しません。');
                 } else {
                     $ProductClass->setClassCategory2($ClassCategory);
                 }
             } else {
-                $this->addErrors(($data->key() + 1) . '行目の規格分類2(ID)が存在しません。');
+                $this->addErrors(($data->key() + 1).'行目の規格分類2(ID)が存在しません。');
             }
         }
 
@@ -1079,12 +1079,12 @@ 
             if (preg_match('/^\d+$/', $row['発送日目安(ID)'])) {
                 $DeliveryDate = $this->deliveryDateRepository->find($row['発送日目安(ID)']);
                 if (!$DeliveryDate) {
-                    $this->addErrors(($data->key() + 1) . '行目の発送日目安(ID)が存在しません。');
+                    $this->addErrors(($data->key() + 1).'行目の発送日目安(ID)が存在しません。');
                 } else {
                     $ProductClass->setDeliveryDate($DeliveryDate);
                 }
             } else {
-                $this->addErrors(($data->key() + 1) . '行目の発送日目安(ID)が存在しません。');
+                $this->addErrors(($data->key() + 1).'行目の発送日目安(ID)が存在しません。');
             }
         }
 
@@ -1095,19 +1095,19 @@ 
         }
 
         if ($row['在庫数無制限フラグ'] == '') {
-            $this->addErrors(($data->key() + 1) . '行目の在庫数無制限フラグが設定されていません。');
+            $this->addErrors(($data->key() + 1).'行目の在庫数無制限フラグが設定されていません。');
         } else {
             if ($row['在庫数無制限フラグ'] == (string) Constant::DISABLED) {
                 $ProductClass->setStockUnlimited(Constant::DISABLED);
                 // 在庫数が設定されていなければエラー
                 if ($row['在庫数'] == '') {
-                    $this->addErrors(($data->key() + 1) . '行目の在庫数が設定されていません。');
+                    $this->addErrors(($data->key() + 1).'行目の在庫数が設定されていません。');
                 } else {
                     $stock = str_replace(',', '', $row['在庫数']);
                     if (preg_match('/^\d+$/', $stock) && $stock >= 0) {
                         $ProductClass->setStock($row['在庫数']);
                     } else {
-                        $this->addErrors(($data->key() + 1) . '行目の在庫数は0以上の数値を設定してください。');
+                        $this->addErrors(($data->key() + 1).'行目の在庫数は0以上の数値を設定してください。');
                     }
                 }
 
@@ -1115,7 +1115,7 @@ 
                 $ProductClass->setStockUnlimited(Constant::ENABLED);
                 $ProductClass->setStock(null);
             } else {
-                $this->addErrors(($data->key() + 1) . '行目の在庫数無制限フラグが設定されていません。');
+                $this->addErrors(($data->key() + 1).'行目の在庫数無制限フラグが設定されていません。');
             }
         }
 
@@ -1124,7 +1124,7 @@ 
             if (preg_match('/^\d+$/', $saleLimit) && $saleLimit >= 0) {
                 $ProductClass->setSaleLimit($saleLimit);
             } else {
-                $this->addErrors(($data->key() + 1) . '行目の販売制限数は0以上の数値を設定してください。');
+                $this->addErrors(($data->key() + 1).'行目の販売制限数は0以上の数値を設定してください。');
             }
         }
 
@@ -1133,18 +1133,18 @@ 
             if (preg_match('/^\d+$/', $price01) && $price01 >= 0) {
                 $ProductClass->setPrice01($price01);
             } else {
-                $this->addErrors(($data->key() + 1) . '行目の通常価格は0以上の数値を設定してください。');
+                $this->addErrors(($data->key() + 1).'行目の通常価格は0以上の数値を設定してください。');
             }
         }
 
         if ($row['販売価格'] == '') {
-            $this->addErrors(($data->key() + 1) . '行目の販売価格が設定されていません。');
+            $this->addErrors(($data->key() + 1).'行目の販売価格が設定されていません。');
         } else {
             $price02 = str_replace(',', '', $row['販売価格']);
             if (preg_match('/^\d+$/', $price02) && $price02 >= 0) {
                 $ProductClass->setPrice02($price02);
             } else {
-                $this->addErrors(($data->key() + 1) . '行目の販売価格は0以上の数値を設定してください。');
+                $this->addErrors(($data->key() + 1).'行目の販売価格は0以上の数値を設定してください。');
             }
         }
 
@@ -1154,7 +1154,7 @@ 
             if ($row['商品規格削除フラグ'] == (string) Constant::DISABLED || $row['商品規格削除フラグ'] == (string) Constant::ENABLED) {
                 $ProductClass->setDelFlg($row['商品規格削除フラグ']);
             } else {
-                $this->addErrors(($data->key() + 1) . '行目の商品規格削除フラグが設定されていません。');
+                $this->addErrors(($data->key() + 1).'行目の商品規格削除フラグが設定されていません。');
             }
         }
 

src/Eccube/InstallApplication.php

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
 
         if (is_writable($logDir)) {
             if (file_exists($installLog) && !is_writable($installLog)) {
-                die($installLog . ' の書込権限を変更して下さい。');
+                die($installLog.' の書込権限を変更して下さい。');
             }
             // install step2 でログディレクトリに書き込み権限が付与されればログ出力を開始する.
             $app->register(new \Silex\Provider\MonologServiceProvider(), array(

src/Eccube/Repository/CustomerRepository.php

Spacing +5 added lines, -5 removed lines

@@ -153,9 +153,9 @@ 
             $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 . '%');
+                ->setParameter('name', '%'.$clean_key_multi.'%')
+                ->setParameter('kana', '%'.$clean_key_multi.'%')
+                ->setParameter('email', '%'.$clean_key_multi.'%');
         }
 
         // Pref
@@ -205,7 +205,7 @@ 
         if (isset($searchData['tel']) && Str::isNotBlank($searchData['tel'])) {
             $qb
                 ->andWhere('CONCAT(c.tel01, c.tel02, c.tel03) LIKE :tel')
-                ->setParameter('tel', '%' . $searchData['tel'] . '%');
+                ->setParameter('tel', '%'.$searchData['tel'].'%');
         }
 
         // buy_total
@@ -299,7 +299,7 @@ 
                 ->leftJoin('c.Orders', 'o')
                 ->leftJoin('o.OrderDetails', 'od')
                 ->andWhere('od.product_name LIKE :buy_product_name OR od.product_code LIKE :buy_product_name')
-                ->setParameter('buy_product_name', '%' . $searchData['buy_product_code'] . '%');
+                ->setParameter('buy_product_name', '%'.$searchData['buy_product_code'].'%');
         }
 
         // Order By

Doc Comments +5 added lines, -5 removed lines

@@ -322,7 +322,7 @@ 
 
     /**
      * ユニークなシークレットキーを返す
-     * @param $app
+     * @param \Eccube\Application $app
      * @return string
      */
     public function getUniqueSecretKey($app)
@@ -340,7 +340,7 @@ 
 
     /**
      * ユニークなパスワードリセットキーを返す
-     * @param $app
+     * @param \Eccube\Application $app
      * @return string
      */
     public function getUniqueResetKey($app)
@@ -359,7 +359,7 @@ 
     /**
      * saltを生成する
      *
-     * @param $byte
+     * @param integer $byte
      * @return string
      */
     public function createSalt($byte)
@@ -370,7 +370,7 @@ 
     /**
      * 入力されたパスワードをSaltと暗号化する
      *
-     * @param $app
+     * @param \Eccube\Application $app
      * @param  Customer $Customer
      * @return mixed
      */
@@ -430,7 +430,7 @@ 
     /**
      * 会員の初回購入時間、購入時間、購入回数、購入金額を更新する
      *
-     * @param $app
+     * @param \Eccube\Application $app
      * @param  Customer $Customer
      * @param  $orderStatusId
      */

src/Eccube/Command/GeneratorCommand/EntityFromDbGenerator.php

Spacing +1 added lines, -1 removed lines

@@ -164,7 +164,7 @@
             $cmf->setEntityManager($em);
             $metadatas = $cmf->getAllMetadata();
 
-            $filters = array_map(function ($value) {
+            $filters = array_map(function($value) {
                 return ucfirst(Inflector::camelize(str_replace('plg_', '', $value)));
             }, $tableList);
             $metadatas = MetadataFilter::filter($metadatas, $filters);

src/Eccube/Application.php

Doc Comments +1 added lines, -1 removed lines

@@ -964,7 +964,7 @@
      * 失敗 : \Doctrine\DBAL\DBALExceptionエラーが発生( 接続に失敗した場合 )、エラー画面を表示しdie()
      * 備考 : app['debug']がtrueの際は処理を行わない
      *
-     * @return boolean true
+     * @return null|boolean true
      *
      */
     protected function checkDatabaseConnection()

Indentation +7 added lines, -7 removed lines

@@ -543,13 +543,13 @@
         $pluginConfigs = PluginConfigManager::getPluginConfigAll($this['debug']);
         $ormMappings = array();
         $ormMappings[] = array(
-             'type' => 'annotation',
-             'namespace' => 'Eccube\Entity',
-             'path' => array(
-                 __DIR__.'/Entity'
-             ),
-             'use_simple_annotation_reader' => false,
-         );
+                'type' => 'annotation',
+                'namespace' => 'Eccube\Entity',
+                'path' => array(
+                    __DIR__.'/Entity'
+                ),
+                'use_simple_annotation_reader' => false,
+            );
 
         // TODO namespace は暫定
         $ormMappings[] = array(

Unused Use Statements -3 removed lines

@@ -23,9 +23,6 @@
 
 namespace Eccube;
 
-use Doctrine\Common\Annotations\AnnotationReader;
-use Doctrine\Common\Annotations\CachedReader;
-use Doctrine\Common\Cache\ArrayCache;
 use Doctrine\DBAL\Types\Type;
 use Eccube\Doctrine\DBAL\Types\UTCDateTimeType;
 use Eccube\Doctrine\DBAL\Types\UTCDateTimeTzType;

Spacing +27 added lines, -27 removed lines

@@ -170,7 +170,7 @@ 
         $this->register(new \Saxulum\Validator\Provider\SaxulumValidatorProvider());
         $this->register(new MobileDetectServiceProvider());
 
-        $this->error(function (\Exception $e, Request $request, $code) {
+        $this->error(function(\Exception $e, Request $request, $code) {
             if ($this['debug']) {
                 return;
             }
@@ -276,14 +276,14 @@ 
             return $router;
         });
 
-        $this['eccube.router.origin'] = function ($app) {
+        $this['eccube.router.origin'] = function($app) {
             $resource = __DIR__.'/Controller';
             $cachePrefix = 'Origin';
 
             return $app['eccube.router']($resource, $cachePrefix);
         };
 
-        $this['eccube.routers.plugin'] = function ($app) {
+        $this['eccube.routers.plugin'] = function($app) {
             // TODO 有効なプラグインを対象とする必要がある.
             $dirs = Finder::create()
                 ->in($app['config']['root_dir'].'/app/Plugin')
@@ -300,7 +300,7 @@ 
             return $routers;
         };
 
-        $this['eccube.router.extend'] = function ($app) {
+        $this['eccube.router.extend'] = function($app) {
             // TODO ディレクトリ名は暫定
             $resource = $app['config']['root_dir'].'/app/Acme/Controller';
             $cachePrefix = 'Extend';
@@ -310,7 +310,7 @@ 
             return $router;
         };
 
-        $this->extend('request_matcher', function ($matcher, $app) {
+        $this->extend('request_matcher', function($matcher, $app) {
             $matchers = [];
             $matchers[] = $app['eccube.router.extend'];
             foreach ($app['eccube.routers.plugin'] as $router) {
@@ -322,7 +322,7 @@ 
             return new ChainUrlMatcher($matchers, $app['request_context']);
         });
 
-        $this->extend('url_generator', function ($generator, $app) {
+        $this->extend('url_generator', function($generator, $app) {
             $generators = [];
             $generators[] = $app['eccube.router.extend'];
             foreach ($app['eccube.routers.plugin'] as $router) {
@@ -335,7 +335,7 @@ 
         });
 
         // Route CollectionにEC-CUBEで定義したルーティングを追加(debug tool barに出力するため)
-        $this->extend('routes', function ($routes, $app) {
+        $this->extend('routes', function($routes, $app) {
             $routes->addCollection($app['eccube.router.extend']->getRouteCollection());
             foreach ($app['eccube.routers.plugin'] as $router) {
                 $routes->addCollection($router->getRouteCollection());
@@ -368,7 +368,7 @@ 
             'translator.cache_dir' => $this['debug'] ? null : $this['config']['root_dir'].'/app/cache/translator',
             'locale_fallbacks' => ['ja', 'en'],
         ));
-        $this->extend('translator', function ($translator, \Silex\Application $app) {
+        $this->extend('translator', function($translator, \Silex\Application $app) {
             $translator->addLoader('yaml', new \Symfony\Component\Translation\Loader\YamlFileLoader());
 
             $file = __DIR__.'/Resource/locale/validator.'.$app['locale'].'.yml';
@@ -415,14 +415,14 @@ 
         $this->register(new \Silex\Provider\TwigServiceProvider(), array(
             'twig.form.templates' => array('Form/form_layout.twig'),
         ));
-        $this->extend('twig', function (\Twig_Environment $twig, \Silex\Application $app) {
+        $this->extend('twig', function(\Twig_Environment $twig, \Silex\Application $app) {
             $twig->addExtension(new \Eccube\Twig\Extension\EccubeExtension($app));
             $twig->addExtension(new \Twig_Extension_StringLoader());
 
             return $twig;
         });
 
-        $this->before(function (Request $request, \Silex\Application $app) {
+        $this->before(function(Request $request, \Silex\Application $app) {
             $app['admin'] = $app['front'] = false;
             $pathinfo = rawurldecode($request->getPathInfo());
             if (strpos($pathinfo, '/'.trim($app['config']['admin_route'], '/').'/') === 0) {
@@ -438,7 +438,7 @@ 
                 }
                 $paths[] = $app['config']['template_admin_realdir'];
                 $paths[] = __DIR__.'/../../app/Plugin';
-                $cacheDir =  __DIR__.'/../../app/cache/twig/admin';
+                $cacheDir = __DIR__.'/../../app/cache/twig/admin';
             } else {
                 // モバイル端末時、smartphoneディレクトリを探索パスに追加する.
                 if ($app['mobile_detect.device_type'] == \Eccube\Entity\Master\DeviceType::DEVICE_TYPE_SP) {
@@ -453,7 +453,7 @@ 
                 }
                 $paths[] = $app['config']['template_default_realdir'];
                 $paths[] = __DIR__.'/../../app/Plugin';
-                $cacheDir =  __DIR__.'/../../app/cache/twig/'.$app['config']['template_code'];
+                $cacheDir = __DIR__.'/../../app/cache/twig/'.$app['config']['template_code'];
             }
             $app['twig']->setCache($app['debug'] ? null : $cacheDir);
             $app['twig.loader']->addLoader(new \Twig_Loader_Filesystem($paths));
@@ -471,7 +471,7 @@ 
         }, self::EARLY_EVENT);
 
         // twigのグローバル変数を定義.
-        $this->on(\Symfony\Component\HttpKernel\KernelEvents::CONTROLLER, function (\Symfony\Component\HttpKernel\Event\FilterControllerEvent $event) {
+        $this->on(\Symfony\Component\HttpKernel\KernelEvents::CONTROLLER, function(\Symfony\Component\HttpKernel\Event\FilterControllerEvent $event) {
             // 未ログイン時にマイページや管理画面以下にアクセスするとSubRequestで実行されるため,
             // $event->isMasterRequest()ではなく、グローバル変数が初期化済かどうかの判定を行う
             if (isset($this['twig_global_initialized']) && $this['twig_global_initialized'] === true) {
@@ -557,7 +557,7 @@ 
         // メール送信時の文字エンコード指定(デフォルトはUTF-8)
         if (isset($this['config']['mail']['charset_iso_2022_jp']) && is_bool($this['config']['mail']['charset_iso_2022_jp'])) {
             if ($this['config']['mail']['charset_iso_2022_jp'] === true) {
-                \Swift::init(function () {
+                \Swift::init(function() {
                     \Swift_DependencyContainer::getInstance()
                         ->register('mime.qpheaderencoder')
                         ->asAliasOf('mime.base64headerencoder');
@@ -592,7 +592,7 @@ 
         $this->register(new \Saxulum\DoctrineOrmManagerRegistry\Provider\DoctrineOrmManagerRegistryProvider());
 
         $app = $this;
-        $this->extend('db.event_manager', function ($evm) use ($app) {
+        $this->extend('db.event_manager', function($evm) use ($app) {
             $initSubscriber = new InitSubscriber($app);
             $evm->addEventSubscriber($initSubscriber);
 
@@ -687,7 +687,7 @@ 
 
         $this->extend(
             'orm.em.config',
-            function (\Doctrine\ORM\Configuration $config, \Silex\Application $app) {
+            function(\Doctrine\ORM\Configuration $config, \Silex\Application $app) {
 
                 /** @var $chain \Doctrine\Common\Persistence\Mapping\Driver\MappingDriverChain */
                 $chain = $config->getMetadataDriverImpl();
@@ -709,7 +709,7 @@ 
             }
         );
 
-        $this->extend('orm.em', function (\Doctrine\ORM\EntityManager $em, \Silex\Application $app) {
+        $this->extend('orm.em', function(\Doctrine\ORM\EntityManager $em, \Silex\Application $app) {
             // save
             $saveEventSubscriber = new \Eccube\Doctrine\EventSubscriber\SaveEventSubscriber($app);
             $em->getEventManager()->addEventSubscriber($saveEventSubscriber);
@@ -732,7 +732,7 @@ 
             // second level cacheの設定.
             $this->extend(
                 'orm.em.config',
-                function (\Doctrine\ORM\Configuration $config, \Silex\Application $app) {
+                function(\Doctrine\ORM\Configuration $config, \Silex\Application $app) {
                     $config->setSecondLevelCacheEnabled();
                     $cacheConfig = $config->getSecondLevelCacheConfiguration();
                     $regionConfig = $cacheConfig->getRegionsConfiguration();
@@ -821,31 +821,31 @@ 
             array('^/mypage', 'ROLE_USER', $channel),
         );
 
-        $this['eccube.password_encoder'] = function ($app) {
+        $this['eccube.password_encoder'] = function($app) {
             return new \Eccube\Security\Core\Encoder\PasswordEncoder($app['config']);
         };
-        $this['security.encoder_factory'] = function ($app) {
+        $this['security.encoder_factory'] = function($app) {
             return new \Symfony\Component\Security\Core\Encoder\EncoderFactory(array(
                 'Eccube\Entity\Customer' => $app['eccube.password_encoder'],
                 'Eccube\Entity\Member' => $app['eccube.password_encoder'],
             ));
         };
-        $this['eccube.event_listner.security'] = function ($app) {
+        $this['eccube.event_listner.security'] = function($app) {
             return new \Eccube\EventListener\SecurityEventListener($app['orm.em']);
         };
 
         // Voterの設定
-        $this['authority_voter'] = function ($app) {
+        $this['authority_voter'] = function($app) {
             return new \Eccube\Security\Voter\AuthorityVoter($app);
         };
 
-        $this->extend('security.voters', function ($voters, \Silex\Application $app) {
+        $this->extend('security.voters', function($voters, \Silex\Application $app) {
             $voters[] = $app['authority_voter'];
 
             return $voters;
         });
 
-        $this['security.access_manager'] = function ($app) {
+        $this['security.access_manager'] = function($app) {
             return new \Symfony\Component\Security\Core\Authorization\AccessDecisionManager($app['security.voters'], 'unanimous');
         };
 
@@ -859,7 +859,7 @@ 
     {
         $config = $this['config'];
         if (isset($config['trusted_proxies_connection_only']) && !empty($config['trusted_proxies_connection_only'])) {
-            $this->on(KernelEvents::REQUEST, function (GetResponseEvent $event) use ($config) {
+            $this->on(KernelEvents::REQUEST, function(GetResponseEvent $event) use ($config) {
                 // サブリクエストのREMOTE_ADDRも動的に設定を行う必要があるため、KernelEvents::REQUESTを使用する
                 Request::setTrustedProxies(array_merge(array($event->getRequest()->server->get('REMOTE_ADDR')), $config['trusted_proxies']));
             }, self::EARLY_EVENT);
@@ -876,7 +876,7 @@ 
         $this->register(new ServiceProvider\EccubePluginServiceProvider());
 
         // TODO Acme\ServiceProvider の初期化はここで OK?
-        if (array_key_exists('service',$this['config'])) {
+        if (array_key_exists('service', $this['config'])) {
             foreach ($this['config']['service'] as $service) {
                 $this->register(new $service);
             }
@@ -1032,7 +1032,7 @@ 
         $app = $this;
 
         // Response Event(http cache対応、event実行は一番遅く設定)
-        $this->on(\Symfony\Component\HttpKernel\KernelEvents::RESPONSE, function (\Symfony\Component\HttpKernel\Event\FilterResponseEvent $event) use ($app) {
+        $this->on(\Symfony\Component\HttpKernel\KernelEvents::RESPONSE, function(\Symfony\Component\HttpKernel\Event\FilterResponseEvent $event) use ($app) {
 
             if (!$event->isMasterRequest()) {
                 return;

src/Eccube/Form/Type/AddCartType.php

Spacing +1 added lines, -1 removed lines

@@ -112,7 +112,7 @@
                 }
             }
 
-            $builder->addEventListener(FormEvents::PRE_SUBMIT, function (FormEvent $event) use ($Product) {
+            $builder->addEventListener(FormEvents::PRE_SUBMIT, function(FormEvent $event) use ($Product) {
                 $data = $event->getData();
                 $form = $event->getForm();
                 if (!is_null($Product->getClassName2())) {

Unused Use Statements -1 removed lines

@@ -26,7 +26,6 @@
 
 use Eccube\Annotation\FormType;
 use Eccube\Annotation\Inject;
-use Eccube\Application;
 use Symfony\Component\Form\AbstractType;
 use Symfony\Component\Form\Extension\Core\Type\ChoiceType;
 use Symfony\Component\Form\Extension\Core\Type\HiddenType;

src/Eccube/Command/CacheClearCommand.php

Spacing +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
 
         $this->app = $this->getSilexApplication();
         
-        \Eccube\Util\Cache::clear($this->app,$input->getOption('all'));
+        \Eccube\Util\Cache::clear($this->app, $input->getOption('all'));
         $output->writeln(sprintf("%s <info>success</info>", 'cache:clear'));
 
     }

Unused Use Statements -1 removed lines

@@ -24,7 +24,6 @@
 
 namespace Eccube\Command;
 
-use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;

src/Eccube/Log/Monolog/Helper/LogHelper.php

Spacing +1 added lines, -1 removed lines

@@ -111,7 +111,7 @@
         $webProcessor = new WebProcessor();
         $uidProcessor = new UidProcessor(8);
 
-        $FingerCrossedHandler->pushProcessor(function ($record) use ($app, $uidProcessor, $webProcessor) {
+        $FingerCrossedHandler->pushProcessor(function($record) use ($app, $uidProcessor, $webProcessor) {
             // ログフォーマットに出力する値を独自に設定
 
             $record['level_name'] = sprintf("%-5s", $record['level_name']);