Completed
Push — 4.0 ( e20433...6e3305 )
by chihiro
36s queued 10s
created
src/Eccube/Repository/ProductRepository.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -226,7 +226,7 @@
 block discarded – undo
226 226
 
227 227
         // id
228 228
         if (isset($searchData['id']) && StringUtil::isNotBlank($searchData['id'])) {
229
-            $id = preg_match('/^\d{0,10}$/', $searchData['id'])  ? $searchData['id'] : null;
229
+            $id = preg_match('/^\d{0,10}$/', $searchData['id']) ? $searchData['id'] : null;
230 230
             if ($id && $id > '2147483647' && $this->isPostgreSQL()) {
231 231
                 $id = null;
232 232
             }
Please login to merge, or discard this patch.
src/Eccube/Controller/Admin/Setting/System/LogController.php 2 patches
Unused Use Statements   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -17,10 +17,10 @@
 block discarded – undo
17 17
 use Eccube\Event\EccubeEvents;
18 18
 use Eccube\Event\EventArgs;
19 19
 use Eccube\Form\Type\Admin\LogType;
20
-use Symfony\Component\Routing\Annotation\Route;
21 20
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Template;
22 21
 use Symfony\Component\HttpFoundation\Request;
23 22
 use Symfony\Component\HttpFoundation\StreamedResponse;
23
+use Symfony\Component\Routing\Annotation\Route;
24 24
 
25 25
 class LogController extends AbstractController
26 26
 {
Please login to merge, or discard this patch.
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -71,13 +71,13 @@
 block discarded – undo
71 71
         if ($form->getClickedButton() && $form->getClickedButton()->getName() === 'download') {
72 72
             $bufferSize = 1024 * 50;
73 73
             $response = new StreamedResponse();
74
-            $response->headers->set('Content-Length',filesize($logFile));
75
-            $response->headers->set('Content-Disposition','attachment; filename=' . basename($logFile));
76
-            $response->headers->set('Content-Type','application/octet-stream');
77
-            $response->setCallback(function() use($logFile,$bufferSize) {
78
-                if ($fh = fopen($logFile,'r')) {
74
+            $response->headers->set('Content-Length', filesize($logFile));
75
+            $response->headers->set('Content-Disposition', 'attachment; filename='.basename($logFile));
76
+            $response->headers->set('Content-Type', 'application/octet-stream');
77
+            $response->setCallback(function () use($logFile, $bufferSize) {
78
+                if ($fh = fopen($logFile, 'r')) {
79 79
                     while (!feof($fh)) {
80
-                        echo fread($fh,$bufferSize);
80
+                        echo fread($fh, $bufferSize);
81 81
                     }
82 82
                 }
83 83
             });
Please login to merge, or discard this patch.
src/Eccube/Form/Type/PostalType.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -56,7 +56,7 @@
 block discarded – undo
56 56
     public function configureOptions(OptionsResolver $resolver)
57 57
     {
58 58
         $eccubeConfig = $this->eccubeConfig;
59
-        $constraints = function(Options $options) use ($eccubeConfig) {
59
+        $constraints = function (Options $options) use ($eccubeConfig) {
60 60
             $constraints = [];
61 61
             // requiredがtrueに指定されている場合, NotBlankを追加
62 62
             if (isset($options['required']) && true === $options['required']) {
Please login to merge, or discard this patch.
src/Eccube/Form/Type/PhoneNumberType.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -56,7 +56,7 @@
 block discarded – undo
56 56
     public function configureOptions(OptionsResolver $resolver)
57 57
     {
58 58
         $eccubeConfig = $this->eccubeConfig;
59
-        $constraints = function(Options $options) use ($eccubeConfig) {
59
+        $constraints = function (Options $options) use ($eccubeConfig) {
60 60
             $constraints = [];
61 61
             // requiredがtrueに指定されている場合, NotBlankを追加
62 62
             if (isset($options['required']) && true === $options['required']) {
Please login to merge, or discard this patch.
src/Eccube/Controller/Admin/Product/ProductController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -550,7 +550,7 @@
 block discarded – undo
550 550
                     if (!$this->productImageRepository->findOneBy(['file_name' => $delete_image])) {
551 551
                         // 削除
552 552
                         $fs = new Filesystem();
553
-                        $fs->remove($this->eccubeConfig['eccube_save_image_dir'] . '/' . $delete_image);
553
+                        $fs->remove($this->eccubeConfig['eccube_save_image_dir'].'/'.$delete_image);
554 554
                     }
555 555
                 }
556 556
                 $this->entityManager->persist($Product);
Please login to merge, or discard this patch.
src/Eccube/Controller/Admin/Product/CsvImportController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -892,7 +892,7 @@  discard block
 block discarded – undo
892 892
                     '%from%' => $this->convertLineNo(2),
893 893
                     '%to%' => $this->currentLineNo]),
894 894
                 'errors' => $this->errors,
895
-                'error_message' => trans('admin.common.csv_upload_line_error',[
895
+                'error_message' => trans('admin.common.csv_upload_line_error', [
896 896
                     '%from%' => $this->convertLineNo(2)])
897 897
             ]);
898 898
         }
@@ -1641,7 +1641,7 @@  discard block
 block discarded – undo
1641 1641
             return $this->json(['success' => true, 'file_name' => $fileName, 'max_file_no' => $fileNo]);
1642 1642
         }
1643 1643
 
1644
-        return $this->json(['success' => false, 'message' => $form->getErrors(true ,true)]);
1644
+        return $this->json(['success' => false, 'message' => $form->getErrors(true, true)]);
1645 1645
     }
1646 1646
 
1647 1647
     /**
Please login to merge, or discard this patch.
src/Eccube/Controller/Admin/AbstractCsvImportController.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -76,6 +76,9 @@
 block discarded – undo
76 76
         return $data->setHeaderRowNumber(0) ? $data : false;
77 77
     }
78 78
 
79
+    /**
80
+     * @param string $filename
81
+     */
79 82
     protected function sendTemplateResponse(Request $request, $columns, $filename)
80 83
     {
81 84
         set_time_limit(0);
Please login to merge, or discard this patch.
src/Eccube/Form/Type/Admin/MasterdataType.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -66,7 +66,7 @@
 block discarded – undo
66 66
 
67 67
                     // OrderStatus/OrderStatusColorは対象外
68 68
                     // @see https://github.com/EC-CUBE/ec-cube/pull/4844
69
-                    if (in_array($meta->getName(), [OrderStatus::class, OrderStatusColor::class, CustomerOrderStatus::class,])) {
69
+                    if (in_array($meta->getName(), [OrderStatus::class, OrderStatusColor::class, CustomerOrderStatus::class, ])) {
70 70
                         continue;
71 71
                     }
72 72
 
Please login to merge, or discard this patch.