phpffcms / ffcms

Apps/Model/Install/Main/FormInstall.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use Apps\ActiveRecord\Profile;
 use Apps\ActiveRecord\User;
-use Apps\Controller\Console\Db;
 use Ffcms\Core\App;
 use Ffcms\Core\Arch\Model;
 use Ffcms\Core\Helper\FileSystem\File;

Spacing +4 added lines, -4 removed lines

@@ -71,14 +71,14 @@
         $cfg['database'] = $this->db;
         $cfg['adminEmail'] = $this->email;
         $cfg['singleLanguage'] = $this->singleLanguage;
-        $cfg['multiLanguage'] = (bool)$this->multiLanguage;
-        $cfg['passwordSalt'] = '$2a$07$' . Str::randomLatinNumeric(mt_rand(21, 30)) . '$';
-        $cfg['debug']['cookie']['key'] = 'fdebug_' . Str::randomLatinNumeric(mt_rand(4, 16));
+        $cfg['multiLanguage'] = (bool) $this->multiLanguage;
+        $cfg['passwordSalt'] = '$2a$07$'.Str::randomLatinNumeric(mt_rand(21, 30)).'$';
+        $cfg['debug']['cookie']['key'] = 'fdebug_'.Str::randomLatinNumeric(mt_rand(4, 16));
         $cfg['debug']['cookie']['value'] = Str::randomLatinNumeric(mt_rand(32, 128));
 
         // import database tables
         $connectName = 'install';
-        include(root . '/Private/Database/install.php');
+        include(root.'/Private/Database/install.php');
 
         // insert admin user
         $user = new User();

Apps/Model/Install/Main/EntityCheck.php

Indentation +2 added lines, -2 removed lines

@@ -20,8 +20,8 @@
     public $chmodCheck = [];
 
     /**
-    * Get default server information and prepare chmod info
-    */
+     * Get default server information and prepare chmod info
+     */
     public function before()
     {
         $this->phpVersion = phpversion();

Apps/View/Install/default/main/index.php

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
 ]);
 ?>
 
-<?php if($model->checkAll()): ?>
+<?php if ($model->checkAll()): ?>
     <?= \Ffcms\Core\Helper\Url::link('main/install', __('Start install'), ['class' => 'btn btn-success btn-block']) ?>
 <?php else: ?>
     <?= \Ffcms\Core\Helper\Url::link('main/index', __('Check again'), ['class' => 'btn btn-warning btn-block']) ?>

Braces +5 added lines, -2 removed lines

@@ -56,7 +56,10 @@
 
 <?php if($model->checkAll()): ?>
     <?= \Ffcms\Core\Helper\Url::link('main/install', __('Start install'), ['class' => 'btn btn-success btn-block']) ?>
-<?php else: ?>
+<?php else {
+    : ?>
     <?= \Ffcms\Core\Helper\Url::link('main/index', __('Check again'), ['class' => 'btn btn-warning btn-block']) ?>
-<?php endif; ?>
+<?php endif;
+}
+?>
 

Private/Database/Tables/Session.php

Spacing +1 added lines, -1 removed lines

