phpsrbija / unfinished

packages/Article/tests/View/Helper/EventHelperTest.php

Spacing +3 added lines, -3 removed lines

@@ -16,13 +16,13 @@
     public function testAllShouldReturnArray()
     {
         $eventService = $this->getMockBuilder('Article\Service\EventService')
-            ->setMethods(['fetchLatest'])
+            ->setMethods([ 'fetchLatest' ])
             ->disableOriginalConstructor()
             ->getMockForAbstractClass();
         $eventService->expects(static::once())
             ->method('fetchLatest')
-            ->willReturn([]);
+            ->willReturn([ ]);
         $eventHelper = new \Article\View\Helper\EventHelper($eventService);
-        static::assertSame([], $eventHelper->getLatest());
+        static::assertSame([ ], $eventHelper->getLatest());
     }
 }

packages/Article/tests/Factory/View/Helper/EventHelperFactoryTest.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/Admin/tests/Controller/UserControllerTest.php

Indentation +5 added lines, -5 removed lines

@@ -147,7 +147,7 @@ 
         /**
 * 
          *
- * @var \Zend\Diactoros\Response $returnedResponse 
+         * @var \Zend\Diactoros\Response $returnedResponse 
 */
         $returnedResponse = $userController($request, $response);
         static::assertSame(302, $returnedResponse->getStatusCode());
@@ -186,7 +186,7 @@ 
         /**
 * 
          *
- * @var \Zend\Diactoros\Response $returnedResponse 
+         * @var \Zend\Diactoros\Response $returnedResponse 
 */
         $returnedResponse = $userController($request, $response);
         static::assertSame(302, $returnedResponse->getStatusCode());
@@ -228,7 +228,7 @@ 
         /**
 * 
          *
- * @var \Zend\Diactoros\Response $returnedResponse 
+         * @var \Zend\Diactoros\Response $returnedResponse 
 */
         $returnedResponse = $userController($request, $response);
         static::assertSame(200, $returnedResponse->getStatusCode());
@@ -303,7 +303,7 @@ 
         /**
 * 
          *
- * @var \Zend\Diactoros\Response $returnedResponse 
+         * @var \Zend\Diactoros\Response $returnedResponse 
 */
         $returnedResponse = $userController($request, $response);
         static::assertSame(302, $returnedResponse->getStatusCode());
@@ -345,7 +345,7 @@ 
         /**
 * 
          *
- * @var \Zend\Diactoros\Response $returnedResponse 
+         * @var \Zend\Diactoros\Response $returnedResponse 
 */
         $returnedResponse = $userController($request, $response);
         static::assertSame(302, $returnedResponse->getStatusCode());

Spacing +13 added lines, -13 removed lines

@@ -13,7 +13,7 @@ 
         $sessionStorage = new \Zend\Session\Storage\ArrayStorage();
         $sessionStorage->user = $user;
         $template = $this->getMockBuilder(\Zend\Expressive\Template\TemplateRendererInterface::class)
-            ->setMethods(['render'])
+            ->setMethods([ 'render' ])
             ->getMockForAbstractClass();
         $template->expects(static::once())
             ->method('render')
@@ -23,7 +23,7 @@ 
             ->getMock();
         $sessionManager = $this->getMockBuilder(\Zend\Session\SessionManager::class)
             ->disableOriginalConstructor()
-            ->setMethods(['getStorage'])
+            ->setMethods([ 'getStorage' ])
             ->getMock();
         $sessionManager->expects(static::any())
             ->method('getStorage')
@@ -49,14 +49,14 @@ 
         $sessionStorage = new \Zend\Session\Storage\ArrayStorage();
         $sessionStorage->user = $user;
         $template = $this->getMockBuilder(\Zend\Expressive\Template\TemplateRendererInterface::class)
-            ->setMethods(['render'])
+            ->setMethods([ 'render' ])
             ->getMockForAbstractClass();
         $template->expects(static::once())
             ->method('render')
             ->will(static::returnValue('test'));
         $adminUserService = $this->getMockBuilder(\Admin\Service\AdminUserService::class)
             ->disableOriginalConstructor()
-            ->setMethods(['getUser'])
+            ->setMethods([ 'getUser' ])
             ->getMock();
         $adminUserService->expects(static::once())
             ->method('getUser')
@@ -86,14 +86,14 @@ 
         $sessionStorage = new \Zend\Session\Storage\ArrayStorage();
         $sessionStorage->user = $user;
         $template = $this->getMockBuilder(\Zend\Expressive\Template\TemplateRendererInterface::class)
