Failed Conditions
Push — experimental/3.1 ( 923e5e...2823d3 )
by Yangsin
18s
created
src/Eccube/Controller/Admin/Product/CsvImportController.php 3 patches
Doc Comments   +11 added lines patch added patch discarded remove patch
@@ -590,6 +590,8 @@  discard block
 block discarded – undo
590 590
     /**
591 591
      * 登録、更新時のエラー画面表示
592 592
      *
593
+     * @param Application $app
594
+     * @param string $twig
593 595
      */
594 596
     protected function render($app, $form, $headers, $twig)
595 597
     {
@@ -621,6 +623,7 @@  discard block
 block discarded – undo
621 623
      * アップロードされたCSVファイルの行ごとの処理
622 624
      *
623 625
      * @param $formFile
626
+     * @param Application $app
624 627
      * @return CsvImportService
625 628
      */
626 629
     protected function getImportData($app, $formFile)
@@ -698,6 +701,8 @@  discard block
 block discarded – undo
698 701
 
699 702
     /**
700 703
      * 商品カテゴリの削除、登録
704
+     * @param Application $app
705
+     * @param CsvImportService $data
701 706
      */
702 707
     protected function createProductCategory($row, Product $Product, $app, $data)
703 708
     {
@@ -797,6 +802,8 @@  discard block
 block discarded – undo
797 802
 
798 803
     /**
799 804
      * 商品規格分類1、商品規格分類2がnullとなる商品規格情報を作成
805
+     * @param Application $app
806
+     * @param CsvImportService $data
800 807
      */
801 808
     protected function createProductClass($row, Product $Product, $app, $data, $ClassCategory1 = null, $ClassCategory2 = null)
802 809
     {
@@ -938,6 +945,8 @@  discard block
 block discarded – undo
938 945
 
939 946
     /**
940 947
      * 商品規格情報を更新
948
+     * @param Application $app
949
+     * @param CsvImportService $data
941 950
      */
942 951
     protected function updateProductClass($row, Product $Product, ProductClass $ProductClass, $app, $data)
943 952
     {
@@ -1084,6 +1093,7 @@  discard block
 block discarded – undo
1084 1093
     /**
1085 1094
      * 登録、更新時のエラー画面表示
1086 1095
      *
1096
+     * @param string $message
1087 1097
      */
1088 1098
     protected function addErrors($message)
1089 1099
     {
@@ -1158,6 +1168,7 @@  discard block
 block discarded – undo
1158 1168
      * ProductCategory作成
1159 1169
      * @param \Eccube\Entity\Product $Product
1160 1170
      * @param \Eccube\Entity\Category $Category
1171
+     * @param integer $rank
1161 1172
      * @return ProductCategory
1162 1173
      */
1163 1174
     private function makeProductCategory($Product, $Category, $rank)
Please login to merge, or discard this patch.
Indentation   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -1155,11 +1155,11 @@
 block discarded – undo
1155 1155
     }
1156 1156
     
1157 1157
         /**
1158
-     * ProductCategory作成
1159
-     * @param \Eccube\Entity\Product $Product
1160
-     * @param \Eccube\Entity\Category $Category
1161
-     * @return ProductCategory
1162
-     */
1158
+         * ProductCategory作成
1159
+         * @param \Eccube\Entity\Product $Product
1160
+         * @param \Eccube\Entity\Category $Category
1161
+         * @return ProductCategory
1162
+         */
1163 1163
     private function makeProductCategory($Product, $Category, $rank)
1164 1164
     {
1165 1165
         $ProductCategory = new ProductCategory();
Please login to merge, or discard this patch.
Spacing   +78 added lines, -78 removed lines patch added patch discarded remove patch
@@ -108,7 +108,7 @@  discard block
 block discarded – undo
108 108
                     foreach ($data as $row) {
109 109
 
110 110
                         if ($headerSize != count($row)) {
111
-                            $this->addErrors(($data->key() + 1) . '行目のCSVフォーマットが一致しません。');
111
+                            $this->addErrors(($data->key() + 1).'行目のCSVフォーマットが一致しません。');
112 112
                             return $this->render($app, $form, $headers, $this->productTwig);
113 113
                         }
114 114
 
@@ -119,33 +119,33 @@  discard block
 block discarded – undo
119 119
                             if (preg_match('/^\d+$/', $row['商品ID'])) {
120 120
                                 $Product = $app['eccube.repository.product']->find($row['商品ID']);
121 121
                                 if (!$Product) {
122
-                                    $this->addErrors(($data->key() + 1) . '行目の商品IDが存在しません。');
122
+                                    $this->addErrors(($data->key() + 1).'行目の商品IDが存在しません。');
123 123
                                     return $this->render($app, $form, $headers, $this->productTwig);
124 124
                                 }
125 125
                             } else {
126
-                                $this->addErrors(($data->key() + 1) . '行目の商品IDが存在しません。');
126
+                                $this->addErrors(($data->key() + 1).'行目の商品IDが存在しません。');
127 127
                                 return $this->render($app, $form, $headers, $this->productTwig);
128 128
                             }
129 129
 
130 130
                         }
131 131
 
132 132
                         if ($row['公開ステータス(ID)'] == '') {
133
-                            $this->addErrors(($data->key() + 1) . '行目の公開ステータス(ID)が設定されていません。');
133
+                            $this->addErrors(($data->key() + 1).'行目の公開ステータス(ID)が設定されていません。');
134 134
                         } else {
135 135
                             if (preg_match('/^\d+$/', $row['公開ステータス(ID)'])) {
136 136
                                 $Disp = $app['eccube.repository.master.disp']->find($row['公開ステータス(ID)']);
137 137
                                 if (!$Disp) {
138
-                                    $this->addErrors(($data->key() + 1) . '行目の公開ステータス(ID)が存在しません。');
138
+                                    $this->addErrors(($data->key() + 1).'行目の公開ステータス(ID)が存在しません。');
139 139
                                 } else {
140 140
                                     $Product->setStatus($Disp);
141 141
                                 }
142 142
                             } else {
143
-                                $this->addErrors(($data->key() + 1) . '行目の公開ステータス(ID)が存在しません。');
143
+                                $this->addErrors(($data->key() + 1).'行目の公開ステータス(ID)が存在しません。');
144 144
                             }
145 145
                         }
146 146
 
147 147
                         if (Str::isBlank($row['商品名'])) {
148
-                            $this->addErrors(($data->key() + 1) . '行目の商品名が設定されていません。');
148
+                            $this->addErrors(($data->key() + 1).'行目の商品名が設定されていません。');
149 149
                             return $this->render($app, $form, $headers, $this->productTwig);
150 150
                         } else {
151 151
                             $Product->setName(Str::trimAll($row['商品名']));
@@ -184,10 +184,10 @@  discard block
 block discarded – undo
184 184
                         if ($row['商品削除フラグ'] == '') {
185 185
                             $Product->setDelFlg(Constant::DISABLED);
186 186
                         } else {
187
-                            if ($row['商品削除フラグ'] == (string)Constant::DISABLED || $row['商品削除フラグ'] == (string)Constant::ENABLED) {
187
+                            if ($row['商品削除フラグ'] == (string) Constant::DISABLED || $row['商品削除フラグ'] == (string) Constant::ENABLED) {
188 188
                                 $Product->setDelFlg($row['商品削除フラグ']);
189 189
                             } else {
190
-                                $this->addErrors(($data->key() + 1) . '行目の商品削除フラグが設定されていません。');
190
+                                $this->addErrors(($data->key() + 1).'行目の商品削除フラグが設定されていません。');
191 191
                                 return $this->render($app, $form, $headers, $this->productTwig);
192 192
                             }
193 193
                         }
@@ -214,7 +214,7 @@  discard block
 block discarded – undo
214 214
                                     if (preg_match('/^\d+$/', $deliveryFee) && $deliveryFee >= 0) {
215 215
                                         $ProductClassOrg->setDeliveryFee($deliveryFee);
216 216
                                     } else {
217
-                                        $this->addErrors(($data->key() + 1) . '行目の送料は0以上の数値を設定してください。');
217
+                                        $this->addErrors(($data->key() + 1).'行目の送料は0以上の数値を設定してください。');
218 218
                                     }
219 219
                                 }
220 220
                             }
@@ -222,7 +222,7 @@  discard block
 block discarded – undo
222 222
                             if ($row['規格分類1(ID)'] != '') {
223 223
 
224 224
                                 if ($row['規格分類1(ID)'] == $row['規格分類2(ID)']) {
225
-                                    $this->addErrors(($data->key() + 1) . '行目の規格分類1(ID)と規格分類2(ID)には同じ値を使用できません。');
225
+                                    $this->addErrors(($data->key() + 1).'行目の規格分類1(ID)と規格分類2(ID)には同じ値を使用できません。');
226 226
                                 } else {
227 227
                                     // 商品規格あり
228 228
                                     // 企画分類あり商品を作成
@@ -237,30 +237,30 @@  discard block
 block discarded – undo
237 237
                                     if (preg_match('/^\d+$/', $row['規格分類1(ID)'])) {
238 238
                                         $ClassCategory1 = $app['eccube.repository.class_category']->find($row['規格分類1(ID)']);
239 239
                                         if (!$ClassCategory1) {
240
-                                            $this->addErrors(($data->key() + 1) . '行目の規格分類1(ID)が存在しません。');
240
+                                            $this->addErrors(($data->key() + 1).'行目の規格分類1(ID)が存在しません。');
241 241
                                         } else {
242 242
                                             $ProductClass->setClassCategory1($ClassCategory1);
243 243
                                         }
244 244
                                     } else {
245
-                                        $this->addErrors(($data->key() + 1) . '行目の規格分類1(ID)が存在しません。');
245
+                                        $this->addErrors(($data->key() + 1).'行目の規格分類1(ID)が存在しません。');
246 246
                                     }
247 247
 
248 248
                                     if ($row['規格分類2(ID)'] != '') {
249 249
                                         if (preg_match('/^\d+$/', $row['規格分類2(ID)'])) {
250 250
                                             $ClassCategory2 = $app['eccube.repository.class_category']->find($row['規格分類2(ID)']);
251 251
                                             if (!$ClassCategory2) {
252
-                                                $this->addErrors(($data->key() + 1) . '行目の規格分類2(ID)が存在しません。');
252
+                                                $this->addErrors(($data->key() + 1).'行目の規格分類2(ID)が存在しません。');
253 253
                                             } else {
254 254
                                                 if ($ClassCategory1 &&
255 255
                                                     ($ClassCategory1->getClassName()->getId() == $ClassCategory2->getClassName()->getId())
256 256
                                                 ) {
257
-                                                    $this->addErrors(($data->key() + 1) . '行目の規格分類1(ID)と規格分類2(ID)の規格名が同じです。');
257
+                                                    $this->addErrors(($data->key() + 1).'行目の規格分類1(ID)と規格分類2(ID)の規格名が同じです。');
258 258
                                                 } else {
259 259
                                                     $ProductClass->setClassCategory2($ClassCategory2);
260 260
                                                 }
261 261
                                             }
262 262
                                         } else {
263
-                                            $this->addErrors(($data->key() + 1) . '行目の規格分類2(ID)が存在しません。');
263
+                                            $this->addErrors(($data->key() + 1).'行目の規格分類2(ID)が存在しません。');
264 264
                                         }
265 265
                                     }
266 266
                                     $ProductClass->setProductStock($ProductStock);
@@ -272,7 +272,7 @@  discard block
 block discarded – undo
272 272
 
273 273
                             } else {
274 274
                                 if ($row['規格分類2(ID)'] != '') {
275
-                                    $this->addErrors(($data->key() + 1) . '行目の規格分類1(ID)が存在しません。');
275
+                                    $this->addErrors(($data->key() + 1).'行目の規格分類1(ID)が存在しません。');
276 276
                                 }
277 277
                             }
278 278
 
@@ -300,7 +300,7 @@  discard block
 block discarded – undo
300 300
                                             if (preg_match('/^\d+$/', $deliveryFee) && $deliveryFee >= 0) {
301 301
                                                 $pc->setDeliveryFee($deliveryFee);
302 302
                                             } else {
303
-                                                $this->addErrors(($data->key() + 1) . '行目の送料は0以上の数値を設定してください。');
303
+                                                $this->addErrors(($data->key() + 1).'行目の送料は0以上の数値を設定してください。');
304 304
                                             }
305 305
                                         }
306 306
                                     }
@@ -322,7 +322,7 @@  discard block
 block discarded – undo
322 322
                                 }
323 323
 
324 324
                                 if ($row['規格分類1(ID)'] == $row['規格分類2(ID)']) {
325
-                                    $this->addErrors(($data->key() + 1) . '行目の規格分類1(ID)と規格分類2(ID)には同じ値を使用できません。');
325
+                                    $this->addErrors(($data->key() + 1).'行目の規格分類1(ID)と規格分類2(ID)には同じ値を使用できません。');
326 326
                                 } else {
327 327
 
328 328
                                     // 必ず規格分類1がセットされている
@@ -331,36 +331,36 @@  discard block
 block discarded – undo
331 331
                                     if (preg_match('/^\d+$/', $classCategoryId1)) {
332 332
                                         $ClassCategory1 = $app['eccube.repository.class_category']->find($classCategoryId1);
333 333
                                         if (!$ClassCategory1) {
334
-                                            $this->addErrors(($data->key() + 1) . '行目の規格分類1(ID)が存在しません。');
334
+                                            $this->addErrors(($data->key() + 1).'行目の規格分類1(ID)が存在しません。');
335 335
                                         }
336 336
                                     } else {
337
-                                        $this->addErrors(($data->key() + 1) . '行目の規格分類1(ID)が存在しません。');
337
+                                        $this->addErrors(($data->key() + 1).'行目の規格分類1(ID)が存在しません。');
338 338
                                     }
339 339
 
340 340
                                     $ClassCategory2 = null;
341 341
                                     if ($row['規格分類2(ID)'] != '') {
342 342
                                         if ($pc->getClassCategory1() != null && $pc->getClassCategory2() == null) {
343
-                                            $this->addErrors(($data->key() + 1) . '行目の規格分類2(ID)は設定できません。');
343
+                                            $this->addErrors(($data->key() + 1).'行目の規格分類2(ID)は設定できません。');
344 344
                                         } else {
345 345
                                             if (preg_match('/^\d+$/', $classCategoryId2)) {
346 346
                                                 $ClassCategory2 = $app['eccube.repository.class_category']->find($classCategoryId2);
347 347
                                                 if (!$ClassCategory2) {
348
-                                                    $this->addErrors(($data->key() + 1) . '行目の規格分類2(ID)が存在しません。');
348
+                                                    $this->addErrors(($data->key() + 1).'行目の規格分類2(ID)が存在しません。');
349 349
                                                 } else {
350 350
                                                     if ($ClassCategory1 &&
351 351
                                                         ($ClassCategory1->getClassName()->getId() == $ClassCategory2->getClassName()->getId())
352 352
                                                     ) {
353
-                                                        $this->addErrors(($data->key() + 1) . '行目の規格分類1(ID)と規格分類2(ID)の規格名が同じです。');
353
+                                                        $this->addErrors(($data->key() + 1).'行目の規格分類1(ID)と規格分類2(ID)の規格名が同じです。');
354 354
                                                     }
355 355
                                                 }
356 356
                                             } else {
357
-                                                $this->addErrors(($data->key() + 1) . '行目の規格分類2(ID)が存在しません。');
357
+                                                $this->addErrors(($data->key() + 1).'行目の規格分類2(ID)が存在しません。');
358 358
                                             }
359 359
 
360 360
                                         }
361 361
                                     } else {
362 362
                                         if ($pc->getClassCategory1() != null && $pc->getClassCategory2() != null) {
363
-                                            $this->addErrors(($data->key() + 1) . '行目の規格分類2(ID)に値を設定してください。');
363
+                                            $this->addErrors(($data->key() + 1).'行目の規格分類2(ID)に値を設定してください。');
364 364
                                         }
365 365
                                     }
366 366
                                     $ProductClass = $this->createProductClass($row, $Product, $app, $data, $ClassCategory1, $ClassCategory2);
@@ -371,7 +371,7 @@  discard block
 block discarded – undo
371 371
                                             if (preg_match('/^\d+$/', $deliveryFee) && $deliveryFee >= 0) {
372 372
                                                 $ProductClass->setDeliveryFee($deliveryFee);
373 373
                                             } else {
374
-                                                $this->addErrors(($data->key() + 1) . '行目の送料は0以上の数値を設定してください。');
374
+                                                $this->addErrors(($data->key() + 1).'行目の送料は0以上の数値を設定してください。');
375 375
                                             }
376 376
                                         }
377 377
                                     }
@@ -458,7 +458,7 @@  discard block
 block discarded – undo
458 458
                     foreach ($data as $row) {
459 459
 
460 460
                         if ($headerSize != count($row)) {
461
-                            $this->addErrors(($data->key() + 1) . '行目のCSVフォーマットが一致しません。');
461
+                            $this->addErrors(($data->key() + 1).'行目のCSVフォーマットが一致しません。');
462 462
                             return $this->render($app, $form, $headers, $this->categoryTwig);
463 463
                         }
464 464
 
@@ -466,23 +466,23 @@  discard block
 block discarded – undo
466 466
                             $Category = new Category();
467 467
                         } else {
468 468
                             if (!preg_match('/^\d+$/', $row['カテゴリID'])) {
469
-                                $this->addErrors(($data->key() + 1) . '行目のカテゴリIDが存在しません。');
469
+                                $this->addErrors(($data->key() + 1).'行目のカテゴリIDが存在しません。');
470 470
                                 return $this->render($app, $form, $headers, $this->categoryTwig);
471 471
                             }
472 472
                             $Category = $app['eccube.repository.category']->find($row['カテゴリID']);
473 473
                             if (!$Category) {
474
-                                $this->addErrors(($data->key() + 1) . '行目のカテゴリIDが存在しません。');
474
+                                $this->addErrors(($data->key() + 1).'行目のカテゴリIDが存在しません。');
475 475
                                 return $this->render($app, $form, $headers, $this->categoryTwig);
476 476
                             }
477 477
                             if ($row['カテゴリID'] == $row['親カテゴリID']) {
478
-                                $this->addErrors(($data->key() + 1) . '行目のカテゴリIDと親カテゴリIDが同じです。');
478
+                                $this->addErrors(($data->key() + 1).'行目のカテゴリIDと親カテゴリIDが同じです。');
479 479
                                 return $this->render($app, $form, $headers, $this->categoryTwig);
480 480
                             }
481 481
 
482 482
                         }
483 483
 
484 484
                         if (Str::isBlank($row['カテゴリ名'])) {
485
-                            $this->addErrors(($data->key() + 1) . '行目のカテゴリ名が設定されていません。');
485
+                            $this->addErrors(($data->key() + 1).'行目のカテゴリ名が設定されていません。');
486 486
                             return $this->render($app, $form, $headers, $this->categoryTwig);
487 487
                         } else {
488 488
                             $Category->setName(Str::trimAll($row['カテゴリ名']));
@@ -491,13 +491,13 @@  discard block
 block discarded – undo
491 491
                         if ($row['親カテゴリID'] != '') {
492 492
 
493 493
                             if (!preg_match('/^\d+$/', $row['親カテゴリID'])) {
494
-                                $this->addErrors(($data->key() + 1) . '行目の親カテゴリIDが存在しません。');
494
+                                $this->addErrors(($data->key() + 1).'行目の親カテゴリIDが存在しません。');
495 495
                                 return $this->render($app, $form, $headers, $this->categoryTwig);
496 496
                             }
497 497
 
498 498
                             $ParentCategory = $app['eccube.repository.category']->find($row['親カテゴリID']);
499 499
                             if (!$ParentCategory) {
500
-                                $this->addErrors(($data->key() + 1) . '行目の親カテゴリIDが存在しません。');
500
+                                $this->addErrors(($data->key() + 1).'行目の親カテゴリIDが存在しません。');
501 501
                                 return $this->render($app, $form, $headers, $this->categoryTwig);
502 502
                             }
503 503
 
@@ -513,14 +513,14 @@  discard block
 block discarded – undo
513 513
                         }
514 514
 
515 515
                         if ($app['config']['category_nest_level'] < $Category->getLevel()) {
516
-                            $this->addErrors(($data->key() + 1) . '行目のカテゴリが最大レベルを超えているため設定できません。');
516
+                            $this->addErrors(($data->key() + 1).'行目のカテゴリが最大レベルを超えているため設定できません。');
517 517
                             return $this->render($app, $form, $headers, $this->categoryTwig);
518 518
                         }
519 519
 
520 520
                         $status = $app['eccube.repository.category']->save($Category);
521 521
 
522 522
                         if (!$status) {
523
-                            $this->addErrors(($data->key() + 1) . '行目のカテゴリが設定できません。');
523
+                            $this->addErrors(($data->key() + 1).'行目のカテゴリが設定できません。');
524 524
                         }
525 525
 
526 526
                         if ($this->hasErrors()) {
@@ -565,7 +565,7 @@  discard block
 block discarded – undo
565 565
             throw new NotFoundHttpException();
566 566
         }
567 567
 
568
-        $response->setCallback(function () use ($app, $request, $headers) {
568
+        $response->setCallback(function() use ($app, $request, $headers) {
569 569
 
570 570
             // ヘッダ行の出力
571 571
             $row = array();
@@ -580,7 +580,7 @@  discard block
 block discarded – undo
580 580
         });
581 581
 
582 582
         $response->headers->set('Content-Type', 'application/octet-stream');
583
-        $response->headers->set('Content-Disposition', 'attachment; filename=' . $filename);
583
+        $response->headers->set('Content-Disposition', 'attachment; filename='.$filename);
584 584
         $response->send();
585 585
 
586 586
         return $response;
@@ -603,7 +603,7 @@  discard block
 block discarded – undo
603 603
         if (!empty($this->fileName)) {
604 604
             try {
605 605
                 $fs = new Filesystem();
606
-                $fs->remove($app['config']['csv_temp_realdir'] . '/' . $this->fileName);
606
+                $fs->remove($app['config']['csv_temp_realdir'].'/'.$this->fileName);
607 607
             } catch (\Exception $e) {
608 608
                 // エラーが発生しても無視する
609 609
             }
@@ -626,10 +626,10 @@  discard block
 block discarded – undo
626 626
     protected function getImportData($app, $formFile)
627 627
     {
628 628
         // アップロードされたCSVファイルを一時ディレクトリに保存
629
-        $this->fileName = 'upload_' . Str::random() . '.' . $formFile->getClientOriginalExtension();
629
+        $this->fileName = 'upload_'.Str::random().'.'.$formFile->getClientOriginalExtension();
630 630
         $formFile->move($app['config']['csv_temp_realdir'], $this->fileName);
631 631
 
632
-        $file = file_get_contents($app['config']['csv_temp_realdir'] . '/' . $this->fileName);
632
+        $file = file_get_contents($app['config']['csv_temp_realdir'].'/'.$this->fileName);
633 633
 
634 634
         if ('\\' === DIRECTORY_SEPARATOR && PHP_VERSION_ID >= 70000) {
635 635
             // Windows 環境の PHP7 の場合はファイルエンコーディングを CP932 に合わせる
@@ -725,8 +725,8 @@  discard block
 block discarded – undo
725 725
                 if (!$Category) {
726 726
                     $this->addErrors(($data->key() + 1).'行目の商品カテゴリ(ID)「'.$category.'」が存在しません。');
727 727
                 } else {
728
-                    foreach($Category->getPath() as $ParentCategory){
729
-                        if (!isset($categoriesIdList[$ParentCategory->getId()])){
728
+                    foreach ($Category->getPath() as $ParentCategory) {
729
+                        if (!isset($categoriesIdList[$ParentCategory->getId()])) {
730 730
                             $ProductCategory = $this->makeProductCategory($Product, $ParentCategory, $rank);
731 731
                             $app['orm.em']->persist($ProductCategory);
732 732
                             $rank++;
@@ -734,7 +734,7 @@  discard block
 block discarded – undo
734 734
                             $categoriesIdList[$ParentCategory->getId()] = true;
735 735
                         }
736 736
                     }
737
-                    if (!isset($categoriesIdList[$Category->getId()])){
737
+                    if (!isset($categoriesIdList[$Category->getId()])) {
738 738
                         $ProductCategory = $this->makeProductCategory($Product, $Category, $rank);
739 739
                         $rank++;
740 740
                         $this->em->persist($ProductCategory);
@@ -789,7 +789,7 @@  discard block
 block discarded – undo
789 789
                 }
790 790
             }
791 791
             if (!$Tag) {
792
-                $this->addErrors(($data->key() + 1) . '行目のタグ(ID)「' . $tag_id . '」が存在しません。');
792
+                $this->addErrors(($data->key() + 1).'行目のタグ(ID)「'.$tag_id.'」が存在しません。');
793 793
             }
794 794
         }
795 795
     }
@@ -807,17 +807,17 @@  discard block
 block discarded – undo
807 807
 
808 808
 
809 809
         if ($row['商品種別(ID)'] == '') {
810
-            $this->addErrors(($data->key() + 1) . '行目の商品種別(ID)が設定されていません。');
810
+            $this->addErrors(($data->key() + 1).'行目の商品種別(ID)が設定されていません。');
811 811
         } else {
812 812
             if (preg_match('/^\d+$/', $row['商品種別(ID)'])) {
813 813
                 $ProductType = $app['eccube.repository.master.product_type']->find($row['商品種別(ID)']);
814 814
                 if (!$ProductType) {
815
-                    $this->addErrors(($data->key() + 1) . '行目の商品種別(ID)が存在しません。');
815
+                    $this->addErrors(($data->key() + 1).'行目の商品種別(ID)が存在しません。');
816 816
                 } else {
817 817
                     $ProductClass->setProductType($ProductType);
818 818
                 }
819 819
             } else {
820
-                $this->addErrors(($data->key() + 1) . '行目の商品種別(ID)が存在しません。');
820
+                $this->addErrors(($data->key() + 1).'行目の商品種別(ID)が存在しません。');
821 821
             }
822 822
         }
823 823
 
@@ -828,12 +828,12 @@  discard block
 block discarded – undo
828 828
             if (preg_match('/^\d+$/', $row['発送日目安(ID)'])) {
829 829
                 $DeliveryDate = $app['eccube.repository.delivery_date']->find($row['発送日目安(ID)']);
830 830
                 if (!$DeliveryDate) {
831
-                    $this->addErrors(($data->key() + 1) . '行目の発送日目安(ID)が存在しません。');
831
+                    $this->addErrors(($data->key() + 1).'行目の発送日目安(ID)が存在しません。');
832 832
                 } else {
833 833
                     $ProductClass->setDeliveryDate($DeliveryDate);
834 834
                 }
835 835
             } else {
836
-                $this->addErrors(($data->key() + 1) . '行目の発送日目安(ID)が存在しません。');
836
+                $this->addErrors(($data->key() + 1).'行目の発送日目安(ID)が存在しません。');
837 837
             }
838 838
         }
839 839
 
@@ -844,19 +844,19 @@  discard block
 block discarded – undo
844 844
         }
845 845
 
846 846
         if ($row['在庫数無制限フラグ'] == '') {
847
-            $this->addErrors(($data->key() + 1) . '行目の在庫数無制限フラグが設定されていません。');
847
+            $this->addErrors(($data->key() + 1).'行目の在庫数無制限フラグが設定されていません。');
848 848
         } else {
849 849
             if ($row['在庫数無制限フラグ'] == (string) Constant::DISABLED) {
850 850
                 $ProductClass->setStockUnlimited(Constant::DISABLED);
851 851
                 // 在庫数が設定されていなければエラー
852 852
                 if ($row['在庫数'] == '') {
853
-                    $this->addErrors(($data->key() + 1) . '行目の在庫数が設定されていません。');
853
+                    $this->addErrors(($data->key() + 1).'行目の在庫数が設定されていません。');
854 854
                 } else {
855 855
                     $stock = str_replace(',', '', $row['在庫数']);
856 856
                     if (preg_match('/^\d+$/', $stock) && $stock >= 0) {
857 857
                         $ProductClass->setStock($stock);
858 858
                     } else {
859
-                        $this->addErrors(($data->key() + 1) . '行目の在庫数は0以上の数値を設定してください。');
859
+                        $this->addErrors(($data->key() + 1).'行目の在庫数は0以上の数値を設定してください。');
860 860
                     }
861 861
                 }
862 862
 
@@ -864,7 +864,7 @@  discard block
 block discarded – undo
864 864
                 $ProductClass->setStockUnlimited(Constant::ENABLED);
865 865
                 $ProductClass->setStock(null);
866 866
             } else {
867
-                $this->addErrors(($data->key() + 1) . '行目の在庫数無制限フラグが設定されていません。');
867
+                $this->addErrors(($data->key() + 1).'行目の在庫数無制限フラグが設定されていません。');
868 868
             }
869 869
         }
870 870
 
@@ -873,7 +873,7 @@  discard block
 block discarded – undo
873 873
             if (preg_match('/^\d+$/', $saleLimit) && $saleLimit >= 0) {
874 874
                 $ProductClass->setSaleLimit($saleLimit);
875 875
             } else {
876
-                $this->addErrors(($data->key() + 1) . '行目の販売制限数は0以上の数値を設定してください。');
876
+                $this->addErrors(($data->key() + 1).'行目の販売制限数は0以上の数値を設定してください。');
877 877
             }
878 878
         }
879 879
 
@@ -882,18 +882,18 @@  discard block
 block discarded – undo
882 882
             if (preg_match('/^\d+$/', $price01) && $price01 >= 0) {
883 883
                 $ProductClass->setPrice01($price01);
884 884
             } else {
885
-                $this->addErrors(($data->key() + 1) . '行目の通常価格は0以上の数値を設定してください。');
885
+                $this->addErrors(($data->key() + 1).'行目の通常価格は0以上の数値を設定してください。');
886 886
             }
887 887
         }
888 888
 
889 889
         if ($row['販売価格'] == '') {
890
-            $this->addErrors(($data->key() + 1) . '行目の販売価格が設定されていません。');
890
+            $this->addErrors(($data->key() + 1).'行目の販売価格が設定されていません。');
891 891
         } else {
892 892
             $price02 = str_replace(',', '', $row['販売価格']);
893 893
             if (preg_match('/^\d+$/', $price02) && $price02 >= 0) {
894 894
                 $ProductClass->setPrice02($price02);
895 895
             } else {
896
-                $this->addErrors(($data->key() + 1) . '行目の販売価格は0以上の数値を設定してください。');
896
+                $this->addErrors(($data->key() + 1).'行目の販売価格は0以上の数値を設定してください。');
897 897
             }
898 898
         }
899 899
 
@@ -902,7 +902,7 @@  discard block
 block discarded – undo
902 902
             if (preg_match('/^\d+$/', $delivery_fee) && $delivery_fee >= 0) {
903 903
                 $ProductClass->setDeliveryFee($delivery_fee);
904 904
             } else {
905
-                $this->addErrors(($data->key() + 1) . '行目の送料は0以上の数値を設定してください。');
905
+                $this->addErrors(($data->key() + 1).'行目の送料は0以上の数値を設定してください。');
906 906
             }
907 907
         }
908 908
 
@@ -912,7 +912,7 @@  discard block
 block discarded – undo
912 912
             if ($row['商品規格削除フラグ'] == (string) Constant::DISABLED || $row['商品規格削除フラグ'] == (string) Constant::ENABLED) {
913 913
                 $ProductClass->setDelFlg($row['商品規格削除フラグ']);
914 914
             } else {
915
-                $this->addErrors(($data->key() + 1) . '行目の商品規格削除フラグが設定されていません。');
915
+                $this->addErrors(($data->key() + 1).'行目の商品規格削除フラグが設定されていません。');
916 916
             }
917 917
         }
918 918
 
@@ -945,17 +945,17 @@  discard block
 block discarded – undo
945 945
         $ProductClass->setProduct($Product);
946 946
 
947 947
         if ($row['商品種別(ID)'] == '') {
948
-            $this->addErrors(($data->key() + 1) . '行目の商品種別(ID)が設定されていません。');
948
+            $this->addErrors(($data->key() + 1).'行目の商品種別(ID)が設定されていません。');
949 949
         } else {
950 950
             if (preg_match('/^\d+$/', $row['商品種別(ID)'])) {
951 951
                 $ProductType = $app['eccube.repository.master.product_type']->find($row['商品種別(ID)']);
952 952
                 if (!$ProductType) {
953
-                    $this->addErrors(($data->key() + 1) . '行目の商品種別(ID)が存在しません。');
953
+                    $this->addErrors(($data->key() + 1).'行目の商品種別(ID)が存在しません。');
954 954
                 } else {
955 955
                     $ProductClass->setProductType($ProductType);
956 956
                 }
957 957
             } else {
958
-                $this->addErrors(($data->key() + 1) . '行目の商品種別(ID)が存在しません。');
958
+                $this->addErrors(($data->key() + 1).'行目の商品種別(ID)が存在しません。');
959 959
             }
960 960
         }
961 961
 
@@ -964,12 +964,12 @@  discard block
 block discarded – undo
964 964
             if (preg_match('/^\d+$/', $row['規格分類1(ID)'])) {
965 965
                 $ClassCategory = $app['eccube.repository.class_category']->find($row['規格分類1(ID)']);
966 966
                 if (!$ClassCategory) {
967
-                    $this->addErrors(($data->key() + 1) . '行目の規格分類1(ID)が存在しません。');
967
+                    $this->addErrors(($data->key() + 1).'行目の規格分類1(ID)が存在しません。');
968 968
                 } else {
969 969
                     $ProductClass->setClassCategory1($ClassCategory);
970 970
                 }
971 971
             } else {
972
-                $this->addErrors(($data->key() + 1) . '行目の規格分類1(ID)が存在しません。');
972
+                $this->addErrors(($data->key() + 1).'行目の規格分類1(ID)が存在しません。');
973 973
             }
974 974
         }
975 975
 
@@ -977,12 +977,12 @@  discard block
 block discarded – undo
977 977
             if (preg_match('/^\d+$/', $row['規格分類2(ID)'])) {
978 978
                 $ClassCategory = $app['eccube.repository.class_category']->find($row['規格分類2(ID)']);
979 979
                 if (!$ClassCategory) {
980
-                    $this->addErrors(($data->key() + 1) . '行目の規格分類2(ID)が存在しません。');
980
+                    $this->addErrors(($data->key() + 1).'行目の規格分類2(ID)が存在しません。');
981 981
                 } else {
982 982
                     $ProductClass->setClassCategory2($ClassCategory);
983 983
                 }
984 984
             } else {
985
-                $this->addErrors(($data->key() + 1) . '行目の規格分類2(ID)が存在しません。');
985
+                $this->addErrors(($data->key() + 1).'行目の規格分類2(ID)が存在しません。');
986 986
             }
987 987
         }
988 988
 
@@ -990,12 +990,12 @@  discard block
 block discarded – undo
990 990
             if (preg_match('/^\d+$/', $row['発送日目安(ID)'])) {
991 991
                 $DeliveryDate = $app['eccube.repository.delivery_date']->find($row['発送日目安(ID)']);
992 992
                 if (!$DeliveryDate) {
993
-                    $this->addErrors(($data->key() + 1) . '行目の発送日目安(ID)が存在しません。');
993
+                    $this->addErrors(($data->key() + 1).'行目の発送日目安(ID)が存在しません。');
994 994
                 } else {
995 995
                     $ProductClass->setDeliveryDate($DeliveryDate);
996 996
                 }
997 997
             } else {
998
-                $this->addErrors(($data->key() + 1) . '行目の発送日目安(ID)が存在しません。');
998
+                $this->addErrors(($data->key() + 1).'行目の発送日目安(ID)が存在しません。');
999 999
             }
1000 1000
         }