@@ -1,7 +1,7 @@
 <?php
 
 Illuminate\Database\Capsule\Manager::schema($connectName)->create('sessions', function($table) {
-    $table->string('sess_id', 128)->index()->primary()  ;
+    $table->string('sess_id', 128)->index()->primary();
     $table->binary('sess_data');
     $table->string('sess_lifetime', 16); // shutout to laravel, if i make it like "integer" it automaticlly add "primary" key for it, hate this!!!
     $table->string('sess_time', 16);

Apps/View/Front/default/content/read.php

Braces +10 added lines, -4 removed lines

@@ -67,17 +67,23 @@
         <?php if ($properties['author'] === true): ?>
             <?php if ($model->authorId !== null && $model->authorId > 0): ?>
                 <span><i class="fa fa-user"></i><?= Url::link(['profile/show', $model->authorId], $model->authorName, ['itemprop' => 'author']) ?></span>
-            <?php else: ?>
+            <?php else {
+    : ?>
                 <span><i class="fa fa-user"></i><s><?= $model->authorName ?></s></span>
-            <?php endif; ?>
+            <?php endif;
+}
+?>
         <?php endif; ?>
         <?php if ($properties['views'] === true): ?>
         <span><i class="fa fa-eye"></i><?= $model->views ?></span>
         <?php endif ?>
     </div>
-    <?php else: ?>
+    <?php else {
+    : ?>
     <hr />
-    <?php endif; ?>
+    <?php endif;
+}
+?>
     <?php if ($trash): ?>
     <p class="alert alert-danger"><i class="fa fa-trash-o"></i> <?= __('This content is placed in trash') ?></p>
     <?php endif; ?>

Indentation +6 added lines, -6 removed lines

@@ -159,12 +159,12 @@
     	<div class="col-md-12">
     		<div class="pull-left">
         	<?php if ($properties['rating']) {
-        	    echo \App::$View->render('content/_rate', [
-        	        'id' => $model->id,
-        	        'rating' => $model->rating,
-        	        'canRate' => $model->canRate
-        	    ]);
-        	} ?>
+                echo \App::$View->render('content/_rate', [
+                    'id' => $model->id,
+                    'rating' => $model->rating,
+                    'canRate' => $model->canRate
+                ]);
+            } ?>
         	<?= \App::$View->render('content/_rateJs') ?>
     		</div>
     		<div class="pull-right">

Spacing +18 added lines, -18 removed lines

@@ -23,7 +23,7 @@ 
 }
 
 // don't use breadcrumbs on injected pathway rule
-if (!\App::$Request->isPathInjected() && (bool)$model->getCategory()->getProperty('showCategory')) {
+if (!\App::$Request->isPathInjected() && (bool) $model->getCategory()->getProperty('showCategory')) {
     $breadcrumbs = [
             Url::to('/') => __('Home')
     ];
@@ -35,22 +35,22 @@ 
                 $breadcrumbs[Url::to('content/list', $cat['path'], null, [], false)] = $cat['name'];
             }
         }
-        $breadcrumbs[] = __('Content') . ': ' . Str::sub($this->title, 0, 40);
+        $breadcrumbs[] = __('Content').': '.Str::sub($this->title, 0, 40);
     }
 
     $this->breadcrumbs = $breadcrumbs;
 }
 
 $properties = [
-    'date' => (bool)$model->getCategory()->getProperty('showDate'),
-    'author' => (bool)$model->getCategory()->getProperty('showAuthor'),
-    'views' => (bool)$model->getCategory()->getProperty('showViews'),
-    'category' => (bool)$model->getCategory()->getProperty('showCategory'),
-    'rating' => (bool)$model->getCategory()->getProperty('showRating'),
-    'tags' => (bool)$model->getCategory()->getProperty('showTags')
+    'date' => (bool) $model->getCategory()->getProperty('showDate'),
+    'author' => (bool) $model->getCategory()->getProperty('showAuthor'),
+    'views' => (bool) $model->getCategory()->getProperty('showViews'),
+    'category' => (bool) $model->getCategory()->getProperty('showCategory'),
+    'rating' => (bool) $model->getCategory()->getProperty('showRating'),
+    'tags' => (bool) $model->getCategory()->getProperty('showTags')
 ];
-$showComments = (bool)$model->getCategory()->getProperty('showComments');
-$showPoster = (bool)$model->getCategory()->getProperty('showPoster');
+$showComments = (bool) $model->getCategory()->getProperty('showComments');
+$showPoster = (bool) $model->getCategory()->getProperty('showPoster');
 
 ?>
 
@@ -75,7 +75,7 @@ 
         <span class="spaced"><i class="fa fa-eye"></i> <?= $model->views ?></span>
         <?php endif ?>
         <?php if (\App::$User->isAuth() && \App::$User->identity()->getRole()->can('Admin/Content/Update')): ?>
-        <span class="pull-right"><a href="<?= \App::$Alias->scriptUrl . '/admin/content/update/' . $model->id ?>" target="_blank"><i class="fa fa-pencil" style="color: #ff0000;"></i></a></span>
+        <span class="pull-right"><a href="<?= \App::$Alias->scriptUrl.'/admin/content/update/'.$model->id ?>" target="_blank"><i class="fa fa-pencil" style="color: #ff0000;"></i></a></span>
         <?php endif; ?>
     </div>
     <?php else: ?>
