phpffcms / ffcms

Apps/ActiveRecord/Role.php

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\Arr;
-use Ffcms\Core\Helper\Type\Str;
 
 class Role extends ActiveModel
 {

Spacing +2 added lines, -2 removed lines

@@ -26,12 +26,12 @@
      */
     public static function get($role_id)
     {
-        $role = MainApp::$Memory->get('user.role.cache.' . $role_id);
+        $role = MainApp::$Memory->get('user.role.cache.'.$role_id);
 
         // not founded in cache
         if ($role === null) {
             $role = self::find($role_id);
-            MainApp::$Memory->set('user.role.cache.' . $role_id, $role);
+            MainApp::$Memory->set('user.role.cache.'.$role_id, $role);
         }
         return $role;
     }

Apps/Controller/Admin/Main.php

Unused Use Statements -1 removed lines

@@ -9,7 +9,6 @@
 use Ffcms\Core\App;
 use Ffcms\Core\Exception\SyntaxException;
 use Ffcms\Core\Helper\FileSystem\File;
-use Ffcms\Core\Helper\Type\Arr;
 use Ffcms\Core\Helper\Type\Integer;
 use Ffcms\Core\Helper\Type\Str;
 

Spacing +10 added lines, -10 removed lines

@@ -37,21 +37,21 @@ 
         // cache some data
         $rootSize = App::$Cache->get('root.size');
         if ($rootSize === null) {
-            $rootSize = round(Directory::getSize('/') / (1024*1000), 2) . ' mb';
+            $rootSize = round(Directory::getSize('/') / (1024 * 1000), 2).' mb';
             App::$Cache->set('root.size', $rootSize, 86400); // 24 hours caching = 60 * 60 * 24
         }
         $loadAvg = App::$Cache->get('load.average');
         if ($loadAvg === null) {
             $loadAvg = Environment::loadAverage();
-            App::$Cache->set('load.average', $loadAvg, 60*5); // 5 min cache
+            App::$Cache->set('load.average', $loadAvg, 60 * 5); // 5 min cache
         }
 
         // prepare system statistic
         $stats = [
-            'ff_version' => App::$Properties->version['num'] . ' (' . App::$Properties->version['date'] . ')',
-            'php_version' => Environment::phpVersion() . ' (' . Environment::phpSAPI() . ')',
+            'ff_version' => App::$Properties->version['num'].' ('.App::$Properties->version['date'].')',
+            'php_version' => Environment::phpVersion().' ('.Environment::phpSAPI().')',
             'os_name' => Environment::osName(),
-            'database_name' => App::$Database->connection()->getDatabaseName() . ' (' . App::$Database->connection()->getDriverName() . ')',
+            'database_name' => App::$Database->connection()->getDatabaseName().' ('.App::$Database->connection()->getDriverName().')',
             'file_size' => $rootSize,
             'load_avg' => $loadAvg
         ];
@@ -103,7 +103,7 @@ 
     public function actionFiles()
     {
         return App::$View->render('files', [
-            'connector' => App::$Alias->scriptUrl . '/api/main/files?lang=' . App::$Request->getLanguage()
+            'connector' => App::$Alias->scriptUrl.'/api/main/files?lang='.App::$Request->getLanguage()
         ]);
     }
 
@@ -174,9 +174,9 @@ 
      */
     public function actionDeleteroute()
     {
-        $type = (string)App::$Request->query->get('type');
-        $loader = (string)App::$Request->query->get('loader');
-        $source = Str::lowerCase((string)App::$Request->query->get('path'));
+        $type = (string) App::$Request->query->get('type');
+        $loader = (string) App::$Request->query->get('loader');
+        $source = Str::lowerCase((string) App::$Request->query->get('path'));
 
         $model = new EntityDeleteRoute($type, $loader, $source);
         if ($model->send()) {
@@ -199,7 +199,7 @@ 
     {
         $stats = App::$Cache->stats();
         // get size in mb from cache stats
-        $size = round((int)$stats['size'] / (1024*1024), 2);
+        $size = round((int) $stats['size'] / (1024 * 1024), 2);
 
         // check if submited
         if (App::$Request->request->get('clearcache', false)) {

Apps/Controller/Front/Main.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use Extend\Core\Arch\FrontAppController;
 use Ffcms\Core\Arch\View;
-use Apps\Model\Front\User;
 
 
 class Main extends FrontAppController

Spacing +1 added lines, -1 removed lines

@@ -25,6 +25,6 @@
 
     public function actionRead($id)
     {
-        echo "Action read called" . $id;
+        echo "Action read called".$id;
     }
 }
\ No newline at end of file

Apps/Model/Admin/Content/FormContentUpdate.php

Unused Use Statements -1 removed lines

@@ -9,7 +9,6 @@
 use Ffcms\Core\Helper\Date;
 use Ffcms\Core\Helper\FileSystem\Directory;
 use Ffcms\Core\Helper\FileSystem\File;
-use Ffcms\Core\Helper\Type\Integer;
 use Ffcms\Core\Helper\Type\Obj;
 use Ffcms\Core\Helper\Serialize;
 use Ffcms\Core\Helper\Type\Str;

Indentation +4 added lines, -4 removed lines

@@ -43,8 +43,8 @@ 
     }
 
     /**
-    * Set model properties from active record data
-    */
+     * Set model properties from active record data
+     */
     public function before()
     {
         // is new item?
@@ -105,8 +105,8 @@ 
     }
 
     /**
-    * Labels
-    */
+     * Labels
+     */
     public function labels()
     {
         return [

Spacing +12 added lines, -12 removed lines

@@ -51,7 +51,7 @@ 
         if ($this->_content->id === null) {
             $this->_new = true;
             if (null === $this->galleryFreeId) {
-                $this->galleryFreeId = '_tmp_' . Str::randomLatin(mt_rand(16, 32));
+                $this->galleryFreeId = '_tmp_'.Str::randomLatin(mt_rand(16, 32));
             }
             if (null === $this->authorId) {
                 $this->authorId = App::$User->identity()->getId();
@@ -81,9 +81,9 @@ 
      */
     public function rules()
     {
-        $res =  [
-            ['title.' . App::$Request->getLanguage(), 'required'],
-            ['text.' . App::$Request->getLanguage(), 'required', null, true, true],
+        $res = [
+            ['title.'.App::$Request->getLanguage(), 'required'],
+            ['text.'.App::$Request->getLanguage(), 'required', null, true, true],
             ['text', 'used', null, true, true],
             ['path', 'reverse_match', '/[\/\'~`\!@#\$%\^&\*\(\)+=\{\}\[\]\|;:"\<\>,\?\\\]/'],
             [['path', 'categoryId', 'authorId', 'display', 'galleryFreeId', 'title'], 'required'],
@@ -96,9 +96,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;
@@ -142,8 +142,8 @@ 
         $this->_content->meta_description = Serialize::encode(App::$Security->strip_tags($this->metaDescription));
         $this->_content->source = App::$Security->strip_tags($this->source);
         // 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) {
@@ -155,7 +155,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;
         }
@@ -168,7 +168,7 @@ 
 
         // move files
         if ($tmpGalleryId !== $this->_content->id) {
-            Directory::rename('/upload/gallery/' . $tmpGalleryId, $this->_content->id);
+            Directory::rename('/upload/gallery/'.$tmpGalleryId, $this->_content->id);
         }
     }
 
@@ -181,7 +181,7 @@ 
         $data = ContentCategory::getSortedCategories();
         $response = [];
         foreach ($data as $key=>$val) {
-            $response[] = (string)$key;
+            $response[] = (string) $key;
         }
         return $response;
     }

Apps/Model/Console/ArchBuilder.php

Doc Comments +3 added lines

@@ -10,6 +10,9 @@
 {
     public $message;
 
+    /**
+     * @param string $type
+     */
     public function createObject($name, $type)
     {
         $singleName = false;

Spacing +16 added lines, -16 removed lines

@@ -29,8 +29,8 @@ 
         $objectTemplate = null;
 
         if ($singleName) {
-            $objectDirPath = root . '/Apps/' . $type . '/';
-            $objectNamespace = 'Apps\\' . $type;
+            $objectDirPath = root.'/Apps/'.$type.'/';
+            $objectNamespace = 'Apps\\'.$type;
             $objectName = ucfirst($name);
         } else {
             $split = explode('/', $name);
@@ -47,46 +47,46 @@ 
             }
             
             if ($type === 'Widget') {
-                $objectDirPath = root . '/Widgets/' . $workground;
-                $objectNamespace = 'Widgets\\' . $workground;
+                $objectDirPath = root.'/Widgets/'.$workground;
+                $objectNamespace = 'Widgets\\'.$workground;
             } else {
-                $objectDirPath = root . '/Apps/' . $type . '/' . $workground;
-                $objectNamespace = 'Apps\\' . $type . '\\' . $workground;
+                $objectDirPath = root.'/Apps/'.$type.'/'.$workground;
+                $objectNamespace = 'Apps\\'.$type.'\\'.$workground;
             }
             
             if (false !== $subName) {
-                $objectDirPath .= '/' . implode('/', $subName);
-                $objectNamespace .= '\\' . implode('\\', $subName);
+                $objectDirPath .= '/'.implode('/', $subName);
+                $objectNamespace .= '\\'.implode('\\', $subName);
             }
 
 
             // try to find workground-based controller
-            if (File::exist('/Private/Carcase/' . $workground . '/' . $type . '.tphp')) {
-                $objectTemplate = File::read('/Private/Carcase/' . $workground . '/' . $type . '.tphp');
+            if (File::exist('/Private/Carcase/'.$workground.'/'.$type.'.tphp')) {
+                $objectTemplate = File::read('/Private/Carcase/'.$workground.'/'.$type.'.tphp');
             }
         }
 
         if (!Directory::exist($objectDirPath) && !Directory::create($objectDirPath)) {
-            $this->message = 'Directory could not be created: ' . $objectDirPath;
+            $this->message = 'Directory could not be created: '.$objectDirPath;
             return false;
         }
 
         if ($objectTemplate === null) {
-            $objectTemplate = File::read('/Private/Carcase/' . $type . '.tphp');
+            $objectTemplate = File::read('/Private/Carcase/'.$type.'.tphp');
             if (false === $objectTemplate) {
-                $this->message = 'Php template file is not founded: /Private/Carcase/' . $type . '.tphp';
+                $this->message = 'Php template file is not founded: /Private/Carcase/'.$type.'.tphp';
                 return false;
             }
         }
 
         $objectContent = Str::replace(['%namespace%', '%name%'], [$objectNamespace, $objectName], $objectTemplate);
-        $objectFullPath = $objectDirPath . '/' . $objectName . '.php';
+        $objectFullPath = $objectDirPath.'/'.$objectName.'.php';
         if (File::exist($objectFullPath)) {
-            $this->message = $type . ' is always exist: ' . $objectFullPath;
+            $this->message = $type.' is always exist: '.$objectFullPath;
             return false;
         }
         File::write($objectFullPath, $objectContent);
-        $this->message = $type . ' template was created: [' . $objectName . '] in path: ' . Str::replace(root, '', $objectDirPath);
+        $this->message = $type.' template was created: ['.$objectName.'] in path: '.Str::replace(root, '', $objectDirPath);
         return true;
     }
 }
\ No newline at end of file

Apps/Model/Front/Profile/FormWallPost.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace Apps\Model\Front\Profile;
 
-use Apps\ActiveRecord\Blacklist;
 use Apps\ActiveRecord\WallPost as WallRecords;
 use Ffcms\Core\App;
 use Ffcms\Core\Interfaces\iUser;

Apps/Controller/Api/Profile.php

Indentation +1 added lines, -1 removed lines

@@ -202,7 +202,7 @@
         $findAnswer->delete();
 
         $this->response = json_encode([
-           'status' => 1,
+            'status' => 1,
             'message' => 'ok'
         ]);
     }

Spacing +12 added lines, -12 removed lines

@@ -149,7 +149,7 @@ 
             $answerTime = Date::convertToTimestamp($lastAnswer->created_at);
             $cfgs = \Apps\ActiveRecord\App::getConfigs('app', 'Profile');
             // hmm, maybe past less then delay required?
-            if ($now - (int)$cfgs['delayBetweenPost'] < $answerTime) {
+            if ($now - (int) $cfgs['delayBetweenPost'] < $answerTime) {
                 throw new JsonException('Delay between answers not pass');
             }
         }
@@ -196,7 +196,7 @@ 
         $postInfo = $findAnswer->getWallPost();
 
         // if not a target user of answer and not answer owner - lets throw exception
-        if($postInfo->target_id !== $viewer->id && $findAnswer->user_id !== $viewer->id) {
+        if ($postInfo->target_id !== $viewer->id && $findAnswer->user_id !== $viewer->id) {
             throw new JsonException('Access declined!');
         }
 
@@ -256,7 +256,7 @@ 
             // sender is not myself? then i'm - target (remote user is sender user->to_me)
             if ($row->sender_id !== $user->id) {
                 $userList[] = $row->sender_id;
-                if ((int)$row->tread === 0) {
+                if ((int) $row->tread === 0) {
                     $unreadList[] = $row->sender_id;
                 }
             }
@@ -324,7 +324,7 @@ 
 
         // get special types for this action
         $queryType = App::$Request->get('type');
-        $queryId = (int)App::$Request->get('id');
+        $queryId = (int) App::$Request->get('id');
         // get current user object
         $user = App::$User->identity();
 
@@ -340,11 +340,11 @@ 
         switch ($queryType) {
             case 'after':
                 $messages = Message::where('id', '>', $queryId)
-                    ->where(function ($query) use ($cor_id, $user) {
-                        $query->where(function ($q) use ($cor_id, $user){
+                    ->where(function($query) use ($cor_id, $user) {
+                        $query->where(function($q) use ($cor_id, $user){
                             $q->where('target_id', '=', $user->getId())
                                 ->where('sender_id', '=', $cor_id);
-                        })->orWhere(function ($q) use ($cor_id, $user){
+                        })->orWhere(function($q) use ($cor_id, $user){
                             $q->where('target_id', '=', $cor_id)
                                 ->where('sender_id', '=', $user->getId());
                         });
@@ -352,11 +352,11 @@ 
                 break;
             case 'before':
                 $messages = Message::where('id', '<', $queryId)
-                    ->where(function ($query) use ($cor_id, $user) {
-                        $query->where(function ($q) use ($cor_id, $user){
+                    ->where(function($query) use ($cor_id, $user) {
+                        $query->where(function($q) use ($cor_id, $user){
                             $q->where('target_id', '=', $user->getId())
                                 ->where('sender_id', '=', $cor_id);
-                        })->orWhere(function ($q) use ($cor_id, $user){
+                        })->orWhere(function($q) use ($cor_id, $user){
                             $q->where('target_id', '=', $cor_id)
                                 ->where('sender_id', '=', $user->getId());
                         });
@@ -455,7 +455,7 @@ 
         $this->setJsonHeader();
 
         // get operation type and target user id
-        $target_id = (int)App::$Request->get('target');
+        $target_id = (int) App::$Request->get('target');
         $type = App::$Request->get('type');
 
         // check type of query
@@ -470,7 +470,7 @@ 
 
         $cfg = \Apps\ActiveRecord\App::getConfigs('app', 'Profile');
         // check if rating is enabled for website
-        if ((int)$cfg['rating'] !== 1) {
+        if ((int) $cfg['rating'] !== 1) {
             throw new JsonException('Rating is disabled');
         }
 

Apps/Model/Admin/Content/FormCategoryDelete.php

Indentation +6 added lines, -6 removed lines

@@ -29,8 +29,8 @@ 
     }
 
     /**
-    * Pass properties from record construction
-    */
+     * Pass properties from record construction
+     */
     public function before()
     {
         $this->title = Serialize::getDecodeLocale($this->_record->title);
@@ -38,8 +38,8 @@ 
     }
 
     /**
-    * Form labels
-    */
+     * Form labels
+     */
     public function labels()
     {
         return [
@@ -50,8 +50,8 @@ 
     }
 
     /**
-    * Validation rules
-    */
+     * Validation rules
+     */
     public function rules()
     {
         return [

Spacing +2 added lines, -2 removed lines

@@ -75,7 +75,7 @@ 
         }
 
         // get all depended category ids
-        $cats = ContentCategory::where('path', 'like', $this->_record->path . '%')->get(['id'])->toArray();
+        $cats = ContentCategory::where('path', 'like', $this->_record->path.'%')->get(['id'])->toArray();
         $allCategoryIds = Arr::ploke('id', $cats);
 
         // update category_id in content
@@ -110,7 +110,7 @@ 
         $response = [];
         foreach ($data as $key=>$val) {
             if ($this->_record->id !== $key) {
-                $response[] = (string)$key;
+                $response[] = (string) $key;
             }
         }
         return $response;

Apps/Model/Admin/Content/FormCategoryUpdate.php

Indentation +6 added lines, -6 removed lines

@@ -38,8 +38,8 @@ 
     }
 
     /**
-    * Pass properties from default record values
-    */
+     * Pass properties from default record values
+     */
     public function before()
     {
         if ($this->_record->id === null) {
@@ -74,8 +74,8 @@ 
     }
 
     /**
-    * Form labels
-    */
+     * Form labels
+     */
     public function labels()
     {
         return [
@@ -93,8 +93,8 @@ 
     }
 
     /**
-    * Validation rules
-    */
+     * Validation rules
+     */
     public function rules()
     {
         $rules = [

Unused Use Statements -1 removed lines

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

Spacing +4 added lines, -4 removed lines

@@ -107,7 +107,7 @@ 
         ];
 
         // general category
-        if ($this->_new === false && (int)$this->_record->id === 1) {
+        if ($this->_new === false && (int) $this->_record->id === 1) {
             $rules[] = ['path', 'used'];
         } else {
             $rules[] = ['path', 'required'];
@@ -116,7 +116,7 @@ 
             $rules[] = ['path', 'reverse_match', '/[\/\'~`\!@#\$%\^&\*\(\)+=\{\}\[\]\|;:"\<\>,\?\\\]/'];
         }
 
-        $rules[] = ['title.' . App::$Request->getLanguage(), 'required'];
+        $rules[] = ['title.'.App::$Request->getLanguage(), 'required'];
 
 
         return $rules;
@@ -129,7 +129,7 @@ 
     {
         $this->_record->title = Serialize::encode(App::$Security->strip_tags($this->title));
         $this->_record->description = Serialize::encode(App::$Security->strip_tags($this->description));
-        $savePath = trim($this->_pathNested . '/' . $this->path, '/');
+        $savePath = trim($this->_pathNested.'/'.$this->path, '/');
         $this->_record->path = $savePath;
         $this->_record->configs = Serialize::encode($this->configs);
         $this->_record->save();
@@ -164,7 +164,7 @@ 
         // build path with owner category
         $this->_pathNested = $owner->path;
         if (Str::length($this->_pathNested) > 0) {
-            $path = $this->_pathNested . '/' . $path;
+            $path = $this->_pathNested.'/'.$path;
         }
 
         // make select for check