phpsrbija / unfinished

packages/Page/tests/Controller/PageControllerTest.php

Spacing +8 added lines, -8 removed lines

@@ -7,7 +7,7 @@ 
     public function testIndexMethodShouldReturnHtmlResponse()
     {
         $template = $this->getMockBuilder(\Zend\Expressive\Template\TemplateRendererInterface::class)
-            ->setMethods(['render'])
+            ->setMethods([ 'render' ])
             ->getMockForAbstractClass();
         $template->expects(static::once())
             ->method('render')
@@ -31,7 +31,7 @@ 
     public function testEditMethodShouldReturnHtmlResponse()
     {
         $template = $this->getMockBuilder(\Zend\Expressive\Template\TemplateRendererInterface::class)
-            ->setMethods(['render'])
+            ->setMethods([ 'render' ])
             ->getMockForAbstractClass();
         $template->expects(static::once())
             ->method('render')
@@ -44,7 +44,7 @@ 
         $request = new \Zend\Diactoros\ServerRequest();
         $request = $request->withAttribute('action', 'edit');
         $request = $request->withAttribute('id', 1);
-        $request = $request->withParsedBody(['page' => 'test']);
+        $request = $request->withParsedBody([ 'page' => 'test' ]);
         $response = new \Zend\Diactoros\Response();
         $pageController = new \Page\Controller\PageController(
             $template,
@@ -68,7 +68,7 @@ 
             ->will(static::returnValue('http://unfinished.dev/admin'));
         $request = new \Zend\Diactoros\ServerRequest();
         $request = $request->withAttribute('action', 'save');
-        $request = $request->withParsedBody(['user' => 'test']);
+        $request = $request->withParsedBody([ 'user' => 'test' ]);
         $response = new \Zend\Diactoros\Response();
         $pageController = new \Page\Controller\PageController(
             $template,
@@ -94,7 +94,7 @@ 
         $request = new \Zend\Diactoros\ServerRequest();
         $request = $request->withAttribute('action', 'save');
         $request = $request->withAttribute('id', 2);
-        $request = $request->withParsedBody(['user' => 'test']);
+        $request = $request->withParsedBody([ 'user' => 'test' ]);
         $response = new \Zend\Diactoros\Response();
         $pageController = new \Page\Controller\PageController(
             $template,
@@ -117,13 +117,13 @@ 
             ->getMock();
         $pageService->expects(static::once())
             ->method('updatePage')
-            ->willThrowException(new \Std\FilterException(['test error']));
+            ->willThrowException(new \Std\FilterException([ 'test error' ]));
         $router = $this->getMockBuilder(\Zend\Expressive\Router\RouterInterface::class)
             ->getMockForAbstractClass();
         $request = new \Zend\Diactoros\ServerRequest();
         $request = $request->withAttribute('action', 'save');
         $request = $request->withAttribute('id', 2);
-        $request = $request->withParsedBody(['user' => 'test']);
+        $request = $request->withParsedBody([ 'user' => 'test' ]);
         $response = new \Zend\Diactoros\Response();
         $pageController = new \Page\Controller\PageController(
             $template,
@@ -163,7 +163,7 @@ 
         $request = new \Zend\Diactoros\ServerRequest();
         $request = $request->withAttribute('action', 'save');
         $request = $request->withAttribute('id', 2);
-        $request = $request->withParsedBody(['user' => 'test']);
+        $request = $request->withParsedBody([ 'user' => 'test' ]);
         $response = new \Zend\Diactoros\Response();
         $pageController = new \Page\Controller\PageController(
             $template,

packages/Page/tests/Service/PageServiceFactoryTest.php

Spacing +3 added lines, -3 removed lines

@@ -13,7 +13,7 @@ 
             ->disableOriginalConstructor()
             ->getMockForAbstractClass();
         $pageMapper = $this->getMockBuilder(\Page\Mapper\PageMapper::class)
-            ->setMethods(['getPaginationSelect', 'getAdapter'])
+            ->setMethods([ 'getPaginationSelect', 'getAdapter' ])
             ->disableOriginalConstructor()
             ->getMockForAbstractClass();
         $pageMapper->expects(static::once())
@@ -27,11 +27,11 @@ 
         $pageFilter = $this->getMockBuilder(\Page\Filter\PageFilter::class)
             ->getMockForAbstractClass();
         $container = $this->getMockBuilder(\Interop\Container\ContainerInterface::class)
-            ->setMethods(['get'])
+            ->setMethods([ 'get' ])
             ->getMockForAbstractClass();
         $container->expects(static::at(0))
             ->method('get')
-            ->will(static::returnValue(['upload' => ['public_path' => 'test', 'non_public_path' => 'test']]));
+            ->will(static::returnValue([ 'upload' => [ 'public_path' => 'test', 'non_public_path' => 'test' ] ]));
         $container->expects(static::at(1))
             ->method('get')
             ->will(static::returnValue($pageMapper));

packages/Page/tests/Service/PageServiceTest.php

Spacing +19 added lines, -19 removed lines

@@ -29,7 +29,7 @@ 
             ->disableOriginalConstructor()
             ->getMock();
         $pageMapper = $this->getMockBuilder(\Page\Mapper\PageMapper::class)
-            ->setMethods(['select', 'current'])
+            ->setMethods([ 'select', 'current' ])
             ->disableOriginalConstructor()
             ->getMock();
         $pageMapper->expects(static::once())
@@ -55,7 +55,7 @@ 
             ->disableOriginalConstructor()
             ->getMock();
         $pageMapper = $this->getMockBuilder(\Page\Mapper\PageMapper::class)
-            ->setMethods(['getActivePage', 'current'])
+            ->setMethods([ 'getActivePage', 'current' ])
             ->disableOriginalConstructor()
             ->getMock();
         $pageMapper->expects(static::once())
@@ -81,7 +81,7 @@ 
             ->disableOriginalConstructor()
             ->getMock();
         $pageMapper = $this->getMockBuilder(\Page\Mapper\PageMapper::class)
-            ->setMethods(['select', 'current'])
+            ->setMethods([ 'select', 'current' ])
             ->disableOriginalConstructor()
             ->getMock();
         $pageMapper->expects(static::once())
@@ -107,7 +107,7 @@ 
             'is_homepage' => 1
         ];
         $pageFilter = $this->getMockBuilder(\Page\Filter\PageFilter::class)
-            ->setMethods(['getInputFilter', 'setData', 'isValid', 'getValues'])
+            ->setMethods([ 'getInputFilter', 'setData', 'isValid', 'getValues' ])
             ->disableOriginalConstructor()
             ->getMock();
         $pageFilter->expects(static::once())
@@ -123,7 +123,7 @@ 
             ->method('setData')
             ->willReturnSelf();
         $pageMapper = $this->getMockBuilder(\Page\Mapper\PageMapper::class)
-            ->setMethods(['update', 'insert'])
+            ->setMethods([ 'update', 'insert' ])
             ->disableOriginalConstructor()
             ->getMock();
         $pageMapper->expects(static::once())
@@ -151,7 +151,7 @@ 
         ];
 
         $pageFilter = $this->getMockBuilder(\Page\Filter\PageFilter::class)
-            ->setMethods(['getInputFilter', 'setData', 'isValid', 'getMessages'])
+            ->setMethods([ 'getInputFilter', 'setData', 'isValid', 'getMessages' ])
             ->disableOriginalConstructor()
             ->getMock();
         $pageFilter->expects(static::once())
@@ -162,12 +162,12 @@ 
             ->willReturn(false);
         $pageFilter->expects(static::once())
             ->method('getMessages')
-            ->willReturn(['test error']);
+            ->willReturn([ 'test error' ]);
         $pageFilter->expects(static::once())
             ->method('setData')
             ->willReturnSelf();
         $pageMapper = $this->getMockBuilder(\Page\Mapper\PageMapper::class)
-            ->setMethods(['update', 'insert'])
+            ->setMethods([ 'update', 'insert' ])
             ->disableOriginalConstructor()
             ->getMock();
         $paginator = $this->getMockBuilder(\Zend\Paginator\Paginator::class)
@@ -191,11 +191,11 @@ 
             'is_homepage' => 1
         ];
         $pageFilter = $this->getMockBuilder(\Page\Filter\PageFilter::class)
-            ->setMethods(['select'])
+            ->setMethods([ 'select' ])
             ->disableOriginalConstructor()
             ->getMock();
         $pageMapper = $this->getMockBuilder(\Page\Mapper\PageMapper::class)
-            ->setMethods(['select', 'current'])
+            ->setMethods([ 'select', 'current' ])
             ->disableOriginalConstructor()
             ->getMock();
         $pageMapper->expects(static::once())
@@ -225,7 +225,7 @@ 
             'is_homepage' => 1
         ];
         $pageFilter = $this->getMockBuilder(\Page\Filter\PageFilter::class)
-            ->setMethods(['getInputFilter', 'setData', 'isValid', 'getMessages'])
+            ->setMethods([ 'getInputFilter', 'setData', 'isValid', 'getMessages' ])
             ->disableOriginalConstructor()
             ->getMock();
         $pageFilter->expects(static::once())
@@ -239,9 +239,9 @@ 
             ->willReturn(false);
         $pageFilter->expects(static::once())
             ->method('getMessages')
-            ->willReturn(['test error']);
+            ->willReturn([ 'test error' ]);
         $pageMapper = $this->getMockBuilder(\Page\Mapper\PageMapper::class)
-            ->setMethods(['select', 'current'])
+            ->setMethods([ 'select', 'current' ])
             ->disableOriginalConstructor()
             ->getMock();
         $pageMapper->expects(static::once())
@@ -267,7 +267,7 @@ 
             'is_homepage' => 1
         ];
         $pageFilter = $this->getMockBuilder(\Page\Filter\PageFilter::class)
-            ->setMethods(['getInputFilter', 'setData', 'isValid', 'getValues', 'select'])
+            ->setMethods([ 'getInputFilter', 'setData', 'isValid', 'getValues', 'select' ])
             ->disableOriginalConstructor()
             ->getMock();
         $pageFilter->expects(static::once())
@@ -283,7 +283,7 @@ 
             ->method('setData')
             ->willReturnSelf();
         $pageMapper = $this->getMockBuilder(\Page\Mapper\PageMapper::class)
-            ->setMethods(['update', 'select', 'current'])
+            ->setMethods([ 'update', 'select', 'current' ])
             ->disableOriginalConstructor()
             ->getMock();
         $pageMapper->expects(static::exactly(2))
@@ -313,7 +313,7 @@ 
             'main_img' => 'test_path'
         ];
         $pageFilter = $this->getMockBuilder(\Page\Filter\PageFilter::class)
-            ->setMethods(['getInputFilter', 'setData', 'isValid', 'getValues', 'select'])
+            ->setMethods([ 'getInputFilter', 'setData', 'isValid', 'getValues', 'select' ])
             ->disableOriginalConstructor()
             ->getMock();
         $pageFilter->expects(static::once())
@@ -329,7 +329,7 @@ 
             ->method('setData')
             ->willReturnSelf();
         $pageMapper = $this->getMockBuilder(\Page\Mapper\PageMapper::class)
-            ->setMethods(['update', 'select', 'current'])
+            ->setMethods([ 'update', 'select', 'current' ])
             ->disableOriginalConstructor()
             ->getMock();
         $pageMapper->expects(static::exactly(2))
@@ -358,7 +358,7 @@ 
             ->disableOriginalConstructor()
             ->getMock();
         $pageMapper = $this->getMockBuilder(\Page\Mapper\PageMapper::class)
-            ->setMethods(['select', 'current', 'delete'])
+            ->setMethods([ 'select', 'current', 'delete' ])
             ->disableOriginalConstructor()
             ->getMock();
         $pageMapper->expects(static::once())
@@ -391,7 +391,7 @@ 
             ->disableOriginalConstructor()
             ->getMock();
         $pageMapper = $this->getMockBuilder(\Page\Mapper\PageMapper::class)
-            ->setMethods(['select', 'current'])
+            ->setMethods([ 'select', 'current' ])
             ->disableOriginalConstructor()
             ->getMock();
         $pageMapper->expects(static::once())

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/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
         ];
     }
 }

