Completed
Pull Request — master (#1181)
by NOBU
18:05
created
src/Eccube/Service/ShoppingService.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -137,7 +137,7 @@
 block discarded – undo
137 137
      * 仮受注情報作成
138 138
      *
139 139
      * @param $Customer
140
-     * @param $preOrderId
140
+     * @param string $preOrderId
141 141
      * @return mixed
142 142
      * @throws \Doctrine\ORM\NoResultException
143 143
      * @throws \Doctrine\ORM\NonUniqueResultException
Please login to merge, or discard this patch.
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -270,7 +270,7 @@  discard block
 block discarded – undo
270 270
             ->setFax03($Customer->getFax03())
271 271
             ->setZip01($Customer->getZip01())
272 272
             ->setZip02($Customer->getZip02())
273
-            ->setZipCode($Customer->getZip01() . $Customer->getZip02())
273
+            ->setZipCode($Customer->getZip01().$Customer->getZip02())
274 274
             ->setPref($Customer->getPref())
275 275
             ->setAddr01($Customer->getAddr01())
276 276
             ->setAddr02($Customer->getAddr02())
@@ -409,7 +409,7 @@  discard block
 block discarded – undo
409 409
                 ->setFax03($CustomerAddress->getFax03())
410 410
                 ->setZip01($CustomerAddress->getZip01())
411 411
                 ->setZip02($CustomerAddress->getZip02())
412
-                ->setZipCode($CustomerAddress->getZip01() . $CustomerAddress->getZip02())
412
+                ->setZipCode($CustomerAddress->getZip01().$CustomerAddress->getZip02())
413 413
                 ->setPref($CustomerAddress->getPref())
414 414
                 ->setAddr01($CustomerAddress->getAddr01())
415 415
                 ->setAddr02($CustomerAddress->getAddr02());
@@ -428,7 +428,7 @@  discard block
 block discarded – undo
428 428
                 ->setFax03($Customer->getFax03())
429 429
                 ->setZip01($Customer->getZip01())
430 430
                 ->setZip02($Customer->getZip02())
431
-                ->setZipCode($Customer->getZip01() . $Customer->getZip02())
431
+                ->setZipCode($Customer->getZip01().$Customer->getZip02())
432 432
                 ->setPref($Customer->getPref())
433 433
                 ->setAddr01($Customer->getAddr01())
434 434
                 ->setAddr02($Customer->getAddr02());
@@ -913,10 +913,10 @@  discard block
 block discarded – undo
913 913
 
914 914
         // 配送日数が設定されている
915 915
         if ($deliveryDateFlag) {
916
-            $period = new \DatePeriod (
917
-                new \DateTime($minDate . ' day'),
916
+            $period = new \DatePeriod(
917
+                new \DateTime($minDate.' day'),
918 918
                 new \DateInterval('P1D'),
919
-                new \DateTime($minDate + $this->app['config']['deliv_date_end_max'] . ' day')
919
+                new \DateTime($minDate + $this->app['config']['deliv_date_end_max'].' day')
920 920
             );
921 921
 
922 922
             foreach ($period as $day) {
Please login to merge, or discard this patch.
html/index.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@
 block discarded – undo
25 25
 require __DIR__.'/../autoload.php';
26 26
 
27 27
 ini_set('display_errors', 'Off');
28
-error_reporting(E_ALL & ~E_DEPRECATED & ~E_STRICT);
28
+error_reporting(E_ALL&~E_DEPRECATED&~E_STRICT);
29 29
 
30 30
 $app = new Eccube\Application();
31 31
 
Please login to merge, or discard this patch.
html/index_dev.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -58,7 +58,7 @@
 block discarded – undo
58 58
 
59 59
 // load config dev
60 60
 $conf = $app['config'];
61
-$app['config'] = $app->share(function () use ($conf) {
61
+$app['config'] = $app->share(function() use ($conf) {
62 62
     $confarray = array();
63 63
     $config_dev_file = __DIR__.'/../app/config/eccube/config_dev.yml';
64 64
     if (file_exists($config_dev_file)) {
Please login to merge, or discard this patch.
html/install.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -26,11 +26,11 @@
 block discarded – undo
26 26
     apc_clear_cache();
27 27
 }
28 28
 
29
-require __DIR__ . '/../autoload.php';
29
+require __DIR__.'/../autoload.php';
30 30
 
31 31
 $app = new Eccube\InstallApplication();
32 32
 $app['debug'] = true;
33
-$app->before(function (\Symfony\Component\HttpFoundation\Request $request, \Silex\Application $app) {
33
+$app->before(function(\Symfony\Component\HttpFoundation\Request $request, \Silex\Application $app) {
34 34
     if (!$request->getSession()->isStarted()) {
35 35
         $request->getSession()->start();
36 36
     }
Please login to merge, or discard this patch.
src/Eccube/Application/ApplicationTrait.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -19,32 +19,32 @@  discard block
 block discarded – undo
19 19
      */
20 20
     public function addSuccess($message, $namespace = 'front')
21 21
     {
22
-        $this['session']->getFlashBag()->add('eccube.' . $namespace . '.success', $message);
22
+        $this['session']->getFlashBag()->add('eccube.'.$namespace.'.success', $message);
23 23
     }
24 24
 
25 25
     public function addError($message, $namespace = 'front')
26 26
     {
27
-        $this['session']->getFlashBag()->add('eccube.' . $namespace . '.error', $message);
27
+        $this['session']->getFlashBag()->add('eccube.'.$namespace.'.error', $message);
28 28
     }
29 29
 
30 30
     public function addDanger($message, $namespace = 'front')
31 31
     {
32
-        $this['session']->getFlashBag()->add('eccube.' . $namespace . '.danger', $message);
32
+        $this['session']->getFlashBag()->add('eccube.'.$namespace.'.danger', $message);
33 33
     }
34 34
 
35 35
     public function addWarning($message, $namespace = 'front')
36 36
     {
37
-        $this['session']->getFlashBag()->add('eccube.' . $namespace . '.warning', $message);
37
+        $this['session']->getFlashBag()->add('eccube.'.$namespace.'.warning', $message);
38 38
     }
39 39
 
40 40
     public function addInfo($message, $namespace = 'front')
41 41
     {
42
-        $this['session']->getFlashBag()->add('eccube.' . $namespace . '.info', $message);
42
+        $this['session']->getFlashBag()->add('eccube.'.$namespace.'.info', $message);
43 43
     }
44 44
 
45 45
     public function addRequestError($message, $namespace = 'front')
46 46
     {
47
-        $this['session']->getFlashBag()->set('eccube.' . $namespace . '.request.error', $message);
47
+        $this['session']->getFlashBag()->set('eccube.'.$namespace.'.request.error', $message);
48 48
     }
49 49
 
50 50
     public function clearMessage()
@@ -210,7 +210,7 @@  discard block
 block discarded – undo
210 210
         $twig = $this['twig'];
211 211
 
212 212
         if ($response instanceof StreamedResponse) {
213
-            $response->setCallback(function () use ($twig, $view, $parameters) {
213
+            $response->setCallback(function() use ($twig, $view, $parameters) {
214 214
                 $twig->display($view, $parameters);
215 215
             });
216 216
         } else {
Please login to merge, or discard this patch.
src/Eccube/ConsoleApplication.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
 
38 38
         // load databtase config
39 39
         $config = array();
40
-        $file = __DIR__ . '/../../app/config/eccube/database.yml';
40
+        $file = __DIR__.'/../../app/config/eccube/database.yml';
41 41
         if (file_exists($file)) {
42 42
             $config = Yaml::parse($file);
43 43
         }
@@ -50,15 +50,15 @@  discard block
 block discarded – undo
50 50
         $app->register(new \Saxulum\DoctrineOrmManagerRegistry\Silex\Provider\DoctrineOrmManagerRegistryProvider());
51 51
 
52 52
         $app->register(new \Dflydev\Silex\Provider\DoctrineOrm\DoctrineOrmServiceProvider(), array(
53
-            "orm.proxies_dir" => __DIR__ . '/../../app/cache/doctrine',
53
+            "orm.proxies_dir" => __DIR__.'/../../app/cache/doctrine',
54 54
             'orm.em.options' => array(
55 55
                 'mappings' => array(
56 56
                     array(
57 57
                         'type' => 'yml',
58 58
                         'namespace' => 'Eccube\Entity',
59 59
                         'path' => array(
60
-                            __DIR__ . '/Resource/doctrine',
61
-                            __DIR__ . '/Resource/doctrine/master',
60
+                            __DIR__.'/Resource/doctrine',
61
+                            __DIR__.'/Resource/doctrine/master',
62 62
                         ),
63 63
                     ),
64 64
                 ),
@@ -71,16 +71,16 @@  discard block
 block discarded – undo
71 71
             array(
72 72
                 'console.name' => 'EC-CUBE',
73 73
                 'console.version' => '3.0.0',
74
-                'console.project_directory' => __DIR__ . "/.."
74
+                'console.project_directory' => __DIR__."/.."
75 75
             )
76 76
         );
77 77
 
78 78
         $app->register(new \Dbtlr\MigrationProvider\Provider\MigrationServiceProvider(), array(
79
-            'db.migrations.path' => __DIR__ . '/Resource/doctrine/migration',
79
+            'db.migrations.path' => __DIR__.'/Resource/doctrine/migration',
80 80
         ));
81 81
 
82 82
         $app->register(new \Silex\Provider\MonologServiceProvider(), array(
83
-            'monolog.logfile' => __DIR__ . '/../../app/log/miglation.log',
83
+            'monolog.logfile' => __DIR__.'/../../app/log/miglation.log',
84 84
         ));
85 85
     }
86 86
 }
Please login to merge, or discard this patch.
src/Eccube/Controller/AbstractController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -38,7 +38,7 @@
 block discarded – undo
38 38
     protected function getBoundForm(Application $app, $type)
39 39
     {
40 40
         $form = $app['form.factory']
41
-            ->createBuilder($app['eccube.form.type.' . $type], $app['eccube.entity.' . $type])
41
+            ->createBuilder($app['eccube.form.type.'.$type], $app['eccube.entity.'.$type])
42 42
             ->getForm();
43 43
         $form->handleRequest($app['request']);
44 44
 
Please login to merge, or discard this patch.
src/Eccube/Controller/Admin/Content/CssController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
             if ($form->isValid()) {
48 48
                 $data = $form->getData();
49 49
                 $fs = new Filesystem();
50
-                $fs->dumpFile($this->getCssDir($app) . $data['file_name'], $data['content']);
50
+                $fs->dumpFile($this->getCssDir($app).$data['file_name'], $data['content']);
51 51
                 $app['session']->getFlashBag()->add('admin.content.css.complete', 'admin.register.complete');
52 52
 
53 53
                 return $app->redirect($app['url_generator']->generate('admin_content_css'));
@@ -113,6 +113,6 @@  discard block
 block discarded – undo
113 113
 
114 114
     protected function getCssDir($app)
115 115
     {
116
-        return $app['config']['user_template_realdir'] . $this->cssDir;
116
+        return $app['config']['user_template_realdir'].$this->cssDir;
117 117
     }
118 118
 }
Please login to merge, or discard this patch.
src/Eccube/Controller/Admin/Content/LayoutController.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -89,20 +89,20 @@  discard block
 block discarded – undo
89 89
                 $max = count($Blocks);
90 90
                 for ($i = 0; $i < $max; $i++) {
91 91
                     // block_id が取得できない場合は INSERT しない
92
-                    if (!isset($data['id_' . $i])) {
92
+                    if (!isset($data['id_'.$i])) {
93 93
                         continue;
94 94
                     }
95 95
                     // 未使用は INSERT しない
96
-                    if ($data['target_id_' . $i] == \Eccube\Entity\PageLayout::TARGET_ID_UNUSED) {
96
+                    if ($data['target_id_'.$i] == \Eccube\Entity\PageLayout::TARGET_ID_UNUSED) {
97 97
                         continue;
98 98
                     }
99 99
                     // 他のページに anywhere が存在する場合は INSERT しない
100
-                    $anywhere = (isset($data['anywhere_' . $i]) && $data['anywhere_' . $i] == 1) ? 1 : 0;
101
-                    if (isset($data['anywhere_' . $i]) && $data['anywhere_' . $i] == 1) {
100
+                    $anywhere = (isset($data['anywhere_'.$i]) && $data['anywhere_'.$i] == 1) ? 1 : 0;
101
+                    if (isset($data['anywhere_'.$i]) && $data['anywhere_'.$i] == 1) {
102 102
                         $Other = $app['orm.em']->getRepository('Eccube\Entity\BlockPosition')
103 103
                             ->findBy(array(
104 104
                                 'anywhere' => 1,
105
-                                'block_id' => $data['id_' . $i],
105
+                                'block_id' => $data['id_'.$i],
106 106
                             ));
107 107
                         if (count($Other) > 0) {
108 108
                             continue;
@@ -112,14 +112,14 @@  discard block
 block discarded – undo
112 112
                     $BlockPosition = new \Eccube\Entity\BlockPosition();
113 113
                     $Block = $app['orm.em']->getRepository('Eccube\Entity\Block')
114 114
                         ->findOneBy(array(
115
-                            'id' => $data['id_' . $i],
115
+                            'id' => $data['id_'.$i],
116 116
                             'DeviceType' => $DeviceType,
117 117
                         ));
118 118
                     $BlockPosition
119 119
                         ->setPageId($id)
120
-                        ->setBlockId($data['id_' . $i])
121
-                        ->setBlockRow($data['top_' . $i])
122
-                        ->setTargetId($data['target_id_' . $i])
120
+                        ->setBlockId($data['id_'.$i])
121
+                        ->setBlockRow($data['top_'.$i])
122
+                        ->setTargetId($data['target_id_'.$i])
123 123
                         ->setBlock($Block)
124 124
                         ->setPageLayout($TargetPageLayout)
125 125
                         ->setAnywhere($anywhere);
Please login to merge, or discard this patch.