phpsrbija / unfinished

src/Web/Factory/Action/VideoActionFactory.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Web\Factory\Action;
 

src/Menu/src/View/Helper/MenuUrlHelper.php

Spacing +10 added lines, -10 removed lines

@@ -33,24 +33,24 @@
      */
     public function __invoke($menuItem)
     {
-        if($menuItem['href']) {
-            return strpos($menuItem['href'], 'http') === 0 ? $menuItem['href'] : '/' . $menuItem['href'];
+        if ($menuItem[ 'href' ]) {
+            return strpos($menuItem[ 'href' ], 'http') === 0 ? $menuItem[ 'href' ] : '/' . $menuItem[ 'href' ];
         }
-        elseif($menuItem['article_slug']) {
+        elseif ($menuItem[ 'article_slug' ]) {
             // @todo refactor - break categories into single category per article...
-            $category = $this->postService->getCategories($menuItem['article_id'])->current();
+            $category = $this->postService->getCategories($menuItem[ 'article_id' ])->current();
 
-            if($category){
-                $params = ['segment_1' => $category->slug, 'segment_2' => $menuItem['article_slug']];
+            if ($category) {
+                $params = [ 'segment_1' => $category->slug, 'segment_2' => $menuItem[ 'article_slug' ] ];
             }
-            else{
-                $params = ['segment_1' => $menuItem['article_slug']];
+            else {
+                $params = [ 'segment_1' => $menuItem[ 'article_slug' ] ];
             }
 
             return $this->url->__invoke('post', $params);
         }
-        elseif($menuItem['category_slug']) {
-            return $this->url->__invoke('category', ['category' => $menuItem['category_slug']]);
+        elseif ($menuItem[ 'category_slug' ]) {
+            return $this->url->__invoke('category', [ 'category' => $menuItem[ 'category_slug' ] ]);
         }
         else {
             return '#';

Braces +4 added lines, -8 removed lines

@@ -35,24 +35,20 @@
     {
         if($menuItem['href']) {
             return strpos($menuItem['href'], 'http') === 0 ? $menuItem['href'] : '/' . $menuItem['href'];
-        }
-        elseif($menuItem['article_slug']) {
+        } elseif($menuItem['article_slug']) {
             // @todo refactor - break categories into single category per article...
             $category = $this->postService->getCategories($menuItem['article_id'])->current();
 
             if($category){
                 $params = ['segment_1' => $category->slug, 'segment_2' => $menuItem['article_slug']];
-            }
-            else{
+            } else{
                 $params = ['segment_1' => $menuItem['article_slug']];
             }
 
             return $this->url->__invoke('post', $params);
-        }
-        elseif($menuItem['category_slug']) {
+        } elseif($menuItem['category_slug']) {
             return $this->url->__invoke('category', ['category' => $menuItem['category_slug']]);
-        }
-        else {
+        } else {
             return '#';
         }
     }

src/Web/Action/VideoAction.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Web\Action;
 

src/Web/Action/EventAction.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Web\Action;
 

src/Article/src/Filter/ArticleFilter.php

Spacing +8 added lines, -8 removed lines

@@ -12,36 +12,36 @@
 
     public function getInputFilter()
     {
-        if(!$this->inputFilter) {
+        if (!$this->inputFilter) {
             $inputFilter = new InputFilter();
 
             $inputFilter->add([
                 '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 ] ],
                 ],
             ]);
 
             $inputFilter->add([
                 'name'       => 'published_at',
                 'required'   => true,
-                'filters'    => [['name' => 'StringTrim']],
-                'validators' => [['name' => 'NotEmpty'], ['name' => 'Date', 'options' => ['format' => 'Y-m-d H:i:s']]]
+                'filters'    => [ [ 'name' => 'StringTrim' ] ],
+                'validators' => [ [ 'name' => 'NotEmpty' ], [ 'name' => 'Date', 'options' => [ 'format' => 'Y-m-d H:i:s' ] ] ]
             ]);
 
             $inputFilter->add([
                 'name'     => 'status',
                 'required' => false,
-                'filters'  => [['name' => 'Boolean']],
+                'filters'  => [ [ 'name' => 'Boolean' ] ],
             ]);
 
             $inputFilter->add([
                 'name'     => 'is_wysiwyg_editor',
                 'required' => false,
-                'filters'  => [['name' => 'Boolean']],
+                'filters'  => [ [ 'name' => 'Boolean' ] ],
             ]);
 
             $this->inputFilter = $inputFilter;

data/phinx/migrations/20161003183353_create_articles_table.php

Spacing +10 added lines, -10 removed lines

@@ -6,19 +6,19 @@
 {
     public function up()
     {
-        $this->table('articles', ['id' => false, 'primary_key' => 'article_uuid'])
-            ->addColumn('article_uuid', 'binary', ['limit' => 16])
+        $this->table('articles', [ 'id' => false, 'primary_key' => 'article_uuid' ])
+            ->addColumn('article_uuid', 'binary', [ 'limit' => 16 ])
             ->addColumn('article_id', 'text')
-            ->addColumn('slug', 'text', ['null' => true])
-            ->addColumn('created_at', 'datetime', ['default' => 'CURRENT_TIMESTAMP'])
-            ->addColumn('published_at', 'datetime', ['default' => 'CURRENT_TIMESTAMP'])
+            ->addColumn('slug', 'text', [ 'null' => true ])
+            ->addColumn('created_at', 'datetime', [ 'default' => 'CURRENT_TIMESTAMP' ])
+            ->addColumn('published_at', 'datetime', [ 'default' => 'CURRENT_TIMESTAMP' ])
             ->addColumn('type', 'integer')// see Core\Entity\ArticleType
             ->addColumn('status', 'integer')// active, not active, ...
-            ->addColumn('admin_user_uuid', 'binary', ['limit' => 16])
-            ->addColumn('is_wysiwyg_editor', 'boolean', ['default' => false])
-            ->addForeignKey('admin_user_uuid', 'admin_users', 'admin_user_uuid', ['delete' => 'NO_ACTION', 'update' => 'NO_ACTION'])
-            ->addIndex('type', ['name' => 'type_INDEX'])
-            ->addIndex('published_at', ['name' => 'published_at_INDEX'])
+            ->addColumn('admin_user_uuid', 'binary', [ 'limit' => 16 ])
+            ->addColumn('is_wysiwyg_editor', 'boolean', [ 'default' => false ])
+            ->addForeignKey('admin_user_uuid', 'admin_users', 'admin_user_uuid', [ 'delete' => 'NO_ACTION', 'update' => 'NO_ACTION' ])
+            ->addIndex('type', [ 'name' => 'type_INDEX' ])
+            ->addIndex('published_at', [ 'name' => 'published_at_INDEX' ])
             ->create();
     }
 

src/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 +28 added lines, -28 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Article\Service;
 
@@ -71,8 +71,8 @@ 
     {
         $article = $this->articlePostsMapper->getBySlug($slug);
 
-        if($article) {
-            $article['categories'] = $this->getCategoryIds($article->article_id);
+        if ($article) {
+            $article[ 'categories' ] = $this->getCategoryIds($article->article_id);
         }
 
         return $article;
@@ -82,8 +82,8 @@ 
     {
         $article = $this->articlePostsMapper->get($articleId);
 
-        if($article) {
-            $article['categories'] = $this->getCategoryIds($articleId);
+        if ($article) {
+            $article[ 'categories' ] = $this->getCategoryIds($articleId);
         }
 
         return $article;
@@ -102,7 +102,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());
         }
 
@@ -119,19 +119,19 @@ 
         $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' ]
             ];
 
-        if($post['is_homepage']) {
-            $this->articlePostsMapper->update(['is_homepage' => false]);
+        if ($post[ 'is_homepage' ]) {
+            $this->articlePostsMapper->update([ 'is_homepage' => false ]);
         }
 
         $this->articleMapper->insert($article);
         $this->articlePostsMapper->insert($post);
 
-        if(isset($data['categories']) && $data['categories']) {
-            $categories = $this->categoryMapper->select(['category_id' => $data['categories']]);
-            $this->articleMapper->insertCategories($categories, $article['article_uuid']);
+        if (isset($data[ 'categories' ]) && $data[ 'categories' ]) {
+            $categories = $this->categoryMapper->select([ 'category_id' => $data[ 'categories' ] ]);
+            $this->articleMapper->insertCategories($categories, $article[ 'article_uuid' ]);
         }
     }
 
@@ -141,36 +141,36 @@ 
         $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' => $article->article_uuid];
+        $article = $articleFilter->getValues() + [ 'article_uuid' => $article->article_uuid ];
         $post    = $postFilter->getValues() + [
                 'featured_img' => $this->upload->uploadImage($data, 'featured_img'),
                 'main_img'     => $this->upload->uploadImage($data, 'main_img')
             ];
 
         // We dont 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' ]);
         }
 
-        if(!$post['main_img']) {
-            unset($post['main_img']);
+        if (!$post[ 'main_img' ]) {
+            unset($post[ 'main_img' ]);
         }
 
-        if($post['is_homepage']) {
-            $this->articlePostsMapper->update(['is_homepage' => false]);
+        if ($post[ 'is_homepage' ]) {
+            $this->articlePostsMapper->update([ 'is_homepage' => false ]);
         }
 
-        $this->articleMapper->update($article, ['article_uuid' => $article['article_uuid']]);
-        $this->articlePostsMapper->update($post, ['article_uuid' => $article['article_uuid']]);
-        $this->articleMapper->deleteCategories($article['article_uuid']);
+        $this->articleMapper->update($article, [ 'article_uuid' => $article[ 'article_uuid' ] ]);
+        $this->articlePostsMapper->update($post, [ 'article_uuid' => $article[ 'article_uuid' ] ]);
+        $this->articleMapper->deleteCategories($article[ 'article_uuid' ]);
 
-        if(isset($data['categories']) && $data['categories']) {
-            $categories = $this->categoryMapper->select(['category_id' => $data['categories']]);
-            $this->articleMapper->insertCategories($categories, $article['article_uuid']);
+        if (isset($data[ 'categories' ]) && $data[ 'categories' ]) {
+            $categories = $this->categoryMapper->select([ 'category_id' => $data[ 'categories' ] ]);
+            $this->articleMapper->insertCategories($categories, $article[ 'article_uuid' ]);
         }
     }
 
@@ -178,11 +178,11 @@ 
     {
         $post = $this->articlePostsMapper->get($id);
 
-        if(!$post) {
+        if (!$post) {
             throw new \Exception('Article not found!');
         }
 
-        $this->articlePostsMapper->delete(['article_uuid' => $post->article_uuid]);
+        $this->articlePostsMapper->delete([ 'article_uuid' => $post->article_uuid ]);
         $this->delete($post->article_uuid);
     }
 

src/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 +6 added lines, -6 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);
 

src/Web/ConfigProvider.php

Spacing +2 added lines, -2 removed lines

@@ -13,8 +13,8 @@
                     'article/pagination' => 'templates/partial/pagination.phtml',
                 ],
                 'paths' => [
-                    'templates' => ['templates'],
-                    'web'       => ['templates/web'],
+                    'templates' => [ 'templates' ],
+                    'web'       => [ 'templates/web' ],
                 ],
             ],