Schlaefer / Saito

src/Controller/EntriesController.php

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/AppController.php

Unused Use Statements -1 removed lines

@@ -9,7 +9,6 @@
 use Cake\Event\Event;
 use Cake\Http\Response;
 use Cake\I18n\I18n;
-use Cake\Routing\Router;
 use Saito\App\SettingsImmutable;
 use Saito\Event\SaitoEventManager;
 use Saito\User\CurrentUser\CurrentUser;

src/View/Helper/LayoutHelper.php

Unused Use Statements -1 removed lines

@@ -9,7 +9,6 @@
 
 namespace App\View\Helper;
 
-use Cake\Core\Configure;
 use Cake\View\Helper;
 use Cake\View\StringTemplateTrait;
 

Spacing +3 added lines, -3 removed lines

@@ -64,7 +64,7 @@ 
     {
         $html = "<i class=\"saito-icon fa fa-{$icon}\"></i>";
         if (!empty($text)) {
-                $html .= '&nbsp;<span class="saito-icon-text">' . $text . '</span>';
+                $html .= '&nbsp;<span class="saito-icon-text">'.$text.'</span>';
         }
 
         return $html;
@@ -80,7 +80,7 @@ 
     public function dropdownMenuButton(array $menuItems, array $options = [])
     {
         $options += ['class' => 'btn btn-primary'];
-        $options['class'] = $options['class'] . ' dropdown-toggle';
+        $options['class'] = $options['class'].' dropdown-toggle';
         $menu = [];
         foreach ($menuItems as $menuItem) {
             if ($menuItem === 'divider') {
@@ -173,7 +173,7 @@ 
         $contentArray += ['first' => '', 'middle' => '', 'last' => ''];
         $out = '';
         foreach (['first', 'middle', 'last'] as $key) {
-            $out .= '<div class="' . $key . '">';
+            $out .= '<div class="'.$key.'">';
             $out .= $options['escape'] ? h($contentArray[$key]) : $contentArray[$key];
             $out .= '</div>';
         }

src/Controller/UsersController.php

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/Saito/Posting/Decorator/AbstractPostingDecorator.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
             return call_user_func_array([$this->_Posting, $method], $args);
         }
         throw new \RuntimeException(
-            'Undefined method ' . get_class($this) . '::' . $method
+            'Undefined method '.get_class($this).'::'.$method
         );
     }
 

src/Lib/Saito/User/Blocker/ManualBlocker.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 /**
  * Saito - The Threaded Web Forum
@@ -70,6 +70,6 @@ 
 
         $entity = $this->Table->newEntity($conditions);
 
-        return (bool)$this->Table->save($entity);
+        return (bool) $this->Table->save($entity);
     }
 }

src/Lib/Saito/Exception/SaitoBlackholeException.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
      */
     public function __construct($type = null, $data = [])
     {
-        $message = 'Request was blackholed. Type: ' . $type;
+        $message = 'Request was blackholed. Type: '.$type;
         $logger = new ExceptionLogger;
         $logger->write($message, $data);
         parent::__construct($message, 400);

src/Lib/Saito/Test/AssertTrait.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 /**
  * Saito - The Threaded Web Forum
@@ -89,7 +89,7 @@ 
     {
         $document = new \DOMDocument;
         libxml_use_internal_errors(true);
-        $document->loadHTML('<!DOCTYPE html>' . $html);
+        $document->loadHTML('<!DOCTYPE html>'.$html);
         $xpath = new \DOMXPath($document);
         libxml_clear_errors();
 

src/Lib/Saito/Test/IntegrationTestCase.php

Spacing +4 added lines, -4 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 /**
  * Saito - The Threaded Web Forum
@@ -136,7 +136,7 @@ 
         $Mock = $this->getMockForTableParent($table, $methods);
         EventManager::instance()->on(
             'Controller.initialize',
-            function (Event $event) use ($table, $Mock) {
+            function(Event $event) use ($table, $Mock) {
                 $Controller = $event->getSubject();
                 $Controller->{$table} = $Mock;
             }
@@ -160,7 +160,7 @@ 
         $this->configRequest([
             'headers' => [
                 'Accept' => 'application/json',
-                'Authorization' => 'bearer ' . $jwtToken,
+                'Authorization' => 'bearer '.$jwtToken,
             ]
         ]);
     }
@@ -314,7 +314,7 @@ 
     {
         $this->assertContainsTag(
             $expected,
-            (string)$this->_controller->response->getBody()
+            (string) $this->_controller->response->getBody()
         );
     }
 }