1001 1001
 
@@ -1006,19 +1006,19 @@  discard block
 block discarded – undo
1006 1006
         }
1007 1007
 
1008 1008
         if ($row['在庫数無制限フラグ'] == '') {
1009
-            $this->addErrors(($data->key() + 1) . '行目の在庫数無制限フラグが設定されていません。');
1009
+            $this->addErrors(($data->key() + 1).'行目の在庫数無制限フラグが設定されていません。');
1010 1010
         } else {
1011 1011
             if ($row['在庫数無制限フラグ'] == (string) Constant::DISABLED) {
1012 1012
                 $ProductClass->setStockUnlimited(Constant::DISABLED);
1013 1013
                 // 在庫数が設定されていなければエラー
1014 1014
                 if ($row['在庫数'] == '') {
1015
-                    $this->addErrors(($data->key() + 1) . '行目の在庫数が設定されていません。');
1015
+                    $this->addErrors(($data->key() + 1).'行目の在庫数が設定されていません。');
1016 1016
                 } else {
1017 1017
                     $stock = str_replace(',', '', $row['在庫数']);
1018 1018
                     if (preg_match('/^\d+$/', $stock) && $stock >= 0) {
1019 1019
                         $ProductClass->setStock($row['在庫数']);
1020 1020
                     } else {
1021
-                        $this->addErrors(($data->key() + 1) . '行目の在庫数は0以上の数値を設定してください。');
1021
+                        $this->addErrors(($data->key() + 1).'行目の在庫数は0以上の数値を設定してください。');
1022 1022
                     }
1023 1023
                 }
1024 1024
 
@@ -1026,7 +1026,7 @@  discard block
 block discarded – undo
1026 1026
                 $ProductClass->setStockUnlimited(Constant::ENABLED);
1027 1027
                 $ProductClass->setStock(null);
1028 1028
             } else {
1029
-                $this->addErrors(($data->key() + 1) . '行目の在庫数無制限フラグが設定されていません。');
1029
+                $this->addErrors(($data->key() + 1).'行目の在庫数無制限フラグが設定されていません。');
1030 1030
             }
1031 1031
         }
