View GitHub Repository unfinished

Schedule Inspection
Subscribe
Completed
Push — master ( fd2751...f5e198 )
by Aleksandar
35:50 queued 08:32
created
Status
Category
Toggle Patches
packages/Page/tests/Controller/PageFactoryTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -10,7 +10,7 @@
 
 block discarded – undo
10 10 
             ->disableOriginalConstructor()
11 11 
             ->getMockForAbstractClass();
12 12 
         $container = $this->getMockBuilder(\Interop\Container\ContainerInterface::class)
13 
-            ->setMethods(['get'])
13 
+            ->setMethods([ 'get' ])
14 14 
             ->getMockForAbstractClass();
15 15 
         $container->expects(static::at(0))
16 16 
             ->method('get')
Please login to merge, or discard this patch.
packages/Page/tests/Service/PageServiceFactoryTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch 
@@ -13,7 +13,7 @@  discard block
 
 block discarded – undo
13 13 
             ->disableOriginalConstructor()
14 14 
             ->getMockForAbstractClass();
15 15 
         $pageMapper = $this->getMockBuilder(\Page\Mapper\PageMapper::class)
16 
-            ->setMethods(['getPaginationSelect', 'getAdapter'])
16 
+            ->setMethods([ 'getPaginationSelect', 'getAdapter' ])
17 17 
             ->disableOriginalConstructor()
18 18 
             ->getMockForAbstractClass();
19 19 
         $pageMapper->expects(static::once())
 
@@ -27,11 +27,11 @@  discard block
 
 block discarded – undo
27 27 
         $pageFilter = $this->getMockBuilder(\Page\Filter\PageFilter::class)
28 28 
             ->getMockForAbstractClass();
29 29 
         $container = $this->getMockBuilder(\Interop\Container\ContainerInterface::class)
30 
-            ->setMethods(['get'])
30 
+            ->setMethods([ 'get' ])
31 31 
             ->getMockForAbstractClass();
32 32 
         $container->expects(static::at(0))
33 33 
             ->method('get')
34 
-            ->will(static::returnValue(['upload' => ['public_path' => 'test', 'non_public_path' => 'test']]));
34 
+            ->will(static::returnValue([ 'upload' => [ 'public_path' => 'test', 'non_public_path' => 'test' ] ]));
35 35 
         $container->expects(static::at(1))
36 36 
             ->method('get')
37 37 
             ->will(static::returnValue($pageMapper));
Please login to merge, or discard this patch.
packages/Page/tests/Service/PageServiceTest.php 1 patch
Spacing   +19 added lines, -19 removed lines patch added patch discarded remove patch 
@@ -29,7 +29,7 @@  discard block
 
 block discarded – undo
29 29 
             ->disableOriginalConstructor()
30 30 
             ->getMock();
31 31 
         $pageMapper = $this->getMockBuilder(\Page\Mapper\PageMapper::class)
32 
-            ->setMethods(['select', 'current'])
32 
+            ->setMethods([ 'select', 'current' ])
33 33 
             ->disableOriginalConstructor()
34 34 
             ->getMock();
35 35 
         $pageMapper->expects(static::once())
 
@@ -55,7 +55,7 @@  discard block
 
 block discarded – undo
55 55 
             ->disableOriginalConstructor()
56 56 
             ->getMock();
57 57 
         $pageMapper = $this->getMockBuilder(\Page\Mapper\PageMapper::class)
58 
-            ->setMethods(['getActivePage', 'current'])
58 
+            ->setMethods([ 'getActivePage', 'current' ])
59 59 
             ->disableOriginalConstructor()
60 60 
             ->getMock();
61 61 
         $pageMapper->expects(static::once())
 
@@ -81,7 +81,7 @@  discard block
 
 block discarded – undo
81 81 
             ->disableOriginalConstructor()
82 82 
             ->getMock();
83 83 
         $pageMapper = $this->getMockBuilder(\Page\Mapper\PageMapper::class)
84 
-            ->setMethods(['select', 'current'])
84 
+            ->setMethods([ 'select', 'current' ])
85 85 
             ->disableOriginalConstructor()
86 86 
             ->getMock();
87 87 
         $pageMapper->expects(static::once())
 
