Completed
Pull Request — master (#2152)
by NOBU
82:06 queued 38:49
created
src/Eccube/Plugin/AbstractPluginManager.php 2 patches
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -17,8 +17,8 @@
 block discarded – undo
17 17
         $config->registerMigrationsFromDirectory($migrationFilePath );
18 18
         $config->setMigrationsTableName(self::MIGRATION_TABLE_PREFIX.$pluginCode);
19 19
         $migration = new Migration($config);
20
-                                  // null 又は 'last' を渡すと最新バージョンまでマイグレートする
21
-                                  // 0か'first'を渡すと最初に戻る
20
+                                    // null 又は 'last' を渡すと最新バージョンまでマイグレートする
21
+                                    // 0か'first'を渡すと最初に戻る
22 22
         $migration->migrate($version, false); 
23 23
 
24 24
 
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -9,12 +9,12 @@
 block discarded – undo
9 9
 
10 10
     CONST MIGRATION_TABLE_PREFIX = 'migration_';
11 11
 
12
-    public function migrationSchema($app,$migrationFilePath,$pluginCode,$version=null)
12
+    public function migrationSchema($app, $migrationFilePath, $pluginCode, $version = null)
13 13
     {
14 14
         $config = new Configuration($app['db']);
15 15
         $config->setMigrationsNamespace('DoctrineMigrations');
16 16
         $config->setMigrationsDirectory($migrationFilePath);
17
-        $config->registerMigrationsFromDirectory($migrationFilePath );
17
+        $config->registerMigrationsFromDirectory($migrationFilePath);
18 18
         $config->setMigrationsTableName(self::MIGRATION_TABLE_PREFIX.$pluginCode);
19 19
         $migration = new Migration($config);
20 20
                                   // null 又は 'last' を渡すと最新バージョンまでマイグレートする
Please login to merge, or discard this patch.
src/Eccube/Repository/BlockRepository.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -187,7 +187,7 @@
 block discarded – undo
187 187
             $this->app['config']['block_default_realdir'],
188 188
         );
189 189
         foreach ($readPaths as $readPath) {
190
-            $filePath = $readPath . '/' . $fileName . '.twig';
190
+            $filePath = $readPath.'/'.$fileName.'.twig';
191 191
             $fs = new Filesystem();
192 192
             if ($fs->exists($filePath)) {
193 193
                 return array(
Please login to merge, or discard this patch.
src/Eccube/Repository/OrderRepository.php 2 patches
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -274,7 +274,7 @@
 block discarded – undo
274 274
             $multi = preg_match('/^\d+$/', $searchData['multi']) ? $searchData['multi'] : null;
275 275
             $qb
276 276
                 ->andWhere('o.id = :multi OR o.name01 LIKE :likemulti OR o.name02 LIKE :likemulti OR ' .
277
-                           'o.kana01 LIKE :likemulti OR o.kana02 LIKE :likemulti OR o.company_name LIKE :likemulti')
277
+                            'o.kana01 LIKE :likemulti OR o.kana02 LIKE :likemulti OR o.company_name LIKE :likemulti')
278 278
                 ->setParameter('multi', $multi)
279 279
                 ->setParameter('likemulti', '%' . $searchData['multi'] . '%');
280 280
         }
Please login to merge, or discard this patch.
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -100,14 +100,14 @@  discard block
 block discarded – undo
100 100
         if (isset($searchData['name']) && Str::isNotBlank($searchData['name'])) {
101 101
             $qb
102 102
                 ->andWhere('CONCAT(o.name01, o.name02) LIKE :name')
103
-                ->setParameter('name', '%' . $searchData['name'] . '%');
103
+                ->setParameter('name', '%'.$searchData['name'].'%');
104 104
         }
105 105
 
106 106
         // kana
107 107
         if (isset($searchData['kana']) && Str::isNotBlank($searchData['kana'])) {
108 108
             $qb
109 109
                 ->andWhere('CONCAT(o.kana01, o.kana02) LIKE :kana')
110
-                ->setParameter('kana', '%' . $searchData['kana'] . '%');
110
+                ->setParameter('kana', '%'.$searchData['kana'].'%');
111 111
         }
112 112
 
113 113
         // email
@@ -244,7 +244,7 @@  discard block
 block discarded – undo
244 244
             $qb
245 245
                 ->leftJoin('o.OrderDetails', 'od')
246 246
                 ->andWhere('od.product_name LIKE :buy_product_name')
247
-                ->setParameter('buy_product_name', '%' . $searchData['buy_product_name'] . '%');
247
+                ->setParameter('buy_product_name', '%'.$searchData['buy_product_name'].'%');
248 248
         }
249 249
 
250 250
         // Order By
@@ -270,13 +270,13 @@  discard block
 block discarded – undo
270 270
                 ->setParameter('order_id_start', $searchData['order_id_start']);
271 271
         }
