phpsrbija / unfinished

data/phinx/migrations/20170214131105_menu.php

Spacing +14 added lines, -14 removed lines

@@ -7,23 +7,23 @@
 
     public function up()
     {
-        $this->table('menu', ['id' => false, 'primary_key' => 'menu_uuid'])
-            ->addColumn('menu_uuid', 'binary', ['limit' => 16])
+        $this->table('menu', [ 'id' => false, 'primary_key' => 'menu_uuid' ])
+            ->addColumn('menu_uuid', 'binary', [ 'limit' => 16 ])
             ->addColumn('menu_id', 'text')
-            ->addColumn('parent_id', 'text', ['null' => true])
+            ->addColumn('parent_id', 'text', [ 'null' => true ])
             ->addColumn('title', 'text')
-            ->addColumn('href', 'text', ['null' => true])
-            ->addColumn('is_active', 'boolean', ['default' => false])
-            ->addColumn('is_in_header', 'boolean', ['default' => true])
-            ->addColumn('is_in_footer', 'boolean', ['default' => true])
-            ->addColumn('is_in_side', 'boolean', ['default' => true])
-            ->addColumn('order_no', 'integer', ['default' => 0])
-            ->addColumn('created_at', 'datetime', ['null' => true])
+            ->addColumn('href', 'text', [ 'null' => true ])
+            ->addColumn('is_active', 'boolean', [ 'default' => false ])
+            ->addColumn('is_in_header', 'boolean', [ 'default' => true ])
+            ->addColumn('is_in_footer', 'boolean', [ 'default' => true ])
+            ->addColumn('is_in_side', 'boolean', [ 'default' => true ])
+            ->addColumn('order_no', 'integer', [ 'default' => 0 ])
+            ->addColumn('created_at', 'datetime', [ 'null' => true ])
             //->addForeignKey('parent_id', 'menu', 'id', ['delete' => 'NO_ACTION', 'update' => 'NO_ACTION']) # we can not add constraints
-            ->addColumn('page_uuid', 'binary', ['limit' => 16, 'null' => true])
-            ->addColumn('category_uuid', 'binary', ['limit' => 16, 'null' => true])
-            ->addForeignKey('page_uuid', 'page', 'page_uuid', ['delete' => 'NO_ACTION', 'update' => 'NO_ACTION'])
-            ->addForeignKey('category_uuid', 'category', 'category_uuid', ['delete' => 'NO_ACTION', 'update' => 'NO_ACTION'])
+            ->addColumn('page_uuid', 'binary', [ 'limit' => 16, 'null' => true ])
+            ->addColumn('category_uuid', 'binary', [ 'limit' => 16, 'null' => true ])
+            ->addForeignKey('page_uuid', 'page', 'page_uuid', [ 'delete' => 'NO_ACTION', 'update' => 'NO_ACTION' ])
+            ->addForeignKey('category_uuid', 'category', 'category_uuid', [ 'delete' => 'NO_ACTION', 'update' => 'NO_ACTION' ])
             ->create();
     }
 

data/phinx/seeds/Menu.php

Spacing +2 added lines, -2 removed lines

@@ -40,7 +40,7 @@ 
             'is_in_side'    => false,
             'order_no'      => 2,
             'page_uuid'     => null,
-            'category_uuid' => $categoryVideo['category_uuid']
+            'category_uuid' => $categoryVideo[ 'category_uuid' ]
         ])->save();
 
         // Insert events category in Menu
@@ -57,7 +57,7 @@ 
             'is_in_side'    => false,
             'order_no'      => 3,
             'page_uuid'     => null,
-            'category_uuid' => $categoryEvent['category_uuid']
+            'category_uuid' => $categoryEvent[ 'category_uuid' ]
         ])->save();
 
         // Insert link to external web site

packages/Newsletter/tests/Service/NewsletterServiceTest.php

Spacing +2 added lines, -2 removed lines