@@ -107,7 +107,7 @@  discard block
 
 block discarded – undo
107 107 
             'is_homepage' => 1
108 108 
         ];
109 109 
         $pageFilter = $this->getMockBuilder(\Page\Filter\PageFilter::class)
110 
-            ->setMethods(['getInputFilter', 'setData', 'isValid', 'getValues'])
110 
+            ->setMethods([ 'getInputFilter', 'setData', 'isValid', 'getValues' ])
111 111 
             ->disableOriginalConstructor()
112 112 
             ->getMock();
113 113 
         $pageFilter->expects(static::once())
 
@@ -123,7 +123,7 @@  discard block
 
 block discarded – undo
123 123 
             ->method('setData')
124 124 
             ->willReturnSelf();
125 125 
         $pageMapper = $this->getMockBuilder(\Page\Mapper\PageMapper::class)
126 
-            ->setMethods(['update', 'insert'])
126 
+            ->setMethods([ 'update', 'insert' ])
127 127 
             ->disableOriginalConstructor()
128 128 
             ->getMock();
129 129 
         $pageMapper->expects(static::once())
 
@@ -151,7 +151,7 @@  discard block
 
 block discarded – undo
151 151 
         ];
152 152
153 153 
         $pageFilter = $this->getMockBuilder(\Page\Filter\PageFilter::class)
154 
-            ->setMethods(['getInputFilter', 'setData', 'isValid', 'getMessages'])
154 
+            ->setMethods([ 'getInputFilter', 'setData', 'isValid', 'getMessages' ])
155 155 
             ->disableOriginalConstructor()
156 156 
             ->getMock();
157 157 
         $pageFilter->expects(static::once())
 
@@ -162,12 +162,12 @@  discard block
 
 block discarded – undo
162 162 
             ->willReturn(false);
163 163 
         $pageFilter->expects(static::once())
164 164 
             ->method('getMessages')
165 
-            ->willReturn(['test error']);
165 
+            ->willReturn([ 'test error' ]);
166 166 
         $pageFilter->expects(static::once())
167 167 
             ->method('setData')
168 168 
             ->willReturnSelf();
169 169 
         $pageMapper = $this->getMockBuilder(\Page\Mapper\PageMapper::class)
170 
-            ->setMethods(['update', 'insert'])
170 
+            ->setMethods([ 'update', 'insert' ])
171 171 
             ->disableOriginalConstructor()
172 172 
             ->getMock();
173 173 
         $paginator = $this->getMockBuilder(\Zend\Paginator\Paginator::class)
 
@@ -191,11 +191,11 @@  discard block
 
 block discarded – undo
191 191 
             'is_homepage' => 1
192 192 
         ];
193 193 
         $pageFilter = $this->getMockBuilder(\Page\Filter\PageFilter::class)
194 
-            ->setMethods(['select'])
194 
+            ->setMethods([ 'select' ])
195 195 
             ->disableOriginalConstructor()
196 196 
             ->getMock();
197 197 
         $pageMapper = $this->getMockBuilder(\Page\Mapper\PageMapper::class)
198 
-            ->setMethods(['select', 'current'])
198 
+            ->setMethods([ 'select', 'current' ])
199 199 
             ->disableOriginalConstructor()
200 200 
             ->getMock();
201 201 
         $pageMapper->expects(static::once())
 
@@ -225,7 +225,7 @@  discard block
 
 block discarded – undo
225 225 
             'is_homepage' => 1
226 226 
         ];
227 227 
         $pageFilter = $this->getMockBuilder(\Page\Filter\PageFilter::class)
228 
-            ->setMethods(['getInputFilter', 'setData', 'isValid', 'getMessages'])
228 
+            ->setMethods([ 'getInputFilter', 'setData', 'isValid', 'getMessages' ])
229 229 
             ->disableOriginalConstructor()
230 230 
             ->getMock();
231 231 
         $pageFilter->expects(static::once())
 
@@ -239,9 +239,9 @@  discard block
 
 block discarded – undo
239 239 
             ->willReturn(false);
240 240 
         $pageFilter->expects(static::once())
241 241 
             ->method('getMessages')
