Schlaefer / Saito

plugins/Admin/src/Controller/CategoriesController.php

Doc Comments +3 added lines, -3 removed lines

@@ -35,7 +35,7 @@ 
     /**
      * add new category
      *
-     * @return \Cake\Network\Response|void
+     * @return \Cake\Http\Response|null
      */
     public function add()
     {
@@ -66,7 +66,7 @@ 
      * edit category
      *
      * @param string $id category-ID
-     * @return \Cake\Network\Response|void
+     * @return \Cake\Http\Response|null
      */
     public function edit($id)
     {
@@ -106,7 +106,7 @@ 
      *
      * @param string $id category-ID
      *
-     * @return \Cake\Network\Response|void
+     * @return \Cake\Http\Response|null
      */
     public function delete($id)
     {

Spacing +1 added lines, -1 removed lines

@@ -123,7 +123,7 @@
 
         switch ($this->request->getData('mode')) {
             case ('move'):
-                $targetId = (int)$this->request->getData('targetCategory');
+                $targetId = (int) $this->request->getData('targetCategory');
                 try {
                     $this->Categories->merge($category->get('id'), $targetId);
                     $this->Flash->set(

plugins/Admin/src/Controller/SettingsController.php

Doc Comments +1 added lines, -1 removed lines

@@ -89,7 +89,7 @@
      *
      * @param null $id settings-ID
      *
-     * @return \Cake\Network\Response|void
+     * @return \Cake\Http\Response|null
      */
     public function edit($id = null)
     {

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 /**
  * Saito - The Threaded Web Forum

plugins/Admin/src/Controller/UsersController.php

Doc Comments +2 added lines, -2 removed lines

@@ -45,7 +45,7 @@ 
     /**
      * add user
      *
-     * @return \Cake\Network\Response|void
+     * @return \Cake\Http\Response|null
      */
     public function add()
     {
@@ -68,7 +68,7 @@ 
      * delete user
      *
      * @param string $id user-ID
-     * @return \Cake\Network\Response|void
+     * @return \Cake\Http\Response|null
      */
     public function delete($id)
     {

Spacing +1 added lines, -1 removed lines

@@ -72,7 +72,7 @@
      */
     public function delete($id)
     {
-        $id = (int)$id;
+        $id = (int) $id;
         $exists = $this->Users->exists($id);
         if (!$exists) {
             $this->Flash->set(__('User not found.'), ['element' => 'error']);

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace Admin\Controller;
 
-use App\Controller\AppController;
 use App\Model\Table\UsersTable;
 
 /**

plugins/Api/src/Controller/ApiAppController.php

Unused Use Statements -1 removed lines

@@ -15,7 +15,6 @@
 use App\Controller\AppController;
 use Cake\Controller\Component\AuthComponent;
 use Cake\Core\Configure;
-use Cake\Event\Event;
 
 /**
  * Api App Controller

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 /**
  * Saito - The Threaded Web Forum

plugins/BbcodeParser/src/Lib/jBBCode/Definitions/JbbCodeDefinitions.php

Doc Comments +1 added lines

@@ -566,6 +566,7 @@
 
     /**
      * {@inheritDoc}
+     * @param string $content
      */
     protected function _getUrl($content, $attributes)
     {

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace Plugin\BbcodeParser\src\Lib\jBBCode\Definitions;
 
 use Cake\Cache\Cache;
-use Cake\Core\Configure;
 use Plugin\BbcodeParser\src\Lib\Helper\Message;
 use Plugin\BbcodeParser\src\Lib\Helper\UrlParserTrait;
 use Saito\DomainParser;

Spacing +16 added lines, -16 removed lines

@@ -71,7 +71,7 @@ 
             return $this->Html->link($url, $url, ['target' => '_blank']);
         }
 
-        $loader = function () use ($url) {
+        $loader = function() use ($url) {
             $embed = ['url' => $url];
 
             try {
@@ -107,7 +107,7 @@ 
 
         $callable = \Closure::fromCallable($loader);
 
-        $uid = 'embed-' . md5($url);
+        $uid = 'embed-'.md5($url);
         $info = Cache::remember($uid, $callable, 'bbcodeParserEmbed');
 
         return $this->_sHelper->Html->div('js-embed', '', ['id' => $uid, 'data-embed' => json_encode($info)]);
@@ -186,7 +186,7 @@ 
         }
 
         $ad = explode('|', $this->_sOptions->get('video_domains_allowed'));
-        $trim = function ($v) {
+        $trim = function($v) {
             return trim($v);
         };
         $this->_allowedVideoDomains = array_fill_keys(array_map($trim, $ad), 1);
@@ -271,9 +271,9 @@ 
             }
         }
 
-        $out = '<object classid="clsid:d27cdb6e-ae6d-11cf-96b8-444553540000" width="' . $width . '" height="' . $height . '">
-									<param name="movie" value="' . $url . '"></param>
-									<embed src="' . $url . '" width="' . $width . '" height="' . $height . '" type="application/x-shockwave-flash" wmode="opaque" style="width:' . $width . 'px; height:' . $height . 'px;" id="VideoPlayback" flashvars=""> </embed> </object>';
+        $out = '<object classid="clsid:d27cdb6e-ae6d-11cf-96b8-444553540000" width="'.$width.'" height="'.$height.'">
+									<param name="movie" value="' . $url.'"></param>
+									<embed src="' . $url.'" width="'.$width.'" height="'.$height.'" type="application/x-shockwave-flash" wmode="opaque" style="width:'.$width.'px; height:'.$height.'px;" id="VideoPlayback" flashvars=""> </embed> </object>';
 
         return $out;
     }
@@ -393,12 +393,12 @@ 
     {
         $listPieces = explode('[*]', $content);
         unset($listPieces[0]);
-        $listPieceProcessor = function ($li) {
-            return '<li>' . $li . '</li>' . "\n";
+        $listPieceProcessor = function($li) {
+            return '<li>'.$li.'</li>'."\n";
         };
         $listPieces = array_map($listPieceProcessor, $listPieces);
 
-        return '<ul>' . implode('', $listPieces) . '</ul>';
+        return '<ul>'.implode('', $listPieces).'</ul>';
     }
 }
 
@@ -420,14 +420,14 @@ 
         }
 
         $title = $this->_mbStrpad(
-            ' ' . __('Spoiler') . ' ',
+            ' '.__('Spoiler').' ',
             $length,
             '▇',
             STR_PAD_BOTH
         );
 
         $json = json_encode(['string' => $content]);
-        $id = 'spoiler_' . rand(0, 9999999999999);
+        $id = 'spoiler_'.rand(0, 9999999999999);
 
         $out = <<<EOF
 <div class="richtext-spoiler" style="display: inline;">
@@ -475,18 +475,18 @@ 
         $result = null;
         $repeat = ceil($strLen - $padStrLen + $padLen);
         if ($dir == STR_PAD_RIGHT) {
-            $result = $str . str_repeat($padStr, $repeat);
+            $result = $str.str_repeat($padStr, $repeat);
             $result = mb_substr($result, 0, $padLen);
         } else {
             if ($dir == STR_PAD_LEFT) {
-                $result = str_repeat($padStr, $repeat) . $str;
+                $result = str_repeat($padStr, $repeat).$str;
                 $result = mb_substr($result, -$padLen);
             } else {
                 if ($dir == STR_PAD_BOTH) {
                     $length = ($padLen - $strLen) / 2;
                     $repeat = ceil($length / $padStrLen);
-                    $result = mb_substr(str_repeat($padStr, $repeat), 0, floor($length)) .
-                        $str .
+                    $result = mb_substr(str_repeat($padStr, $repeat), 0, floor($length)).
+                        $str.
                         mb_substr(str_repeat($padStr, $repeat), 0, ceil($length));
                 }
             }
@@ -517,7 +517,7 @@ 
             $attributes['img'] = $attributes['width'];
         }
         if (!empty($attributes['height'])) {
-            $attributes['img'] .= 'x' . $attributes['height'];
+            $attributes['img'] .= 'x'.$attributes['height'];
         }
 
         return parent::_parse($content, $attributes, $node);

plugins/BbcodeParser/src/Lib/Parser.php

Doc Comments +3 added lines, -3 removed lines

@@ -208,7 +208,7 @@ 
      * Constructor
      *
      * @param Helper $Helper helper
-     * @param array $settings settings
+     * @param MarkupSettings $settings settings
      */
     public function __construct(Helper $Helper, MarkupSettings $settings)
     {
@@ -225,7 +225,7 @@ 
      * - `multimedia` bool true|false parse or ignore multimedia content
      * - `embed` bool true|false parse or ignore embed content
      *
-     * @return mixed|string
+     * @return string
      */
     public function parse($string, array $options = [])
     {
@@ -315,7 +315,7 @@ 
      * Add definitin set
      *
      * @param string $set set
-     * @param array $options options
+     * @param MarkupSettings $options options
      *
      * @return void
      * @throws \Exception

Spacing +7 added lines, -7 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 /**
  * Saito - The Threaded Web Forum
@@ -281,7 +281,7 @@ 
         // serializing complex objects kills PHP
         $serializable = array_filter(
             $options,
-            function ($value) {
+            function($value) {
                 return !is_object($value);
             }
         );
@@ -335,7 +335,7 @@ 
                     $this->_Parser->addCodeDefinition($builder->build());
                     break;
                 case 'class':
-                    $class = '\Plugin\BbcodeParser\src\Lib\jBBCode\Definitions\\' . ucfirst($title);
+                    $class = '\Plugin\BbcodeParser\src\Lib\jBBCode\Definitions\\'.ucfirst($title);
                     $this->_Parser->addCodeDefinition(new $class($this->_Helper, $options));
                     break;
                 default:
@@ -355,10 +355,10 @@ 
             return;
         }
 
-        $folder = __DIR__ . '/jBBCode/Definitions/';
-        require_once $folder . 'JbbCodeDefinitions.php';
-        require_once $folder . 'JbbHtml5MediaCodeDefinition.php';
-        require_once $folder . 'JbbCodeCodeDefinition.php';
+        $folder = __DIR__.'/jBBCode/Definitions/';
+        require_once $folder.'JbbCodeDefinitions.php';
+        require_once $folder.'JbbHtml5MediaCodeDefinition.php';
+        require_once $folder.'JbbCodeCodeDefinition.php';
 
         $this->areCombinedClassFilesLoaded = true;
     }

plugins/Bookmarks/src/Lib/Bookmarks.php

Unused Use Statements -1 removed lines

@@ -12,7 +12,6 @@
 
 namespace Bookmarks\Lib;
 
-use App\Controller\Component\CurrentUserComponent;
 use Cake\ORM\TableRegistry;
 use Saito\User\CurrentUser\CurrentUserInterface;
 

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 /**
  * Saito - The Threaded Web Forum

plugins/Cron/src/Lib/Cron.php

Doc Comments +1 added lines, -1 removed lines

@@ -137,7 +137,7 @@
      *
      * @param string $msg message
      *
-     * @return bool
+     * @return boolean|null
      */
     protected function _log($msg)
     {

Spacing +1 added lines, -1 removed lines

@@ -74,7 +74,7 @@
             }
             $jobsExecuted++;
             $job->execute();
-            $this->_log('Run cron-job ' . $job->uid);
+            $this->_log('Run cron-job '.$job->uid);
             $this->_lastRuns[$job->due][$job->uid] = $this->_now;
         }
         if ($jobsExecuted === 0) {

plugins/Feeds/src/Controller/PostingsController.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use App\Controller\AppController;
 use App\Model\Table\EntriesTable;
-use Cake\Core\Configure;
 use Cake\Event\Event;
 use Cake\Http\Exception\BadRequestException;
 use Feeds\Model\Behavior\FeedsPostingBehavior;