1032 1032
 
@@ -1035,7 +1035,7 @@  discard block
 block discarded – undo
1035 1035
             if (preg_match('/^\d+$/', $saleLimit) && $saleLimit >= 0) {
1036 1036
                 $ProductClass->setSaleLimit($saleLimit);
1037 1037
             } else {
1038
-                $this->addErrors(($data->key() + 1) . '行目の販売制限数は0以上の数値を設定してください。');
1038
+                $this->addErrors(($data->key() + 1).'行目の販売制限数は0以上の数値を設定してください。');
1039 1039
             }
1040 1040
         }
1041 1041
 
@@ -1044,18 +1044,18 @@  discard block
 block discarded – undo
1044 1044
             if (preg_match('/^\d+$/', $price01) && $price01 >= 0) {
1045 1045
                 $ProductClass->setPrice01($price01);
1046 1046
             } else {
1047
-                $this->addErrors(($data->key() + 1) . '行目の通常価格は0以上の数値を設定してください。');
1047
+                $this->addErrors(($data->key() + 1).'行目の通常価格は0以上の数値を設定してください。');
1048 1048
             }
1049 1049
         }
1050 1050
 
1051 1051
         if ($row['販売価格'] == '') {
1052
-            $this->addErrors(($data->key() + 1) . '行目の販売価格が設定されていません。');
1052
+            $this->addErrors(($data->key() + 1).'行目の販売価格が設定されていません。');
1053 1053
         } else {
1054 1054
             $price02 = str_replace(',', '', $row['販売価格']);
1055 1055
             if (preg_match('/^\d+$/', $price02) && $price02 >= 0) {
1056 1056
                 $ProductClass->setPrice02($price02);
1057 1057
             } else {
1058
-                $this->addErrors(($data->key() + 1) . '行目の販売価格は0以上の数値を設定してください。');
1058
+                $this->addErrors(($data->key() + 1).'行目の販売価格は0以上の数値を設定してください。');
1059 1059
             }
1060 1060
         }