272 272
         // multi
273
-        if (isset( $searchData['multi']) && Str::isNotBlank($searchData['multi'])) {
273
+        if (isset($searchData['multi']) && Str::isNotBlank($searchData['multi'])) {
274 274
             $multi = preg_match('/^\d+$/', $searchData['multi']) ? $searchData['multi'] : null;
275 275
             $qb
276
-                ->andWhere('o.id = :multi OR o.name01 LIKE :likemulti OR o.name02 LIKE :likemulti OR ' .
276
+                ->andWhere('o.id = :multi OR o.name01 LIKE :likemulti OR o.name02 LIKE :likemulti OR '.
277 277
                            'o.kana01 LIKE :likemulti OR o.kana02 LIKE :likemulti OR o.company_name LIKE :likemulti')
278 278
                 ->setParameter('multi', $multi)
279
-                ->setParameter('likemulti', '%' . $searchData['multi'] . '%');
279
+                ->setParameter('likemulti', '%'.$searchData['multi'].'%');
280 280
         }
281 281
 
282 282
         // order_id_end
@@ -313,28 +313,28 @@  discard block
 block discarded – undo
313 313
         if (isset($searchData['name']) && Str::isNotBlank($searchData['name'])) {
314 314
             $qb
315 315
                 ->andWhere('CONCAT(o.name01, o.name02) LIKE :name')
316
-                ->setParameter('name', '%' . $searchData['name'] . '%');
316
+                ->setParameter('name', '%'.$searchData['name'].'%');
317 317
         }
318 318
 
319 319
         // kana
320 320
         if (isset($searchData['kana']) && Str::isNotBlank($searchData['kana'])) {
321 321
             $qb
322 322
                 ->andWhere('CONCAT(o.kana01, o.kana02) LIKE :kana')
323
-                ->setParameter('kana', '%' . $searchData['kana'] . '%');
323
+                ->setParameter('kana', '%'.$searchData['kana'].'%');
324 324
         }
325 325
 
326 326
         // email
327 327
         if (isset($searchData['email']) && Str::isNotBlank($searchData['email'])) {
328 328
             $qb
329 329
                 ->andWhere('o.email like :email')
330
-                ->setParameter('email', '%' . $searchData['email'] . '%');
330
+                ->setParameter('email', '%'.$searchData['email'].'%');
331 331
         }
332 332
 
333 333
         // tel
334 334
         if (isset($searchData['tel']) && Str::isNotBlank($searchData['tel'])) {
335 335
             $qb
336 336
                 ->andWhere('CONCAT(o.tel01, o.tel02, o.tel03) LIKE :tel')
337
-                ->setParameter('tel', '%' . $searchData['tel'] . '%');
337
+                ->setParameter('tel', '%'.$searchData['tel'].'%');
338 338
         }
339 339
 
340 340
         // sex
@@ -446,7 +446,7 @@  discard block
 block discarded – undo
446 446
             $qb
447 447
                 ->leftJoin('o.OrderDetails', 'od')
448 448
                 ->andWhere('od.product_name LIKE :buy_product_name')
449
-                ->setParameter('buy_product_name', '%' . $searchData['buy_product_name'] . '%');
449
+                ->setParameter('buy_product_name', '%'.$searchData['buy_product_name'].'%');
450 450
         }
451 451
 
452 452
         // Order By
Please login to merge, or discard this patch.
src/Eccube/Repository/PageLayoutRepository.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -257,7 +257,7 @@
 block discarded – undo
257 257
         }
258 258
 
