phpffcms / ffcms

Apps/ActiveRecord/App.php

Doc Comments +2 added lines, -2 removed lines

@@ -58,7 +58,7 @@ 
 
     /**
      * @deprecated
-     * @return \Illuminate\Database\Eloquent\Collection|mixed
+     * @return \Illuminate\Database\Eloquent\Collection
      * @throws SyntaxException
      */
     public static function getAll()
@@ -88,7 +88,7 @@ 
      * Get single row by defined type and sys_name with query caching
      * @param string $type
      * @param string|array $name
-     * @return mixed|null
+     * @return App
      * @throws SyntaxException
      */
     public static function getItem($type, $name)

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 use Ffcms\Core\Exception\SyntaxException;
 use Ffcms\Core\Helper\Type\Any;
 use Ffcms\Core\Helper\Type\Arr;
-use Ffcms\Core\Helper\Type\Obj;
 use Ffcms\Core\Helper\Type\Str;
 
 /**

Spacing +4 added lines, -4 removed lines

@@ -43,7 +43,7 @@ 
      */
     public static function all($columns = ['*'])
     {
-        $cacheName = 'activercord.app.all.' . implode('.', $columns);
+        $cacheName = 'activercord.app.all.'.implode('.', $columns);
         $records = MemoryObject::instance()->get($cacheName);
         if ($records === null) {
             $records = parent::all($columns);
@@ -181,15 +181,15 @@ 
      */
     public function getScriptVersion()
     {
-        $class = 'Apps\Controller\Admin\\' . $this->sys_name;
+        $class = 'Apps\Controller\Admin\\'.$this->sys_name;
         if (!class_exists($class)) {
             return false;
         }
 
-        if (!defined($class . '::VERSION')) {
+        if (!defined($class.'::VERSION')) {
             return false;
         }
 
-        return constant($class . '::VERSION');
+        return constant($class.'::VERSION');
     }
 }

Apps/ActiveRecord/Profile.php

Unused Use Statements -1 removed lines

@@ -7,7 +7,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;
 use Ffcms\Core\Interfaces\iProfile;
 

Spacing +9 added lines, -9 removed lines

@@ -61,8 +61,8 @@ 
         }
 
         // check in cache
-        if (MainApp::$Memory->get('profile.object.cache.' . $userId) !== null) {
-            return MainApp::$Memory->get('profile.object.cache.' . $userId);
+        if (MainApp::$Memory->get('profile.object.cache.'.$userId) !== null) {
+            return MainApp::$Memory->get('profile.object.cache.'.$userId);
         }
 
         // find row
@@ -75,7 +75,7 @@ 
 
         $object = $profile->first();
 
-        MainApp::$Memory->set('profile.object.cache.' . $userId, $object);
+        MainApp::$Memory->set('profile.object.cache.'.$userId, $object);
         return $object;
     }
 
@@ -86,17 +86,17 @@ 
      */
     public function getAvatarUrl($type = 'small')
     {
-        $default = '/upload/user/avatar/' . $type . '/default.jpg';
+        $default = '/upload/user/avatar/'.$type.'/default.jpg';
         if (!Arr::in($type, ['small', 'big', 'medium'])) {
-            return MainApp::$Alias->scriptUrl . $default;
+            return MainApp::$Alias->scriptUrl.$default;
         }
 
-        $route = '/upload/user/avatar/' . $type . '/' . $this->user_id . '.jpg';
+        $route = '/upload/user/avatar/'.$type.'/'.$this->user_id.'.jpg';
         if (File::exist($route)) {
-            return MainApp::$Alias->scriptUrl . $route . '?mtime=' . File::mTime($route);
+            return MainApp::$Alias->scriptUrl.$route.'?mtime='.File::mTime($route);
         }
 
-        return MainApp::$Alias->scriptUrl . $default;
+        return MainApp::$Alias->scriptUrl.$default;
     }
 
     /**
@@ -107,7 +107,7 @@ 
     {
         $userNick = $this->nick;
         if ($userNick === null || Str::likeEmpty($userNick)) {
-            $userNick = 'id' . $this->id;
+            $userNick = 'id'.$this->id;
         }
 
         return $userNick;

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/Controller/Admin/Comments.php

Unused Use Statements -1 removed lines

@@ -14,7 +14,6 @@
 use Ffcms\Core\Helper\HTML\SimplePagination;
 use Ffcms\Core\Helper\Type\Any;
 use Ffcms\Core\Helper\Type\Arr;
-use Ffcms\Core\Helper\Type\Obj;
 
 /**
  * Class Comments. Admin controller for management user comments.

Spacing +6 added lines, -6 removed lines

@@ -40,7 +40,7 @@ 
     public function actionIndex()
     {
         // set current page and offset
-        $page = (int)$this->request->query->get('page');
+        $page = (int) $this->request->query->get('page');
         $offset = $page * self::ITEM_PER_PAGE;
 
         // initialize active record model
@@ -143,7 +143,7 @@ 
     public function actionDelete($type, $id = 0)
     {
         // sounds like a multiply delete definition
-        if ($id === 0 || (int)$id < 1) {
+        if ($id === 0 || (int) $id < 1) {
             $ids = $this->request->query->get('selected');
             if (!Any::isArray($ids) || !Arr::onlyNumericValues($ids)) {
                 throw new NotFoundException('Bad conditions');
@@ -177,7 +177,7 @@ 
         if ($model->send() && $model->validate()) {
             $model->make();
             App::$Session->getFlashBag()->add('success', __('Comments or answers are successful deleted!'));
-            $this->response->redirect('comments/' . ($type === 'answer' ? 'answerlist' : 'index'));
+            $this->response->redirect('comments/'.($type === 'answer' ? 'answerlist' : 'index'));
         }
 
         // render view
@@ -198,7 +198,7 @@ 
     public function actionPublish($type, $id = 0)
     {
         // check if it multiple accept ids
-        if ($id === 0 || (int)$id < 1) {
+        if ($id === 0 || (int) $id < 1) {
             $ids = $this->request->query->get('selected');
             if (!Any::isArray($ids) || !Arr::onlyNumericValues($ids)) {
                 throw new NotFoundException('Bad conditions');
@@ -231,7 +231,7 @@ 
         if ($model->send()) {
             $model->make();
             App::$Session->getFlashBag()->add('success', __('Comments or answers are successful published'));
-            $this->response->redirect('comments/' . ($type === 'answer' ? 'answerlist' : 'index'));
+            $this->response->redirect('comments/'.($type === 'answer' ? 'answerlist' : 'index'));
         }
 
         return $this->view->render('publish', [
@@ -248,7 +248,7 @@ 
     public function actionAnswerlist()
     {
         // set current page and offset
-        $page = (int)$this->request->query->get('page');
+        $page = (int) $this->request->query->get('page');
         $offset = $page * self::ITEM_PER_PAGE;
 
         // initialize ar answers model

Apps/Controller/Admin/Content.php

Unused Use Statements -1 removed lines

@@ -21,7 +21,6 @@
 use Ffcms\Core\Exception\SyntaxException;
 use Ffcms\Core\Helper\HTML\SimplePagination;
 use Ffcms\Core\Helper\Type\Any;
-use Ffcms\Core\Helper\Type\Obj;
 
 /**
  * Class Content. Admin controller to manage & control contents

Spacing +3 added lines, -3 removed lines

@@ -43,7 +43,7 @@ 
     public function actionIndex()
     {
         // set current page and offset
-        $page = (int)$this->request->query->get('page');
+        $page = (int) $this->request->query->get('page');
         $offset = $page * self::ITEM_PER_PAGE;
 
         $query = null;
@@ -54,7 +54,7 @@ 
         } elseif ($type === 'moderate') { // only items on moderate
             $query = ContentEntity::where('display', '=', 0);
         } elseif (Any::isInt($type)) { // sounds like category id ;)
-            $query = ContentEntity::where('category_id', '=', (int)$type);
+            $query = ContentEntity::where('category_id', '=', (int) $type);
         } else {
             $query = new ContentEntity();
             $type = 'all';
@@ -264,7 +264,7 @@ 
     public function actionCategoryupdate($id = null)
     {
         // get owner id for new rows
-        $parentId = (int)$this->request->query->get('parent');
+        $parentId = (int) $this->request->query->get('parent');
 
         // get relation and pass to model
         $record = ContentCategory::findOrNew($id);

Apps/Controller/Api/Comments.php

Unused Use Statements -1 removed lines

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

Spacing +10 added lines, -10 removed lines

@@ -33,7 +33,7 @@ 
         $this->setJsonHeader();
         $configs = AppRecord::getConfigs('widget', 'Comments');
 
-        $replayTo = (int)$this->request->request->get('replay-to');
+        $replayTo = (int) $this->request->request->get('replay-to');
         $model = null;
         // check if its a answer (comment answer type)
         if ($replayTo > 0) {
@@ -45,7 +45,7 @@ 
         }
 
         // pass general comment params to model
-        $model->message = App::$Security->secureHtml((string)$this->request->request->get('message'));
+        $model->message = App::$Security->secureHtml((string) $this->request->request->get('message'));
         $model->guestName = App::$Security->strip_tags($this->request->request->get('guest-name'));
 
         // check model conditions before add new row
@@ -77,12 +77,12 @@ 
         // get configs
         $configs = AppRecord::getConfigs('widget', 'Comments');
         // items per page
-        $perPage = (int)$configs['perPage'];
+        $perPage = (int) $configs['perPage'];
         // offset can be only integer
-        $index = (int)$index;
+        $index = (int) $index;
         $offset = $perPage * $index;
         // get comment target path and check
-        $path = (string)$this->request->query->get('path');
+        $path = (string) $this->request->query->get('path');
         if (Str::likeEmpty($path)) {
             throw new NotFoundException('Wrong path');
         }
@@ -92,7 +92,7 @@ 
             ->where('moderate', '=', 0);
 
         // check if comments is depend of language locale
-        if ((bool)$configs['onlyLocale'] === true) {
+        if ((bool) $configs['onlyLocale'] === true) {
             $query = $query->where('lang', '=', $this->request->getLanguage());
         }
 
@@ -121,7 +121,7 @@ 
         // calculate comments left count
         $countQuery = CommentPost::where('pathway', '=', $path)->where('moderate', '=', 0);
         // check if comments is depend of language locale
-        if ((bool)$configs['onlyLocale'] === true) {
+        if ((bool) $configs['onlyLocale'] === true) {
             $countQuery = $countQuery->where('lang', '=', $this->request->getLanguage());
         }
         $count = $countQuery->count();
@@ -160,7 +160,7 @@ 
         $records = CommentAnswer::with(['user', 'user.profile', 'user.role'])
             ->where('comment_id', '=', $commentId)
             ->where('moderate', '=', 0);
-        if ((int)$configs['onlyLocale'] === 1) {
+        if ((int) $configs['onlyLocale'] === 1) {
             $records = $records->where('lang', '=', $this->request->getLanguage());
         }
 
@@ -205,11 +205,11 @@ 
         foreach ($path as $id => $uri) {
             $query = CommentPost::where('pathway', '=', $uri)->where('moderate', '=', 0);
             // check if comments is depend of language locale
-            if ((int)$configs['onlyLocale'] === 1) {
+            if ((int) $configs['onlyLocale'] === 1) {
                 $query = $query->where('lang', '=', $this->request->getLanguage());
             }
             // set itemId => count
-            $count[(int)$id] = $query->count();
+            $count[(int) $id] = $query->count();
         }
         // render json response
         return json_encode(['status' => 1, 'count' => $count]);