phpsrbija / unfinished

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/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/Admin/src/Mapper/AdminUsersMapper.php

Doc Comments +12 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,11 @@ 
         return $this->update(['last_login' => date('Y-m-d H:i:s')], ['admin_user_id' => $userId]);
     }
 
+    /**
+     * @param string $userId
+     *
+     * @return \Zend\Db\Sql\Select
+     */
     public function getPaginationSelect($userId)
     {
         $select = $this->getSql()->select()->order(['created_at' => 'desc']);
@@ -67,6 +74,9 @@ 
         return $select;
     }
 
+    /**
+     * @param integer $limit
+     */
     public function getRandom($limit)
     {
         $select = $this->getSql()->select()

Spacing +6 added lines, -6 removed lines

@@ -32,7 +32,7 @@ 
 
     public function get($id)
     {
-        return $this->select(['admin_user_id' => $id])->current();
+        return $this->select([ 'admin_user_id' => $id ])->current();
     }
 
     /**
@@ -43,7 +43,7 @@ 
      */
     public function getByEmail(string $email)
     {
-        return $this->select(['email' => $email])->current();
+        return $this->select([ 'email' => $email ])->current();
     }
 
     /**
@@ -54,12 +54,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);
 
@@ -69,7 +69,7 @@ 
     public function getRandom($limit)
     {
         $select = $this->getSql()->select()
-            ->where(['status' => 1])
+            ->where([ 'status' => 1 ])
             ->order(new Expression('rand()'))
             ->limit($limit);
 
@@ -78,7 +78,7 @@ 
 
     public function getUuid($adminUserId)
     {
-        $user = $this->select(['admin_user_id' => $adminUserId])->current();
+        $user = $this->select([ 'admin_user_id' => $adminUserId ])->current();
 
         if (!$user) {
             throw new \Exception('Admin user does not exist!', 400);

packages/Article/src/Mapper/ArticleEventsMapper.php

Doc Comments +3 added lines

@@ -33,6 +33,9 @@
         $this->adapter = $adapter;
     }
 
+    /**
+     * @param integer $status
+     */
     public function getPaginationSelect($status = null)
     {
         $select = $this->getSql()->select()

Spacing +18 added lines, -18 removed lines

@@ -34,17 +34,17 @@ 
     public function getPaginationSelect($status = null)
     {
         $select = $this->getSql()->select()
-            ->columns(['title', 'body', 'longitude', 'latitude'])
+            ->columns([ 'title', 'body', 'longitude', 'latitude' ])
             ->join('articles', 'article_events.article_uuid = articles.article_uuid')
             ->join(
                 'admin_users',
                 'admin_users.admin_user_uuid = articles.admin_user_uuid',
-                ['admin_user_id', 'first_name', 'last_name']
-            )->where(['articles.type' => ArticleType::EVENT])
-            ->order(['created_at' => 'desc']);
+                [ 'admin_user_id', 'first_name', 'last_name' ]
+            )->where([ 'articles.type' => ArticleType::EVENT ])
+            ->order([ 'created_at' => 'desc' ]);
 
         if ($status) {
-            $select->where(['articles.status' => (int)$status]);
+            $select->where([ 'articles.status' => (int) $status ]);
         }
 
         return $select;
@@ -56,10 +56,10 @@ 
             ->join('articles', 'article_events.article_uuid = articles.article_uuid')
             ->join(
                 'category', 'category.category_uuid = articles.category_uuid',
-                ['category_slug' => 'slug', 'category_name' => 'name', 'category_id'], 'left'
+                [ 'category_slug' => 'slug', 'category_name' => 'name', 'category_id' ], 'left'
             )
-            ->join('admin_users', 'admin_users.admin_user_uuid = articles.admin_user_uuid', ['admin_user_id'], 'left')
-            ->where(['articles.article_id' => $id]);
+            ->join('admin_users', 'admin_users.admin_user_uuid = articles.admin_user_uuid', [ 'admin_user_id' ], 'left')
+            ->where([ 'articles.article_id' => $id ]);
 
         return $this->selectWith($select)->current();
     }
@@ -70,9 +70,9 @@ 
             ->join('articles', 'article_events.article_uuid = articles.article_uuid')
             ->join(
                 'category', 'category.category_uuid = articles.category_uuid',
-                ['category_slug' => 'slug', 'category_name' => 'name', 'category_id'], 'left'
+                [ 'category_slug' => 'slug', 'category_name' => 'name', 'category_id' ], 'left'
             )
-            ->where(['articles.slug' => $slug]);
+            ->where([ 'articles.slug' => $slug ]);
 
         return $this->selectWith($select)->current();
     }
@@ -83,9 +83,9 @@ 
             ->join(
                 'articles',
                 'article_events.article_uuid = articles.article_uuid',
-                ['article_id', 'slug', 'published_at']
-            )->where(['articles.status' => 1])
-            ->order(['published_at' => 'desc'])
+                [ 'article_id', 'slug', 'published_at' ]
+            )->where([ 'articles.status' => 1 ])
+            ->order([ 'published_at' => 'desc' ])
             ->limit($limit);
 
         return $this->selectWith($select);
@@ -94,11 +94,11 @@ 
     public function getFuture()
     {
         $select = $this->getSql()->select()
-            ->where(['articles.status' => 1])
+            ->where([ 'articles.status' => 1 ])
             ->join(
                 'articles',
                 'articles.article_uuid = article_events.article_uuid',
-                ['article_id', 'slug', 'published_at']
+                [ 'article_id', 'slug', 'published_at' ]
             )->order(new Expression('rand()'));
 
         $select->where->greaterThanOrEqualTo('end_at', date('Y-m-d H:i:s'));
@@ -109,12 +109,12 @@ 
     public function getPastSelect()
     {
         $select = $this->getSql()->select()
-            ->where(['articles.status' => 1])
+            ->where([ 'articles.status' => 1 ])
             ->join(
                 'articles',
                 'articles.article_uuid = article_events.article_uuid',
-                ['article_id', 'slug', 'published_at']
-            )->order(['start_at' => 'desc']);
+                [ 'article_id', 'slug', 'published_at' ]
+            )->order([ 'start_at' => 'desc' ]);
         $select->where->lessThan('end_at', date('Y-m-d H:i:s'));
 
         return $select;

packages/Admin/src/Middleware/AdminAuth.php

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
         if (!$user) {
             return $response->withStatus(302)->withHeader(
                 'Location',
-                $this->router->generateUri('auth', ['action' => 'login'])
+                $this->router->generateUri('auth', [ 'action' => 'login' ])
             );
         }
 

packages/Admin/src/Factory/Service/AdminUserServiceFactory.php

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Admin\Factory\Service;
 
@@ -21,8 +21,8 @@ 
      */
     public function __invoke(ContainerInterface $container): AdminUserService
     {
-        $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' ]);
 
         return new AdminUserService(
             new Bcrypt(),

packages/Category/tests/View/Helper/CategoryHelperFactoryTest.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')