242 
-            ->willReturn(['test error']);
242 
+            ->willReturn([ 'test error' ]);
243 243 
         $pageMapper = $this->getMockBuilder(\Page\Mapper\PageMapper::class)
244 
-            ->setMethods(['select', 'current'])
244 
+            ->setMethods([ 'select', 'current' ])
245 245 
             ->disableOriginalConstructor()
246 246 
             ->getMock();
247 247 
         $pageMapper->expects(static::once())
 
@@ -267,7 +267,7 @@  discard block
 
 block discarded – undo
267 267 
             'is_homepage' => 1
268 268 
         ];
269 269 
         $pageFilter = $this->getMockBuilder(\Page\Filter\PageFilter::class)
270 
-            ->setMethods(['getInputFilter', 'setData', 'isValid', 'getValues', 'select'])
270 
+            ->setMethods([ 'getInputFilter', 'setData', 'isValid', 'getValues', 'select' ])
271 271 
             ->disableOriginalConstructor()
272 272 
             ->getMock();
273 273 
         $pageFilter->expects(static::once())
 
@@ -283,7 +283,7 @@  discard block
 
 block discarded – undo
283 283 
             ->method('setData')
284 284 
             ->willReturnSelf();
285 285 
         $pageMapper = $this->getMockBuilder(\Page\Mapper\PageMapper::class)
286 
-            ->setMethods(['update', 'select', 'current'])
286 
+            ->setMethods([ 'update', 'select', 'current' ])
287 287 
             ->disableOriginalConstructor()
288 288 
             ->getMock();
289 289 
         $pageMapper->expects(static::exactly(2))
 
@@ -313,7 +313,7 @@  discard block
 
 block discarded – undo
313 313 
             'main_img' => 'test_path'
314 314 
         ];
315 315 
         $pageFilter = $this->getMockBuilder(\Page\Filter\PageFilter::class)
316 
-            ->setMethods(['getInputFilter', 'setData', 'isValid', 'getValues', 'select'])
316 
+            ->setMethods([ 'getInputFilter', 'setData', 'isValid', 'getValues', 'select' ])
317 317 
             ->disableOriginalConstructor()
318 318 
             ->getMock();
319 319 
         $pageFilter->expects(static::once())
 
@@ -329,7 +329,7 @@  discard block
 
 block discarded – undo
329 329 
             ->method('setData')
330 330 
             ->willReturnSelf();
331 331 
         $pageMapper = $this->getMockBuilder(\Page\Mapper\PageMapper::class)
332 
-            ->setMethods(['update', 'select', 'current'])
332 
+            ->setMethods([ 'update', 'select', 'current' ])
333 333 
             ->disableOriginalConstructor()
334 334 
             ->getMock();
335 335 
         $pageMapper->expects(static::exactly(2))
 
@@ -358,7 +358,7 @@  discard block
 
 block discarded – undo
358 358 
             ->disableOriginalConstructor()
359 359 
             ->getMock();
360 360 
         $pageMapper = $this->getMockBuilder(\Page\Mapper\PageMapper::class)
361 
-            ->setMethods(['select', 'current', 'delete'])
361 
+            ->setMethods([ 'select', 'current', 'delete' ])
362 362 
             ->disableOriginalConstructor()
363 363 
             ->getMock();
364 364 
         $pageMapper->expects(static::once())
 
@@ -391,7 +391,7 @@  discard block
 
 block discarded – undo
391 391 
             ->disableOriginalConstructor()
392 392 
             ->getMock();
393 393 
         $pageMapper = $this->getMockBuilder(\Page\Mapper\PageMapper::class)
394 
-            ->setMethods(['select', 'current'])
394 
+            ->setMethods([ 'select', 'current' ])
395 395 
             ->disableOriginalConstructor()
396 396 
             ->getMock();
397 397 
         $pageMapper->expects(static::once())
Please login to merge, or discard this patch.
data/phinx/migrations/20161003183351_page.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch 
@@ -6,19 +6,19 @@
 
 block discarded – undo