1061 1061
 
@@ -1065,7 +1065,7 @@  discard block
 block discarded – undo
1065 1065
             if ($row['商品規格削除フラグ'] == (string) Constant::DISABLED || $row['商品規格削除フラグ'] == (string) Constant::ENABLED) {
1066 1066
                 $ProductClass->setDelFlg($row['商品規格削除フラグ']);
1067 1067
             } else {
1068
-                $this->addErrors(($data->key() + 1) . '行目の商品規格削除フラグが設定されていません。');
1068
+                $this->addErrors(($data->key() + 1).'行目の商品規格削除フラグが設定されていません。');
1069 1069
             }
1070 1070
         }
1071 1071
 
Please login to merge, or discard this patch.
src/Eccube/Controller/Admin/Product/ProductController.php 2 patches
Doc Comments   +1 added lines patch added patch discarded remove patch
@@ -792,6 +792,7 @@
 block discarded – undo
792 792
      * ProductCategory作成
793 793
      * @param \Eccube\Entity\Product $Product
794 794
      * @param \Eccube\Entity\Category $Category
795
+     * @param integer $count
795 796
      * @return \Eccube\Entity\ProductCategory
796 797
      */
797 798
     private function createProductCategory($Product, $Category, $count)
Please login to merge, or discard this patch.
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -203,7 +203,7 @@  discard block
 block discarded – undo
