Schlaefer / Saito

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;

plugins/Feeds/src/View/Helper/FeedsHelper.php

Doc Comments +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
     /**
      * Get RssWriter feed
      *
-     * @return Channel
+     * @return Feed
      */
     public function getFeed(): Feed
     {

plugins/ImageUploader/src/Model/Table/UploadsTable.php

Unused Use Statements -1 removed lines

@@ -14,7 +14,6 @@
 
 use App\Lib\Model\Table\AppTable;
 use Cake\Core\Configure;
-use Cake\Datasource\EntityInterface;
 use Cake\Event\Event;
 use Cake\Filesystem\File;
 use Cake\ORM\RulesChecker;

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
@@ -51,12 +51,12 @@ 
             ->notBlank('user_id')
             ->requirePresence(['name', 'size', 'type', 'user_id'], 'create');
 
-        $maxUploadSize = (int)Configure::read('Saito.Settings.upload_max_img_size');
+        $maxUploadSize = (int) Configure::read('Saito.Settings.upload_max_img_size');
         $validator->add(
             'document',
             [
                 'fileSize' => [
-                    'rule' => ['fileSize', '<', $maxUploadSize . 'kB'],
+                    'rule' => ['fileSize', '<', $maxUploadSize.'kB'],
                     'message' => __d(
                         'image_uploader',
                         'validation.error.fileSize',
@@ -83,9 +83,9 @@ 
     public function buildRules(RulesChecker $rules)
     {
         // check max allowed number of uploads per user
-        $nMax = (int)Configure::read('Saito.Settings.upload_max_number_of_uploads');
+        $nMax = (int) Configure::read('Saito.Settings.upload_max_number_of_uploads');
         $rules->add(
-            function (Upload $entity, array $options) use ($nMax) {
+            function(Upload $entity, array $options) use ($nMax) {
                 $count = $this->findByUserId($entity->get('user_id'))->count();
 
                 return $count < $nMax;
@@ -205,8 +205,8 @@ 
         $ratio = $size / $target;
         $ratio = sqrt($ratio);
 
-        $newwidth = (int)($width / $ratio);
-        $newheight = (int)($height / $ratio);
+        $newwidth = (int) ($width / $ratio);
+        $newheight = (int) ($height / $ratio);
         $destination = imagecreatetruecolor($newwidth, $newheight);
 
         $source = imagecreatefromstring($raw);

plugins/ImageUploader/tests/TestCase/Controller/UploadsControllerTest.php

Unused Use Statements -2 removed lines

@@ -13,13 +13,11 @@
 namespace ImageUploader\Test\TestCase\Controller;
 
 use Api\Error\Exception\GenericApiException;
-use Cake\Cache\Cache;
 use Cake\Core\Configure;
 use Cake\Core\Plugin;
 use Cake\Filesystem\File;
 use Cake\Http\Exception\UnauthorizedException;
 use Cake\ORM\TableRegistry;
-use claviska\SimpleImage;
 use Saito\Exception\SaitoForbiddenException;
 use Saito\Test\IntegrationTestCase;
 

Spacing +11 added lines, -11 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 /**
  * Saito - The Threaded Web Forum
@@ -45,7 +45,7 @@ 
     {
         parent::setUp();
 
-        $this->file = new File(TMP . 'my new-upload.png');
+        $this->file = new File(TMP.'my new-upload.png');
         $this->mockMediaFile($this->file);
     }
 
@@ -73,7 +73,7 @@ 
                 0 => [
                     'file' => [
                         'tmp_name' => $this->file->path,
-                        'name' => $this->file->name() . '.' . $this->file->ext(),
+                        'name' => $this->file->name().'.'.$this->file->ext(),
                         'size' => $this->file->size(),
                         'type' => $this->file->mime(),
                     ]
@@ -82,7 +82,7 @@ 
         ];
         $this->post('api/v2/uploads.json', $data);
 
-        $response = json_decode((string)$this->_response->getBody(), true);
+        $response = json_decode((string) $this->_response->getBody(), true);
 
         $this->assertResponseCode(200);
 
@@ -112,12 +112,12 @@ 
     {
         $this->loginJwt(1);
         unset($this->file);
-        $this->file = new File(TMP . 'tmp_exif.jpg');
+        $this->file = new File(TMP.'tmp_exif.jpg');
 
-        $fixture = new File($path = Plugin::path('ImageUploader') . 'tests/Fixture/exif-with-location.jpg');
+        $fixture = new File($path = Plugin::path('ImageUploader').'tests/Fixture/exif-with-location.jpg');
         $fixture->copy($this->file->path);
 
-        $readExif = function (File $file) {
+        $readExif = function(File $file) {
             //@codingStandardsIgnoreStart
             return @exif_read_data($file->path);
             //@codingStandardsIgnoreEnd
@@ -132,7 +132,7 @@ 
                 0 => [
                     'file' => [
                         'tmp_name' => $this->file->path,
-                        'name' => $this->file->name() . '.' . $this->file->ext(),
+                        'name' => $this->file->name().'.'.$this->file->ext(),
                         'size' => $this->file->size(),
                         'type' => $this->file->mime(),
                     ]
@@ -141,7 +141,7 @@ 
         ];
         $this->post('api/v2/uploads.json', $data);
 
-        $response = json_decode((string)$this->_response->getBody(), true);
+        $response = json_decode((string) $this->_response->getBody(), true);
 
         $this->assertResponseCode(200);
 
@@ -222,7 +222,7 @@ 
 
         $this->get('api/v2/uploads');
 
-        $response = json_decode((string)$this->_response->getBody(), true);
+        $response = json_decode((string) $this->_response->getBody(), true);
 
         $this->assertResponseCode(200);
 
@@ -260,7 +260,7 @@ 
 
         $this->delete('api/v2/uploads/1');
 
-        $response = json_decode((string)$this->_response->getBody(), true);
+        $response = json_decode((string) $this->_response->getBody(), true);
 
         $this->assertResponseCode(204);
 

plugins/Installer/src/Controller/InstallController.php

Unused Use Statements -2 removed lines

@@ -14,8 +14,6 @@
 
 use Cake\Core\Configure;
 use Cake\Datasource\ConnectionManager;
-use Cake\Datasource\ConnectionRegistry;
-use Cake\Event\Event;
 use Cake\Filesystem\File;
 use Cake\ORM\Table;
 use Cake\ORM\TableRegistry;

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
@@ -64,7 +64,7 @@ 
         if ($hasSettingsTable) {
             $this->log('Installer found Settings-table. Moving on to Updater.', LogLevel::INFO, ['saito.install']);
             //// disable installer and move on to updater instead
-            $token = (new File(CONFIG . 'installer'))
+            $token = (new File(CONFIG.'installer'))
                 ->delete();
 
             return $this->redirect('/');

plugins/Installer/src/Controller/UpdaterController.php

Unused Use Statements -4 removed lines

@@ -13,11 +13,7 @@
 namespace Installer\Controller;
 
 use Cake\Core\Configure;
-use Cake\Event\Event;
 use Cake\Filesystem\File;
-use Cake\I18n\I18n;
-use Cake\ORM\Table;
-use Cake\ORM\TableRegistry;
 use Installer\Form\UpdaterStartForm;
 use Installer\Lib\DbVersion;
 use Psr\Log\LogLevel;

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 /**
  * Saito - The Threaded Web Forum
@@ -50,7 +50,7 @@ 
      */
     public function start()
     {
-        $token = new File(CONFIG . 'updater');
+        $token = new File(CONFIG.'updater');
 
         if ($token->exists()) {
             $this->renderFailure(__d('installer', 'update.failure.explanation'), 1529737182);
@@ -106,7 +106,7 @@ 
             (new DbVersion($this->Settings))->set($this->saitoVersion);
             $this->logCurrentState('Post upgrade state.');
         } catch (\Throwable $e) {
-            $this->logCurrentState('Migration failed: ' . $e->getMessage());
+            $this->logCurrentState('Migration failed: '.$e->getMessage());
 
             return $this->redirect('/');
         }

plugins/Installer/tests/TestCase/Controller/UpdaterControllerTest.php

Unused Use Statements -1 removed lines

@@ -12,7 +12,6 @@
 
 namespace Installer\Test\TestCase\Controller;
 
-use App\Model\Table\SettingsTable;
 use Cake\Core\Configure;
 use Cake\Datasource\ConnectionManager;
 use Cake\Filesystem\File;

Spacing +5 added lines, -5 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 /**
  * Saito - The Threaded Web Forum
@@ -35,7 +35,7 @@ 
     public function setUp()
     {
         parent::setUp();
-        $this->token = new File(CONFIG . 'updater');
+        $this->token = new File(CONFIG.'updater');
         $this->dbVersion = (new DbVersion(TableRegistry::get('Settings')));
         Configure::write('Saito.updated', false);
     }
@@ -54,7 +54,7 @@ 
 
         $this->assertResponseOk();
         $this->assertEquals('failure', $this->_controller->viewBuilder()->getTemplate());
-        $this->assertResponseContains((string)1529737182);
+        $this->assertResponseContains((string) 1529737182);
     }
 
     public function testUpdaterShowFailureNoDbVersionString()
@@ -64,7 +64,7 @@ 
 
         $this->assertResponseOk();
         $this->assertEquals('failure', $this->_controller->viewBuilder()->getTemplate());
-        $this->assertResponseContains((string)1529737397);
+        $this->assertResponseContains((string) 1529737397);
     }
 
     public function testUpdaterShowFailureWrongDbVersionString()
@@ -74,7 +74,7 @@ 
 
         $this->assertResponseOk();
         $this->assertEquals('failure', $this->_controller->viewBuilder()->getTemplate());
-        $this->assertResponseContains((string)1529737648);
+        $this->assertResponseContains((string) 1529737648);
     }
 
     public function testUpdaterInitMigrationsFormEmpty()

plugins/Sitemap/src/Lib/SitemapGenerator.php

Doc Comments +2 added lines, -2 removed lines

@@ -43,7 +43,7 @@ 
 
     /**
      * @param string $file filename
-     * @return mixed
+     * @return false|string
      */
     public function content($file)
     {
@@ -94,7 +94,7 @@ 
     /**
      * Creates name for sitemap-file
      *
-     * @param array $params additional name parameters
+     * @param integer[] $params additional name parameters
      * @return string
      */
     protected function _filename($params = [])

Spacing +1 added lines, -1 removed lines

@@ -101,7 +101,7 @@
     {
         $filename = "sitemap-{$this->_type}";
         if ($params) {
-            $filename .= '-' . implode('-', $params);
+            $filename .= '-'.implode('-', $params);
         }
 
         return $filename;

plugins/Stopwatch/src/Lib/Stopwatch.php

Doc Comments +2 added lines, -2 removed lines

@@ -167,7 +167,7 @@ 
     /**
      * time to Cake start
      *
-     * @return mixed
+     * @return double
      */
     protected static function _timeToCake()
     {
@@ -177,7 +177,7 @@ 
     /**
      * time from cake to stopwatch
      *
-     * @return int
+     * @return double
      */
     protected static function _timeFromCakeToStopwatch()
     {

Spacing +8 added lines, -8 removed lines

@@ -141,10 +141,10 @@ 
 
         switch ($event) {
             case 'start':
-                $x = '* ' . $x;
+                $x = '* '.$x;
                 break;
             case 'stop':
-                $x = '† ' . $x;
+                $x = '† '.$x;
                 break;
         }
 
@@ -208,13 +208,13 @@ 
         self::start('now');
 
         $out = "";
-        $out .= 'Time to Cake: ' . sprintf('%05.3f', self::_timeToCake()) . " s\n";
-        $out .= 'Cake bootstrap: ' . sprintf('%05.3f', self::_timeFromCakeToStopwatch()) . " s\n";
+        $out .= 'Time to Cake: '.sprintf('%05.3f', self::_timeToCake())." s\n";
+        $out .= 'Cake bootstrap: '.sprintf('%05.3f', self::_timeFromCakeToStopwatch())." s\n";
 
         $out .= "W\tU\tW_delta\tU_delta\tMem [MB]\n";
         $_seriesIndex = 1;
         foreach (self::$_events as $k => $v) {
-            $out .= '<span id="stopwatch-' . $_seriesIndex++ . '" class="stopwatch-row">';
+            $out .= '<span id="stopwatch-'.$_seriesIndex++.'" class="stopwatch-row">';
             $out .= sprintf(
                 "%05.3f\t%05.3f\t%05.3f\t%05.3f\t%5.1f\t%s\n",
                 $v['wtime'],
@@ -265,7 +265,7 @@ 
             );
         }
 
-        $out .= "\n\n" . self::printStatistic();
+        $out .= "\n\n".self::printStatistic();
 
         return $out;
     }
@@ -354,10 +354,10 @@ 
      */
     public static function printStatistic()
     {
-        return self::$_stopwatchCalls . " calls with ca " . sprintf(
+        return self::$_stopwatchCalls." calls with ca ".sprintf(
             "%05.3f",
             self::$_stopwatchTime
-        ) . ' sec overhead.';
+        ).' sec overhead.';
     }
 
     /**