phpffcms / ffcms-core

src/Ffcms/Core/Helper/Text.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
                 $breakerPos = mb_strpos($text, '<br', null, 'UTF-8');
             } else {
                 // add length('</p>')
-                $breakerPos+= 4;
+                $breakerPos += 4;
             }
             // cut text from position caret before </p> (+4 symbols to save item as valid)
             if ($breakerPos !== false) {

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

Doc Comments +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
      * @param string|null $add
      * @param array $params
      * @param bool $encode
-     * @return null|string
+     * @return string
      */
     public static function to($controller_action, $id = null, $add = null, array $params = null, $encode = true)
     {

Spacing +9 added lines, -9 removed lines

@@ -24,7 +24,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
@@ -64,9 +64,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;
@@ -80,7 +80,7 @@ 
      */
     public static function buildSingleTag($tagName, array $property = null)
     {
-        return '<' . self::nohtml($tagName) . self::applyProperty($property) . '/>';
+        return '<'.self::nohtml($tagName).self::applyProperty($property).'/>';
     }
 
     /**
@@ -98,7 +98,7 @@ 
             $value = self::nohtml($value);
         }
         
-        return '<' . $tagName . self::applyProperty($property) . '>' . $value . '</' . $tagName . '>';
+        return '<'.$tagName.self::applyProperty($property).'>'.$value.'</'.$tagName.'>';
     }
 
     /**
@@ -140,15 +140,15 @@ 
                 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 (null !== $elementArray[2]) {
-                    $elementPoint .= '/' . $elementArray[2];
+                    $elementPoint .= '/'.$elementArray[2];
                 }
 
                 $active = Str::startsWith($elementPoint, $currentPoint);
@@ -204,7 +204,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/Helper/HTML/Form/Constructor.php

Doc Comments +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
     
     /**
      * Initialize Constructor. Pass model and type inside of current field inside.
-     * @param Ffcms\Core\Arch\Model $model
+     * @param \Ffcms\Core\Arch\Model $model
      * @param string $type
      */
     public function __construct($model, $formName = false, $type = 'text')

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/MultiCheckboxField.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace Ffcms\Core\Helper\FileSystem;
 
-use Ffcms\Core\Helper\Type\Str;
 use FilesystemIterator;
 use RecursiveDirectoryIterator;
 use RecursiveIteratorIterator;

Spacing +2 added lines, -2 removed lines

@@ -44,7 +44,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 field type
@@ -64,7 +64,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/Security.php

Doc Comments +2 added lines, -2 removed lines

@@ -47,7 +47,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)
     {
@@ -87,7 +87,7 @@ 
      * @deprecated
      * @param $var
      * @param null $indent
-     * @return mixed|string
+     * @return string
      */
     public function var_export54($var, $indent = null, $guessTypes = false) {
         return Arr::exportVar($var, $indent, $guessTypes);

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,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');
         //$config->set('URI.Base', 'http://www.example.com');
         //$config->set('URI.MakeAbsolute', true);

tests/_support/_generated/AcceptanceTesterActions.php

Unused Use Statements -1 removed lines

@@ -6,7 +6,6 @@
 // @codingStandardsIgnoreFile
 
 use Codeception\Module\PhpBrowser;
-use Ffcms\Core\Helper\Acceptance;
 
 trait AcceptanceTesterActions
 {

tests/_support/_generated/UnitTesterActions.php

Unused Use Statements -1 removed lines

@@ -6,7 +6,6 @@
 // @codingStandardsIgnoreFile
 
 use Codeception\Module\Asserts;
-use Ffcms\Core\Helper\Unit;
 
 trait UnitTesterActions
 {

tests/_support/FunctionalTester.php

Indentation +3 added lines, -3 removed lines

@@ -20,7 +20,7 @@
 {
     use _generated\FunctionalTesterActions;
 
-   /**
-    * Define custom actions here
-    */
+    /**
+     * Define custom actions here
+     */
 }

src/Ffcms/Core/Exception/NativeException.php

Doc Comments +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
     /**
      * Display native exception
      * @param string $message
-     * @return string|unknown
+     * @return string
      */
     public function display($message = null)
     {

Spacing +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
     protected function sendHTML($message = null)
     {
         header('HTTP/1.1 404 Not Found');
-        return '<!DOCTYPE html><html><head><title>An error has occurred</title></head><body><div style="width:60%; margin: auto; background-color: #fcc;border: 1px solid #faa; padding: 0.5em 1em;"><h1 style="font-size: 120%">Runtime error</h1><p>' . $message . '</p></div></body></html>';
+        return '<!DOCTYPE html><html><head><title>An error has occurred</title></head><body><div style="width:60%; margin: auto; background-color: #fcc;border: 1px solid #faa; padding: 0.5em 1em;"><h1 style="font-size: 120%">Runtime error</h1><p>'.$message.'</p></div></body></html>';
     }
     
     /**