6 6 
 {
7 7 
     public function change()
8 8 
     {
9 
-        $this->table('page', ['id' => false, 'primary_key' => 'page_uuid'])
10 
-            ->addColumn('page_uuid', 'binary', ['limit' => 16])
9 
+        $this->table('page', [ 'id' => false, 'primary_key' => 'page_uuid' ])
10 
+            ->addColumn('page_uuid', 'binary', [ 'limit' => 16 ])
11 11 
             ->addColumn('page_id', 'text')
12 12 
             ->addColumn('title', 'text')
13 13 
             ->addColumn('body', 'text')
14 14 
             ->addColumn('description', 'text')
15 15 
             ->addColumn('slug', 'text')
16 
-            ->addColumn('main_img', 'text', ['null' => true])
17 
-            ->addColumn('has_layout', 'boolean', ['default' => true])
18 
-            ->addColumn('is_homepage', 'boolean', ['default' => false])
19 
-            ->addColumn('is_active', 'boolean', ['default' => false])
20 
-            ->addColumn('is_wysiwyg_editor', 'boolean', ['default' => false])
21 
-            ->addColumn('created_at', 'timestamp', ['default' => 'CURRENT_TIMESTAMP'])
16 
+            ->addColumn('main_img', 'text', [ 'null' => true ])
17 
+            ->addColumn('has_layout', 'boolean', [ 'default' => true ])
18 
+            ->addColumn('is_homepage', 'boolean', [ 'default' => false ])
19 
+            ->addColumn('is_active', 'boolean', [ 'default' => false ])
20 
+            ->addColumn('is_wysiwyg_editor', 'boolean', [ 'default' => false ])
21 
+            ->addColumn('created_at', 'timestamp', [ 'default' => 'CURRENT_TIMESTAMP' ])
22 22 
             ->create();
23 23 
     }
24 24 
 }
Please login to merge, or discard this patch.
data/phinx/migrations/20170214131105_menu.php 1 patch
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch 
@@ -7,23 +7,23 @@
 
 block discarded – undo
7 7
8 8 
     public function up()
9 9 
     {
10 
-        $this->table('menu', ['id' => false, 'primary_key' => 'menu_uuid'])
11 
-            ->addColumn('menu_uuid', 'binary', ['limit' => 16])
10 
+        $this->table('menu', [ 'id' => false, 'primary_key' => 'menu_uuid' ])
11 
+            ->addColumn('menu_uuid', 'binary', [ 'limit' => 16 ])
12 12 
             ->addColumn('menu_id', 'text')
13 
-            ->addColumn('parent_id', 'text', ['null' => true])
13 
+            ->addColumn('parent_id', 'text', [ 'null' => true ])
14 14 
             ->addColumn('title', 'text')
15 
-            ->addColumn('href', 'text', ['null' => true])
16 
-            ->addColumn('is_active', 'boolean', ['default' => false])
17 
-            ->addColumn('is_in_header', 'boolean', ['default' => true])
18 
-            ->addColumn('is_in_footer', 'boolean', ['default' => true])
19 
-            ->addColumn('is_in_side', 'boolean', ['default' => true])
20 
-            ->addColumn('order_no', 'integer', ['default' => 0])
21 
-            ->addColumn('created_at', 'datetime', ['null' => true])
15 
+            ->addColumn('href', 'text', [ 'null' => true ])
16 
+            ->addColumn('is_active', 'boolean', [ 'default' => false ])
17 
+            ->addColumn('is_in_header', 'boolean', [ 'default' => true ])
18 
+            ->addColumn('is_in_footer', 'boolean', [ 'default' => true ])
19 
+            ->addColumn('is_in_side', 'boolean', [ 'default' => true ])
20 
+            ->addColumn('order_no', 'integer', [ 'default' => 0 ])
21 
+            ->addColumn('created_at', 'datetime', [ 'null' => true ])
22 22 
             //->addForeignKey('parent_id', 'menu', 'id', ['delete' => 'NO_ACTION', 'update' => 'NO_ACTION']) # we can not add constraints
23 
-            ->addColumn('page_uuid', 'binary', ['limit' => 16, 'null' => true])
24 
-            ->addColumn('category_uuid', 'binary', ['limit' => 16, 'null' => true])
25 
-            ->addForeignKey('page_uuid', 'page', 'page_uuid', ['delete' => 'NO_ACTION', 'update' => 'NO_ACTION'])
26 
-            ->addForeignKey('category_uuid', 'category', 'category_uuid', ['delete' => 'NO_ACTION', 'update' => 'NO_ACTION'])
23 
+            ->addColumn('page_uuid', 'binary', [ 'limit' => 16, 'null' => true ])
24 
+            ->addColumn('category_uuid', 'binary', [ 'limit' => 16, 'null' => true ])
25 
+            ->addForeignKey('page_uuid', 'page', 'page_uuid', [ 'delete' => 'NO_ACTION', 'update' => 'NO_ACTION' ])
26 
+            ->addForeignKey('category_uuid', 'category', 'category_uuid', [ 'delete' => 'NO_ACTION', 'update' => 'NO_ACTION' ])
27 27 
             ->create();
28 28 
     }
