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

src/Ffcms/Core/App.php

Spacing +5 added lines, -5 removed lines

@@ -177,7 +177,7 @@ 
         try {
             /** @var \Ffcms\Core\Arch\Controller $callClass */
             $callClass = null;
-            $callMethod = 'action' . self::$Request->getAction();
+            $callMethod = 'action'.self::$Request->getAction();
 
             // founded callback injection alias
             if (self::$Request->getCallbackAlias() !== false) {
@@ -185,16 +185,16 @@ 
                 if (class_exists($cName)) {
                     $callClass = new $cName;
                 } else {
-                    throw new NotFoundException('Callback alias of class "' . App::$Security->strip_tags($cName) . '" is not founded');
+                    throw new NotFoundException('Callback alias of class "'.App::$Security->strip_tags($cName).'" is not founded');
                 }
             } else { // typical parsing of native apps
-                $cName = '\Apps\Controller\\' . env_name . '\\' . self::$Request->getController();
+                $cName = '\Apps\Controller\\'.env_name.'\\'.self::$Request->getController();
 
                 // try to initialize class object
                 if (class_exists($cName)) {
                     $callClass = new $cName;
                 } else {
-                    throw new NotFoundException('Application can not be runned. Initialized class not founded: ' . App::$Security->strip_tags($cName));
+                    throw new NotFoundException('Application can not be runned. Initialized class not founded: '.App::$Security->strip_tags($cName));
                 }
             }
 
@@ -238,7 +238,7 @@ 
                 // build full compiled output html data
                 $html = $callClass->buildOutput();
             } else {
-                throw new NotFoundException('Method "' . App::$Security->strip_tags($callMethod) . '()" not founded in "' . get_class($callClass) . '"');
+                throw new NotFoundException('Method "'.App::$Security->strip_tags($callMethod).'()" not founded in "'.get_class($callClass).'"');
             }
         } catch (NotFoundException $e) { // catch exceptions and set output
             $html = $e->display();