Doc Comments +10 added lines, -10 removed lines

@@ -31,7 +31,7 @@ 
     }
 
     /**
-     * @param mixed $is_wysiwyg_editor
+     * @param integer $is_wysiwyg_editor
      */
     public function setIsWysiwygEditor($is_wysiwyg_editor)
     {
@@ -47,7 +47,7 @@ 
     }
 
     /**
-     * @param mixed $slug
+     * @param string $slug
      */
     public function setSlug($slug)
     {
@@ -70,7 +70,7 @@ 
     }
 
     /**
-     * @param mixed $is_active
+     * @param integer $is_active
      */
     public function setIsActive($is_active)
     {
@@ -86,7 +86,7 @@ 
     }
 
     /**
-     * @param $page_uuid
+     * @param string $page_uuid
      */
     public function setPageUuid($page_uuid)
     {
@@ -102,7 +102,7 @@ 
     }
 
     /**
-     * @param $page_id
+     * @param integer $page_id
      */
     public function setPageId($page_id)
     {
@@ -118,7 +118,7 @@ 
     }
 
     /**
-     * @param mixed $title
+     * @param string $title
      */
     public function setTitle($title)
     {
@@ -134,7 +134,7 @@ 
     }
 
     /**
-     * @param mixed $body
+     * @param string $body
      */
     public function setBody($body)
     {
@@ -156,7 +156,7 @@ 
     }
 
     /**
-     * @param mixed $description
+     * @param string $description
      */
     public function setDescription($description)
     {
@@ -172,7 +172,7 @@ 
     }
 
     /**
-     * @param mixed $main_img
+     * @param string $main_img
      */
     public function setMainImg($main_img)
     {
@@ -202,7 +202,7 @@ 
     }
 
     /**
-     * @param mixed $is_homepage
+     * @param integer $is_homepage
      */
     public function setIsHomepage($is_homepage)
     {

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

Spacing +3 added lines, -3 removed lines

@@ -16,13 +16,13 @@
     public function testForSelectShouldReturnArray()
     {
         $pageService = $this->getMockBuilder(\Page\Service\PageService::class)
-            ->setMethods(['getForSelect'])
+            ->setMethods([ 'getForSelect' ])
             ->disableOriginalConstructor()
             ->getMockForAbstractClass();
         $pageService->expects(static::once())
             ->method('getForSelect')
-            ->willReturn([]);
+            ->willReturn([ ]);
         $pageHelper = new \Page\View\Helper\PageHelper($pageService);
-        static::assertSame([], $pageHelper->forSelect());
+        static::assertSame([ ], $pageHelper->forSelect());
     }
 }

packages/Page/tests/Mapper/PageMapperTest.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
             ->method('createStatement')
             ->willReturn($statementMock);
         $adapterMock = $this->getMockBuilder(\Zend\Db\Adapter\Adapter::class)
-            ->setConstructorArgs([$driverMock])
+            ->setConstructorArgs([ $driverMock ])
             ->getMockForAbstractClass();
         $resultSetMock = $this->getMockBuilder(\Zend\Db\ResultSet\HydratingResultSet::class)
             ->getMockForAbstractClass();