phpffcms / ffcms-core

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/Traits/ModelValidator.php

Spacing +5 added lines, -5 removed lines

@@ -138,7 +138,7 @@ 
                     } elseif (method_exists($class, $obj)) { // maybe its a function?
                         $class = $class::$obj; // call function
                     } else {
-                        throw new SyntaxException('Filter callback execution failed: ' . $filterName);
+                        throw new SyntaxException('Filter callback execution failed: '.$filterName);
                     }
 
                 }
@@ -148,7 +148,7 @@ 
             if (method_exists($class, $method)) {
                 $check = @$class::$method($fieldValue, $filterArgs);
             } else {
-                throw new SyntaxException('Filter callback execution failed: ' . $filterName);
+                throw new SyntaxException('Filter callback execution failed: '.$filterName);
             }
         } elseif (method_exists('Ffcms\Core\Helper\ModelFilters', $filterName)) { // only full namespace\class path based :(
             if ($filterArgs != null) {
@@ -157,7 +157,7 @@ 
                 $check = ModelFilters::$filterName($fieldValue);
             }
         } else {
-            throw new SyntaxException('Filter "' . $filterName . '" is not exist');
+            throw new SyntaxException('Filter "'.$filterName.'" is not exist');
         }
         if ($check !== true) { // switch only on fail check.
             $this->_badAttr[] = $propertyName;
@@ -323,10 +323,10 @@ 
         $paramQuery = $this->getFormName();
         if (Str::contains('.', $param)) {
             foreach (explode('.', $param) as $item) {
-                $paramQuery .= '[' . $item . ']';
+                $paramQuery .= '['.$item.']';
             }
         } else {
-            $paramQuery .= '[' . $param . ']';
+            $paramQuery .= '['.$param.']';
         }
 
         if ($method === null) {

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

src/Ffcms/Core/Arch/ActiveModel.php

Doc Comments +1 added lines, -1 removed lines

@@ -91,7 +91,7 @@
     /**
      * Serialize value
      * @param $value
-     * @return Serialize
+     * @return string
      */
     public function asSerialize($value)
     {

src/Ffcms/Core/Helper/HTML/Bootstrap/Navbar.php

Spacing +8 added lines, -8 removed lines

@@ -37,7 +37,7 @@ 
         // set mobile collapse id for toggle
         $mobCollapseId = $elements['collapseId'];
         if (Str::likeEmpty($mobCollapseId)) {
-            $mobCollapseId = Str::randomLatin(mt_rand(6,12)) . mt_rand(1, 99);
+            $mobCollapseId = Str::randomLatin(mt_rand(6, 12)).mt_rand(1, 99);
         }
 
         // set element id for toggle
@@ -99,7 +99,7 @@ 
         // generate output dom of bootstrap navbar
         $dom = new Dom();
         $body = $dom->div(function() use ($leftBuild, $rightBuild, $itemsStatic){
-            return $leftBuild . $itemsStatic . $rightBuild;
+            return $leftBuild.$itemsStatic.$rightBuild;
         }, ['class' => 'collapse navbar-collapse', 'id' => $mobCollapseId]);
 
         // drow <nav @properties>@next</nav>
@@ -110,17 +110,17 @@ 
                 $header = $dom->div(function() use ($dom, $elements, $mobCollapseId){
                     // drow <button @collapse>@next</button>
                     $collapseButton = $dom->button(function() use ($dom){
-                        $toggleItem = $dom->span(function(){
+                        $toggleItem = $dom->span(function() {
                             return 'Toggle menu';
                         }, ['class' => 'sr-only']);
                         $toggleIcon = null;
                         for ($i = 0; $i < 3; $i++) {
-                            $toggleIcon .= $dom->span(function(){
+                            $toggleIcon .= $dom->span(function() {
                                 return null;
                             }, ['class' => 'icon-bar']);
                         }
-                        return $toggleItem . $toggleIcon;
-                    }, ['type' => 'button', 'class' => 'navbar-toggle collapsed', 'data-toggle' => 'collapse', 'data-target' => '#' . $mobCollapseId]);
+                        return $toggleItem.$toggleIcon;
+                    }, ['type' => 'button', 'class' => 'navbar-toggle collapsed', 'data-toggle' => 'collapse', 'data-target' => '#'.$mobCollapseId]);
                     // drow <div @brand>@brandtext<?div>
                     $brand = null;
                     if (isset($elements['brand'])) {
@@ -132,10 +132,10 @@ 
                             }, ['class' => 'navbar-brand']);
                         }
                     }
-                    return $collapseButton . $brand;
+                    return $collapseButton.$brand;
                 }, ['class' => 'navbar-header']);
                 // return header and body concat
-                return $header . $body;
+                return $header.$body;
             }, $elements['container']);
         }, $elements['nav']);
     }

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

Spacing +7 added lines, -7 removed lines

