phpffcms / ffcms-core

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

Spacing +2 added lines, -2 removed lines

@@ -91,7 +91,7 @@ 
             return false;
         }
 
-        $pattern = rtrim($path, '/') . '/*';
+        $pattern = rtrim($path, '/').'/*';
         $entry = glob($pattern, $mod);
 
         if ($returnRelative === true) {
@@ -122,7 +122,7 @@ 
         array_pop($separatedPath);
         $clearPath = implode(DIRECTORY_SEPARATOR, $separatedPath);
 
-        @rename($path, $clearPath . DIRECTORY_SEPARATOR . $newDirName);
+        @rename($path, $clearPath.DIRECTORY_SEPARATOR.$newDirName);
 
         return true;
     }

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

Spacing +3 added lines, -3 removed lines

@@ -165,15 +165,15 @@
     {
         $path = Normalize::diskFullPath($path);
 
-        $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/Helper/HTML/Form/Constructor.php

Spacing +7 added lines, -7 removed lines

@@ -44,7 +44,7 @@ 
     {
         // check if properties is passed well
         if (!Obj::isArray($properties) && $properties !== null) {
-            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)
@@ -94,9 +94,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);
     }
     
     /**
@@ -143,12 +143,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/Form/MultiSelectField.php

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
         if (isset($this->properties['options']) && Obj::isArray($this->properties['options'])) {
             $options = $this->properties['options'];
         } else {
-            throw new SyntaxException('Options for field ' . self::nohtml($this->name) . ' is not defined');
+            throw new SyntaxException('Options for field '.self::nohtml($this->name).' is not defined');
         }
         
         // set global field type

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::isArray($options) || count($options) < 1) {
-            throw new SyntaxException('Select field ' . self::nohtml($this->name) . ' have no options passed');
+            throw new SyntaxException('Select field '.self::nohtml($this->name).' have no options passed');
         }
         // value is not used there
         unset($this->properties['value']);

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

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
             'id' => 'src-secure-image',
             'src' => $image,
             'alt' => 'secure captcha image',
-            'onClick' => 'this.src=\'' . $image . '&rnd=\'+Math.random()'
+            'onClick' => 'this.src=\''.$image.'&rnd=\'+Math.random()'
         ]);
         // render response tag with image
         $this->properties['type'] = 'text';

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

Spacing +5 added lines, -5 removed lines

@@ -97,9 +97,9 @@ 
     {
         // looks like a single tag, <img src="" class="" />, <hr class="" />
         if (Arr::in($name, self::$singleTags)) {
-            return '<' . $name . self::applyProperties($properties) . ' />';
-        } elseif(Arr::in($name, self::$containerTags)) { // looks like a container tag, <div class=""></div>
-            return '<' . $name . self::applyProperties($properties) . '>' . $content . '</' . $name . '>';
+            return '<'.$name.self::applyProperties($properties).' />';
+        } elseif (Arr::in($name, self::$containerTags)) { // looks like a container tag, <div class=""></div>
+            return '<'.$name.self::applyProperties($properties).'>'.$content.'</'.$name.'>';
         }
 
         // empty response
@@ -126,9 +126,9 @@ 
             }
             // sounds like single standalone property, ex required, selected etc
             if ($value === null || $value === false) {
-                $build .= ' ' . htmlentities($property, ENT_QUOTES);
+                $build .= ' '.htmlentities($property, ENT_QUOTES);
             } else { // sounds like a classic key="value" property
-                $build .= ' ' . htmlentities($property, ENT_QUOTES) . '="' . htmlentities($value, ENT_QUOTES) . '"';
+                $build .= ' '.htmlentities($property, ENT_QUOTES).'="'.htmlentities($value, ENT_QUOTES).'"';
             }
         }
         return $build;

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

Spacing +9 added lines, -9 removed lines

@@ -74,7 +74,7 @@ 
      */
     public function start()
     {
-        return '<form' . self::applyProperty($this->formProperty) . '>';
+        return '<form'.self::applyProperty($this->formProperty).'>';
     }
 
     /**
@@ -90,7 +90,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;
         }
@@ -104,7 +104,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;
         }
@@ -119,13 +119,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);
         }
         //$itemBody = $this->dataTypeTag($type, $object, $itemValue, $property);
@@ -155,7 +155,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);
     }
@@ -169,10 +169,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);
                 }
@@ -183,7 +183,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($this->structLayer['jsnotify'], ['itemId' => $itemId]);
                         App::$Alias->addPlainCode('js', $render);
                     }

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

Unused Use Statements -1 removed lines

@@ -6,7 +6,6 @@
 use Ffcms\Core\Exception\NativeException;
 use Ffcms\Core\Exception\SyntaxException;
 use Ffcms\Core\Helper\HTML\System\NativeGenerator;
-use Ffcms\Core\Helper\Security;
 use Ffcms\Core\Helper\Type\Arr;
 use Ffcms\Core\Helper\FileSystem\File;
 use Ffcms\Core\Helper\Type\Obj;

src/Ffcms/Core/Helper/Type/Arr.php

Spacing +5 added lines, -5 removed lines

@@ -118,16 +118,16 @@
                         return 'true';
                     }
                 }
-                return '\'' . $var . '\'';
+                return '\''.$var.'\'';
             case 'array':
                 $indexed = array_keys($var) === range(0, count($var) - 1);
                 $row = [];
                 foreach ($var as $key => $value) {
-                    $row[] = $indent . "\t"
-                        . ($indexed ? null : self::exportVar($key, null, $guessTypes) . ' => ')
-                        . self::exportVar($value, $indent . "\t", $guessTypes);
+                    $row[] = $indent."\t"
+                        . ($indexed ? null : self::exportVar($key, null, $guessTypes).' => ')
+                        . self::exportVar($value, $indent."\t", $guessTypes);
                 }
-                return "[\n" . implode(",\n", $row) . "\n" . $indent . ']';
+                return "[\n".implode(",\n", $row)."\n".$indent.']';
             case 'boolean':
                 return $var ? 'true' : 'false';
             default: