phpsrbija / unfinished

src/Admin/src/Controller/AuthController.php

Doc Comments +1 added lines, -1 removed lines

@@ -98,7 +98,7 @@
     /**
      * Clears user session.
      *
-     * @return static
+     * @return \Psr\Http\Message\ResponseInterface
      */
     public function logout(): \Psr\Http\Message\ResponseInterface
     {

Spacing +7 added lines, -7 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Admin\Controller;
 
@@ -60,11 +60,11 @@ 
      */
     public function login($error = false): \Psr\Http\Message\ResponseInterface
     {
-        if($this->session->getStorage()->user) {
+        if ($this->session->getStorage()->user) {
             return $this->response->withStatus(302)->withHeader('Location', $this->router->generateUri('admin'));
         }
 
-        return new HtmlResponse($this->template->render('admin::login', ['layout' => false, 'error' => $error]));
+        return new HtmlResponse($this->template->render('admin::login', [ 'layout' => false, 'error' => $error ]));
     }
 
     /**
@@ -74,20 +74,20 @@ 
      */
     public function loginHandle(): \Psr\Http\Message\ResponseInterface
     {
-        if($this->session->getStorage()->user) {
+        if ($this->session->getStorage()->user) {
             return $this->response->withStatus(302)->withHeader('Location', $this->router->generateUri('admin'));
         }
 
         $data     = $this->request->getParsedBody();
-        $email    = isset($data['email']) ? $data['email'] : null;
-        $password = isset($data['password']) ? $data['password'] : null;
+        $email    = isset($data[ 'email' ]) ? $data[ 'email' ] : null;
+        $password = isset($data[ 'password' ]) ? $data[ 'password' ] : null;
 
         try {
             $this->session->getStorage()->user = $this->adminUserService->loginUser($email, $password);
 
             return $this->response->withStatus(302)->withHeader('Location', $this->router->generateUri('admin'));
         }
-        catch(\Exception $e) {
+        catch (\Exception $e) {
             return $this->login($e->getMessage());
 
             //@todo set $e->getMessage() to flash messanger and print messages in next page

Braces +1 added lines, -2 removed lines

@@ -86,8 +86,7 @@
             $this->session->getStorage()->user = $this->adminUserService->loginUser($email, $password);
 
             return $this->response->withStatus(302)->withHeader('Location', $this->router->generateUri('admin'));
-        }
-        catch(\Exception $e) {
+        } catch(\Exception $e) {
             return $this->login($e->getMessage());
 
             //@todo set $e->getMessage() to flash messanger and print messages in next page

src/Menu/src/ConfigProvider.php

Spacing +3 added lines, -3 removed lines

@@ -12,7 +12,7 @@ 
                     'partial/menu-level' => __DIR__ . '/../templates/partial/menu-level.phtml',
                 ],
                 'paths' => [
-                    'menu' => [__DIR__ . '/../templates/menu'],
+                    'menu' => [ __DIR__ . '/../templates/menu' ],
                 ],
             ],
 
@@ -33,13 +33,13 @@ 
                     'name'            => 'admin.menu',
                     'path'            => '/admin/menu',
                     'middleware'      => Controller\IndexController::class,
-                    'allowed_methods' => ['GET'],
+                    'allowed_methods' => [ 'GET' ],
                 ],
                 [
                     'name'            => 'admin.menu.action',
                     'path'            => '/admin/menu/:action/:id',
                     'middleware'      => Controller\IndexController::class,
-                    'allowed_methods' => ['GET', 'POST'],
+                    'allowed_methods' => [ 'GET', 'POST' ],
                 ],
             ],
 

src/Core/Filter/AbstractFilter.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
     public function getMessages(array $msgs)
     {
         $allMsgs = '';
-        array_walk_recursive($msgs, function ($value) use (&$allMsgs){
+        array_walk_recursive($msgs, function($value) use (&$allMsgs){
             $allMsgs .= "- " . $value . "<br/>";
         });
 

src/Category/src/ConfigProvider.php

Spacing +3 added lines, -3 removed lines

@@ -9,7 +9,7 @@ 
         return [
             'templates' => [
                 'paths' => [
-                    'category' => [__DIR__ . '/../templates/category'],
+                    'category' => [ __DIR__ . '/../templates/category' ],
                 ],
             ],
 
@@ -30,13 +30,13 @@ 
                     'name'            => 'admin.categories',
                     'path'            => '/admin/categories',
                     'middleware'      => Controller\IndexController::class,
-                    'allowed_methods' => ['GET'],
+                    'allowed_methods' => [ 'GET' ],
                 ],
                 [
                     'name'            => 'admin.categories.action',
                     'path'            => '/admin/categories/:action/:id',
                     'middleware'      => Controller\IndexController::class,
-                    'allowed_methods' => ['GET', 'POST']
+                    'allowed_methods' => [ 'GET', 'POST' ]
                 ],
             ],
 

src/Category/src/Controller/IndexController.php

Spacing +13 added lines, -13 removed lines

@@ -47,12 +47,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' ]));
     }
 
     /**
@@ -60,13 +60,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;
         }
 
@@ -79,36 +79,36 @@ 
 
     public function save()
     {
-        try{
+        try {
             $id   = $this->request->getAttribute('id');
             $data = $this->request->getParsedBody();
 
-            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

@@ -85,17 +85,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;
         }
     }
@@ -107,8 +104,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'));
         }
     }

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('article_uuid', 'binary', ['limit' => 16, 'null' => true])
-            ->addColumn('category_uuid', 'binary', ['limit' => 16, 'null' => true])
-            ->addForeignKey('article_uuid', 'articles', 'article_uuid', ['delete' => 'NO_ACTION', 'update' => 'NO_ACTION'])
-            ->addForeignKey('category_uuid', 'category', 'category_uuid', ['delete' => 'NO_ACTION', 'update' => 'NO_ACTION'])
+            ->addColumn('article_uuid', 'binary', [ 'limit' => 16, 'null' => true ])
+            ->addColumn('category_uuid', 'binary', [ 'limit' => 16, 'null' => true ])
+            ->addForeignKey('article_uuid', 'articles', 'article_uuid', [ 'delete' => 'NO_ACTION', 'update' => 'NO_ACTION' ])
+            ->addForeignKey('category_uuid', 'category', 'category_uuid', [ 'delete' => 'NO_ACTION', 'update' => 'NO_ACTION' ])
             ->create();
     }
 

src/Menu/src/Controller/IndexController.php

Spacing +12 added lines, -12 removed lines

@@ -37,13 +37,13 @@ 
         ]));
     }
 
-    public function edit($errors = [])
+    public function edit($errors = [ ])
     {
         $id   = $this->request->getAttribute('id');
         $item = $this->menuService->get($id);
 
-        if($this->request->getParsedBody()){
-            $item          = (object)($this->request->getParsedBody() + (array)$item);
+        if ($this->request->getParsedBody()) {
+            $item          = (object) ($this->request->getParsedBody() + (array) $item);
             $item->menu_id = $id;
         }
 
@@ -58,36 +58,36 @@ 
 
     public function save()
     {
-        try{
+        try {
             $id   = $this->request->getAttribute('id');
             $data = $this->request->getParsedBody();
 
-            if($id){
+            if ($id) {
                 $this->menuService->updateMenuItem($data, $id);
             }
-            else{
+            else {
                 $this->menuService->addMenuItem($data);
             }
 
             return $this->response->withStatus(302)->withHeader('Location', $this->router->generateUri('admin.menu'));
         }
-        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->menuService->delete($id);
 
             return $this->response->withStatus(302)->withHeader('Location', $this->router->generateUri('admin.menu'));
         }
-        catch(\Exception $e){
+        catch (\Exception $e) {
             throw $e;
         }
     }
@@ -95,9 +95,9 @@ 
     public function updateOrder()
     {
         $data      = $this->request->getParsedBody();
-        $menuOrder = isset($data['order']) ? json_decode($data['order']) : [];
+        $menuOrder = isset($data[ 'order' ]) ? json_decode($data[ 'order' ]) : [ ];
         $status    = $this->menuService->updateMenuOrder($menuOrder);
 
-        return new JsonResponse(['status' => $status]);
+        return new JsonResponse([ 'status' => $status ]);
     }
 }
\ No newline at end of file

Braces +4 added lines, -8 removed lines

@@ -64,17 +64,14 @@ 
 
             if($id){
                 $this->menuService->updateMenuItem($data, $id);
-            }
-            else{
+            } else{
                 $this->menuService->addMenuItem($data);
             }
 
             return $this->response->withStatus(302)->withHeader('Location', $this->router->generateUri('admin.menu'));
-        }
-        catch(FilterException $fe){
+        } catch(FilterException $fe){
             return $this->edit($fe->getArrayMessages());
-        }
-        catch(\Exception $e){
+        } catch(\Exception $e){
             throw $e;
         }
     }
@@ -86,8 +83,7 @@ 
             $this->menuService->delete($id);
 
             return $this->response->withStatus(302)->withHeader('Location', $this->router->generateUri('admin.menu'));
-        }
-        catch(\Exception $e){
+        } catch(\Exception $e){
             throw $e;
         }
     }

src/Menu/src/Filter/MenuFilter.php

Spacing +13 added lines, -13 removed lines

@@ -20,65 +20,65 @@
 
     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' ] ],
                 ],
             ]);
 
             $inputFilter->add([
                 'name'     => 'href',
                 'required' => false,
-                'filters'  => [['name' => 'StripTags'], ['name' => 'StringTrim']]
+                'filters'  => [ [ 'name' => 'StripTags' ], [ 'name' => 'StringTrim' ] ]
             ]);
 
             $inputFilter->add([
                 'name'       => 'article_id',
                 'required'   => false,
-                'filters'    => [['name' => 'Null']],
+                'filters'    => [ [ 'name' => 'Null' ] ],
                 'validators' => [
-                    ['name' => RecordExists::class, 'options' => ['table' => 'articles', 'field' => 'article_id', 'adapter' => $this->db]]
+                    [ 'name' => RecordExists::class, 'options' => [ 'table' => 'articles', 'field' => 'article_id', 'adapter' => $this->db ] ]
                 ]
             ]);
 
             $inputFilter->add([
                 '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 ] ]
                 ]
             ]);
 
             $inputFilter->add([
                 'name'     => 'is_active',
                 'required' => false,
-                'filters'  => [['name' => 'Boolean']]
+                'filters'  => [ [ 'name' => 'Boolean' ] ]
             ]);
 
             $inputFilter->add([
                 'name'     => 'is_in_header',
                 'required' => false,
-                'filters'  => [['name' => 'Boolean']]
+                'filters'  => [ [ 'name' => 'Boolean' ] ]
             ]);
 
             $inputFilter->add([
                 'name'     => 'is_in_footer',
                 'required' => false,
-                'filters'  => [['name' => 'Boolean']]
+                'filters'  => [ [ 'name' => 'Boolean' ] ]
             ]);
 
             $inputFilter->add([
                 'name'     => 'is_in_side',
                 'required' => false,
-                'filters'  => [['name' => 'Boolean']]
+                'filters'  => [ [ 'name' => 'Boolean' ] ]
             ]);
 
             $this->inputFilter = $inputFilter;

src/Article/src/Service/ArticleServiceInterface.php

Doc Comments -1 removed lines

@@ -12,7 +12,6 @@
     /**
      * Fetches a list of ArticleEntity models.
      *
-     * @param array $params
      * @return ArrayObject
      */
     public function fetchAllArticles($page, $limit);