phpffcms / ffcms

Apps/ActiveRecord/User.php

Doc Comments +1 added lines, -1 removed lines

@@ -168,7 +168,7 @@
     /**
      * Get user person like a object via email
      * @param string|null $email
-     * @return null|self
+     * @return null|ActiveModel
      */
     public static function getIdentityViaEmail(?string $email = null)
     {

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use Ffcms\Core\App as MainApp;
 use Ffcms\Core\Arch\ActiveModel;
 use Ffcms\Core\Helper\Type\Any;
-use Ffcms\Core\Helper\Type\Obj;
 use Ffcms\Core\Helper\Type\Str;
 use Ffcms\Core\Interfaces\iUser;
 

Spacing +9 added lines, -9 removed lines

@@ -50,13 +50,13 @@ 
         }
 
         // check if id is looks like integer
-        if (!Any::isInt($id) || (int)$id < 1) {
+        if (!Any::isInt($id) || (int) $id < 1) {
             return null;
         }
 
         // check in memory cache object
-        if (MainApp::$Memory->get('user.object.cache.' . $id) !== null) {
-            return MainApp::$Memory->get('user.object.cache.' . $id);
+        if (MainApp::$Memory->get('user.object.cache.'.$id) !== null) {
+            return MainApp::$Memory->get('user.object.cache.'.$id);
         }
 
         // not founded in memory? lets make query
@@ -64,7 +64,7 @@ 
             ->find($id);
 
         // store cache and return object
-        MainApp::$Memory->set('user.object.cache.' . $user->id, $user);
+        MainApp::$Memory->set('user.object.cache.'.$user->id, $user);
         return $user;
     }
 
@@ -74,7 +74,7 @@ 
      */
     public function getId(): ?int
     {
-        return (int)$this->id;
+        return (int) $this->id;
     }
 
     /**
@@ -95,7 +95,7 @@ 
     public static function isAuth(): bool
     {
         // get data from session
-        $sessionUserId = (int)MainApp::$Session->get('ff_user_id', 0);
+        $sessionUserId = (int) MainApp::$Session->get('ff_user_id', 0);
 
         // check if session contains user id data
         if ($sessionUserId < 1) {
@@ -128,10 +128,10 @@ 
             return false;
         }
 
-        $find = MainApp::$Memory->get('user.counter.cache.' . $id);
+        $find = MainApp::$Memory->get('user.counter.cache.'.$id);
         if (!$find) {
             $find = self::where('id', $id)->count();
-            MainApp::$Memory->set('user.counter.cache.' . $id, $find);
+            MainApp::$Memory->set('user.counter.cache.'.$id, $find);
         }
 
         return $find === 1;
@@ -231,7 +231,7 @@ 
      */
     public function inBlacklist(?string $target = null): bool
     {
-        if ($target === null || (int)$target < 1) {
+        if ($target === null || (int) $target < 1) {
             return false;
         }
 

Apps/Console/MigrationCreateCommand.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 
 use Ffcms\Console\Command;
 use Ffcms\Core\Helper\FileSystem\File;
-use Ffcms\Core\Helper\Type\Obj;
 use Ffcms\Core\Helper\Type\Str;
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;

Spacing +2 added lines, -2 removed lines

@@ -58,9 +58,9 @@
         // work with migration template: read & parse & save
         $tpl = File::read('/Private/Carcase/Migration.tphp');
         $classContent = Str::replace(['%class%', '%table%'], [$name, $table], $tpl);
-        $fullPath = $dir . $name . '-' . $suffix . '.php';
+        $fullPath = $dir.$name.'-'.$suffix.'.php';
         File::write($fullPath, $classContent);
         // show success msg
-        $output->write('New migration is created: ' . $fullPath);
+        $output->write('New migration is created: '.$fullPath);
     }
 }
\ No newline at end of file

Apps/Console/MigrationDownCommand.php

Unused Use Statements -3 removed lines

@@ -5,10 +5,7 @@
 
 use Apps\ActiveRecord\Migration;
 use Ffcms\Console\Command;
-use Ffcms\Core\Helper\FileSystem\File;
 use Ffcms\Core\Helper\Type\Any;
-use Ffcms\Core\Helper\Type\Obj;
-use Ffcms\Core\Helper\Type\Str;
 use Ffcms\Core\Managers\MigrationsManager;
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
         // list found migrations and ask to revert each one
         $fired = false;
         foreach ($search as $migration) {
-            if (!$this->confirm('Are you sure to revert: ' . $migration)) {
+            if (!$this->confirm('Are you sure to revert: '.$migration)) {
                 continue;
             }
             // run down migration

Apps/Console/MigrationUpCommand.php

Unused Use Statements -5 removed lines

@@ -3,13 +3,8 @@
 namespace Apps\Console;
 
 
-use Apps\ActiveRecord\Migration;
 use Ffcms\Console\Command;
-use Ffcms\Core\Helper\FileSystem\File;
 use Ffcms\Core\Helper\Type\Any;
-use Ffcms\Core\Helper\Type\Arr;
-use Ffcms\Core\Helper\Type\Obj;
-use Ffcms\Core\Helper\Type\Str;
 use Ffcms\Core\Managers\MigrationsManager;
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;

Spacing +2 added lines, -2 removed lines

@@ -51,11 +51,11 @@
         // require confirmation from user each ever migration file
         $fired = false;
         foreach ($search as $migration) {
-            if (!$this->confirm('Are you sure to apply migration: ' . $migration, true)) {
+            if (!$this->confirm('Are you sure to apply migration: '.$migration, true)) {
                 continue;
             }
             $manager->makeUp($migration);
-            $fired =  true;
+            $fired = true;
         }
 
         if ($fired) {

Apps/Model/Admin/Content/FormContentUpdate.php

Unused Use Statements -1 removed lines

@@ -11,7 +11,6 @@
 use Ffcms\Core\Helper\FileSystem\Directory;
 use Ffcms\Core\Helper\FileSystem\File;
 use Ffcms\Core\Helper\Type\Any;
-use Ffcms\Core\Helper\Type\Obj;
 use Ffcms\Core\Helper\Type\Str;
 
 /**

Spacing +11 added lines, -11 removed lines

@@ -59,7 +59,7 @@ 
         if ($this->_content->id === null) {
             $this->_new = true;
             if (!$this->galleryFreeId) {
-                $this->galleryFreeId = '_tmp_' . Str::randomLatin(mt_rand(16, 32));
+                $this->galleryFreeId = '_tmp_'.Str::randomLatin(mt_rand(16, 32));
             }
 
             if (!$this->authorId) {
@@ -94,8 +94,8 @@ 
     public function rules(): array
     {
         $res = [
-            ['title.' . App::$Request->getLanguage(), 'required'],
-            ['text.' . App::$Request->getLanguage(), 'required'],
+            ['title.'.App::$Request->getLanguage(), 'required'],
+            ['text.'.App::$Request->getLanguage(), 'required'],
             ['text', 'used'],
             ['path', 'reverse_match', '/[\/\'~`\!@#\$%\^&\*\(\)+=\{\}\[\]\|;:"\<\>,\?\\\]/'],
             [['path', 'categoryId', 'authorId', 'display', 'galleryFreeId', 'title', 'important'], 'required'],
@@ -108,9 +108,9 @@ 
         ];
 
         foreach (App::$Properties->get('languages') as $lang) {
-            $res[] = ['title.' . $lang, 'length_max', 120, null, true, true];
-            $res[] = ['keywords.' . $lang, 'length_max', 150];
-            $res[] = ['description.' . $lang, 'length_max', 250];
+            $res[] = ['title.'.$lang, 'length_max', 120, null, true, true];
+            $res[] = ['keywords.'.$lang, 'length_max', 150];
+            $res[] = ['description.'.$lang, 'length_max', 250];
         }
 
         return $res;
@@ -166,10 +166,10 @@ 
         $this->_content->meta_keywords = $this->metaKeywords;
         $this->_content->meta_description = $this->metaDescription;
         $this->_content->source = $this->source;
-        $this->_content->important = (int)$this->important;
+        $this->_content->important = (int) $this->important;
         // check if rating is changed
-        if ((int)$this->addRating !== 0) {
-            $this->_content->rating += (int)$this->addRating;
+        if ((int) $this->addRating !== 0) {
+            $this->_content->rating += (int) $this->addRating;
         }
         // check if special comment hash is exist
         if ($this->_new || Str::length($this->_content->comment_hash) < 32) {
@@ -182,7 +182,7 @@ 
         }
 
         // save poster data
-        $posterPath = '/upload/gallery/' . $this->galleryFreeId . '/orig/' . $this->poster;
+        $posterPath = '/upload/gallery/'.$this->galleryFreeId.'/orig/'.$this->poster;
         if (File::exist($posterPath)) {
             $this->_content->poster = $this->poster;
         }
@@ -217,7 +217,7 @@ 
 
         // move files
         if ($tmpGalleryId !== $this->_content->id) {
-            Directory::rename('/upload/gallery/' . $tmpGalleryId, $this->_content->id);
+            Directory::rename('/upload/gallery/'.$tmpGalleryId, $this->_content->id);
         }
     }
 

Apps/Model/Admin/Main/FormUpdateDatabase.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use Apps\ActiveRecord\System;
 use Ffcms\Core\Arch\Model;
-use Ffcms\Core\Helper\FileSystem\File;
 use Ffcms\Core\Helper\Type\Str;
 use Ffcms\Core\Managers\MigrationsManager;
 

Spacing +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
             $name = Str::firstIn($fullName, '-');
             // get update version number from migration name
             list($type, $obj, $version) = explode('_', $name);
-            $intVersion = (int)Str::replace('.', '', $this->dbVersion);
+            $intVersion = (int) Str::replace('.', '', $this->dbVersion);
             // if migration version > db version - implement it
             if ($version > $intVersion) {
                 $this->updateQueries[] = $file;

Apps/Model/Admin/User/FormUserUpdate.php

Unused Use Statements -1 removed lines

@@ -8,7 +8,6 @@
 use Ffcms\Core\App;
 use Ffcms\Core\Arch\Model;
 use Ffcms\Core\Helper\Type\Any;
-use Ffcms\Core\Helper\Type\Obj;
 use Ffcms\Core\Helper\Type\Str;
 use Ffcms\Core\Interfaces\iUser;
 

Apps/Model/Front/Content/EntityContentRead.php

Unused Use Statements -1 removed lines

@@ -12,7 +12,6 @@
 use Ffcms\Core\Helper\FileSystem\File;
 use Ffcms\Core\Helper\Type\Any;
 use Ffcms\Core\Helper\Type\Arr;
-use Ffcms\Core\Helper\Type\Obj;
 use Ffcms\Core\Helper\Type\Str;
 
 /**

Indentation +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
     /**
      * Prepare model attributes from passed objects
      * @throws ForbiddenException
-    */
+     */
     public function before()
     {
         $this->id = $this->_content->id;

Spacing +9 added lines, -9 removed lines

@@ -72,7 +72,7 @@ 
         $this->id = $this->_content->id;
         $this->title = $this->_content->getLocaled('title');
         $this->text = $this->_content->getLocaled('text');
-        $this->display = (bool)$this->_content->display;
+        $this->display = (bool) $this->_content->display;
 
         // check if title and text are exists
         if (Str::length($this->title) < 1 || Str::length($this->text) < 1) {
@@ -102,7 +102,7 @@ 
         }
 
         $this->source = $this->_content->source;
-        $this->views = $this->_content->views+1;
+        $this->views = $this->_content->views + 1;
         // check for dependence, add '' for general cat, ex: general/depend1/depend2/.../depend-n
         $catNestingArray = Arr::merge([0 => ''], explode('/', $this->catPath));
         if ($catNestingArray > 1) {
@@ -134,15 +134,15 @@ 
         ];
 
         // get gallery images and poster data
-        $galleryPath = '/upload/gallery/' . $this->_content->id;
+        $galleryPath = '/upload/gallery/'.$this->_content->id;
         // check if gallery folder is exist
         if (Directory::exist($galleryPath)) {
-            $originImages = File::listFiles($galleryPath . '/orig/', ['.jpg', '.png', '.gif', '.jpeg', '.bmp', '.webp'], true);
+            $originImages = File::listFiles($galleryPath.'/orig/', ['.jpg', '.png', '.gif', '.jpeg', '.bmp', '.webp'], true);
             // generate poster data
             if (Arr::in($this->_content->poster, $originImages)) {
                 // original poster
                 $posterName = $this->_content->poster;
-                $this->posterFull = $galleryPath . '/orig/' . $posterName;
+                $this->posterFull = $galleryPath.'/orig/'.$posterName;
                 if (!File::exist($this->posterFull)) {
                     $this->posterFull = null;
                 }
@@ -151,7 +151,7 @@ 
                 $posterSplit = explode('.', $posterName);
                 array_pop($posterSplit);
                 $posterCleanName = implode('.', $posterSplit);
-                $this->posterThumb = $galleryPath . '/thumb/' . $posterCleanName . '.jpg';
+                $this->posterThumb = $galleryPath.'/thumb/'.$posterCleanName.'.jpg';
                 if (!File::exist($this->posterThumb)) {
                     $this->posterThumb = null;
                 }
@@ -167,9 +167,9 @@ 
                     continue;
                 }
 
-                $thumbPath = $galleryPath . '/thumb/' . $imageClearName . '.jpg';
+                $thumbPath = $galleryPath.'/thumb/'.$imageClearName.'.jpg';
                 if (File::exist($thumbPath)) {
-                    $this->galleryItems[$thumbPath] = $galleryPath . '/orig/' . $image;
+                    $this->galleryItems[$thumbPath] = $galleryPath.'/orig/'.$image;
                 }
             }
         }
@@ -178,7 +178,7 @@ 
         $this->rating = $this->_content->rating;
         $ignoredRate = App::$Session->get('content.rate.ignore');
         $this->canRate = true;
-        if (Any::isArray($ignoredRate) && Arr::in((string)$this->id, $ignoredRate)) {
+        if (Any::isArray($ignoredRate) && Arr::in((string) $this->id, $ignoredRate)) {
             $this->canRate = false;
         }
         if (!App::$User->isAuth()) {

Apps/Model/Front/Content/EntityContentSearch.php

Doc Comments +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
     /**
      * EntityContentSearch constructor. Pass search terms (query string) to model and used items to skip it by id.
      * @param $terms
-     * @param int|array $skipIds
+     * @param integer $skipIds
      * @param int|null $categoryId
      */
     public function __construct($terms, $skipIds = 0, $categoryId = null)

Unused Use Statements -1 removed lines

@@ -8,7 +8,6 @@
 use Ffcms\Core\Exception\NotFoundException;
 use Ffcms\Core\Helper\Text;
 use Ffcms\Core\Helper\Type\Any;
-use Ffcms\Core\Helper\Type\Obj;
 use Ffcms\Core\Helper\Type\Str;
 
 /**

Spacing +2 added lines, -2 removed lines

@@ -61,7 +61,7 @@ 
 
         $index = implode('-', $this->_skip);
         // try to get this slow query from cache
-        $cache = App::$Cache->getItem('entity.content.search.index.' . $index);
+        $cache = App::$Cache->getItem('entity.content.search.index.'.$index);
         if (!$cache->isHit()) {
             $cache->set($this->makeSearch());
             $cache->expiresAfter(static::CACHE_TIME);
@@ -94,7 +94,7 @@ 
             $this->items[] = [
                 'title' => $item->getLocaled('title'),
                 'snippet' => Text::snippet($text),
-                'uri' => '/content/read/' . $item->getPath(),
+                'uri' => '/content/read/'.$item->getPath(),
                 'thumb' => $item->getPosterThumbUri()
             ];
         }