@@ -45,7 +45,7 @@ 
     {
         // check if properties is passed well
         if ($properties !== null && !Obj::isArray($properties)) {
-            throw new SyntaxException('Property must be passed as array or null! Field: ' . $name);
+            throw new SyntaxException('Property must be passed as array or null! Field: '.$name);
         }
         
         // add properties to autovalidation by js (properties passed by ref)
@@ -98,9 +98,9 @@ 
         
         // if field is unknown type add notification in debugbar
         if (App::$Debug !== null) {
-            App::$Debug->addMessage('Field with name [' . App::$Security->strip_tags($name) . '] have unknown type [' . $this->type . ']', 'error');
+            App::$Debug->addMessage('Field with name ['.App::$Security->strip_tags($name).'] have unknown type ['.$this->type.']', 'error');
         }
-        return 'No data: ' . App::$Security->strip_tags($name);
+        return 'No data: '.App::$Security->strip_tags($name);
     }
     
     /**
@@ -147,12 +147,12 @@ 
         if (Str::contains('.', $name)) {
             $splitedName = explode('.', $name);
             foreach ($splitedName as $nameKey) {
-                $properties['name'] .= '[' . $nameKey . ']';
-                $properties['id'] .= '-' . $nameKey;
+                $properties['name'] .= '['.$nameKey.']';
+                $properties['id'] .= '-'.$nameKey;
             }
         } else { // standard property definition - add field name
-            $properties['name'] .= '[' . $name . ']';
-            $properties['id'] .= '-' . $name;
+            $properties['name'] .= '['.$name.']';
+            $properties['id'] .= '-'.$name;
         }
     }
 }
\ No newline at end of file

src/Ffcms/Core/Helper/HTML/System/NativeGenerator.php

Spacing +9 added lines, -9 removed lines

@@ -23,7 +23,7 @@ 
             $purifier = App::$Memory->get('object.purifier.helpers');
         } else {
             $config = \HTMLPurifier_Config::createDefault();
-            $config->set('Cache.SerializerPath', root . '/Private/Cache/HTMLPurifier/');
+            $config->set('Cache.SerializerPath', root.'/Private/Cache/HTMLPurifier/');
             $config->set('AutoFormat.AutoParagraph', false);
 
             // allow use target=_blank for links
@@ -65,9 +65,9 @@ 
         $build = null;
         foreach ($property as $p => $v) {
             if ($v === null || $v === false || $v === true) {
-                $build .= ' ' . self::nohtml($p);
+                $build .= ' '.self::nohtml($p);
             } else {
-                $build .= ' ' . self::nohtml($p) . '="' . self::nohtml($v) . '"';
+                $build .= ' '.self::nohtml($p).'="'.self::nohtml($v).'"';
             }
         }
         return $build;
@@ -82,7 +82,7 @@ 
      */
     public static function buildSingleTag($tagName, array $property = null, $closeSlash = true)
     {
-        return '<' . self::nohtml($tagName) . self::applyProperty($property) . ($closeSlash ? '/>' : '>');
+        return '<'.self::nohtml($tagName).self::applyProperty($property).($closeSlash ? '/>' : '>');
     }
 
     /**
@@ -100,7 +100,7 @@ 
             $value = self::nohtml($value);
         }
         
-        return '<' . $tagName . self::applyProperty($property) . '>' . $value . '</' . $tagName . '>';
+        return '<'.$tagName.self::applyProperty($property).'>'.$value.'</'.$tagName.'>';
     }
 
     /**
@@ -142,13 +142,13 @@ 
                 if (!Str::contains('/', $elementPoint) || count($elementArray) < 2) {
                     $active = $elementPoint === $currentPoint;
                 } else {
-                    $elementPoint = $elementArray[0] . '/' . $elementArray[1];
+                    $elementPoint = $elementArray[0].'/'.$elementArray[1];
                     $active = Str::startsWith($elementPoint, $currentPoint);
                 }
                 break;
             case 'id':
                 $elementArray = explode('/', $elementPoint);
-                $elementPoint = $elementArray[0] . '/' . $elementArray[1];
+                $elementPoint = $elementArray[0].'/'.$elementArray[1];
                 if ($elementArray[2] === null) { // looks like id is not defined in element
                     if (Str::contains('?', $currentPoint)) {
                         $currentPoint = Str::firstIn($currentPoint, '?');
@@ -157,7 +157,7 @@ 
                     $currentToId = implode('/', array_slice($currentArray, 0, 3));
                     $active = $elementPoint === $currentToId;
                 } else {
-                    $elementPoint .= '/' . $elementArray[2];
+                    $elementPoint .= '/'.$elementArray[2];
                     $active = Str::startsWith($elementPoint, $currentPoint);
                 }
                 break;
@@ -212,7 +212,7 @@ 
      */
     public static function convertLink($uri)
     {
-        $link = App::$Alias->baseUrl . '/';
+        $link = App::$Alias->baseUrl.'/';
         if (Obj::isArray($uri)) {
             $link .= Url::buildPathway($uri);
         } elseif (Str::startsWith('http', $uri)) {