-            ->setMethods(['render'])
+            ->setMethods([ 'render' ])
             ->getMockForAbstractClass();
         $template->expects(static::once())
             ->method('render')
             ->will(static::returnValue('test'));
         $adminUserService = $this->getMockBuilder(\Admin\Service\AdminUserService::class)
             ->disableOriginalConstructor()
-            ->setMethods(['getUser'])
+            ->setMethods([ 'getUser' ])
             ->getMock();
         $adminUserService->expects(static::once())
             ->method('getUser')
@@ -106,7 +106,7 @@ 
         $request = new \Zend\Diactoros\ServerRequest();
         $request = $request->withAttribute('action', 'edit');
         $request = $request->withAttribute('id', 1);
-        $request = $request->withParsedBody(['user' => 'test']);
+        $request = $request->withParsedBody([ 'user' => 'test' ]);
         $response = new \Zend\Diactoros\Response();
         $userController = new \Admin\Controller\UserController(
             $template,
@@ -138,7 +138,7 @@ 
             ->will(static::returnValue('http://unfinished.dev/admin/users'));
         $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();
         $userController = new \Admin\Controller\UserController(
             $template,
@@ -177,7 +177,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();
         $userController = new \Admin\Controller\UserController(
             $template,
@@ -215,11 +215,11 @@ 
             ->getMockForAbstractClass();
         $router->expects(static::at(0))
             ->method('generateUri')
-            ->willThrowException(new \Std\FilterException(['test error']));
+            ->willThrowException(new \Std\FilterException([ 'test error' ]));
         $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();
         $userController = new \Admin\Controller\UserController(
             $template,
@@ -262,7 +262,7 @@ 
             ->willThrowException(new \Exception('test error'));
         $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();
         $userController = new \Admin\Controller\UserController(
             $template,
@@ -320,7 +320,7 @@ 
         $template = $this->getMockBuilder(\Zend\Expressive\Template\TemplateRendererInterface::class)
             ->getMockForAbstractClass();
         $adminUserService = $this->getMockBuilder(\Admin\Service\AdminUserService::class)
-            ->setMethods(['delete'])
+            ->setMethods([ 'delete' ])
             ->disableOriginalConstructor()
             ->getMock();
         $adminUserService->expects(static::once())

packages/Category/src/Controller/IndexController.php

Spacing +13 added lines, -13 removed lines

@@ -60,12 +60,12 @@ 
     public function index() : \Psr\Http\Message\ResponseInterface
     {
         $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;
 
         $categories = $this->categoryService->getPagination($page, $limit);
 
-        return new HtmlResponse($this->template->render('category::index/index', ['list' => $categories, 'layout' => 'layout/admin']));
+        return new HtmlResponse($this->template->render('category::index/index', [ 'list' => $categories, 'layout' => 'layout/admin' ]));
     }
 
     /**
@@ -73,13 +73,13 @@ 
      *
      * @return \Psr\Http\Message\ResponseInterface
      */
-    public function edit($errors = []): \Psr\Http\Message\ResponseInterface
+    public function edit($errors = [ ]): \Psr\Http\Message\ResponseInterface
     {
         $id       = $this->request->getAttribute('id');
         $category = $this->categoryService->getCategory($id);
 
-        if($this->request->getParsedBody()) {
-            $category              = (object)($this->request->getParsedBody() + (array)$category);
+        if ($this->request->getParsedBody()) {
+            $category              = (object) ($this->request->getParsedBody() + (array) $category);
             $category->category_id = $id;
         }
 
@@ -96,37 +96,37 @@ 
 
     public function save()
     {
-        try{
+        try {
             $id   = $this->request->getAttribute('id');
             $data = $this->request->getParsedBody();
             $data += (new Request())->getFiles()->toArray();
 
-            if($id) {
+            if ($id) {
                 $this->categoryService->updateCategory($data, $id);
             }
-            else{
+            else {
                 $this->categoryService->createCategory($data);
             }
 
             return $this->response->withStatus(302)->withHeader('Location', $this->router->generateUri('admin.categories'));
         }
-        catch(FilterException $fe){
+        catch (FilterException $fe) {
             return $this->edit($fe->getArrayMessages());
         }
-        catch(\Exception $e){
+        catch (\Exception $e) {
             throw $e;
         }
     }
 
     public function delete()
     {
-        try{
+        try {
             $id = $this->request->getAttribute('id');
             $this->categoryService->delete($id);
 
             return $this->response->withStatus(302)->withHeader('Location', $this->router->generateUri('admin.categories'));
         }
-        catch(\Exception $e){
+        catch (\Exception $e) {
             return $this->response->withStatus(302)->withHeader('Location', $this->router->generateUri('admin.categories'));
         }
     }

Braces +4 added lines, -8 removed lines

@@ -103,17 +103,14 @@ 
 
             if($id) {
                 $this->categoryService->updateCategory($data, $id);
-            }
-            else{
+            } else{
                 $this->categoryService->createCategory($data);
             }
 
             return $this->response->withStatus(302)->withHeader('Location', $this->router->generateUri('admin.categories'));
-        }
-        catch(FilterException $fe){
+        } catch(FilterException $fe){
             return $this->edit($fe->getArrayMessages());
-        }
-        catch(\Exception $e){
+        } catch(\Exception $e){
             throw $e;
         }
     }
@@ -125,8 +122,7 @@ 
             $this->categoryService->delete($id);
 
             return $this->response->withStatus(302)->withHeader('Location', $this->router->generateUri('admin.categories'));
-        }
-        catch(\Exception $e){
+        } catch(\Exception $e){
             return $this->response->withStatus(302)->withHeader('Location', $this->router->generateUri('admin.categories'));
         }
     }

packages/Category/src/Filter/CategoryFilter.php

Spacing +15 added lines, -15 removed lines

@@ -12,17 +12,17 @@ 
 
     public function getInputFilter()
     {
-        if(!$this->inputFilter) {
+        if (!$this->inputFilter) {
             $inputFilter = new InputFilter();
 
             $inputFilter->add(
                 [
                 'name'       => 'name',
                 'required'   => true,
-                'filters'    => [['name' => 'StringTrim']],
+                'filters'    => [ [ 'name' => 'StringTrim' ] ],
                 'validators' => [
-                    ['name' => 'NotEmpty'],
-                    ['name' => 'StringLength', 'options' => ['min' => 2, 'max' => 500]],
+                    [ 'name' => 'NotEmpty' ],
+                    [ 'name' => 'StringLength', 'options' => [ 'min' => 2, 'max' => 500 ] ],
                 ],
                 ]
             );
@@ -31,10 +31,10 @@ 
                 [
                 'name'       => 'slug',
                 'required'   => true,
-                'filters'    => [['name' => 'StringTrim', 'options' => ['charlist' => '/']]],
+                'filters'    => [ [ 'name' => 'StringTrim', 'options' => [ 'charlist' => '/' ] ] ],
                 'validators' => [
-                    ['name' => 'NotEmpty'],
-                    ['name' => 'StringLength', 'options' => ['min' => 2, 'max' => 100]],
+                    [ 'name' => 'NotEmpty' ],
+                    [ 'name' => 'StringLength', 'options' => [ 'min' => 2, 'max' => 100 ] ],
                 ],
                 ]
             );
@@ -43,10 +43,10 @@ 
                 [
                 'name'       => 'title',
                 'required'   => false,
-                'filters'    => [['name' => 'StringTrim']],
+                'filters'    => [ [ 'name' => 'StringTrim' ] ],
                 'validators' => [
-                    ['name' => 'NotEmpty'],
-                    ['name' => 'StringLength', 'options' => ['min' => 2, 'max' => 100]],
+                    [ 'name' => 'NotEmpty' ],
+                    [ 'name' => 'StringLength', 'options' => [ 'min' => 2, 'max' => 100 ] ],
                 ],
                 ]
             );
@@ -55,10 +55,10 @@ 
                 [
                 'name'       => 'description',
                 'required'   => false,
-                'filters'    => [['name' => 'StringTrim']],
+                'filters'    => [ [ 'name' => 'StringTrim' ] ],
                 'validators' => [
-                    ['name' => 'NotEmpty'],
-                    ['name' => 'StringLength', 'options' => ['min' => 2, 'max' => 1000]],
+                    [ 'name' => 'NotEmpty' ],
+                    [ 'name' => 'StringLength', 'options' => [ 'min' => 2, 'max' => 1000 ] ],
                 ],
                 ]
             );
@@ -67,7 +67,7 @@ 
                 [
                 'name'     => 'is_in_homepage',
                 'required' => false,
-                'filters'  => [['name' => 'Boolean']],
+                'filters'  => [ [ 'name' => 'Boolean' ] ],
                 ]
             );
 
@@ -75,7 +75,7 @@ 
                 [
                 'name'     => 'is_in_category_list',
                 'required' => false,
-                'filters'  => [['name' => 'Boolean']],
+                'filters'  => [ [ 'name' => 'Boolean' ] ],
                 ]
             );
 

packages/Page/src/Mapper/PageMapper.php

Spacing +1 added lines, -1 removed lines

@@ -35,6 +35,6 @@
 
     public function getActivePage($urlSlug)
     {
-        return $this->select(['slug' => $urlSlug, 'is_active' => true]);
+        return $this->select([ 'slug' => $urlSlug, 'is_active' => true ]);
     }
 }

packages/Page/src/Filter/PageFilter.php

Spacing +16 added lines, -16 removed lines

@@ -18,10 +18,10 @@ 
             [
             'name'       => 'title',
             'required'   => true,
-            'filters'    => [['name' => 'StringTrim']],
+            'filters'    => [ [ 'name' => 'StringTrim' ] ],
             'validators' => [
-                ['name' => 'NotEmpty'],
-                ['name' => 'StringLength', 'options' => ['min' => 2, 'max' => 100]],
+                [ 'name' => 'NotEmpty' ],
+                [ 'name' => 'StringLength', 'options' => [ 'min' => 2, 'max' => 100 ] ],
             ],
             ]
         );
@@ -30,10 +30,10 @@ 
             [
             'name'       => 'slug',
             'required'   => true,
-            'filters'    => [['name' => 'StringTrim']],
+            'filters'    => [ [ 'name' => 'StringTrim' ] ],
             'validators' => [
-                ['name' => 'NotEmpty'],
-                ['name' => 'StringLength', 'options' => ['min' => 2, 'max' => 100]],
+                [ 'name' => 'NotEmpty' ],
+                [ 'name' => 'StringLength', 'options' => [ 'min' => 2, 'max' => 100 ] ],
             ],
             ]
         );
@@ -42,10 +42,10 @@ 
             [
             'name'       => 'body',
             'required'   => true,
-            'filters'    => [['name' => 'StringTrim']],
+            'filters'    => [ [ 'name' => 'StringTrim' ] ],
             'validators' => [
-                ['name' => 'NotEmpty'],
-                ['name' => 'StringLength', 'options' => ['min' => 2, 'max' => 100000]],
+                [ 'name' => 'NotEmpty' ],
+                [ 'name' => 'StringLength', 'options' => [ 'min' => 2, 'max' => 100000 ] ],
             ],
             ]
         );
@@ -54,10 +54,10 @@ 
             [
             'name'       => 'description',
             'required'   => true,
-            'filters'    => [['name' => 'StringTrim']],
+            'filters'    => [ [ 'name' => 'StringTrim' ] ],
             'validators' => [
-                ['name' => 'NotEmpty'],
-                ['name' => 'StringLength', 'options' => ['min' => 2, 'max' => 50000]],
+                [ 'name' => 'NotEmpty' ],
+                [ 'name' => 'StringLength', 'options' => [ 'min' => 2, 'max' => 50000 ] ],
             ],
             ]
         );
@@ -66,7 +66,7 @@ 
             [
             'name'     => 'has_layout',
             'required' => false,
-            'filters'  => [['name' => 'Boolean']],
+            'filters'  => [ [ 'name' => 'Boolean' ] ],
             ]
         );
 
@@ -74,7 +74,7 @@ 
             [
             'name'     => 'is_homepage',
             'required' => false,
-            'filters'  => [['name' => 'Boolean']],
+            'filters'  => [ [ 'name' => 'Boolean' ] ],
             ]
         );
 
@@ -82,7 +82,7 @@ 
             [
             'name'     => 'is_active',
             'required' => false,
-            'filters'  => [['name' => 'Boolean']],
+            'filters'  => [ [ 'name' => 'Boolean' ] ],
             ]
         );
 