@@ -87,7 +87,7 @@ 
     <div id="content-text">
         <?php if ($showPoster === true && $model->posterFull !== null && $model->posterThumb !== null): ?>
             <a href="#showPoster" data-toggle="modal" data-target="#showPoster">
-                <img alt="<?= __('Poster for') ?>: <?= Str::lowerCase(\App::$Security->strip_tags($model->title)) ?>" src="<?= \App::$Alias->scriptUrl . $model->posterThumb ?>" class="image_poster img-thumbnail" />
+                <img alt="<?= __('Poster for') ?>: <?= Str::lowerCase(\App::$Security->strip_tags($model->title)) ?>" src="<?= \App::$Alias->scriptUrl.$model->posterThumb ?>" class="image_poster img-thumbnail" />
             </a>
 
             <!-- Modal poster pop-up -->
@@ -99,19 +99,19 @@ 
                             <h4 class="modal-title" id="myModalLabel"><?= __('View poster') ?></h4>
                         </div>
                         <div class="modal-body">
-                            <img class="img-responsive" src="<?= \App::$Alias->scriptUrl . $model->posterFull ?>" alt="<?= __('Poster image') ?>" style="margin: 0 auto;" />
+                            <img class="img-responsive" src="<?= \App::$Alias->scriptUrl.$model->posterFull ?>" alt="<?= __('Poster image') ?>" style="margin: 0 auto;" />
                         </div>
                     </div>
                 </div>
             </div>
-        <?php endif ;?>
+        <?php endif; ?>
         <?= $model->text ?>
     </div>
     <?php if ($model->galleryItems !== null && Obj::isArray($model->galleryItems)): ?>
         <div class="row">
         <?php foreach ($model->galleryItems as $thumbPic => $fullPic): ?>
             <div class="col-md-2 well">
-                <a href="#showGallery" class="modalGallery" content="<?= \App::$Alias->scriptUrl . $fullPic ?>"><img src="<?= \App::$Alias->scriptUrl . $thumbPic ?>" class="img-responsive image-item" /></a>
+                <a href="#showGallery" class="modalGallery" content="<?= \App::$Alias->scriptUrl.$fullPic ?>"><img src="<?= \App::$Alias->scriptUrl.$thumbPic ?>" class="img-responsive image-item" /></a>
             </div>
         <?php endforeach; ?>
         </div>
@@ -123,7 +123,7 @@ 
                         <h4 class="modal-title" id="showModalLabel"><?= __('View picture') ?></h4>
                     </div>
                     <div class="modal-body" id="modal-gallery-body">
-                        <img class="img-responsive" src="<?= \App::$Alias->scriptUrl . $model->posterFull ?>" alt="<?= __('Gallery picture') ?>" style="margin: 0 auto;" />
+                        <img class="img-responsive" src="<?= \App::$Alias->scriptUrl.$model->posterFull ?>" alt="<?= __('Gallery picture') ?>" style="margin: 0 auto;" />
                     </div>
                 </div>
             </div>
@@ -144,7 +144,7 @@ 
                 </div>
                 <div id="similar<?= $idx ?>" class="panel-collapse collapse">
                     <div class="panel-body">
-                        <a href="<?= \App::$Alias->baseUrl . $item['uri'] ?>">
+                        <a href="<?= \App::$Alias->baseUrl.$item['uri'] ?>">
                             <?= $item['snippet'] ?>
                         </a>
                     </div>
@@ -175,7 +175,7 @@ 
                         echo '<i class="fa fa-tags hidden-xs"></i> ';
                         foreach ($model->metaKeywords as $tag) {
                             $tag = \App::$Security->strip_tags(trim($tag));
-                            echo Url::link(['content/tag', $tag], $tag, ['class' => 'label label-default']) . "&nbsp;";
+                            echo Url::link(['content/tag', $tag], $tag, ['class' => 'label label-default'])."&nbsp;";
                         }
                     }
                     ?>

Apps/Model/Admin/Application/FormInstall.php

Indentation +2 added lines, -2 removed lines

