phpsrbija / unfinished

packages/Page/src/Controller/PageController.php

Spacing +6 added lines, -6 removed lines

@@ -47,8 +47,8 @@ 
     public function index(): HtmlResponse
     {
         $params     = $this->request->getQueryParams();
-        $page       = isset($params['page']) ? $params['page'] : 1;
-        $limit      = isset($params['limit']) ? $params['limit'] : 15;
+        $page       = isset($params[ 'page' ]) ? $params[ 'page' ] : 1;
+        $limit      = isset($params[ 'limit' ]) ? $params[ 'limit' ] : 15;
         $pagination = $this->pageService->getPagination($page, $limit);
 
         return new HtmlResponse(
@@ -61,14 +61,14 @@ 
         );
     }
 
-    public function edit($errors = []): HtmlResponse
+    public function edit($errors = [ ]): HtmlResponse
     {
         $id   = $this->request->getAttribute('id');
         $page = $this->pageService->getPage($id);
 
-        if($this->request->getParsedBody()) {
+        if ($this->request->getParsedBody()) {
             $page = new \Page\Entity\Page();
-            $page->exchangeArray($this->request->getParsedBody() + (array)$page);
+            $page->exchangeArray($this->request->getParsedBody() + (array) $page);
         }
 
         return new HtmlResponse(
@@ -98,7 +98,7 @@ 
             return $this->response->withStatus(302)->withHeader('Location', $this->router->generateUri('admin.pages'));
         } catch (FilterException $fe) {
             return $this->edit($fe->getArrayMessages());
-        } catch(\Exception $e) {
+        } catch (\Exception $e) {
             throw $e;
         }
     }

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/Controller/PageFactoryTest.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/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/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')