phpsrbija / unfinished

packages/Menu/src/Filter/MenuFilter.php

Spacing +13 added lines, -13 removed lines

@@ -20,17 +20,17 @@ 
 
     public function getInputFilter()
     {
-        if(!$this->inputFilter) {
+        if (!$this->inputFilter) {
             $inputFilter = new InputFilter();
 
             $inputFilter->add(
                 [
                 'name'       => 'title',
                 'required'   => true,
-                'filters'    => [['name' => 'StripTags'], ['name' => 'StringTrim']],
+                'filters'    => [ [ 'name' => 'StripTags' ], [ 'name' => 'StringTrim' ] ],
                 'validators' => [
-                    ['name' => 'NotEmpty'],
-                    ['name' => 'StringLength', 'options' => ['max' => '255']],
+                    [ 'name' => 'NotEmpty' ],
+                    [ 'name' => 'StringLength', 'options' => [ 'max' => '255' ] ],
                 ],
                 ]
             );
@@ -39,7 +39,7 @@ 
                 [
                 'name'     => 'href',
                 'required' => false,
-                'filters'  => [['name' => 'StripTags'], ['name' => 'StringTrim']]
+                'filters'  => [ [ 'name' => 'StripTags' ], [ 'name' => 'StringTrim' ] ]
                 ]
             );
 
@@ -47,9 +47,9 @@ 
                 [
                 'name'       => 'page_id',
                 'required'   => false,
-                'filters'    => [['name' => 'Null']],
+                'filters'    => [ [ 'name' => 'Null' ] ],
                 'validators' => [
-                    ['name' => RecordExists::class, 'options' => ['table' => 'page', 'field' => 'page_id', 'adapter' => $this->db]]
+                    [ 'name' => RecordExists::class, 'options' => [ 'table' => 'page', 'field' => 'page_id', 'adapter' => $this->db ] ]
                 ]
                 ]
             );
@@ -58,9 +58,9 @@ 
                 [
                 'name'       => 'category_id',
                 'required'   => false,
-                'filters'    => [['name' => 'Null']],
+                'filters'    => [ [ 'name' => 'Null' ] ],
                 'validators' => [
-                    ['name' => RecordExists::class, 'options' => ['table' => 'category', 'field' => 'category_id', 'adapter' => $this->db]]
+                    [ 'name' => RecordExists::class, 'options' => [ 'table' => 'category', 'field' => 'category_id', 'adapter' => $this->db ] ]
                 ]
                 ]
             );
@@ -69,7 +69,7 @@ 
                 [
                 'name'     => 'is_active',
                 'required' => false,
-                'filters'  => [['name' => 'Boolean']]
+                'filters'  => [ [ 'name' => 'Boolean' ] ]
                 ]
             );
 
@@ -77,7 +77,7 @@ 
                 [
                 'name'     => 'is_in_header',
                 'required' => false,
-                'filters'  => [['name' => 'Boolean']]
+                'filters'  => [ [ 'name' => 'Boolean' ] ]
                 ]
             );
 
@@ -85,7 +85,7 @@ 
                 [
                 'name'     => 'is_in_footer',
                 'required' => false,
-                'filters'  => [['name' => 'Boolean']]
+                'filters'  => [ [ 'name' => 'Boolean' ] ]
                 ]
             );
 
@@ -93,7 +93,7 @@ 
                 [
                 'name'     => 'is_in_side',
                 'required' => false,
-                'filters'  => [['name' => 'Boolean']]
+                'filters'  => [ [ 'name' => 'Boolean' ] ]
                 ]
             );
 

data/phinx/migrations/20161003183351_page.php

Spacing +8 added lines, -8 removed lines

@@ -6,19 +6,19 @@
 {
     public function change()
     {
-        $this->table('page', ['id' => false, 'primary_key' => 'page_uuid'])
-            ->addColumn('page_uuid', 'binary', ['limit' => 16])
+        $this->table('page', [ 'id' => false, 'primary_key' => 'page_uuid' ])
+            ->addColumn('page_uuid', 'binary', [ 'limit' => 16 ])
             ->addColumn('page_id', 'text')
             ->addColumn('title', 'text')
             ->addColumn('body', 'text')
             ->addColumn('description', 'text')
             ->addColumn('slug', 'text')
-            ->addColumn('main_img', 'text', ['null' => true])
-            ->addColumn('has_layout', 'boolean', ['default' => true])
-            ->addColumn('is_homepage', 'boolean', ['default' => false])
-            ->addColumn('is_active', 'boolean', ['default' => false])
-            ->addColumn('is_wysiwyg_editor', 'boolean', ['default' => false])
-            ->addColumn('created_at', 'timestamp', ['default' => 'CURRENT_TIMESTAMP'])
+            ->addColumn('main_img', 'text', [ 'null' => true ])
+            ->addColumn('has_layout', 'boolean', [ 'default' => true ])
+            ->addColumn('is_homepage', 'boolean', [ 'default' => false ])
+            ->addColumn('is_active', 'boolean', [ 'default' => false ])
+            ->addColumn('is_wysiwyg_editor', 'boolean', [ 'default' => false ])
+            ->addColumn('created_at', 'timestamp', [ 'default' => 'CURRENT_TIMESTAMP' ])
             ->create();
     }
 }

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/Page/src/Entity/Page.php

Doc Comments +10 added lines, -10 removed lines

@@ -30,7 +30,7 @@ 
     }
 
     /**
-     * @param mixed $is_wysiwyg_editor
+     * @param integer $is_wysiwyg_editor
      */
     public function setIsWysiwygEditor($is_wysiwyg_editor)
     {
@@ -46,7 +46,7 @@ 
     }
 
     /**
-     * @param mixed $slug
+     * @param string $slug
      */
     public function setSlug($slug)
     {
@@ -69,7 +69,7 @@ 
     }
 
     /**
-     * @param mixed $is_active
+     * @param integer $is_active
      */
     public function setIsActive($is_active)
     {
@@ -85,7 +85,7 @@ 
     }
 
     /**
-     * @param $page_uuid
+     * @param string $page_uuid
      */
     public function setPageUuid($page_uuid) 
     {
@@ -101,7 +101,7 @@ 
     }
 
     /**
-     * @param $page_id
+     * @param integer $page_id
      */
     public function setPageId($page_id)
     {
@@ -117,7 +117,7 @@ 
     }
 
     /**
-     * @param mixed $title
+     * @param string $title
      */
     public function setTitle($title)
     {
@@ -133,7 +133,7 @@ 
     }
 
     /**
-     * @param mixed $body
+     * @param string $body
      */
     public function setBody($body)
     {
@@ -155,7 +155,7 @@ 
     }
 
     /**
-     * @param mixed $description
+     * @param string $description
      */
     public function setDescription($description)
     {
@@ -171,7 +171,7 @@ 
     }
 
     /**
-     * @param mixed $main_img
+     * @param string $main_img
      */
     public function setMainImg($main_img)
     {
@@ -201,7 +201,7 @@ 
     }
 
     /**
-     * @param mixed $is_homepage
+     * @param integer $is_homepage
      */
     public function setIsHomepage($is_homepage)
     {

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/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);