phpffcms / ffcms-core

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/Managers/BootManager.php

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)
     {

Spacing +5 added lines, -5 removed lines

@@ -42,7 +42,7 @@ 
         // check if cache is enabled
         if (App::$Cache !== null) {
             // try to get bootable class map from cache, or initialize parsing
-            $cache = App::$Cache->getItem('boot.' . env_name . '.class.map');
+            $cache = App::$Cache->getItem('boot.'.env_name.'.class.map');
             if (!$cache->isHit()) {
                 $this->compileBootableClasses();
                 $cache->set($this->objects)->expiresAfter(static::CACHE_TREE_TIME);
@@ -97,11 +97,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;
@@ -111,14 +111,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;
                 }

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']);
 

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

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
         $options = $this->properties['options'];
         $optionsKey = (bool)$this->properties['optionsKey'];
         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']);
         

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

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
         $options = $this->properties['options'];
         unset($this->properties['options']);
         if (!Obj::isIterable($options)) {
-            throw new SyntaxException('Select field ' . self::nohtml($this->name) . ' have no iterable options');
+            throw new SyntaxException('Select field '.self::nohtml($this->name).' have no iterable options');
         }
         // value is not used there
         unset($this->properties['value']);

src/Ffcms/Core/Arch/Widget.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
 
         // check if class exist
         if (!class_exists(self::$class)) {
-            return 'Error: Widget is not founded: ' . self::$class;
+            return 'Error: Widget is not founded: '.self::$class;
         }
 
         // init class and pass properties

src/Ffcms/Core/Helper/FileSystem/File.php

Doc Comments +1 added lines, -1 removed lines

@@ -210,7 +210,7 @@
     /**
      * Get file md5 hash
      * @param string $path
-     * @return bool|string
+     * @return false|string
      */
     public static function getMd5($path)
     {

Spacing +3 added lines, -3 removed lines

@@ -192,15 +192,15 @@
             return [];
         }
 
-        $dir = opendir($path . '/.');
+        $dir = opendir($path.'/.');
         while ($item = readdir($dir)) {
-            if (is_file($sub = $path . '/' . $item)) {
+            if (is_file($sub = $path.'/'.$item)) {
                 $item_ext = Str::lastIn($item, '.');
                 if ($ext === null || Arr::in($item_ext, $ext)) {
                     if ($returnRelative) {
                         $files[] = $item;
                     } else {
-                        $files[] = $path . DIRECTORY_SEPARATOR . $item;
+                        $files[] = $path.DIRECTORY_SEPARATOR.$item;
                     }
                 }
             } else {

src/Ffcms/Core/Managers/MigrationsManager.php

Spacing +6 added lines, -6 removed lines

@@ -72,7 +72,7 @@ 
                 continue;
             }
             // check initialize conditions (equals to $exist)
-            if (File::exist($this->dir . '/' . $migration)) {
+            if (File::exist($this->dir.'/'.$migration)) {
                 if (Arr::in($fullName, $dbmigrations) === $exist) {
                     $found[] = $migration;
                 }
@@ -102,17 +102,17 @@ 
         }
 
         // check if migration file is exists
-        if (!File::exist($this->dir . '/' . $file)) {
+        if (!File::exist($this->dir.'/'.$file)) {
             return false;
         }
 
         // check if migration file located in extend directory and copy to default
         if (Normalize::diskFullPath($this->dir) !== Normalize::diskFullPath(static::DEFAULT_DIR)) {
-            File::copy($this->dir . DIRECTORY_SEPARATOR . $file, static::DEFAULT_DIR . DIRECTORY_SEPARATOR . $file);
+            File::copy($this->dir.DIRECTORY_SEPARATOR.$file, static::DEFAULT_DIR.DIRECTORY_SEPARATOR.$file);
         }
 
         // include migration and get class name
-        File::inc($this->dir . '/' . $file, false, false);
+        File::inc($this->dir.'/'.$file, false, false);
         $fullName = Str::cleanExtension($file);
         $class = Str::firstIn($fullName, '-');
 
@@ -148,11 +148,11 @@ 
         }
 
         // check if exists
-        if (!File::exist($this->dir . '/' . $file)) {
+        if (!File::exist($this->dir.'/'.$file)) {
             return false;
         }
 
-        File::inc($this->dir . '/' . $file, false, false);
+        File::inc($this->dir.'/'.$file, false, false);
         $fullName = Str::cleanExtension($file);
         $class = Str::firstIn($fullName, '-');