phpffcms / ffcms-core

src/Ffcms/Core/Managers/CronManager.php

Spacing +2 added lines, -2 removed lines

@@ -76,7 +76,7 @@ 
             return false;
         }
 
-        $callback = (string)$class . '::' . (string)$method;
+        $callback = (string)$class.'::'.(string)$method;
 
         // add instance to cron task manager
         if (!isset($this->configs['instances'][$callback])) {
@@ -94,7 +94,7 @@ 
      */
     public function remove($class, $method)
     {
-        $callback = $class . '::' . $method;
+        $callback = $class.'::'.$method;
         if (isset($this->configs['instances'][$callback])) {
             unset($this->configs['instances'][$callback], $this->configs['log'][$callback]);
             App::$Properties->writeConfig('Cron', $this->configs);

src/Ffcms/Core/Helper/HTML/Form.php

Doc Comments +2 added lines, -2 removed lines

@@ -103,8 +103,8 @@
 
     /**
      * Display form field. Allowed type: text, password, textarea, checkbox, select, checkboxes, file, captcha, email, hidden
-     * @param $object
-     * @param $type
+     * @param string $object
+     * @param string $type
      * @param null|array $property
      * @param null|string $helper
      * @param null|string $layerFile

Spacing +9 added lines, -9 removed lines

@@ -81,7 +81,7 @@ 
     {
         $form = self::buildSingleTag('form', $this->formProperty, false);
         if ($this->model->_tokenRequired) {
-            $form .= PHP_EOL . $this->field('_csrf_token', 'hidden', ['value' => $this->model->_csrf_token]);
+            $form .= PHP_EOL.$this->field('_csrf_token', 'hidden', ['value' => $this->model->_csrf_token]);
         }
         return $form;
     }
@@ -101,7 +101,7 @@ 
     {
         if ($this->model === null) {
             if (App::$Debug !== null) {
-                App::$Debug->addMessage('Form model is not defined for field name: [' . strip_tags($object) . ']');
+                App::$Debug->addMessage('Form model is not defined for field name: ['.strip_tags($object).']');
             }
             return null;
         }
@@ -115,7 +115,7 @@ 
         // check if model contains current tag name as property
         if (!property_exists($this->model, $propertyName)) {
             if (App::$Debug !== null) {
-                App::$Debug->addMessage('Form field ["' . $object . '"] is not defined in model: [' . get_class($this->model) . ']', 'error');
+                App::$Debug->addMessage('Form field ["'.$object.'"] is not defined in model: ['.get_class($this->model).']', 'error');
             }
             return null;
         }
@@ -136,13 +136,13 @@ 
         }
         
         // prepare labels text and label "for" attr 
-        $labelFor = $this->name . '-' . $propertyName;
+        $labelFor = $this->name.'-'.$propertyName;
         $labelText = $this->model->getLabel($object);
         $itemValue = $this->model->{$propertyName};
         // sounds like a dot-separated $object
         if ($propertyName !== $object) {
             $nesting = trim(strstr($object, '.'), '.');
-            $labelFor .= '-' . Str::replace('.', '-', $nesting);
+            $labelFor .= '-'.Str::replace('.', '-', $nesting);
             $itemValue = Arr::getByPath($nesting, $itemValue);
         }
 
@@ -173,7 +173,7 @@ 
     public function submitButton($title, array $property = [])
     {
         $property['type'] = 'submit';
-        $property['name'] = $this->name . '[submit]';
+        $property['name'] = $this->name.'[submit]';
         $property['value'] = $title;
         return self::buildSingleTag('input', $property);
     }
@@ -187,10 +187,10 @@ 
     {
         // pre-validate form fields based on model rules and jquery.validation
         if ($validate) {
-            App::$Alias->addPlainCode('js', '$().ready(function() { $("#' . $this->name . '").validate(); });');
+            App::$Alias->addPlainCode('js', '$().ready(function() { $("#'.$this->name.'").validate(); });');
             App::$Alias->setCustomLibrary('js', '/vendor/bower/jquery-validation/dist/jquery.validate.min.js');
             if (App::$Request->getLanguage() !== 'en') {
-                $localeFile = '/vendor/bower/jquery-validation/src/localization/messages_' . App::$Request->getLanguage() . '.js';
+                $localeFile = '/vendor/bower/jquery-validation/src/localization/messages_'.App::$Request->getLanguage().'.js';
                 if (File::exist($localeFile)) {
                     App::$Alias->setCustomLibrary('js', $localeFile);
                 }
@@ -201,7 +201,7 @@ 
                 $formName = $this->model->getFormName();
                 if (Obj::isArray($badAttr) && count($badAttr) > 0) {
                     foreach ($badAttr as $attr) {
-                        $itemId = $formName . '-' . $attr;
+                        $itemId = $formName.'-'.$attr;
                         $render = App::$View->render(static::$structLayer['jsnotify'], ['itemId' => $itemId]);
                         App::$Alias->addPlainCode('js', $render);
                     }

src/Ffcms/Core/Managers/BootManager.php

Spacing +7 added lines, -7 removed lines

@@ -39,11 +39,11 @@ 
         // check if cache is enabled
         if (App::$Cache !== null) {
             // try to get bootable class map from cache, or initialize parsing
-            if (App::$Cache->get('boot.' . env_name . '.class.map') !== null) {
-                $this->objects = App::$Cache->get('boot.' . env_name . '.class.map');
+            if (App::$Cache->get('boot.'.env_name.'.class.map') !== null) {
+                $this->objects = App::$Cache->get('boot.'.env_name.'.class.map');
             } else {
                 $this->compileBootableClasses();
-                App::$Cache->set('boot.' . env_name . '.class.map', $this->objects, static::CACHE_TREE_TIME);
+                App::$Cache->set('boot.'.env_name.'.class.map', $this->objects, static::CACHE_TREE_TIME);
             }
         }
     }
@@ -89,11 +89,11 @@ 
     {
         // list app root's
         foreach ($this->appRoots as $app) {
-            $app .= '/Apps/Controller/' . env_name;
+            $app .= '/Apps/Controller/'.env_name;
             $files = File::listFiles($app, ['.php'], true);
             foreach ($files as $file) {
                 // define full class name with namespace
-                $class = 'Apps\Controller\\' . env_name . '\\' . Str::cleanExtension($file);
+                $class = 'Apps\Controller\\'.env_name.'\\'.Str::cleanExtension($file);
                 // check if class exists (must be loaded over autoloader), boot method exist and this is controller instanceof
                 if (class_exists($class) && method_exists($class, 'boot') && is_a($class, 'Ffcms\Core\Arch\Controller', true)) {
                     $this->objects[] = $class;
@@ -103,14 +103,14 @@ 
 
         // list widget root's
         foreach ($this->widgetRoots as $widget) {
-            $widget .= '/Widgets/' . env_name;
+            $widget .= '/Widgets/'.env_name;
             // widgets are packed in directory, classname should be the same with root directory name
             $dirs = Directory::scan($widget, GLOB_ONLYDIR, true);
             if (!Obj::isArray($dirs)) {
                 continue;
             }
             foreach ($dirs as $instance) {
-                $class = 'Widgets\\' . env_name . '\\' . $instance . '\\' . $instance;
+                $class = 'Widgets\\'.env_name.'\\'.$instance.'\\'.$instance;
                 if (class_exists($class) && method_exists($class, 'boot') && is_a($class, 'Ffcms\Core\Arch\Widget', true)) {
                     $this->objects[] = $class;
                 }

Doc Comments +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
 
     /**
      * BootManager constructor. Pass composer loader inside
-     * @param bool|object $loader
+     * @param boolean $loader
      */
     public function __construct($loader = false)
     {

src/Ffcms/Core/Traits/ClassTools.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
     {
         $hash = null;
         foreach ($this as $property => $value) {
-            $hash = md5($hash . $property . '=' . $value);
+            $hash = md5($hash.$property.'='.$value);
         }
         return $hash;
     }

src/Ffcms/Core/Helper/Url.php

Spacing +14 added lines, -14 removed lines

@@ -23,7 +23,7 @@ 
     public static function to($controller_action, $id = null, $add = null, array $params = null, $encode = true)
     {
         $pathway = self::buildPathway([$controller_action, $id, $add, $params], $encode);
-        return App::$Alias->baseUrl . '/' . $pathway;
+        return App::$Alias->baseUrl.'/'.$pathway;
     }
 
     /**
@@ -60,7 +60,7 @@ 
                 $controller = Str::lastIn($controller, '\\', true);
             }
 
-            $response = $controller . '/' . $action;
+            $response = $controller.'/'.$action;
         }
 
         // check if controller and action is defined
@@ -70,17 +70,17 @@ 
 
         // id is defined?
         if (isset($to[1]) && !Str::likeEmpty($to[1])) {
-            $response .= '/' . self::safeUri($to[1], $encode);
+            $response .= '/'.self::safeUri($to[1], $encode);
         }
 
         // add param is defined?
         if (isset($to[2]) && !Str::likeEmpty($to[2])) {
-            $response .= '/' . self::safeUri($to[2], $encode);
+            $response .= '/'.self::safeUri($to[2], $encode);
         }
 
         // try to find static alias
-        if (isset($routing['Alias'][env_name]) && Arr::in('/' . $response, $routing['Alias'][env_name])) {
-            $pathAlias = array_search('/' . $response, $routing['Alias'][env_name]);
+        if (isset($routing['Alias'][env_name]) && Arr::in('/'.$response, $routing['Alias'][env_name])) {
+            $pathAlias = array_search('/'.$response, $routing['Alias'][env_name]);
             if ($pathAlias !== false) {
                 $response = Str::lowerCase(trim($pathAlias, '/'));
             }
@@ -96,7 +96,7 @@ 
             }
             $queryString = http_build_query($to[3]);
             if (Str::length($queryString) > 0) {
-                $response .= '?' . http_build_query($to[3]);
+                $response .= '?'.http_build_query($to[3]);
             }
             if ($anchor !== false) {
                 $response .= $anchor;
@@ -118,7 +118,7 @@ 
     public static function buildPathwayFromRequest()
     {
         return self::buildPathway([
-            Str::lowerCase(App::$Request->getController()) . '/' . Str::lowerCase(App::$Request->getAction()),
+            Str::lowerCase(App::$Request->getController()).'/'.Str::lowerCase(App::$Request->getAction()),
             App::$Request->getID(),
             App::$Request->getAdd(),
             App::$Request->query->all()
@@ -140,9 +140,9 @@ 
             $to = [$to];
         }
         // call Url::to(args)
-        $callbackTo = call_user_func_array([__NAMESPACE__ . '\Url', 'to'], $to);
+        $callbackTo = call_user_func_array([__NAMESPACE__.'\Url', 'to'], $to);
 
-        return '<a href="' . $callbackTo . '"' . $compile_property . '>' . $name . '</a>';
+        return '<a href="'.$callbackTo.'"'.$compile_property.'>'.$name.'</a>';
     }
 
     /**
@@ -155,19 +155,19 @@ 
     {
         /** @var array $configs */
         $configs = \App::$Properties->getAll('default');
-        $httpHost = $configs['baseProto'] . '://' . $configs['baseDomain'];
+        $httpHost = $configs['baseProto'].'://'.$configs['baseDomain'];
         if ($configs['basePath'] !== '/') {
-            $httpHost .= $configs['basePath'] . '/';
+            $httpHost .= $configs['basePath'].'/';
         }
 
         // check if is this is URI not URL
         if (!Str::startsWith($httpHost, $uri)) {
             // check if lang is defined in URI or define it
             if ($lang !== null && $configs['multiLanguage'] && !Str::startsWith($lang, $uri)) {
-                $uri = $lang . '/' . ltrim($uri, '/');
+                $uri = $lang.'/'.ltrim($uri, '/');
             }
             // add basic httpHost data
-            $uri = rtrim($httpHost, '/') . '/' . ltrim($uri, '/');
+            $uri = rtrim($httpHost, '/').'/'.ltrim($uri, '/');
         }
 
         return $uri;

src/Ffcms/Core/Alias.php

Spacing +10 added lines, -10 removed lines

@@ -92,7 +92,7 @@ 
             $this->baseDomain = App::$Properties->get('baseDomain');
         }
         // build script url
-        $this->scriptUrl = App::$Request->getScheme() . '://' . $this->baseDomain;
+        $this->scriptUrl = App::$Request->getScheme().'://'.$this->baseDomain;
         if (App::$Properties->get('basePath') !== '/') {
             $this->scriptUrl .= rtrim(App::$Properties->get('basePath'), '/');
         }
@@ -104,27 +104,27 @@ 
 
         $this->baseUrlNoLang = $this->baseUrl;
         if (App::$Request->languageInPath() && App::$Request->getLanguage() !== null) {
-            $this->baseUrl .= '/' . App::$Request->getLanguage();
+            $this->baseUrl .= '/'.App::$Request->getLanguage();
         }
 
         // add cron initiation from user if enabled
         if ((bool)App::$Properties->get('userCron') && env_name === 'Front') {
-            $this->addPlainCode('js', 'if(Math.random() > 0.8) { $.get("' . $this->scriptUrl . '/cron.php?rand=" + Math.random()); }');
+            $this->addPlainCode('js', 'if(Math.random() > 0.8) { $.get("'.$this->scriptUrl.'/cron.php?rand=" + Math.random()); }');
         }
 
         // build vendor libs alias
-        $this->vendorNamedLibrary['js']['jquery'] = $this->scriptUrl . '/vendor/bower/jquery/dist/jquery.min.js';
-        $this->vendorNamedLibrary['css']['bootstrap'] = $this->scriptUrl . '/vendor/bower/bootstrap/dist/css/bootstrap.min.css';
-        $this->vendorNamedLibrary['js']['bootstrap'] = $this->scriptUrl . '/vendor/bower/bootstrap/dist/js/bootstrap.min.js';
-        $this->vendorNamedLibrary['css']['fa'] = $this->scriptUrl . '/vendor/bower/components-font-awesome/css/font-awesome.min.css';
-        $this->vendorNamedLibrary['js']['jquery-ui'] = $this->scriptUrl . '/vendor/bower/jquery-ui/jquery-ui.min.js';
-        $this->vendorNamedLibrary['css']['jquery-ui'] = $this->scriptUrl . '/vendor/bower/jquery-ui/themes/base/jquery-ui.min.css';
+        $this->vendorNamedLibrary['js']['jquery'] = $this->scriptUrl.'/vendor/bower/jquery/dist/jquery.min.js';
+        $this->vendorNamedLibrary['css']['bootstrap'] = $this->scriptUrl.'/vendor/bower/bootstrap/dist/css/bootstrap.min.css';
+        $this->vendorNamedLibrary['js']['bootstrap'] = $this->scriptUrl.'/vendor/bower/bootstrap/dist/js/bootstrap.min.js';
+        $this->vendorNamedLibrary['css']['fa'] = $this->scriptUrl.'/vendor/bower/components-font-awesome/css/font-awesome.min.css';
+        $this->vendorNamedLibrary['js']['jquery-ui'] = $this->scriptUrl.'/vendor/bower/jquery-ui/jquery-ui.min.js';
+        $this->vendorNamedLibrary['css']['jquery-ui'] = $this->scriptUrl.'/vendor/bower/jquery-ui/themes/base/jquery-ui.min.css';
 
         $themeAll = App::$Properties->get('theme');
         if (!isset($themeAll[env_name]) || Str::length($themeAll[env_name]) < 1) {
             $themeAll[env_name] = 'default';
         }
-        $this->currentViewUrl = $this->scriptUrl . '/Apps/View/' . env_name . '/' . $themeAll[env_name];
+        $this->currentViewUrl = $this->scriptUrl.'/Apps/View/'.env_name.'/'.$themeAll[env_name];
     }
 
     /**

src/Ffcms/Core/Helper/Security.php

Doc Comments +2 added lines, -2 removed lines

@@ -50,7 +50,7 @@ 
      * String html tags and escape quotes
      * @param string|array $html
      * @param boolean $escapeQuotes
-     * @return string|array|null
+     * @return string|null
      */
     public function strip_tags($html, $escapeQuotes = true)
     {
@@ -90,7 +90,7 @@ 
      * @deprecated
      * @param $var
      * @param null $indent
-     * @return mixed|string
+     * @return string|null
      */
     public function var_export54($var, $indent = null, $guessTypes = false) {
         return Arr::exportVar($var, $indent, $guessTypes);

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
     public function __construct()
     {
         $config = \HTMLPurifier_Config::createDefault();
-        $config->set('Cache.SerializerPath', root . '/Private/Cache/HTMLPurifier/');
+        $config->set('Cache.SerializerPath', root.'/Private/Cache/HTMLPurifier/');
         $config->set('HTML.Allowed', 'p,b,strong,em,a[href],i,span,ul,ol,li,blockquote,h2,h3,pre,code,img[src|alt|width|height]');
         //$config->set('URI.Base', 'http://www.example.com');
         //$config->set('URI.MakeAbsolute', true);

src/Ffcms/Core/Helper/HTML/Form/MultiCheckboxField.php

Spacing +2 added lines, -2 removed lines

@@ -44,7 +44,7 @@ 
         // check if options is defined
         $options = $this->properties['options'];
         if (!Obj::isIterable($options)) {
-            throw new SyntaxException('Options for field ' . self::nohtml($this->name) . ' is not iterable');
+            throw new SyntaxException('Options for field '.self::nohtml($this->name).' is not iterable');
         }
         unset($this->properties['options']);
         
@@ -65,7 +65,7 @@ 
             $this->properties['value'] = $opt;
             // apply structured checkboxes style for each item
             $build .= App::$View->render('native/form/multi_checkboxes_list', [
-                'item' => self::buildSingleTag('input', $this->properties) . self::nohtml($opt)
+                'item' => self::buildSingleTag('input', $this->properties).self::nohtml($opt)
             ]);
         }
         

src/Ffcms/Core/Helper/HTML/Form/RadioField.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
         // get options from properties
         $options = $this->properties['options'];
         if (!Obj::isIterable($options)) {
-            throw new SyntaxException('Radio field ' . self::nohtml($this->name) . ' have no iterable options');
+            throw new SyntaxException('Radio field '.self::nohtml($this->name).' have no iterable options');
         }
         unset($this->properties['options'], $this->properties['value']);