@@ -21,7 +21,7 @@ 
     public function testRegisterNewShouldReturnTrue()
     {
         $newsletterMapper = $this->getMockBuilder(\Newsletter\Mapper\NewsletterMapper::class)
-            ->setMethods(['insert', 'select', 'current'])
+            ->setMethods([ 'insert', 'select', 'current' ])
             ->disableOriginalConstructor()
             ->getMock();
         $newsletterMapper->expects(static::once())
@@ -41,7 +41,7 @@ 
     public function testRegisterNewWithExistingEmailShouldDoNothingAndReturnNull()
     {
         $newsletterMapper = $this->getMockBuilder(\Newsletter\Mapper\NewsletterMapper::class)
-            ->setMethods(['select', 'current'])
+            ->setMethods([ 'select', 'current' ])
             ->disableOriginalConstructor()
             ->getMock();
         $newsletterMapper->expects(static::once())

packages/Newsletter/tests/Service/NewsletterServiceFactoryTest.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
             ->disableOriginalConstructor()
             ->getMockForAbstractClass();
         $container = $this->getMockBuilder(\Interop\Container\ContainerInterface::class)
-            ->setMethods(['get'])
+            ->setMethods([ 'get' ])
             ->getMockForAbstractClass();
         $container->expects(static::at(0))
             ->method('get')

packages/Newsletter/tests/Mapper/PageMapperFactoryTest.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
             ->disableOriginalConstructor()
             ->getMockForAbstractClass();
         $container = $this->getMockBuilder(\Interop\Container\ContainerInterface::class)
-            ->setMethods(['get'])
+            ->setMethods([ 'get' ])
             ->getMockForAbstractClass();
         $container->expects(static::at(0))
             ->method('get')

packages/Newsletter/tests/Web/Action/HelperPostActionTest.php

Spacing +3 added lines, -3 removed lines

@@ -8,7 +8,7 @@ 
     {
         $newsletterService = $this->getMockBuilder(\Newsletter\Service\NewsletterService::class)
             ->disableOriginalConstructor()
-            ->setMethods(['registerNew'])
+            ->setMethods([ 'registerNew' ])
             ->getMockForAbstractClass();
         $newsletterService->expects(static::once())
             ->method('registerNew')
@@ -24,14 +24,14 @@ 
     public function testHandlePostActionShouldReturnJsonResponseWithSuccess()
     {
         $newsletterService = $this->getMockBuilder(\Newsletter\Service\NewsletterService::class)
-            ->setMethods(['registerNew'])
+            ->setMethods([ 'registerNew' ])
             ->disableOriginalConstructor()
             ->getMockForAbstractClass();
         $request = $this->getMockBuilder(\Psr\Http\Message\ServerRequestInterface::class)
             ->getMockForAbstractClass();
         $request->expects(static::once())
             ->method('getParsedBody')
-            ->willReturn(['email' => 'test@example.com']);
+            ->willReturn([ 'email' => 'test@example.com' ]);
         $response = $this->getMockBuilder(\Psr\Http\Message\ResponseInterface::class)
             ->getMockForAbstractClass();
         $newsletterAction = new \Newsletter\Web\Action\HandlePostAction($newsletterService);

packages/Newsletter/tests/Web/Action/HandlePostActionFactoryTest.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
             ->disableOriginalConstructor()
             ->getMockForAbstractClass();
         $container = $this->getMockBuilder(\Interop\Container\ContainerInterface::class)
-            ->setMethods(['get'])
+            ->setMethods([ 'get' ])
             ->getMockForAbstractClass();
         $container->expects(static::once())
             ->method('get')

packages/Page/src/Entity/Page.php

Spacing +14 added lines, -14 removed lines

@@ -235,10 +235,10 @@ 
      *
      * @param array $data
      */
-    public function exchangeArray($data = [])
+    public function exchangeArray($data = [ ])
     {
         foreach (array_keys(get_object_vars($this)) as $property) {
-            $this->{$property} = isset($data[$property]) ? $data[$property]
+            $this->{$property} = isset($data[ $property ]) ? $data[ $property ]
                 : null;
         }
     }
@@ -249,18 +249,18 @@ 
     public function getArrayCopy()
     {
         return [
-            'page_uuid'         => (binary)$this->page_uuid,
-            'page_id'           => (string)$this->page_id,
-            'title'             => (string)$this->title,
-            'body'              => (string)$this->body,
-            'description'       => (string)$this->description,
-            'main_img'          => (string)$this->main_img,
-            'has_layout'        => (boolean)$this->has_layout,
-            'is_homepage'       => (boolean)$this->is_homepage,
-            'is_active'         => (boolean)$this->is_active,
-            'is_wysiwyg_editor' => (boolean)$this->is_wysiwyg_editor,
-            'created_at'        => (string)$this->created_at,
-            'slug'              => (string)$this->slug
+            'page_uuid'         => (binary) $this->page_uuid,
+            'page_id'           => (string) $this->page_id,
+            'title'             => (string) $this->title,
+            'body'              => (string) $this->body,
+            'description'       => (string) $this->description,
+            'main_img'          => (string) $this->main_img,
+            'has_layout'        => (boolean) $this->has_layout,
+            'is_homepage'       => (boolean) $this->is_homepage,
+            'is_active'         => (boolean) $this->is_active,
+            'is_wysiwyg_editor' => (boolean) $this->is_wysiwyg_editor,
+            'created_at'        => (string) $this->created_at,
+            'slug'              => (string) $this->slug
         ];
     }
 }

packages/Page/tests/View/Helper/PageHelperFactoryTest.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
             ->disableOriginalConstructor()
             ->getMockForAbstractClass();
         $container = $this->getMockBuilder(\Interop\Container\ContainerInterface::class)
-            ->setMethods(['get'])
+            ->setMethods([ 'get' ])
             ->getMockForAbstractClass();
         $container->expects(static::at(0))
             ->method('get')