203 203
                     }
204 204
 
205 205
                     $extension = $image->getClientOriginalExtension();
206
-                    $filename = date('mdHis') . uniqid('_') . '.' . $extension;
206
+                    $filename = date('mdHis').uniqid('_').'.'.$extension;
207 207
                     $image->move($app['config']['image_temp_realdir'], $filename);
208 208
                     $files[] = $filename;
209 209
                 }
@@ -279,7 +279,7 @@  discard block
 block discarded – undo
279 279
         $form = $builder->getForm();
280 280
 
281 281
         if (!$has_class) {
282
-            $ProductClass->setStockUnlimited((boolean)$ProductClass->getStockUnlimited());
282
+            $ProductClass->setStockUnlimited((boolean) $ProductClass->getStockUnlimited());
283 283
             $form['class']->setData($ProductClass);
284 284
         }
285 285
 
@@ -365,8 +365,8 @@  discard block
 block discarded – undo
365 365
                 $Categories = $form->get('Category')->getData();
366 366
                 $categoriesIdList = array();
367 367
                 foreach ($Categories as $Category) {
368
-                    foreach($Category->getPath() as $ParentCategory){
369
-                        if (!isset($categoriesIdList[$ParentCategory->getId()])){
368
+                    foreach ($Category->getPath() as $ParentCategory) {
369
+                        if (!isset($categoriesIdList[$ParentCategory->getId()])) {
370 370
                             $ProductCategory = $this->createProductCategory($Product, $ParentCategory, $count);
371 371
                             $app['orm.em']->persist($ProductCategory);
372 372
                             $count++;
@@ -375,7 +375,7 @@  discard block
 block discarded – undo
375 375
                             $categoriesIdList[$ParentCategory->getId()] = true;
376 376
                         }
377 377
                     }
378
-                    if (!isset($categoriesIdList[$Category->getId()])){
378
+                    if (!isset($categoriesIdList[$Category->getId()])) {
379 379
                         $ProductCategory = $this->createProductCategory($Product, $Category, $count);
380 380
                         $app['orm.em']->persist($ProductCategory);
381 381
                         $count++;
@@ -397,7 +397,7 @@  discard block
 block discarded – undo
397 397
                     $app['orm.em']->persist($ProductImage);
398 398
 
399 399
                     // 移動
400
-                    $file = new File($app['config']['image_temp_realdir'] . '/' . $add_image);
400
+                    $file = new File($app['config']['image_temp_realdir'].'/'.$add_image);
401 401
                     $file->move($app['config']['image_save_realdir']);
402 402
                 }
403 403
 
@@ -417,7 +417,7 @@  discard block
 block discarded – undo
417 417
 
418 418
                     // 削除
419 419
                     $fs = new Filesystem();
420
-                    $fs->remove($app['config']['image_save_realdir'] . '/' . $delete_image);
420
+                    $fs->remove($app['config']['image_save_realdir'].'/'.$delete_image);
421 421
                 }
422 422
                 $app['orm.em']->persist($Product);
423 423
                 $app['orm.em']->flush();
@@ -577,7 +577,7 @@  discard block
 block discarded – undo
577 577
                 foreach ($deleteImages as $deleteImage) {
578 578
                     try {
579 579
                         $fs = new Filesystem();
580
-                        $fs->remove($app['config']['image_save_realdir'] . '/' . $deleteImage);
580
+                        $fs->remove($app['config']['image_save_realdir'].'/'.$deleteImage);
581 581
                     } catch (\Exception $e) {
582 582
                         // エラーが発生しても無視する
583 583
                     }
@@ -647,10 +647,10 @@  discard block
 block discarded – undo
647 647
 
648 648
                     // 画像ファイルを新規作成
649 649
                     $extension = pathinfo($Image->getFileName(), PATHINFO_EXTENSION);
650
-                    $filename = date('mdHis') . uniqid('_') . '.' . $extension;
650
+                    $filename = date('mdHis').uniqid('_').'.'.$extension;
651 651
                     try {
652 652
                         $fs = new Filesystem();
653
-                        $fs->copy($app['config']['image_save_realdir'] . '/' . $Image->getFileName(), $app['config']['image_save_realdir'] . '/' . $filename);
653
+                        $fs->copy($app['config']['image_save_realdir'].'/'.$Image->getFileName(), $app['config']['image_save_realdir'].'/'.$filename);
654 654
                     } catch (\Exception $e) {
655 655
                         // エラーが発生しても無視する
656 656
                     }
@@ -725,7 +725,7 @@  discard block
 block discarded – undo
725 725
         $em->getConfiguration()->setSQLLogger(null);
726 726
 
727 727
         $response = new StreamedResponse();
728
-        $response->setCallback(function () use ($app, $request) {
728
+        $response->setCallback(function() use ($app, $request) {
729 729
 
730 730
             // CSV種別を元に初期化.
731 731
             $app['eccube.service.csv.export']->initCsvType(CsvType::CSV_TYPE_PRODUCT);
@@ -765,7 +765,7 @@  discard block
 block discarded – undo
765 765
             // データ行の出力.
766 766
             $app['eccube.service.csv.export']->setExportQueryBuilder($qb);
767 767
 
768
-            $app['eccube.service.csv.export']->exportData(function ($entity, CsvExportService $csvService) use ($app, $request) {
768
+            $app['eccube.service.csv.export']->exportData(function($entity, CsvExportService $csvService) use ($app, $request) {
769 769
                 $Csvs = $csvService->getCsvs();
770 770
 
771 771
                 /** @var $Product \Eccube\Entity\Product */
@@ -808,9 +808,9 @@  discard block
 block discarded – undo
808 808
         });
809 809
 
810 810
         $now = new \DateTime();
811
-        $filename = 'product_' . $now->format('YmdHis') . '.csv';
811
+        $filename = 'product_'.$now->format('YmdHis').'.csv';
812 812
         $response->headers->set('Content-Type', 'application/octet-stream');
813
-        $response->headers->set('Content-Disposition', 'attachment; filename=' . $filename);
813
+        $response->headers->set('Content-Disposition', 'attachment; filename='.$filename);
814 814
         $response->send();
815 815
 
816 816
         log_info('商品CSV出力ファイル名', array($filename));
Please login to merge, or discard this patch.
src/Eccube/Service/Calculator/Strategy/ShippingStrategy.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -21,12 +21,12 @@
 block discarded – undo
21 21
         // 送料をすべて足す
22 22
         $delivery_fee_total = array_reduce(
23 23
             array_map(
24
-                function ($Shipping) {
24
+                function($Shipping) {
25 25
                     return $Shipping->getShippingDeliveryFee();
26 26
                 },
27 27
                 $this->Order->getShippings()->toArray()
28 28
             ),
29
-            function ($carry, $item) {
29
+            function($carry, $item) {
30 30
                 return $carry += $item;
31 31
             }
32 32
         );
Please login to merge, or discard this patch.
src/Eccube/Service/Calculator/CalculateContext.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -18,7 +18,7 @@
 block discarded – undo
18 18
             $Strategy->execute($this->OrderDetails);
19 19
         }
20 20
 
21
-        foreach($this->OrderDetails as $OrderDetail) {
21
+        foreach ($this->OrderDetails as $OrderDetail) {
22 22
             if (!$this->Order->getOrderDetails()->contains($OrderDetail)) {
23 23
                 $OrderDetail->setOrder($this->Order);
24 24
                 $this->Order->addOrderDetail($OrderDetail);
Please login to merge, or discard this patch.
src/Eccube/Form/Type/Shopping/ShippingType.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -54,7 +54,7 @@  discard block
 block discarded – undo
54 54
         // 配送業者のプルダウンを生成
55 55
         $builder->addEventListener(
56 56
             FormEvents::PRE_SET_DATA,
57
-            function (FormEvent $event) {
57
+            function(FormEvent $event) {
58 58
                 $Shipping = $event->getData();
59 59
                 if (is_null($Shipping) || !$Shipping->getId()) {
60 60
                     return;
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
         // お届け日のプルダウンを生成
96 96
         $builder->addEventListener(
97 97
             FormEvents::PRE_SET_DATA,
98
-            function (FormEvent $event) {
98
+            function(FormEvent $event) {
99 99
                 $Shipping = $event->getData();
100 100
                 if (is_null($Shipping) || !$Shipping->getId()) {
101 101
                     return;
@@ -131,7 +131,7 @@  discard block
 block discarded – undo
131 131
 
132 132
                 // 配送日数が設定されている
133 133
                 if ($deliveryDateFlag) {
134
-                    $period = new \DatePeriod (
134
+                    $period = new \DatePeriod(
135 135
                         new \DateTime($minDate.' day'),
136 136
                         new \DateInterval('P1D'),
137 137
                         new \DateTime($minDate + $this->config['deliv_date_end_max'].' day')
@@ -159,7 +159,7 @@  discard block
 block discarded – undo
159 159
         // お届け時間のプルダウンを生成
160 160
         $builder->addEventListener(
161 161
             FormEvents::PRE_SET_DATA,
162
-            function (FormEvent $event) {
162
+            function(FormEvent $event) {
163 163
                 $Shipping = $event->getData();
164 164
                 if (is_null($Shipping) || !$Shipping->getId()) {
165 165
                     return;
Please login to merge, or discard this patch.
src/Eccube/InstallApplication.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -39,7 +39,7 @@
 block discarded – undo
39 39
 
40 40
         if (is_writable($logDir)) {
41 41
             if (file_exists($installLog) && !is_writable($installLog)) {
42
-                die($installLog . ' の書込権限を変更して下さい。');
42
+                die($installLog.' の書込権限を変更して下さい。');
43 43
             }
44 44
             // install step2 でログディレクトリに書き込み権限が付与されればログ出力を開始する.
45 45
             $app->register(new \Silex\Provider\MonologServiceProvider(), array(
Please login to merge, or discard this patch.
src/Eccube/Service/MailService.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -229,7 +229,7 @@
 block discarded – undo
229 229
      * Send order mail.
230 230
      *
231 231
      * @param \Eccube\Entity\Order $Order 受注情報
232
-     * @return string
232
+     * @return \Swift_Mime_MimePart
233 233
      */
234 234
     public function sendOrderMail(\Eccube\Entity\Order $Order)
235 235
     {
Please login to merge, or discard this patch.
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
61 61
         ));
62 62
 
63 63
         $message = \Swift_Message::newInstance()
64
-            ->setSubject('[' . $this->BaseInfo->getShopName() . '] 会員登録のご確認')
64
+            ->setSubject('['.$this->BaseInfo->getShopName().'] 会員登録のご確認')
65 65
             ->setFrom(array($this->BaseInfo->getEmail01() => $this->BaseInfo->getShopName()))
66 66
             ->setTo(array($Customer->getEmail()))
67 67
             ->setBcc($this->BaseInfo->getEmail01())
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
102 102
         ));
103 103
 
104 104
         $message = \Swift_Message::newInstance()
105
-            ->setSubject('[' . $this->BaseInfo->getShopName() . '] 会員登録が完了しました。')
105
+            ->setSubject('['.$this->BaseInfo->getShopName().'] 会員登録が完了しました。')
106 106
             ->setFrom(array($this->BaseInfo->getEmail01() => $this->BaseInfo->getShopName()))
107 107
             ->setTo(array($Customer->getEmail()))
108 108
             ->setBcc($this->BaseInfo->getEmail01())
@@ -144,7 +144,7 @@  discard block
 block discarded – undo
144 144
         ));
145 145
 
146 146
         $message = \Swift_Message::newInstance()
147
-            ->setSubject('[' . $this->BaseInfo->getShopName() . '] 退会手続きのご完了')
147
+            ->setSubject('['.$this->BaseInfo->getShopName().'] 退会手続きのご完了')
148 148
             ->setFrom(array($this->BaseInfo->getEmail01() => $this->BaseInfo->getShopName()))
149 149
             ->setTo(array($email))
150 150
             ->setBcc($this->BaseInfo->getEmail01())
@@ -187,7 +187,7 @@  discard block
 block discarded – undo
187 187
 
188 188
         // 問い合わせ者にメール送信
189 189
         $message = \Swift_Message::newInstance()
190
-            ->setSubject('[' . $this->BaseInfo->getShopName() . '] お問い合わせを受け付けました。')
190
+            ->setSubject('['.$this->BaseInfo->getShopName().'] お問い合わせを受け付けました。')
191 191
             ->setFrom(array($this->BaseInfo->getEmail02() => $this->BaseInfo->getShopName()))
192 192
             ->setTo(array($formData['email']))
193 193
             ->setBcc($this->BaseInfo->getEmail02())
@@ -244,7 +244,7 @@  discard block
 block discarded – undo
244 244
         ));
245 245
 
246 246
         $message = \Swift_Message::newInstance()
247
-            ->setSubject('[' . $this->BaseInfo->getShopName() . '] ' . $MailTemplate->getSubject())
247
+            ->setSubject('['.$this->BaseInfo->getShopName().'] '.$MailTemplate->getSubject())
248 248
             ->setFrom(array($this->BaseInfo->getEmail01() => $this->BaseInfo->getShopName()))
249 249
             ->setTo(array($Order->getEmail()))
250 250
             ->setBcc($this->BaseInfo->getEmail01())
@@ -288,7 +288,7 @@  discard block
 block discarded – undo
288 288
         ));
289 289
 
290 290
         $message = \Swift_Message::newInstance()
291
-            ->setSubject('[' . $this->BaseInfo->getShopName() . '] 会員登録のご確認')
291
+            ->setSubject('['.$this->BaseInfo->getShopName().'] 会員登録のご確認')
292 292
             ->setFrom(array($this->BaseInfo->getEmail03() => $this->BaseInfo->getShopName()))
293 293
             ->setTo(array($Customer->getEmail()))
294 294
             ->setBcc($this->BaseInfo->getEmail01())
@@ -332,7 +332,7 @@  discard block
 block discarded – undo
332 332
         ));
333 333
 
334 334
         $message = \Swift_Message::newInstance()
335
-            ->setSubject('[' . $this->BaseInfo->getShopName() . '] ' . $formData['subject'])
335
+            ->setSubject('['.$this->BaseInfo->getShopName().'] '.$formData['subject'])
336 336
             ->setFrom(array($this->BaseInfo->getEmail01() => $this->BaseInfo->getShopName()))
337 337
             ->setTo(array($Order->getEmail()))
338 338
             ->setBcc($this->BaseInfo->getEmail01())
@@ -373,7 +373,7 @@  discard block
 block discarded – undo
373 373
         ));
374 374
 
375 375
         $message = \Swift_Message::newInstance()
376
-            ->setSubject('[' . $this->BaseInfo->getShopName() . '] パスワード変更のご確認')
376
+            ->setSubject('['.$this->BaseInfo->getShopName().'] パスワード変更のご確認')
377 377
             ->setFrom(array($this->BaseInfo->getEmail01() => $this->BaseInfo->getShopName()))
378 378
             ->setTo(array($Customer->getEmail()))
379 379
             ->setBcc($this->BaseInfo->getEmail01())
@@ -414,7 +414,7 @@  discard block
 block discarded – undo
414 414
         ));
415 415
 
416 416
         $message = \Swift_Message::newInstance()
417
-            ->setSubject('[' . $this->BaseInfo->getShopName() . '] パスワード変更のお知らせ')
417
+            ->setSubject('['.$this->BaseInfo->getShopName().'] パスワード変更のお知らせ')
418 418
             ->setFrom(array($this->BaseInfo->getEmail01() => $this->BaseInfo->getShopName()))
419 419
             ->setTo(array($Customer->getEmail()))
420 420
             ->setBcc($this->BaseInfo->getEmail01())
Please login to merge, or discard this patch.
src/Eccube/Controller/ShoppingController.php 3 patches
Doc Comments   -1 removed lines patch added patch discarded remove patch
@@ -1360,7 +1360,6 @@
 block discarded – undo
1360 1360
     /**
1361 1361
      * フォームの情報からお届け先のインデックスを返す
1362 1362
      *
1363
-     * @param Application $app
1364 1363
      * @param mixed $CustomerAddressData
1365 1364
      * @return int
1366 1365
      */
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1336,7 +1336,7 @@
 block discarded – undo
1336 1336
                 // ステータス履歴も保持しておく? 在庫引き当ての仕様もセットで。
1337 1337
                 if ($dispatcher instanceof Response
1338 1338
                     && ($dispatcher->isRedirection() || $dispatcher->getContent())) { // $paymentMethod->apply() が Response を返した場合は画面遷移
1339
-                    return $dispatcher;                // 画面遷移したいパターンが複数ある場合はどうする? 引数で制御?
1339
+                    return $dispatcher; // 画面遷移したいパターンが複数ある場合はどうする? 引数で制御?
1340 1340
                 }
1341 1341
                 $PaymentResult = $paymentService->doCheckout($paymentMethod); // 決済実行
1342 1342
                 if (!$PaymentResult->isSuccess()) {
Please login to merge, or discard this patch.
Unused Use Statements   -2 removed lines patch added patch discarded remove patch
@@ -40,9 +40,7 @@
 block discarded – undo
40 40
 use Eccube\Form\Type\ShippingMultipleType;
41 41
 use Eccube\Form\Type\Shopping\OrderType;
42 42
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Method;
43
-use Sensio\Bundle\FrameworkExtraBundle\Configuration\ParamConverter;
44 43
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
45
-use Sensio\Bundle\FrameworkExtraBundle\Configuration\Security;
46 44
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Template;
47 45
 use Symfony\Component\Form\Extension\Core\Type\CollectionType;
48 46
 use Symfony\Component\HttpFoundation\Request;
Please login to merge, or discard this patch.
src/Eccube/Repository/CustomerRepository.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -153,9 +153,9 @@  discard block
 block discarded – undo
153 153
             $qb
154 154
                 ->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')
155 155
                 ->setParameter('customer_id', $id)
156
-                ->setParameter('name', '%' . $clean_key_multi . '%')
157
-                ->setParameter('kana', '%' . $clean_key_multi . '%')
158
-                ->setParameter('email', '%' . $clean_key_multi . '%');
156
+                ->setParameter('name', '%'.$clean_key_multi.'%')
157
+                ->setParameter('kana', '%'.$clean_key_multi.'%')
158
+                ->setParameter('email', '%'.$clean_key_multi.'%');
159 159
         }
160 160
 
161 161
         // Pref
@@ -205,7 +205,7 @@  discard block
 block discarded – undo
205 205
         if (isset($searchData['tel']) && Str::isNotBlank($searchData['tel'])) {
206 206
             $qb
207 207
                 ->andWhere('CONCAT(c.tel01, c.tel02, c.tel03) LIKE :tel')
208
-                ->setParameter('tel', '%' . $searchData['tel'] . '%');
208
+                ->setParameter('tel', '%'.$searchData['tel'].'%');
209 209
         }
210 210
 
211 211
         // buy_total
@@ -299,7 +299,7 @@  discard block
 block discarded – undo
299 299
                 ->leftJoin('c.Orders', 'o')
300 300
                 ->leftJoin('o.OrderDetails', 'od')
301 301
                 ->andWhere('od.product_name LIKE :buy_product_name OR od.product_code LIKE :buy_product_name')
302
-                ->setParameter('buy_product_name', '%' . $searchData['buy_product_code'] . '%');
302
+                ->setParameter('buy_product_name', '%'.$searchData['buy_product_code'].'%');
303 303
         }
304 304
 
305 305
         // Order By
Please login to merge, or discard this patch.