phpffcms / ffcms-core

src/Helper/HTML/Form/MultiSelectField.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use Ffcms\Core\Helper\HTML\System\NativeGenerator;
 use Ffcms\Core\Helper\Type\Any;
 use Ffcms\Core\Helper\Type\Arr;
-use Ffcms\Core\Helper\Type\Obj;
 
 class MultiSelectField extends NativeGenerator implements iField
 {

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
         $options = $this->properties['options'];
         $optionsKey = (bool)$this->properties['optionsKey'];
         if (!Any::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/Helper/HTML/SimplePagination.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use Ffcms\Core\Helper\Type\Any;
 use Ffcms\Core\Helper\Type\Arr;
-use Ffcms\Core\Helper\Type\Obj;
 
 /**
  * Class SimplePagination. Build and display html pagination block

src/Helper/HTML/System/NativeGenerator.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use Ffcms\Core\App;
 use Ffcms\Core\Helper\Type\Any;
-use Ffcms\Core\Helper\Type\Obj;
 use Ffcms\Core\Helper\Type\Str;
 use Ffcms\Core\Helper\Url;
 

Spacing +10 added lines, -10 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
@@ -66,9 +66,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;
@@ -83,7 +83,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 ? '/>' : '>');
     }
 
     /**
@@ -101,7 +101,7 @@ 
             $value = self::nohtml($value);
         }
         
-        return '<' . $tagName . self::applyProperty($property) . '>' . $value . '</' . $tagName . '>';
+        return '<'.$tagName.self::applyProperty($property).'>'.$value.'</'.$tagName.'>';
     }
 
     /**
@@ -143,13 +143,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, '?');
@@ -158,7 +158,7 @@ 
                     $currentToId = implode('/', array_slice($currentArray, 0, 3));
                     $active = $elementPoint === $currentToId;
                 } else {
-                    $elementPoint .= '/' . $elementArray[2];
+                    $elementPoint .= '/'.$elementArray[2];
                     $active = Str::startsWith($elementPoint, $currentPoint);
                 }
                 break;
@@ -213,7 +213,7 @@ 
      */
     public static function convertLink($uri)
     {
-        $link = App::$Alias->baseUrl . '/';
+        $link = App::$Alias->baseUrl.'/';
         if (Any::isArray($uri)) {
             $link .= Url::buildPathway($uri);
         } elseif (Str::startsWith('http', $uri)) {
@@ -237,7 +237,7 @@ 
             return null;
         }
 
-        return implode('/', array_map(function ($v) {
+        return implode('/', array_map(function($v) {
             return urlencode($v);
         }, explode('/', $uri)));
     }

src/Helper/HTML/Table.php

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 use Ffcms\Core\Helper\HTML\System\NativeGenerator;
 use Ffcms\Core\Helper\Type\Any;
 use Ffcms\Core\Helper\Type\Arr;
-use Ffcms\Core\Helper\Type\Obj;
 
 /**
  * Class Table. Helper for drawing tables using php static call Table::display([params])

Spacing +15 added lines, -15 removed lines

@@ -36,13 +36,13 @@ 
         // init dom model
         $dom = new Dom();
         // draw response
-        $table = $dom->table(function () use ($dom, $elements, $selectOptions) {
+        $table = $dom->table(function() use ($dom, $elements, $selectOptions) {
             $res = null;
             // check if thead is defined
             if (isset($elements['thead']) && Any::isArray($elements['thead']) && count($elements['thead']) > 0 && Any::isArray($elements['thead']['titles'])) {
                 // add thead container
-                $res .= $dom->thead(function () use ($dom, $elements, $selectOptions) {
-                    return $dom->tr(function () use ($dom, $elements, $selectOptions) {
+                $res .= $dom->thead(function() use ($dom, $elements, $selectOptions) {
+                    return $dom->tr(function() use ($dom, $elements, $selectOptions) {
                         $tr = null;
                         foreach ($elements['thead']['titles'] as $order => $title) {
                             $th = null;
@@ -57,12 +57,12 @@ 
                             }
                             // make global checkbox for selectable columns
                             if ($selectOptions !== false && $order + 1 === $selectOptions['attachOrder']) {
-                                $th = $dom->input(function () {
+                                $th = $dom->input(function() {
                                     return null;
-                                }, ['type' => 'checkbox', 'name' => 'selectAll']) . ' ' . $th;
+                                }, ['type' => 'checkbox', 'name' => 'selectAll']).' '.$th;
                             }
                             // build tr row collecting all th's
-                            $tr .= $dom->th(function () use ($th) {
+                            $tr .= $dom->th(function() use ($th) {
                                 return $th;
                             }, $title['property']);
                         }
@@ -74,14 +74,14 @@ 
             // parse tbody array elements
             if (isset($elements['tbody']) && Any::isArray($elements['tbody']) && isset($elements['tbody']['items']) && Any::isArray($elements['tbody']['items'])) {
                 // add tbody container
-                $res .= $dom->tbody(function () use ($dom, $elements, $selectOptions) {
+                $res .= $dom->tbody(function() use ($dom, $elements, $selectOptions) {
                     $tr = null;
                     // each all items by row (tr)
                     foreach ($elements['tbody']['items'] as $row) {
                         // sort td items inside row by key increment
                         ksort($row);
                         // add data in tr container
-                        $tr .= $dom->tr(function () use ($dom, $row, $selectOptions) {
+                        $tr .= $dom->tr(function() use ($dom, $row, $selectOptions) {
                             $td = null;
                             foreach ($row as $order => $item) {
                                 if (!Any::isInt($order)) {
@@ -89,7 +89,7 @@ 
                                 }
 
                                 // collect td item
-                                $td .= $dom->td(function () use ($dom, $order, $item, $selectOptions) {
+                                $td .= $dom->td(function() use ($dom, $order, $item, $selectOptions) {
                                     $text = null;
                                     // make text secure based on passed options
                                     if ($item['html'] === true) {
@@ -103,9 +103,9 @@ 
                                     }
                                     // check if selectable box is enabled and equal current order id
                                     if ($selectOptions !== false && $order === $selectOptions['attachOrder']) {
-                                        $text = $dom->input(function () {
+                                        $text = $dom->input(function() {
                                             return null;
-                                        }, Arr::merge($selectOptions['selector'], ['value' => htmlentities($text, null, 'UTF-8')])) . ' ' . $text;
+                                        }, Arr::merge($selectOptions['selector'], ['value' => htmlentities($text, null, 'UTF-8')])).' '.$text;
                                     }
                                     return $text;
                                 }, $item['property']);
@@ -128,15 +128,15 @@ 
             // build js code for "selectAll" checkbox
             self::buildSelectorHtml($selectOptions);
             // return response inside "form" tag
-            return $dom->form(function () use ($dom, $selectOptions, $table) {
+            return $dom->form(function() use ($dom, $selectOptions, $table) {
                 foreach ($selectOptions['buttons'] as $btn) {
                     if (!Any::isArray($btn)) {
                         continue;
                     }
 
-                    $table .= $dom->input(function () {
+                    $table .= $dom->input(function() {
                         return null;
-                    }, $btn) . ' ';
+                    }, $btn).' ';
                 }
                 return $table;
             }, $selectOptions['form']);
@@ -152,7 +152,7 @@ 
     private static function buildSelectorHtml(array $opt)
     {
         $js = '$(function () {
-            var targetSwitchbox = $(\'input[name="' . $opt['selector']['name'] . '"]\');
+            var targetSwitchbox = $(\'input[name="' . $opt['selector']['name'].'"]\');
             $(\'input[name="selectAll"]\').change(function() {
                 $(targetSwitchbox).each(function () {
                     $(this).prop(\'checked\', !$(this).is(\':checked\'));

src/Helper/ModelFilters.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use Ffcms\Core\Helper\Type\Any;
 use Ffcms\Core\Helper\Type\Arr;
-use Ffcms\Core\Helper\Type\Obj;
 use Ffcms\Core\Helper\Type\Str;
 
 /**

src/Helper/Security.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use Ffcms\Core\App;
 use Ffcms\Core\Helper\Type\Any;
 use Ffcms\Core\Helper\Type\Arr;
-use Ffcms\Core\Helper\Type\Obj;
 use Ffcms\Core\Helper\Type\Str;
 
 /**

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

Doc Comments +2 added lines, -2 removed lines

@@ -46,7 +46,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|null
      */
     public function var_export54($var, $indent = null, $guessTypes = false)
     {

src/Helper/Type/Str.php

Doc Comments +2 added lines, -5 removed lines

@@ -213,8 +213,8 @@ 
 
     /**
      * Case-ignore replace $needle to $replacement in string $haystack. Alias for function str_ireplace()
-     * @param string|array $needle
-     * @param string|array $replacement
+     * @param string[] $needle
+     * @param string $replacement
      * @param string $haystack
      * @return string
      */
@@ -318,9 +318,6 @@ 
 
     /**
      * Concat string with add string by separator
-     * @param string $separator
-     * @param string $string1
-     * @param string $string2
      * @return string
      */
     public static function concat()

Spacing +1 added lines, -1 removed lines

@@ -297,7 +297,7 @@
 
         $output = null;
         foreach ($args as $string) {
-            $output .= $string . $separator;
+            $output .= $string.$separator;
         }
 
         $output = rtrim($output, $separator);

src/I18n/Translate.php

Unused Use Statements -1 removed lines

@@ -9,7 +9,6 @@
 use Ffcms\Core\Helper\Serialize;
 use Ffcms\Core\Helper\Type\Any;
 use Ffcms\Core\Helper\Type\Arr;
-use Ffcms\Core\Helper\Type\Obj;
 use Ffcms\Core\Helper\Type\Str;
 
 /**

Spacing +4 added lines, -4 removed lines

@@ -55,7 +55,7 @@ 
 
         if (Any::isArray($params) && count($params) > 0) {
             foreach ($params as $var => $value) {
-                $text = Str::replace('%' . $var . '%', $value, $text);
+                $text = Str::replace('%'.$var.'%', $value, $text);
             }
         }
         return $text;
@@ -70,7 +70,7 @@ 
     public function translate(string $text, array $params = null)
     {
         $index = null;
-        $namespace = 'Apps\Controller\\' . env_name . '\\';
+        $namespace = 'Apps\Controller\\'.env_name.'\\';
         foreach (@debug_backtrace() as $caller) {
             if (isset($caller['class']) && Str::startsWith($namespace, $caller['class'])) {
                 $index = Str::sub((string)$caller['class'], Str::length($namespace));
@@ -86,7 +86,7 @@ 
      */
     protected function load(string $index): ?array
     {
-        $file = root . '/I18n/' . env_name . '/' . App::$Request->getLanguage() . '/' . $index . '.php';
+        $file = root.'/I18n/'.env_name.'/'.App::$Request->getLanguage().'/'.$index.'.php';
         if (!File::exist($file)) {
             return [];
         }
@@ -125,7 +125,7 @@ 
     public function getAvailableLangs(): array
     {
         $langs = ['en'];
-        $scan = Directory::scan(root . '/I18n/' . env_name . '/', GLOB_ONLYDIR, true);
+        $scan = Directory::scan(root.'/I18n/'.env_name.'/', GLOB_ONLYDIR, true);
         foreach ($scan as $row) {
             $langs[] = trim($row, '/');
         }

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

Unused Use Statements -1 removed lines

@@ -8,7 +8,6 @@
 use Ffcms\Core\Helper\FileSystem\Directory;
 use Ffcms\Core\Helper\FileSystem\File;
 use Ffcms\Core\Helper\Type\Any;
-use Ffcms\Core\Helper\Type\Obj;
 use Ffcms\Core\Helper\Type\Str;
 
 /**

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,7 +111,7 @@ 
 
         // 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 (!Any::isArray($dirs)) {
@@ -119,7 +119,7 @@ 
             }
 
             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;
                 }