29 29
Please login to merge, or discard this patch.
data/phinx/seeds/Menu.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -40,7 +40,7 @@  discard block
 
 block discarded – undo
40 40 
             'is_in_side'    => false,
41 41 
             'order_no'      => 2,
42 42 
             'page_uuid'     => null,
43 
-            'category_uuid' => $categoryVideo['category_uuid']
43 
+            'category_uuid' => $categoryVideo[ 'category_uuid' ]
44 44 
         ])->save();
45 45
46 46 
         // Insert events category in Menu
 
@@ -57,7 +57,7 @@  discard block
 
 block discarded – undo
57 57 
             'is_in_side'    => false,
58 58 
             'order_no'      => 3,
59 59 
             'page_uuid'     => null,
60 
-            'category_uuid' => $categoryEvent['category_uuid']
60 
+            'category_uuid' => $categoryEvent[ 'category_uuid' ]
61 61 
         ])->save();
62 62
63 63 
         // Insert link to external web site
Please login to merge, or discard this patch.
packages/Newsletter/tests/Service/NewsletterServiceTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -21,7 +21,7 @@  discard block
 
 block discarded – undo
21 21 
     public function testRegisterNewShouldReturnTrue()
22 22 
     {
23 23 
         $newsletterMapper = $this->getMockBuilder(\Newsletter\Mapper\NewsletterMapper::class)
24 
-            ->setMethods(['insert', 'select', 'current'])
24 
+            ->setMethods([ 'insert', 'select', 'current' ])
25 25 
             ->disableOriginalConstructor()
26 26 
             ->getMock();
27 27 
         $newsletterMapper->expects(static::once())
 
@@ -41,7 +41,7 @@  discard block
 
 block discarded – undo
41 41 
     public function testRegisterNewWithExistingEmailShouldDoNothingAndReturnNull()
42 42 
     {
43 43 
         $newsletterMapper = $this->getMockBuilder(\Newsletter\Mapper\NewsletterMapper::class)
44 
-            ->setMethods(['select', 'current'])
44 
+            ->setMethods([ 'select', 'current' ])
45 45 
             ->disableOriginalConstructor()
46 46 
             ->getMock();
47 47 
         $newsletterMapper->expects(static::once())
Please login to merge, or discard this patch.
packages/Newsletter/tests/Service/NewsletterServiceFactoryTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -10,7 +10,7 @@
 
 block discarded – undo
10 10 
             ->disableOriginalConstructor()
11 11 
             ->getMockForAbstractClass();
12 12 
         $container = $this->getMockBuilder(\Interop\Container\ContainerInterface::class)
13 
-            ->setMethods(['get'])
13 
+            ->setMethods([ 'get' ])
14 14 
             ->getMockForAbstractClass();
15 15 
         $container->expects(static::at(0))
16 16 
             ->method('get')
Please login to merge, or discard this patch.
packages/Newsletter/tests/Mapper/PageMapperFactoryTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -10,7 +10,7 @@
 
 block discarded – undo
10 10 
             ->disableOriginalConstructor()
11 11 
             ->getMockForAbstractClass();
12 12 
         $container = $this->getMockBuilder(\Interop\Container\ContainerInterface::class)
13 
-            ->setMethods(['get'])
13 
+            ->setMethods([ 'get' ])
14 14 
             ->getMockForAbstractClass();
15 15 
         $container->expects(static::at(0))
16 16 
             ->method('get')
Please login to merge, or discard this patch.