phpsrbija / unfinished

packages/Article/src/Mapper/ArticleMapper.php

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 use Zend\Db\Adapter\AdapterAwareInterface;
 use Zend\Db\TableGateway\AbstractTableGateway;
 use Zend\Db\Sql\Delete;
-use Zend\Db\Sql\Insert;
 
 /**
  * Class ArticleMapper.

Spacing +4 added lines, -4 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Article\Mapper;
 
@@ -45,9 +45,9 @@ 
     public function getCategories($articleId)
     {
         $select = $this->getSql()->select()
-            ->columns([])
-            ->join('category', 'category.category_uuid = articles.category_uuid', ['name', 'slug', 'category_id'])
-            ->where(['articles.article_id' => $articleId]);
+            ->columns([ ])
+            ->join('category', 'category.category_uuid = articles.category_uuid', [ 'name', 'slug', 'category_id' ])
+            ->where([ 'articles.article_id' => $articleId ]);
 
         return $this->selectWith($select);
     }

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

packages/Article/src/Service/EventService.php

Doc Comments +3 added lines

@@ -63,6 +63,9 @@
         return $this->getPagination($select, $page, $limit);
     }
 
+    /**
+     * @param integer $limit
+     */
     public function fetchLatest($limit)
     {
         return $this->articleEventsMapper->getLatest($limit);

Spacing +18 added lines, -18 removed lines

@@ -87,7 +87,7 @@ 
         $articleFilter = $this->articleFilter->getInputFilter()->setData($data);
         $eventFilter   = $this->eventFilter->getInputFilter()->setData($data);
 
-        if(!$articleFilter->isValid() || !$eventFilter->isValid()) {
+        if (!$articleFilter->isValid() || !$eventFilter->isValid()) {
             throw new FilterException($articleFilter->getMessages() + $eventFilter->getMessages());
         }
 
@@ -96,15 +96,15 @@ 
 
         $article = $articleFilter->getValues();
         $article += [
-            'admin_user_uuid' => $this->adminUsersMapper->getUuid($article['admin_user_id']),
+            'admin_user_uuid' => $this->adminUsersMapper->getUuid($article[ 'admin_user_id' ]),
             'type'            => ArticleType::EVENT,
             'article_id'      => $id,
             'article_uuid'    => $uuId
         ];
-        unset($article['admin_user_id']);
+        unset($article[ 'admin_user_id' ]);
 
-        $article['category_uuid'] = $this->categoryMapper->get($article['category_id'])->category_uuid;
-        unset($article['category_id']);
+        $article[ 'category_uuid' ] = $this->categoryMapper->get($article[ 'category_id' ])->category_uuid;
+        unset($article[ 'category_id' ]);
 
         $event = $eventFilter->getValues() + [
                 'featured_img' => $this->upload->uploadImage($data, 'featured_img'),
@@ -122,50 +122,50 @@ 
         $articleFilter = $this->articleFilter->getInputFilter()->setData($data);
         $eventFilter   = $this->eventFilter->getInputFilter()->setData($data);
 
-        if(!$articleFilter->isValid() || !$eventFilter->isValid()) {
+        if (!$articleFilter->isValid() || !$eventFilter->isValid()) {
             throw new FilterException($articleFilter->getMessages() + $eventFilter->getMessages());
         }
 
-        $article = $articleFilter->getValues() + ['article_uuid' => $oldArticle->article_uuid];
+        $article = $articleFilter->getValues() + [ 'article_uuid' => $oldArticle->article_uuid ];
         $event   = $eventFilter->getValues() + [
                 'featured_img' => $this->upload->uploadImage($data, 'featured_img'),
                 'main_img'     => $this->upload->uploadImage($data, 'main_img')
             ];
 
-        $article['admin_user_uuid'] = $this->adminUsersMapper->getUuid($article['admin_user_id']);
-        $article['category_uuid']   = $this->categoryMapper->get($article['category_id'])->category_uuid;
-        unset($article['category_id'], $article['admin_user_id']);
+        $article[ 'admin_user_uuid' ] = $this->adminUsersMapper->getUuid($article[ 'admin_user_id' ]);
+        $article[ 'category_uuid' ]   = $this->categoryMapper->get($article[ 'category_id' ])->category_uuid;
+        unset($article[ 'category_id' ], $article[ 'admin_user_id' ]);
 
         // We don't want to force user to re-upload image on edit
-        if (!$event['featured_img']) {
-            unset($event['featured_img']);
+        if (!$event[ 'featured_img' ]) {
+            unset($event[ 'featured_img' ]);
         }
         else {
             $this->upload->deleteFile($oldArticle->featured_img);
         }
 
-        if (!$event['main_img']) {
-            unset($event['main_img']);
+        if (!$event[ 'main_img' ]) {
+            unset($event[ 'main_img' ]);
         }
         else {
             $this->upload->deleteFile($oldArticle->main_img);
         }
 
-        $this->articleMapper->update($article, ['article_uuid' => $article['article_uuid']]);
-        $this->articleEventsMapper->update($event, ['article_uuid' => $article['article_uuid']]);
+        $this->articleMapper->update($article, [ 'article_uuid' => $article[ 'article_uuid' ] ]);
+        $this->articleEventsMapper->update($event, [ 'article_uuid' => $article[ 'article_uuid' ] ]);
     }
 
     public function deleteArticle($id)
     {
         $event = $this->articleEventsMapper->get($id);
 
-        if(!$event) {
+        if (!$event) {
             throw new \Exception('Article not found!');
         }
 
         $this->upload->deleteFile($event->main_img);
         $this->upload->deleteFile($event->featured_img);
-        $this->articleEventsMapper->delete(['article_uuid' => $event->article_uuid]);
+        $this->articleEventsMapper->delete([ 'article_uuid' => $event->article_uuid ]);
         $this->delete($event->article_uuid);
     }
 

Braces +2 added lines, -4 removed lines

@@ -139,15 +139,13 @@
         // We don't want to force user to re-upload image on edit
         if (!$event['featured_img']) {
             unset($event['featured_img']);
-        }
-        else {
+        } else {
             $this->upload->deleteFile($oldArticle->featured_img);
         }
 
         if (!$event['main_img']) {
             unset($event['main_img']);
-        }
-        else {
+        } else {
             $this->upload->deleteFile($oldArticle->main_img);
         }
 

packages/Article/src/Service/PostService.php

Doc Comments +3 added lines

@@ -203,6 +203,9 @@
         return $this->articleMapper->getCategories($articleId);
     }
 
+    /**
+     * @param integer $limit
+     */
     public function getLatestWeb($limit)
     {
         return $this->articlePostsMapper->getLatest($limit);

Spacing +18 added lines, -18 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Article\Service;
 
@@ -95,7 +95,7 @@ 
         $articleFilter = $this->articleFilter->getInputFilter()->setData($data);
         $postFilter    = $this->postFilter->getInputFilter()->setData($data);
 
-        if(!$articleFilter->isValid() || !$postFilter->isValid()) {
+        if (!$articleFilter->isValid() || !$postFilter->isValid()) {
             throw new FilterException($articleFilter->getMessages() + $postFilter->getMessages());
         }
 
@@ -104,19 +104,19 @@ 
 
         $article = $articleFilter->getValues();
         $article += [
-            'admin_user_uuid' => $this->adminUsersMapper->getUuid($article['admin_user_id']),
+            'admin_user_uuid' => $this->adminUsersMapper->getUuid($article[ 'admin_user_id' ]),
             'type'            => ArticleType::POST,
             'article_id'      => $id,
             'article_uuid'    => $uuId
         ];
 
-        $article['category_uuid'] = $this->categoryMapper->get($article['category_id'])->category_uuid;
-        unset($article['category_id'], $article['admin_user_id']);
+        $article[ 'category_uuid' ] = $this->categoryMapper->get($article[ 'category_id' ])->category_uuid;
+        unset($article[ 'category_id' ], $article[ 'admin_user_id' ]);
 
         $post = $postFilter->getValues() + [
                 'featured_img' => $this->upload->uploadImage($data, 'featured_img'),
                 'main_img'     => $this->upload->uploadImage($data, 'main_img'),
-                'article_uuid' => $article['article_uuid']
+                'article_uuid' => $article[ 'article_uuid' ]
             ];
 
         $this->articleMapper->insert($article);
@@ -129,37 +129,37 @@ 
         $articleFilter = $this->articleFilter->getInputFilter()->setData($data);
         $postFilter    = $this->postFilter->getInputFilter()->setData($data);
 
-        if(!$articleFilter->isValid() || !$postFilter->isValid()) {
+        if (!$articleFilter->isValid() || !$postFilter->isValid()) {
             throw new FilterException($articleFilter->getMessages() + $postFilter->getMessages());
         }
 
-        $article = $articleFilter->getValues() + ['article_uuid' => $oldArticle->article_uuid];
+        $article = $articleFilter->getValues() + [ 'article_uuid' => $oldArticle->article_uuid ];
         $post    = $postFilter->getValues() + [
                 'featured_img' => $this->upload->uploadImage($data, 'featured_img'),
                 'main_img'     => $this->upload->uploadImage($data, 'main_img')
             ];
 
-        $article['admin_user_uuid'] = $this->adminUsersMapper->getUuid($article['admin_user_id']);
-        $article['category_uuid']   = $this->categoryMapper->get($article['category_id'])->category_uuid;
-        unset($article['category_id'], $article['admin_user_id']);
+        $article[ 'admin_user_uuid' ] = $this->adminUsersMapper->getUuid($article[ 'admin_user_id' ]);
+        $article[ 'category_uuid' ]   = $this->categoryMapper->get($article[ 'category_id' ])->category_uuid;
+        unset($article[ 'category_id' ], $article[ 'admin_user_id' ]);
 
         // We don't want to force user to re-upload image on edit
-        if (!$post['featured_img']) {
-            unset($post['featured_img']);
+        if (!$post[ 'featured_img' ]) {
+            unset($post[ 'featured_img' ]);
         }
         else {
             $this->upload->deleteFile($oldArticle->featured_img);
         }
 
-        if (!$post['main_img']) {
-            unset($post['main_img']);
+        if (!$post[ 'main_img' ]) {
+            unset($post[ 'main_img' ]);
         }
         else {
             $this->upload->deleteFile($oldArticle->main_img);
         }
 
-        $this->articleMapper->update($article, ['article_uuid' => $article['article_uuid']]);
-        $this->articlePostsMapper->update($post, ['article_uuid' => $article['article_uuid']]);
+        $this->articleMapper->update($article, [ 'article_uuid' => $article[ 'article_uuid' ] ]);
+        $this->articlePostsMapper->update($post, [ 'article_uuid' => $article[ 'article_uuid' ] ]);
     }
 
     public function deleteArticle($id)
@@ -172,7 +172,7 @@ 
 
         $this->upload->deleteFile($post->main_img);
         $this->upload->deleteFile($post->featured_img);
-        $this->articlePostsMapper->delete(['article_uuid' => $post->article_uuid]);
+        $this->articlePostsMapper->delete([ 'article_uuid' => $post->article_uuid ]);
         $this->delete($post->article_uuid);
     }
 

Braces +2 added lines, -4 removed lines

@@ -146,15 +146,13 @@
         // We don't want to force user to re-upload image on edit
         if (!$post['featured_img']) {
             unset($post['featured_img']);
-        }
-        else {
+        } else {
             $this->upload->deleteFile($oldArticle->featured_img);
         }
 
         if (!$post['main_img']) {
             unset($post['main_img']);
-        }
-        else {
+        } else {
             $this->upload->deleteFile($oldArticle->main_img);
         }
 

packages/Article/src/Service/VideoService.php

Doc Comments +3 added lines

@@ -91,6 +91,9 @@
         return $this->getPagination($select, $page, $limit);
     }
 
+    /**
+     * @param integer $limit
+     */
     public function fetchLatest($limit)
     {
         return $this->articleVideosMapper->getLatest($limit);

Spacing +19 added lines, -19 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Article\Service;
 
@@ -106,7 +106,7 @@ 
         $articleFilter = $this->articleFilter->getInputFilter()->setData($data);
         $videosFilter  = $this->videosFilter->getInputFilter()->setData($data);
 
-        if(!$articleFilter->isValid() || !$videosFilter->isValid()) {
+        if (!$articleFilter->isValid() || !$videosFilter->isValid()) {
             throw new FilterException($articleFilter->getMessages() + $videosFilter->getMessages());
         }
 
@@ -115,14 +115,14 @@ 
 
         $article = $articleFilter->getValues();
         $article += [
-            'admin_user_uuid' => $this->adminUsersMapper->getUuid($article['admin_user_id']),
+            'admin_user_uuid' => $this->adminUsersMapper->getUuid($article[ 'admin_user_id' ]),
             'type'            => ArticleType::VIDEO,
             'article_id'      => $id,
             'article_uuid'    => $uuId
         ];
 
-        $article['category_uuid'] = $this->categoryMapper->get($article['category_id'])->category_uuid;
-        unset($article['category_id'], $article['admin_user_id']);
+        $article[ 'category_uuid' ] = $this->categoryMapper->get($article[ 'category_id' ])->category_uuid;
+        unset($article[ 'category_id' ], $article[ 'admin_user_id' ]);
 
         $videos = $videosFilter->getValues() + [
                 'featured_img' => $this->upload->uploadImage($data, 'featured_img'),
@@ -140,15 +140,15 @@ 
         $articleFilter = $this->articleFilter->getInputFilter()->setData($data);
         $videosFilter  = $this->videosFilter->getInputFilter()->setData($data);
 
-        if(!$articleFilter->isValid() || !$videosFilter->isValid()) {
+        if (!$articleFilter->isValid() || !$videosFilter->isValid()) {
             throw new FilterException($articleFilter->getMessages() + $videosFilter->getMessages());
         }
 
-        $article                    = $articleFilter->getValues() + ['article_uuid' => $oldArticle->article_uuid];
-        $article['category_uuid']   = $this->categoryMapper->get($article['category_id'])->category_uuid;
-        $article['admin_user_uuid'] = $this->adminUsersMapper->getUuid($article['admin_user_id']);
-        unset($article['category_id']);
-        unset($article['admin_user_id']);
+        $article                    = $articleFilter->getValues() + [ 'article_uuid' => $oldArticle->article_uuid ];
+        $article[ 'category_uuid' ]   = $this->categoryMapper->get($article[ 'category_id' ])->category_uuid;
+        $article[ 'admin_user_uuid' ] = $this->adminUsersMapper->getUuid($article[ 'admin_user_id' ]);
+        unset($article[ 'category_id' ]);
+        unset($article[ 'admin_user_id' ]);
 
         $videos = $videosFilter->getValues() + [
                 'featured_img' => $this->upload->uploadImage($data, 'featured_img'),
@@ -156,35 +156,35 @@ 
             ];
 
         // We don't want to force user to re-upload image on edit
-        if(!$videos['featured_img']) {
-            unset($videos['featured_img']);
+        if (!$videos[ 'featured_img' ]) {
+            unset($videos[ 'featured_img' ]);
         }
         else {
             $this->upload->deleteFile($oldArticle->featured_img);
         }
 
-        if(!$videos['main_img']) {
-            unset($videos['main_img']);
+        if (!$videos[ 'main_img' ]) {
+            unset($videos[ 'main_img' ]);
         }
         else {
             $this->upload->deleteFile($oldArticle->main_img);
         }
 
-        $this->articleMapper->update($article, ['article_uuid' => $article['article_uuid']]);
-        $this->articleVideosMapper->update($videos, ['article_uuid' => $article['article_uuid']]);
+        $this->articleMapper->update($article, [ 'article_uuid' => $article[ 'article_uuid' ] ]);
+        $this->articleVideosMapper->update($videos, [ 'article_uuid' => $article[ 'article_uuid' ] ]);
     }
 
     public function deleteArticle($id)
     {
         $video = $this->articleVideosMapper->get($id);
 
-        if(!$video) {
+        if (!$video) {
             throw new \Exception('Article not found!');
         }
 
         $this->upload->deleteFile($video->main_img);
         $this->upload->deleteFile($video->featured_img);
-        $this->articleVideosMapper->delete(['article_uuid' => $video->article_uuid]);
+        $this->articleVideosMapper->delete([ 'article_uuid' => $video->article_uuid ]);
         $this->delete($video->article_uuid);
     }
 

Braces +2 added lines, -4 removed lines

@@ -158,15 +158,13 @@
         // We don't want to force user to re-upload image on edit
         if(!$videos['featured_img']) {
             unset($videos['featured_img']);
-        }
-        else {
+        } else {
             $this->upload->deleteFile($oldArticle->featured_img);
         }
 
         if(!$videos['main_img']) {
             unset($videos['main_img']);
-        }
-        else {
+        } else {
             $this->upload->deleteFile($oldArticle->main_img);
         }
 

packages/Category/src/Mapper/CategoryMapper.php

Doc Comments +3 added lines

@@ -44,6 +44,9 @@
         return $select;
     }
 
+    /**
+     * @param integer $limit
+     */
     public function getCategoryPostsSelect($categoryId = null, $limit = null)
     {
         $select = $this->getSql()->select()

Spacing +19 added lines, -19 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Category\Mapper;
 
@@ -34,12 +34,12 @@ 
 
     public function get($id)
     {
-        return $this->select(['category_id' => $id])->current();
+        return $this->select([ 'category_id' => $id ])->current();
     }
 
     public function getPaginationSelect()
     {
-        $select = $this->getSql()->select()->order(['name' => 'asc']);
+        $select = $this->getSql()->select()->order([ 'name' => 'asc' ]);
 
         return $select;
     }
@@ -47,18 +47,18 @@ 
     public function getCategoryPostsSelect($categoryId = null, $limit = null)
     {
         $select = $this->getSql()->select()
-            ->columns(['category_name' => 'name', 'category_slug' => 'slug'])
-            ->join('articles', 'articles.category_uuid = category.category_uuid', ['article_id', 'slug', 'admin_user_uuid', 'published_at'])
-            ->join('article_posts', 'article_posts.article_uuid = articles.article_uuid', ['*'], 'right')
-            ->join('admin_users', 'admin_users.admin_user_uuid = articles.admin_user_uuid', ['admin_user_id', 'first_name', 'last_name', 'face_img'])
-            ->where(['articles.status' => 1])
-            ->order(['published_at' => 'desc']);
-
-        if($categoryId) {
-            $select->where(['category_id' => $categoryId]);
+            ->columns([ 'category_name' => 'name', 'category_slug' => 'slug' ])
+            ->join('articles', 'articles.category_uuid = category.category_uuid', [ 'article_id', 'slug', 'admin_user_uuid', 'published_at' ])
+            ->join('article_posts', 'article_posts.article_uuid = articles.article_uuid', [ '*' ], 'right')
+            ->join('admin_users', 'admin_users.admin_user_uuid = articles.admin_user_uuid', [ 'admin_user_id', 'first_name', 'last_name', 'face_img' ])
+            ->where([ 'articles.status' => 1 ])
+            ->order([ 'published_at' => 'desc' ]);
+
+        if ($categoryId) {
+            $select->where([ 'category_id' => $categoryId ]);
         }
 
-        if($limit) {
+        if ($limit) {
             $select->limit($limit);
         }
 
@@ -86,19 +86,19 @@ 
         $select->where->notEqualTo('slug', 'videos');
         $select->where->notEqualTo('slug', 'events');
 
-        if($inHomepage !== null){
-            $select->where(['is_in_homepage' => $inHomepage]);
+        if ($inHomepage !== null) {
+            $select->where([ 'is_in_homepage' => $inHomepage ]);
         }
 
-        if($inCategoryList !== null){
-            $select->where(['is_in_category_list' => $inCategoryList]);
+        if ($inCategoryList !== null) {
+            $select->where([ 'is_in_category_list' => $inCategoryList ]);
         }
 
-        if($limit) {
+        if ($limit) {
             $select->limit($limit);
         }
 
-        if($order) {
+        if ($order) {
             $select->order($order);
         } else {
             $select->order(new Expression('rand()'));

packages/Core/Mapper/AdminUsersMapper.php

Doc Comments +10 added lines, -2 removed lines

@@ -31,6 +31,9 @@ 
         $this->adapter = $adapter;
     }
 
+    /**
+     * @param string $id
+     */
     public function get($id)
     {
         return $this->select(['admin_user_id' => $id])->current();
@@ -40,7 +43,7 @@ 
      * Get admin user by email.
      *
      * @param  string $email email
-     * @return array|\ArrayObject|null
+     * @return string
      */
     public function getByEmail(string $email)
     {
@@ -50,7 +53,6 @@ 
     /**
      * Updates login data.
      *
-     * @param  string $uuid admin user id
      * @return int number of affected rows
      */
     public function updateLogin(string $userId): int
@@ -58,6 +60,9 @@ 
         return $this->update(['last_login' => date('Y-m-d H:i:s')], ['admin_user_id' => $userId]);
     }
 
+    /**
+     * @param string $userId
+     */
     public function getPaginationSelect($userId)
     {
         $select = $this->getSql()->select()->order(['created_at' => 'desc']);
@@ -67,6 +72,9 @@ 
         return $select;
     }
 
+    /**
+     * @param integer $limit
+     */
     public function getRandom($limit)
     {
         $select = $this->getSql()->select()

Spacing +8 added lines, -8 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Core\Mapper;
 
@@ -33,7 +33,7 @@ 
 
     public function get($id)
     {
-        return $this->select(['admin_user_id' => $id])->current();
+        return $this->select([ 'admin_user_id' => $id ])->current();
     }
 
     /**
@@ -44,7 +44,7 @@ 
      */
     public function getByEmail(string $email)
     {
-        return $this->select(['email' => $email])->current();
+        return $this->select([ 'email' => $email ])->current();
     }
 
     /**
@@ -55,12 +55,12 @@ 
      */
     public function updateLogin(string $userId): int
     {
-        return $this->update(['last_login' => date('Y-m-d H:i:s')], ['admin_user_id' => $userId]);
+        return $this->update([ 'last_login' => date('Y-m-d H:i:s') ], [ 'admin_user_id' => $userId ]);
     }
 
     public function getPaginationSelect($userId)
     {
-        $select = $this->getSql()->select()->order(['created_at' => 'desc']);
+        $select = $this->getSql()->select()->order([ 'created_at' => 'desc' ]);
 
         $select->where->notEqualTo('admin_user_id', $userId);
 
@@ -70,7 +70,7 @@ 
     public function getRandom($limit)
     {
         $select = $this->getSql()->select()
-            ->where(['status' => 1])
+            ->where([ 'status' => 1 ])
             ->order(new Expression('rand()'))
             ->limit($limit);
 
@@ -79,9 +79,9 @@ 
 
     public function getUuid($adminUserId)
     {
-        $user = $this->select(['admin_user_id' => $adminUserId])->current();
+        $user = $this->select([ 'admin_user_id' => $adminUserId ])->current();
 
-        if(!$user) {
+        if (!$user) {
             throw new \Exception('Admin user does not exist!', 400);
         }
 

packages/Article/src/Service/DiscussionService.php

Spacing +14 added lines, -14 removed lines

@@ -53,7 +53,7 @@ 
         $articleFilter    = $this->articleFilter->getInputFilter()->setData($data);
         $discussionFilter = $this->discussionFilter->getInputFilter()->setData($data);
 
-        if(!$articleFilter->isValid() || !$discussionFilter->isValid()) {
+        if (!$articleFilter->isValid() || !$discussionFilter->isValid()) {
             throw new FilterException($articleFilter->getMessages() + $discussionFilter->getMessages());
         }
 
@@ -62,16 +62,16 @@ 
 
         $article = $articleFilter->getValues();
         $article += [
-            'admin_user_uuid' => $this->adminUsersMapper->getUuid($article['admin_user_id']),
+            'admin_user_uuid' => $this->adminUsersMapper->getUuid($article[ 'admin_user_id' ]),
             'type'            => ArticleType::DISCUSSION,
             'article_id'      => $id,
             'article_uuid'    => $uuId
         ];
 
-        $article['category_uuid'] = $this->categoryMapper->get($article['category_id'])->category_uuid;
-        unset($article['category_id'], $article['admin_user_id']);
+        $article[ 'category_uuid' ] = $this->categoryMapper->get($article[ 'category_id' ])->category_uuid;
+        unset($article[ 'category_id' ], $article[ 'admin_user_id' ]);
 
-        $discussion = $discussionFilter->getValues() + ['article_uuid' => $uuId];
+        $discussion = $discussionFilter->getValues() + [ 'article_uuid' => $uuId ];
 
         $this->articleMapper->insert($article);
         $this->articleDiscussionsMapper->insert($discussion);
@@ -83,30 +83,30 @@ 
         $articleFilter    = $this->articleFilter->getInputFilter()->setData($data);
         $discussionFilter = $this->discussionFilter->getInputFilter()->setData($data);
 
-        if(!$articleFilter->isValid() || !$discussionFilter->isValid()) {
+        if (!$articleFilter->isValid() || !$discussionFilter->isValid()) {
             throw new FilterException($articleFilter->getMessages() + $discussionFilter->getMessages());
         }
 
-        $article    = $articleFilter->getValues() + ['article_uuid' => $article->article_uuid];
+        $article    = $articleFilter->getValues() + [ 'article_uuid' => $article->article_uuid ];
         $discussion = $discussionFilter->getValues();
 
-        $article['admin_user_uuid'] = $this->adminUsersMapper->getUuid($article['admin_user_id']);
-        $article['category_uuid']   = $this->categoryMapper->get($article['category_id'])->category_uuid;
-        unset($article['category_id'], $article['admin_user_id']);
+        $article[ 'admin_user_uuid' ] = $this->adminUsersMapper->getUuid($article[ 'admin_user_id' ]);
+        $article[ 'category_uuid' ]   = $this->categoryMapper->get($article[ 'category_id' ])->category_uuid;
+        unset($article[ 'category_id' ], $article[ 'admin_user_id' ]);
 
-        $this->articleMapper->update($article, ['article_uuid' => $article['article_uuid']]);
-        $this->articleDiscussionsMapper->update($discussion, ['article_uuid' => $article['article_uuid']]);
+        $this->articleMapper->update($article, [ 'article_uuid' => $article[ 'article_uuid' ] ]);
+        $this->articleDiscussionsMapper->update($discussion, [ 'article_uuid' => $article[ 'article_uuid' ] ]);
     }
 
     public function deleteArticle($id)
     {
         $discussion = $this->articleDiscussionsMapper->get($id);
 
-        if(!$discussion) {
+        if (!$discussion) {
             throw new \Exception('Article not found!');
         }
 
-        $this->articleDiscussionsMapper->delete(['article_uuid' => $discussion->article_uuid]);
+        $this->articleDiscussionsMapper->delete([ 'article_uuid' => $discussion->article_uuid ]);
         $this->delete($discussion->article_uuid);
     }
 }

packages/Article/src/Service/ArticleService.php

Spacing +4 added lines, -4 removed lines

@@ -31,9 +31,9 @@ 
 
     public function getCategoryIds($articleId)
     {
-        $categories = [];
-        foreach($this->articleMapper->getCategories($articleId) as $category) {
-            $categories[] = $category->category_id;
+        $categories = [ ];
+        foreach ($this->articleMapper->getCategories($articleId) as $category) {
+            $categories[ ] = $category->category_id;
         }
 
         return $categories;
@@ -41,6 +41,6 @@ 
 
     public function delete($articleId)
     {
-        $this->articleMapper->delete(['article_uuid' => $articleId]);
+        $this->articleMapper->delete([ 'article_uuid' => $articleId ]);
     }
 }
\ No newline at end of file