@@ -90,7 +90,7 @@ 
             [
             'name'     => 'is_wysiwyg_editor',
             'required' => false,
-            'filters'  => [['name' => 'Boolean']],
+            'filters'  => [ [ 'name' => 'Boolean' ] ],
             ]
         );
 

packages/Page/src/Service/PageServiceFactory.php

Spacing +2 added lines, -2 removed lines

@@ -14,8 +14,8 @@
     public function __invoke(ContainerInterface $container)
     {
 
-        $config = $container->get('config')['upload'];
-        $upload = new Upload($config['public_path'], $config['non_public_path']);
+        $config = $container->get('config')[ 'upload' ];
+        $upload = new Upload($config[ 'public_path' ], $config[ 'non_public_path' ]);
 
         // Create pagination object
         $pageMapper        = $container->get(PageMapper::class);

packages/Page/src/Service/PageService.php

Indentation +1 added lines, -1 removed lines

@@ -77,7 +77,7 @@
             + ['main_img' => $this->upload->uploadImage($data, 'main_img')];
         $data['page_id'] = Uuid::uuid1()->toString();
         $data['page_uuid']
-                         = (new MysqlUuid($data['page_id']))->toFormat(new Binary);
+                            = (new MysqlUuid($data['page_id']))->toFormat(new Binary);
 
         if($data['is_homepage']){
             $this->pageMapper->update(['is_homepage' => false]);

Braces +1 added lines, -2 removed lines

@@ -104,8 +104,7 @@
         // We don't want to force user to re-upload image on edit
         if(!$data['main_img']){
             unset($data['main_img']);
-        }
-        else{
+        } else{
             $this->upload->deleteFile($page->getMainImg());
         }
 

Spacing +20 added lines, -20 removed lines

@@ -40,7 +40,7 @@ 
      */
     public function getPage($pageId)
     {
-        return $this->pageMapper->select(['page_id' => $pageId])->current();
+        return $this->pageMapper->select([ 'page_id' => $pageId ])->current();
     }
 
     /**
@@ -57,7 +57,7 @@ 
      */
     public function getHomepage()
     {
-        return $this->pageMapper->select(['is_homepage' => true])->current();
+        return $this->pageMapper->select([ 'is_homepage' => true ])->current();
     }
 
     /**
@@ -69,18 +69,18 @@ 
     {
         $filter = $this->pageFilter->getInputFilter()->setData($data);
 
-        if(!$filter->isValid()){
+        if (!$filter->isValid()) {
             throw new FilterException($filter->getMessages());
         }
 
         $data            = $filter->getValues()
-            + ['main_img' => $this->upload->uploadImage($data, 'main_img')];
-        $data['page_id'] = Uuid::uuid1()->toString();
-        $data['page_uuid']
-                         = (new MysqlUuid($data['page_id']))->toFormat(new Binary);
+            + [ 'main_img' => $this->upload->uploadImage($data, 'main_img') ];
+        $data[ 'page_id' ] = Uuid::uuid1()->toString();
+        $data[ 'page_uuid' ]
+                         = (new MysqlUuid($data[ 'page_id' ]))->toFormat(new Binary);
 
-        if($data['is_homepage']){
-            $this->pageMapper->update(['is_homepage' => false]);
+        if ($data[ 'is_homepage' ]) {
+            $this->pageMapper->update([ 'is_homepage' => false ]);
         }
 
         return $this->pageMapper->insert($data);
@@ -88,43 +88,43 @@ 
 
     public function updatePage($data, $pageId)
     {
-        if(!($page = $this->getPage($pageId))){
+        if (!($page = $this->getPage($pageId))) {
             throw new \Exception('Page object not found. Page ID:' . $pageId);
         }
 
         $filter = $this->pageFilter->getInputFilter()->setData($data);
 
-        if(!$filter->isValid()){
+        if (!$filter->isValid()) {
             throw new FilterException($filter->getMessages());
         }
 
         $data = $filter->getValues()
-            + ['main_img' => $this->upload->uploadImage($data, 'main_img')];
+            + [ 'main_img' => $this->upload->uploadImage($data, 'main_img') ];
 
         // We don't want to force user to re-upload image on edit
-        if(!$data['main_img']){
-            unset($data['main_img']);
+        if (!$data[ 'main_img' ]) {
+            unset($data[ 'main_img' ]);
         }
-        else{
+        else {
             $this->upload->deleteFile($page->getMainImg());
         }
 
-        if($data['is_homepage']){
-            $this->pageMapper->update(['is_homepage' => false]);
+        if ($data[ 'is_homepage' ]) {
+            $this->pageMapper->update([ 'is_homepage' => false ]);
         }
 
-        return $this->pageMapper->update($data, ['page_id' => $pageId]);
+        return $this->pageMapper->update($data, [ 'page_id' => $pageId ]);
     }
 
     public function delete($pageId)
     {
-        if(!($page = $this->getPage($pageId))){
+        if (!($page = $this->getPage($pageId))) {
             throw new \Exception('Page not found');
         }
 
         $this->upload->deleteFile($page->getMainImg());
 
-        return (bool)$this->pageMapper->delete(['page_id' => $pageId]);
+        return (bool) $this->pageMapper->delete([ 'page_id' => $pageId ]);
     }
 
     public function getForSelect()