259 259
         foreach ($readPaths as $readPath) {
260
-            $filePath = $readPath . '/' . $fileName . '.twig';
260
+            $filePath = $readPath.'/'.$fileName.'.twig';
261 261
             $fs = new Filesystem();
262 262
             if ($fs->exists($filePath)) {
263 263
                 return array(
Please login to merge, or discard this patch.
src/Eccube/Repository/PluginEventHandlerRepository.php 3 patches
Indentation   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -73,17 +73,17 @@  discard block
 block discarded – undo
73 73
 
74 74
         $qb = $this->createQueryBuilder('e');
75 75
         $qb->andWhere("e.priority >= $range_end ")
76
-           ->andWhere("e.priority <= $range_start ")
77
-           ->andWhere('e.event = :event')
78
-           ->setParameter('event',$event)
79
-           ->setMaxResults(1)
80
-           ->orderBy('e.priority','ASC');
76
+            ->andWhere("e.priority <= $range_start ")
77
+            ->andWhere('e.event = :event')
78
+            ->setParameter('event',$event)
79
+            ->setMaxResults(1)
80
+            ->orderBy('e.priority','ASC');
81 81
 
82 82
         $result=$qb->getQuery()->getResult();
83 83
         if(count($result)){
84
-             return $result[0]->getPriority() -1;
84
+                return $result[0]->getPriority() -1;
85 85
         }else{
86
-             return $range_start;
86
+                return $range_start;
87 87
         }
88 88
 
89 89
     }
@@ -96,14 +96,14 @@  discard block
 block discarded – undo
96 96
         $qb = $this->createQueryBuilder('e');
97 97
 
98 98
         $qb->andWhere("e.priority >= $range_end ")
99
-           ->andWhere("e.priority <= $range_start ")
100
-           ->andWhere("e.del_flg = 0 ") 
101
-           ->andWhere('e.priority '.($up ?  '>' : '<' ).' :pri')
102
-           ->andWhere('e.event = :event')
103
-           ->setParameter('event',$pluginEventHandler->getEvent())
104
-           ->setParameter('pri',  $pluginEventHandler->getPriority()  )
105
-           ->setMaxResults(1)
106
-           ->orderBy('e.priority', ($up ? 'ASC':'DESC' )  );
99
+            ->andWhere("e.priority <= $range_start ")
100
+            ->andWhere("e.del_flg = 0 ") 
101
+            ->andWhere('e.priority '.($up ?  '>' : '<' ).' :pri')
102
+            ->andWhere('e.event = :event')
103
+            ->setParameter('event',$pluginEventHandler->getEvent())
104
+            ->setParameter('pri',  $pluginEventHandler->getPriority()  )
105
+            ->setMaxResults(1)
106
+            ->orderBy('e.priority', ($up ? 'ASC':'DESC' )  );
107 107
 
108 108
         $result=$qb->getQuery()->getResult();
109 109
 
Please login to merge, or discard this patch.
Spacing   +31 added lines, -31 removed lines patch added patch discarded remove patch
@@ -42,8 +42,8 @@  discard block
 block discarded – undo
42 42
         $qb = $this->createQueryBuilder('e')
43 43
             ->innerJoin('e.Plugin', 'p')
44 44
             ->andWhere('e.del_flg = 0 ')  
45
-            ->Orderby('e.event','ASC') 
46
-            ->addOrderby('e.priority','DESC');
45
+            ->Orderby('e.event', 'ASC') 
46
+            ->addOrderby('e.priority', 'DESC');
47 47
             ;
48 48
 
49 49
         return $qb->getQuery()->getResult();
@@ -52,66 +52,66 @@  discard block
 block discarded – undo
52 52
     public function getPriorityRange($type)
53 53
     {
54 54
 
55
-        if(PluginEventHandler::EVENT_HANDLER_TYPE_FIRST==$type){
56
-            $range_start=PluginEventHandler::EVENT_PRIORITY_FIRST_START;
57
-            $range_end=PluginEventHandler::EVENT_PRIORITY_FIRST_END;
58
-        }elseif(PluginEventHandler::EVENT_HANDLER_TYPE_LAST==$type){
59
-            $range_start=PluginEventHandler::EVENT_PRIORITY_LAST_START;
60
-            $range_end=PluginEventHandler::EVENT_PRIORITY_LAST_END;
61
-        }else{
62
-            $range_start=PluginEventHandler::EVENT_PRIORITY_NORMAL_START;
63
-            $range_end=PluginEventHandler::EVENT_PRIORITY_NORMAL_END;
55
+        if (PluginEventHandler::EVENT_HANDLER_TYPE_FIRST == $type) {
56
+            $range_start = PluginEventHandler::EVENT_PRIORITY_FIRST_START;
57
+            $range_end = PluginEventHandler::EVENT_PRIORITY_FIRST_END;
58
+        }elseif (PluginEventHandler::EVENT_HANDLER_TYPE_LAST == $type) {
59
+            $range_start = PluginEventHandler::EVENT_PRIORITY_LAST_START;
60
+            $range_end = PluginEventHandler::EVENT_PRIORITY_LAST_END;
61
+        } else {
62
+            $range_start = PluginEventHandler::EVENT_PRIORITY_NORMAL_START;
63
+            $range_end = PluginEventHandler::EVENT_PRIORITY_NORMAL_END;
64 64
         }
65
-        return array($range_start,$range_end);
65
+        return array($range_start, $range_end);
66 66
 
67 67
     }
68 68
 
69
-    public function calcNewPriority($event , $type)
69
+    public function calcNewPriority($event, $type)
70 70
     {
71 71
 
72
-        list($range_start,$range_end) = $this->getPriorityRange($type);
72
+        list($range_start, $range_end) = $this->getPriorityRange($type);
73 73
 
74 74
         $qb = $this->createQueryBuilder('e');
75 75
         $qb->andWhere("e.priority >= $range_end ")
76 76
            ->andWhere("e.priority <= $range_start ")
77 77
            ->andWhere('e.event = :event')
78
-           ->setParameter('event',$event)
78
+           ->setParameter('event', $event)
79 79
            ->setMaxResults(1)
80
-           ->orderBy('e.priority','ASC');
80
+           ->orderBy('e.priority', 'ASC');
81 81
 
82
-        $result=$qb->getQuery()->getResult();
83
-        if(count($result)){
84
-             return $result[0]->getPriority() -1;
85
-        }else{
82
+        $result = $qb->getQuery()->getResult();
83
+        if (count($result)) {
84
+             return $result[0]->getPriority() - 1;
85
+        } else {
86 86
              return $range_start;
87 87
         }
88 88
 
89 89
     }
90 90
 
91
-    public function upPriority($pluginEventHandler,$up=true)
91
+    public function upPriority($pluginEventHandler, $up = true)
92 92
     {
93 93
 
94
-        list($range_start,$range_end) = $this->getPriorityRange($pluginEventHandler->getHandlerType());
94
+        list($range_start, $range_end) = $this->getPriorityRange($pluginEventHandler->getHandlerType());
95 95
 
96 96
         $qb = $this->createQueryBuilder('e');
97 97
 
98 98
         $qb->andWhere("e.priority >= $range_end ")
99 99
            ->andWhere("e.priority <= $range_start ")
100 100
            ->andWhere("e.del_flg = 0 ") 
101
-           ->andWhere('e.priority '.($up ?  '>' : '<' ).' :pri')
101
+           ->andWhere('e.priority '.($up ? '>' : '<').' :pri')
102 102
            ->andWhere('e.event = :event')
103
-           ->setParameter('event',$pluginEventHandler->getEvent())
104
-           ->setParameter('pri',  $pluginEventHandler->getPriority()  )
103
+           ->setParameter('event', $pluginEventHandler->getEvent())
104
+           ->setParameter('pri', $pluginEventHandler->getPriority())
105 105
            ->setMaxResults(1)
106
-           ->orderBy('e.priority', ($up ? 'ASC':'DESC' )  );
106
+           ->orderBy('e.priority', ($up ? 'ASC' : 'DESC'));
107 107
 
108
-        $result=$qb->getQuery()->getResult();
108
+        $result = $qb->getQuery()->getResult();
109 109
 
110
-        if(count($result)){
111
-            $em =$this->getEntityManager();
110
+        if (count($result)) {
111
+            $em = $this->getEntityManager();
112 112
             $em->getConnection()->beginTransaction();
113 113
             // 2個のentityのprioriryを入れ替える
114
-            $tmp=$pluginEventHandler->getPriority();
114
+            $tmp = $pluginEventHandler->getPriority();
115 115
             $pluginEventHandler->setPriority($result[0]->getPriority());
116 116
             $result[0]->setPriority($tmp);
117 117
             $em->persist($result[0]);
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
119 119
             $em->flush();
120 120
             $em->getConnection()->commit();
121 121
             # 移動する
122
-        }else{
122
+        } else {
123 123
             # 移動しない
124 124
             throw new PluginException("Can't swap");
125 125
         }
Please login to merge, or discard this patch.
Braces   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -55,10 +55,10 @@  discard block
 block discarded – undo
55 55
         if(PluginEventHandler::EVENT_HANDLER_TYPE_FIRST==$type){
56 56
             $range_start=PluginEventHandler::EVENT_PRIORITY_FIRST_START;
57 57
             $range_end=PluginEventHandler::EVENT_PRIORITY_FIRST_END;
58
-        }elseif(PluginEventHandler::EVENT_HANDLER_TYPE_LAST==$type){
58
+        } elseif(PluginEventHandler::EVENT_HANDLER_TYPE_LAST==$type){
59 59
             $range_start=PluginEventHandler::EVENT_PRIORITY_LAST_START;
60 60
             $range_end=PluginEventHandler::EVENT_PRIORITY_LAST_END;
61
-        }else{
61
+        } else{
62 62
             $range_start=PluginEventHandler::EVENT_PRIORITY_NORMAL_START;
63 63
             $range_end=PluginEventHandler::EVENT_PRIORITY_NORMAL_END;
64 64
         }
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
         $result=$qb->getQuery()->getResult();
83 83
         if(count($result)){
84 84
              return $result[0]->getPriority() -1;
85
-        }else{
85
+        } else{
86 86
              return $range_start;
87 87
         }
88 88
 
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
119 119
             $em->flush();
120 120
             $em->getConnection()->commit();
121 121
             # 移動する
122
-        }else{
122
+        } else{
123 123
             # 移動しない
124 124
             throw new PluginException("Can't swap");
125 125
         }
Please login to merge, or discard this patch.
src/Eccube/Resource/doctrine/migration/Version20150722170707.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -16,7 +16,7 @@
 block discarded – undo
16 16
     public function up(Schema $schema)
17 17
     {
18 18
 
19
-        if($this->connection->getDatabasePlatform()->getName() == "mysql"){
19
+        if ($this->connection->getDatabasePlatform()->getName() == "mysql") {
20 20
             // this up() migration is auto-generated, please modify it to your needs
21 21
             $this->addSql("alter table  dtb_base_info                 collate utf8_general_ci ;");
22 22
             $this->addSql("alter table  dtb_block                     collate utf8_general_ci ;");
Please login to merge, or discard this patch.
src/Eccube/Resource/doctrine/migration/Version20150806184533.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -42,7 +42,7 @@
 block discarded – undo
42 42
         $t = $schema->getTable('dtb_product_class');
43 43
         $c = $t->getColumn('stock_unlimited');
44 44
 
45
-        if($c->getType()->getName() != 'smallint'){
45
+        if ($c->getType()->getName() != 'smallint') {
46 46
             $this->addSql('ALTER TABLE dtb_product_class ADD stock_unlimited_tmp int ;');
47 47
             $this->addSql('UPDATE dtb_product_class SET stock_unlimited_tmp = 1 where stock_unlimited =  true ');
48 48
             $this->addSql('UPDATE dtb_product_class SET stock_unlimited_tmp = 0 where stock_unlimited <> true ');
Please login to merge, or discard this patch.
src/Eccube/Resource/doctrine/migration/Version20150806220909.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -38,8 +38,8 @@
 block discarded – undo
38 38
     public function up(Schema $schema)
39 39
     {
40 40
         // this up() migration is auto-generated, please modify it to your needs
41
-        $t=$schema->getTable('dtb_product_class');
42
-        if(!$t->hasColumn('stock_unlimited') && $t->hasColumn('stock_unlimited_tmp')){
41
+        $t = $schema->getTable('dtb_product_class');
42
+        if (!$t->hasColumn('stock_unlimited') && $t->hasColumn('stock_unlimited_tmp')) {
43 43
             $t->addColumn('stock_unlimited', 'smallint', array('NotNull'=>true, 'Default'=>0));
44 44
         }
45 45
     }
Please login to merge, or discard this patch.
src/Eccube/Resource/doctrine/migration/Version20150806222639.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -39,8 +39,8 @@
 block discarded – undo
39 39
     public function up(Schema $schema)
40 40
     {
41 41
         // this up() migration is auto-generated, please modify it to your needs
42
-        $t=$schema->getTable('dtb_product_class');
43
-        if($t->hasColumn('stock_unlimited_tmp')){
42
+        $t = $schema->getTable('dtb_product_class');
43
+        if ($t->hasColumn('stock_unlimited_tmp')) {
44 44
             $this->addSql('update dtb_product_class set stock_unlimited = stock_unlimited_tmp;');
45 45
             $t->dropColumn('stock_unlimited_tmp');
46 46
         }
Please login to merge, or discard this patch.