@@ -33,8 +33,8 @@
     }
 
     /**
-    * Validation rules
-    */
+     * Validation rules
+     */
     public function rules()
     {
         return [

Spacing +5 added lines, -5 removed lines

@@ -39,7 +39,7 @@ 
     public function before()
     {
         foreach ($this->_apps as $app) {
-            $this->_definedControllers[] = (string)$app->sys_name;
+            $this->_definedControllers[] = (string) $app->sys_name;
         }
 
         parent::before();
@@ -70,14 +70,14 @@ 
     public function make()
     {
         $cName = ucfirst(Str::lowerCase($this->sysname));
-        $cPath = 'Apps\Controller\Admin\\' . $cName;
+        $cPath = 'Apps\Controller\Admin\\'.$cName;
         // if object class is not loaded - prevent install
-        if (!class_exists($cPath) || !defined($cPath . '::VERSION')) {
+        if (!class_exists($cPath) || !defined($cPath.'::VERSION')) {
             return false;
         }
 
         // get ext version
-        $cVersion = (float)constant($cPath . '::VERSION');
+        $cVersion = (float) constant($cPath.'::VERSION');
         if ($cVersion < 0.1) {
             $cVersion = 0.1;
         }
@@ -93,7 +93,7 @@ 
 
         // callback to install method in extension
         if (method_exists($cPath, 'install')) {
-            call_user_func($cPath . '::install');
+            call_user_func($cPath.'::install');
         }
 
         return true;

Apps/Controller/Admin/Content.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 +4 added lines, -4 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;
 
         $query = null;
@@ -51,7 +51,7 @@ 
         } elseif ($type === 'moderate') { // only items on moderate
             $query = ContentEntity::where('display', '=', 0);
         } elseif (Obj::isLikeInt($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';
@@ -197,7 +197,7 @@ 
         if ($model->send() && $model->validate()) {
             // remove all trashed items
             foreach ($records->get() as $item) {
-                $galleryPath = '/upload/gallery/' . (int)$item->id;
+                $galleryPath = '/upload/gallery/'.(int) $item->id;
                 if (Directory::exist($galleryPath)) {
                     Directory::remove($galleryPath);
                 }
@@ -274,7 +274,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);

Extend/Core/Arch/AdminController.php

Doc Comments +1 added lines, -1 removed lines

@@ -123,7 +123,7 @@
     /**
      * Get current extension active record
      * @param string|null $type
-     * @return mixed
+     * @return AppRecord
      */
     public function getTypeItem($type = null)
     {

Spacing +4 added lines, -4 removed lines

@@ -86,18 +86,18 @@ 
         $user = App::$User->identity();
         // user is not authed ?
         if ($user === null || !App::$User->isAuth()) {
-            $redirectUrl = App::$Alias->scriptUrl . '/user/login';
+            $redirectUrl = App::$Alias->scriptUrl.'/user/login';
             App::$Response->redirect($redirectUrl, true);
             exit();
         }
 
-        $permission = env_name . '/' . App::$Request->getController() . '/' . App::$Request->getAction();
+        $permission = env_name.'/'.App::$Request->getController().'/'.App::$Request->getAction();
 
         // doesn't have permission? get the f*ck out
         if (!$user->getRole()->can($permission)) {
             App::$Session->invalidate();
 
-            $redirectUrl = App::$Alias->scriptUrl . '/user/login';
+            $redirectUrl = App::$Alias->scriptUrl.'/user/login';
             App::$Response->redirect($redirectUrl, true);
             exit();
         }
@@ -144,7 +144,7 @@ 
      */
     public function getConfigs()
     {
-        return $this->type === 'widget' ? (array)Serialize::decode($this->widget->configs) : (array)Serialize::decode($this->application->configs);
+        return $this->type === 'widget' ? (array) Serialize::decode($this->widget->configs) : (array) Serialize::decode($this->application->configs);
     }
 
     /**

Apps/Controller/Admin/Application.php

Spacing +2 added lines, -2 removed lines

@@ -71,7 +71,7 @@ 
         $search = \Apps\ActiveRecord\App::getItem('app', $controller);
 
         // check what we got
-        if ($search === null || (int)$search->id < 1) {
+        if ($search === null || (int) $search->id < 1) {
             throw new ForbiddenException('App is not founded');
         }
 
@@ -96,7 +96,7 @@ 
 
         $search = \Apps\ActiveRecord\App::where('sys_name', '=', $controllerName)->where('type', '=', 'app')->first();
 
-        if ($search === null || (int)$search->id < 1) {
+        if ($search === null || (int) $search->id < 1) {
             throw new ForbiddenException('App is not founded');
         }