Schlaefer / Saito

src/Controller/EntriesController.php

Doc Comments +7 added lines, -7 removed lines

@@ -64,7 +64,7 @@ 
     /**
      * posting index
      *
-     * @return void|\Cake\Network\Response
+     * @return \Cake\Http\Response|null
      */
     public function index()
     {
@@ -111,7 +111,7 @@ 
      * Mix view
      *
      * @param string $tid thread-ID
-     * @return void|Response
+     * @return \Cake\Http\Response|null
      * @throws NotFoundException
      */
     public function mix($tid)
@@ -184,7 +184,7 @@ 
      * View posting.
      *
      * @param string $id posting-ID
-     * @return \Cake\Network\Response|void
+     * @return \Cake\Http\Response|null
      */
     public function view($id = null)
     {
@@ -236,7 +236,7 @@ 
      * Add new posting.
      *
      * @param null|string $id parent-ID if is answer
-     * @return void|\Cake\Network\Response
+     * @return \Cake\Http\Response|null
      * @throws ForbiddenException
      */
     public function add($id = null)
@@ -345,7 +345,7 @@ 
      * Get thread-line to insert after an inline-answer
      *
      * @param string $id posting-ID
-     * @return void|\Cake\Network\Response
+     * @return \Cake\Http\Response|null
      */
     public function threadLine($id = null)
     {
@@ -364,7 +364,7 @@ 
      * Edit posting
      *
      * @param string $id posting-ID
-     * @return void|\Cake\Network\Response
+     * @return \Cake\Http\Response|null
      * @throws NotFoundException
      * @throws BadRequestException
      */
@@ -570,7 +570,7 @@ 
      * @param string $id posting-ID
      * @param string $toggle property
      *
-     * @return \Cake\Network\Response
+     * @return \Cake\Http\Response
      */
     public function ajaxToggle($id = null, $toggle = null)
     {

Spacing +8 added lines, -8 removed lines

@@ -81,9 +81,9 @@ 
         $threads = $this->Threads->paginate($order);
         $this->set('entries', $threads);
 
-        $currentPage = (int)$this->request->getQuery('page') ?: 1;
+        $currentPage = (int) $this->request->getQuery('page') ?: 1;
         if ($currentPage > 1) {
-            $this->set('titleForLayout', __('page') . ' ' . $currentPage);
+            $this->set('titleForLayout', __('page').' '.$currentPage);
         }
         if ($currentPage === 1) {
             if ($this->MarkAsRead->refresh()) {
@@ -116,7 +116,7 @@ 
      */
     public function mix($tid)
     {
-        $tid = (int)$tid;
+        $tid = (int) $tid;
         if ($tid <= 0) {
             throw new BadRequestException();
         }
@@ -485,7 +485,7 @@ 
                 $redirect = '/';
             } else {
                 $message = __('delete_subtree_success');
-                $redirect = '/entries/view/' . $posting->get('pid');
+                $redirect = '/entries/view/'.$posting->get('pid');
             }
         } else {
             $flashType = 'error';
@@ -552,7 +552,7 @@ 
         $targetId = $this->request->getData('targetId');
         if (!empty($targetId)) {
             if ($this->Entries->threadMerge($sourceId, $targetId)) {
-                $this->redirect('/entries/view/' . $sourceId);
+                $this->redirect('/entries/view/'.$sourceId);
 
                 return;
             } else {
@@ -583,11 +583,11 @@ 
             throw new BadRequestException;
         }
 
-        $current = $this->Entries->toggle((int)$id, $toggle);
+        $current = $this->Entries->toggle((int) $id, $toggle);
         if ($current) {
-            $out['html'] = __d('nondynamic', $toggle . '_unset_entry_link');
+            $out['html'] = __d('nondynamic', $toggle.'_unset_entry_link');
         } else {
-            $out['html'] = __d('nondynamic', $toggle . '_set_entry_link');
+            $out['html'] = __d('nondynamic', $toggle.'_set_entry_link');
         }
 
         $this->response = $this->response->withType('json');

Unused Use Statements -3 removed lines

@@ -9,7 +9,6 @@ 
 
 namespace App\Controller;
 
-use App\Controller\Component\CurrentUserComponent;
 use App\Controller\Component\MarkAsReadComponent;
 use App\Controller\Component\ThreadsComponent;
 use App\Model\Entity\Entry;
@@ -21,9 +20,7 @@ 
 use Cake\Http\Exception\ForbiddenException;
 use Cake\Http\Exception\MethodNotAllowedException;
 use Cake\Http\Exception\NotFoundException;
-use Cake\I18n\Time;
 use Cake\Routing\RequestActionTrait;
-use Saito\App\Registry;
 use Saito\Posting\Posting;
 use Saito\User\ForumsUserInterface;
 use Stopwatch\Lib\Stopwatch;

src/Controller/PreviewController.php

Doc Comments +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
     /**
      * Generate posting preview for JSON frontend.
      *
-     * @return \Cake\Network\Response|void
+     * @return null|\Cake\Http\Response
      * @throws BadRequestException
      * @throws ForbiddenException
      */

Spacing +2 added lines, -2 removed lines

@@ -68,9 +68,9 @@
             $jsonApiErrors = ['errors' => []];
             foreach ($errors as $field => $error) {
                 $out = [
-                    'meta' => ['field' => '#' . $this->_domId($field)],
+                    'meta' => ['field' => '#'.$this->_domId($field)],
                     'status' => '400',
-                    'title' => __d('nondynamic', $field) . ": " . __d('nondynamic', current($error)),
+                    'title' => __d('nondynamic', $field).": ".__d('nondynamic', current($error)),
                 ];
 
                 $jsonApiErrors['errors'][] = $out;

src/Controller/StatusController.php

Doc Comments +2 added lines, -2 removed lines

@@ -14,7 +14,7 @@ 
     /**
      * Current app status ping
      *
-     * @return string
+     * @return \Cake\Http\Response
      * @throws BadRequestException
      */
     public function status()
@@ -55,7 +55,7 @@ 
      * Get status as JSON response
      *
      * @param string $data json-encoded data
-     * @return mixed
+     * @return string
      */
     protected function _statusAsJson($data)
     {

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use Cake\Event\Event;
 use Cake\Http\Exception\BadRequestException;
-use Cake\ORM\TableRegistry;
 
 class StatusController extends AppController
 {

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
         $this->response->disableCache();
         $out = '';
         $out .= "retry: $retry\n";
-        $out .= 'data: ' . $data . "\n\n";
+        $out .= 'data: '.$data."\n\n";
 
         return $out;
     }

src/Controller/UsersController.php

Doc Comments +8 added lines, -8 removed lines

@@ -56,7 +56,7 @@ 
     /**
      * Login user.
      *
-     * @return void|\Cake\Network\Response
+     * @return null|Response
      */
     public function login()
     {
@@ -309,7 +309,7 @@ 
      *
      * @param int $blockedId user to ignore
      * @param bool $set block or unblock
-     * @return \Cake\Network\Response
+     * @return Response|null
      */
     protected function _ignore($blockedId, $set)
     {
@@ -359,7 +359,7 @@ 
      * View user profile.
      *
      * @param null $id user-ID
-     * @return \Cake\Network\Response|void
+     * @return null|Response
      */
     public function view($id = null)
     {
@@ -420,8 +420,8 @@ 
     /**
      * Set user avatar.
      *
-     * @param string $userId user-ID
-     * @return void|\Cake\Network\Response
+     * @param integer $userId user-ID
+     * @return Response|null
      */
     public function avatar(int $userId)
     {
@@ -454,7 +454,7 @@ 
      *
      * @param null $id user-ID
      *
-     * @return \Cake\Network\Response|void
+     * @return Response|null
      */
     public function edit($id = null)
     {
@@ -661,7 +661,7 @@ 
     /**
      * Set slidetab-order.
      *
-     * @return \Cake\Network\Response
+     * @return Response
      * @throws BadRequestException
      */
     public function slidetabOrder()
@@ -709,7 +709,7 @@ 
      * Set category for user.
      *
      * @param null $id category-ID
-     * @return \Cake\Network\Response
+     * @return Response|null
      * @throws ForbiddenException
      */
     public function setcategory($id = null)

Unused Use Statements -1 removed lines

@@ -10,7 +10,6 @@
 namespace App\Controller;
 
 use App\Controller\Component\RefererComponent;
-use App\Controller\Component\ThemesComponent;
 use App\Form\BlockForm;
 use App\Model\Entity\User;
 use App\Model\Table\UsersTable;

Spacing +8 added lines, -8 removed lines

@@ -234,7 +234,7 @@ 
         }
         $code = $this->request->getQuery('c');
         try {
-            $activated = $this->Users->activate((int)$id, $code);
+            $activated = $this->Users->activate((int) $id, $code);
         } catch (\Exception $e) {
             $activated = false;
         }
@@ -292,7 +292,7 @@ 
     public function ignore()
     {
         $this->request->allowMethod('POST');
-        $blockedId = (int)$this->request->getData('id');
+        $blockedId = (int) $this->request->getData('id');
         $this->_ignore($blockedId, true);
     }
 
@@ -304,7 +304,7 @@ 
     public function unignore()
     {
         $this->request->allowMethod('POST');
-        $blockedId = (int)$this->request->getData('id');
+        $blockedId = (int) $this->request->getData('id');
         $this->_ignore($blockedId, false);
     }
 
@@ -318,7 +318,7 @@ 
     protected function _ignore($blockedId, $set)
     {
         $userId = $this->CurrentUser->getId();
-        if ((int)$userId === (int)$blockedId) {
+        if ((int) $userId === (int) $blockedId) {
             throw new BadRequestException();
         }
         if ($set) {
@@ -379,7 +379,7 @@ 
         $user = $this->Users->find()
             ->contain(
                 [
-                    'UserBlocks' => function ($q) {
+                    'UserBlocks' => function($q) {
                         return $q->find('assocUsers');
                     },
                     'UserOnline'
@@ -540,7 +540,7 @@ 
             throw new BadRequestException;
         }
 
-        $id = (int)$this->request->getData('lockUserId');
+        $id = (int) $this->request->getData('lockUserId');
         if (!$this->Users->exists($id)) {
             throw new NotFoundException('User does not exist.', 1524298280);
         }
@@ -554,7 +554,7 @@ 
             $this->Flash->set($message, ['element' => 'error']);
         } else {
             try {
-                $duration = (int)$this->request->getData('lockPeriod');
+                $duration = (int) $this->request->getData('lockPeriod');
                 $blocker = new ManualBlocker($this->CurrentUser->getId(), $duration);
                 $status = $this->Users->UserBlocks->block($blocker, $id);
                 if (!$status) {
@@ -679,7 +679,7 @@ 
         $allowed = $this->Slidetabs->getAvailable();
         $order = array_filter(
             $order,
-            function ($item) use ($allowed) {
+            function($item) use ($allowed) {
                 return in_array($item, $allowed);
             }
         );

src/Lib/Model/Table/AppSettingTable.php

Doc Comments +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
      *
      * @param Event $event event
      * @param Entity $entity entity
-     * @param array|\ArrayObject $options options
+     * @param \ArrayObject $options options
      *  - 'clearCache' set to 'false' to prevent cache clearing
      * @return void
      */

src/Lib/Model/Table/AppTable.php

Unused Use Statements -1 removed lines

@@ -6,7 +6,6 @@
 use Cake\Database\Expression\QueryExpression;
 use Cake\Event\Event;
 use Cake\Event\EventManager;
-use Cake\ORM\Entity;
 use Cake\ORM\Table;
 use Saito\Event\SaitoEventManager;
 

Doc Comments +1 added lines

@@ -125,6 +125,7 @@
 
     /**
      * {@inheritdoc}
+     * @param string $key
      */
     public function getConfig($key = null, $default = null)
     {

Spacing +1 added lines, -1 removed lines

@@ -129,7 +129,7 @@
     public function getConfig($key = null, $default = null)
     {
         if (is_string($key)) {
-            $setting = Configure::read('Saito.Settings.' . $key);
+            $setting = Configure::read('Saito.Settings.'.$key);
             if ($setting !== null) {
                 return $setting;
             }

src/Lib/Saito/Event/SaitoEventManager.php

Doc Comments +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
     /**
      * attaches event-listener
      *
-     * @param string|SaitoEventListener $key key
+     * @param \Saito\Cache\EntriesCacheSupportCachelet $key key
      * @param null $callable function if $key is set
      * @return void
      * @throws InvalidArgumentException

Spacing +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
     {
         $data = ($event->getData()) ?: [];
         $data += ['subject' => $event->getSubject()];
-        $name = 'Event.Saito.' . $event->getName();
+        $name = 'Event.Saito.'.$event->getName();
         $this->dispatch($name, $data);
     }
 

src/Lib/Saito/Posting/Decorator/UserPostingTrait.php

Doc Comments +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
     /**
      * Checks if answering an entry is allowed
      *
-     * @return bool
+     * @return string|boolean
      */
     public function isAnsweringForbidden()
     {

Spacing +1 added lines, -1 removed lines

@@ -167,7 +167,7 @@
     {
         if (!$this->isRoot()) {
             throw new \RuntimeException(
-                'Posting with id ' . $this->get('id') . ' is no root posting.'
+                'Posting with id '.$this->get('id').' is no root posting.'
             );
         }
         if (!$this->_CurrentUser->get('last_refresh')) {

src/Lib/Saito/Posting/Posting.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use Saito\Posting\Basic\BasicPostingInterface;
 use Saito\Posting\Basic\BasicPostingTrait;
-use Saito\Posting\Decorator\PostingTrait;
 use Saito\Posting\Decorator\UserPostingTrait;
 use Saito\Thread\Thread;
 use Saito\User\CurrentUser\CurrentUser;

Spacing +1 added lines, -1 removed lines

@@ -102,7 +102,7 @@
     {
         $postings = [];
         $this->map(
-            function ($node) use (&$postings) {
+            function($node) use (&$postings) {
                 $postings[$node->get('id')] = $node;
             },
             false