phpffcms / ffcms-core

src/Ffcms/Core/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

Braces +12 added lines, -8 removed lines

@@ -37,17 +37,20 @@ 
     public function display(array $property = null): ?string
     {
         // total items is less to pagination requirement
-        if (($this->page * $this->step) + 1 > $this->total)
-            return null;
+        if (($this->page * $this->step) + 1 > $this->total) {
+                    return null;
+        }
 
         $lastPage = ceil($this->total / $this->step); // 6/5 ~ 2 = 0..2
 
-        if ($lastPage <= 1)
-            return null;
+        if ($lastPage <= 1) {
+                    return null;
+        }
 
         // prevent hack-boy's any try
-        if ($this->page > $lastPage)
-            return null;
+        if ($this->page > $lastPage) {
+                    return null;
+        }
 
         $items = [];
         // more then 10 items in pagination
@@ -137,8 +140,9 @@ 
     protected function generateItems($start, $end)
     {
         // prevent any shit's
-        if ($end <= $start)
-            return null;
+        if ($end <= $start) {
+                    return null;
+        }
 
         $items = [];
         for ($i = $start; $i < $end; $i++) {

src/Ffcms/Core/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
@@ -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 ? '/>' : '>');
     }
 
     /**
@@ -99,7 +99,7 @@ 
         if ($valueHtml !== true)
             $value = self::nohtml($value);
         
-        return '<' . $tagName . self::applyProperty($property) . '>' . $value . '</' . $tagName . '>';
+        return '<'.$tagName.self::applyProperty($property).'>'.$value.'</'.$tagName.'>';
     }
 
     /**
@@ -140,13 +140,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, '?');
@@ -155,7 +155,7 @@ 
                     $currentToId = implode('/', array_slice($currentArray, 0, 3));
                     $active = $elementPoint === $currentToId;
                 } else {
-                    $elementPoint .= '/' . $elementArray[2];
+                    $elementPoint .= '/'.$elementArray[2];
                     $active = Str::startsWith($elementPoint, $currentPoint);
                 }
                 break;
@@ -210,7 +210,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)) {
@@ -233,7 +233,7 @@ 
         if (Str::likeEmpty($uri))
             return null;
 
-        return implode('/', array_map(function($v){
+        return implode('/', array_map(function($v) {
             return urlencode($v);
         }, explode('/', $uri)));
     }

Braces +12 added lines, -8 removed lines

@@ -59,8 +59,9 @@ 
      */
     public static function applyProperty(array $property = null)
     {
-        if ($property === null || count($property) < 1)
-            return null;
+        if ($property === null || count($property) < 1) {
+                    return null;
+        }
         
         $build = null;
         foreach ($property as $p => $v) {
@@ -96,8 +97,9 @@ 
     public static function buildContainerTag($tagName, array $property = null, $value = null, $valueHtml = false)
     {
         $tagName = self::nohtml($tagName);
-        if ($valueHtml !== true)
-            $value = self::nohtml($value);
+        if ($valueHtml !== true) {
+                    $value = self::nohtml($value);
+        }
         
         return '<' . $tagName . self::applyProperty($property) . '>' . $value . '</' . $tagName . '>';
     }
@@ -111,8 +113,9 @@ 
     public static function safeUri($string, $encode = true)
     {
         $string = self::nohtml($string);
-        if ($encode === true)
-            $string = self::encode($string);
+        if ($encode === true) {
+                    $string = self::encode($string);
+        }
 
         return $string;
     }
@@ -230,8 +233,9 @@ 
      */
     public static function encode(?string $uri = null)
     {
-        if (Str::likeEmpty($uri))
-            return null;
+        if (Str::likeEmpty($uri)) {
+                    return null;
+        }
 
         return implode('/', array_map(function($v){
             return urlencode($v);

src/Ffcms/Core/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 +14 added lines, -14 removed lines

@@ -35,13 +35,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;
@@ -56,12 +56,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']);
                         }
@@ -80,14 +80,14 @@ 
                         // 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))
                                     continue;
 
                                 // 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) {
@@ -101,9 +101,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']);
@@ -126,14 +126,14 @@ 
             // 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']);
@@ -149,7 +149,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\'));

Braces +9 added lines, -6 removed lines

@@ -23,8 +23,9 @@ 
      */
     public static function display(array $elements): ?string
     {
-        if (!array_key_exists('tbody', $elements) || !Any::isArray($elements['tbody']['items']) || count($elements['tbody']['items']) < 1)
-            return null;
+        if (!array_key_exists('tbody', $elements) || !Any::isArray($elements['tbody']['items']) || count($elements['tbody']['items']) < 1) {
+                    return null;
+        }
 
         $selectOptions = false;
         if (isset($elements['selectableBox'])) {
@@ -83,8 +84,9 @@ 
                         $tr .= $dom->tr(function () use ($dom, $row, $selectOptions) {
                             $td = null;
                             foreach ($row as $order => $item) {
-                                if (!Any::isInt($order))
-                                    continue;
+                                if (!Any::isInt($order)) {
+                                                                    continue;
+                                }
 
                                 // collect td item
                                 $td .= $dom->td(function () use ($dom, $order, $item, $selectOptions) {
@@ -128,8 +130,9 @@ 
             // return response inside "form" tag
             return $dom->form(function () use ($dom, $selectOptions, $table) {
                 foreach ($selectOptions['buttons'] as $btn) {
-                    if (!Any::isArray($btn))
-                        continue;
+                    if (!Any::isArray($btn)) {
+                                            continue;
+                    }
 
                     $table .= $dom->input(function(){
                         return null;

src/Ffcms/Core/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;
 
 /**

Braces +66 added lines, -41 removed lines

@@ -23,8 +23,9 @@ 
      */
     public static function length_min($object, $length): bool
     {
-        if (!Any::isLine($object))
-            return false;
+        if (!Any::isLine($object)) {
+                    return false;
+        }
 
         return Str::length((string)$object) >= $length;
     }
@@ -37,8 +38,9 @@ 
      */
     public static function length_max($object, $length): bool
     {
-        if (!Any::isLine($object))
-            return false;
+        if (!Any::isLine($object)) {
+                    return false;
+        }
 
         return Str::length((string)$object) <= $length;
     }
@@ -51,12 +53,14 @@ 
      */
     public static function in($object, array $handle): bool
     {
-        if (!Any::isLine($object))
-            return false;
+        if (!Any::isLine($object)) {
+                    return false;
+        }
 
         // allow empty, validate on required rule
-        if (Any::isEmpty($object))
-            return true;
+        if (Any::isEmpty($object)) {
+                    return true;
+        }
 
         $object = Any::guessValueType($object);
 
@@ -82,8 +86,9 @@ 
      */
     public static function checked($object): bool
     {
-        if (!Any::isBool($object))
-            return false;
+        if (!Any::isBool($object)) {
+                    return false;
+        }
 
         return (bool)$object;
     }
@@ -155,11 +160,14 @@ 
      */
     public static function email($object): bool
     {
-        if (Any::isEmpty($object)) // allow empty
+        if (Any::isEmpty($object)) {
+            // allow empty
             return true;
+        }
 
-        if (!Any::isLine($object))
-            return false;
+        if (!Any::isLine($object)) {
+                    return false;
+        }
 
         return Str::isEmail($object);
     }
@@ -171,11 +179,14 @@ 
      */
     public static function phone($object): bool
     {
-        if (Any::isEmpty($object)) // allow empty
+        if (Any::isEmpty($object)) {
+            // allow empty
             return true;
+        }
 
-        if (!Any::isLine($object))
-            return false;
+        if (!Any::isLine($object)) {
+                    return false;
+        }
 
         return Str::isPhone($object);
     }
@@ -187,11 +198,14 @@ 
      */
     public static function url($object): bool
     {
-        if (Any::isEmpty($object)) // allow empty
+        if (Any::isEmpty($object)) {
+            // allow empty
             return true;
+        }
 
-        if (!Any::isLine($object))
-            return false;
+        if (!Any::isLine($object)) {
+                    return false;
+        }
 
         return Str::isUrl($object);
     }
@@ -204,8 +218,9 @@ 
      */
     public static function equal($object, $value = null): bool
     {
-        if (!Any::isLine($object))
-            return false;
+        if (!Any::isLine($object)) {
+                    return false;
+        }
 
         return $object === $value;
     }
@@ -218,8 +233,9 @@ 
      */
     public static function notequal($object, $value = null): bool
     {
-        if (!Any::isLine($object))
-            return false;
+        if (!Any::isLine($object)) {
+                    return false;
+        }
 
         return $object !== $value;
     }
@@ -254,11 +270,13 @@ 
      */
     public static function reg_match($object, $value): bool
     {
-        if (!Any::isLine($object))
-            return false;
+        if (!Any::isLine($object)) {
+                    return false;
+        }
 
-        if (Str::likeEmpty($object))
-            return true;
+        if (Str::likeEmpty($object)) {
+                    return true;
+        }
 
         // what the f*ck? -> return preg_match($value, $object) > 0;
         return (bool)preg_match($value, $object);
@@ -272,8 +290,9 @@ 
      */
     public static function intList($object, $value): bool
     {
-        if (!Any::isLine($object))
-            return false;
+        if (!Any::isLine($object)) {
+                    return false;
+        }
 
         return !preg_match('/[^0-9\s,]/$', $object);
     }
@@ -287,8 +306,9 @@ 
     public static function isFile($object, $value): bool
     {
         // allow empty fields, "required" option filter that
-        if ($object === null)
-            return true;
+        if ($object === null) {
+                    return true;
+        }
 
         $all = false;
         // if string is given
@@ -301,13 +321,15 @@ 
         }
 
         // input file is not object?
-        if ($object === null || !Any::isObj($object))
-            return false;
+        if ($object === null || !Any::isObj($object)) {
+                    return false;
+        }
 
         // get guess file type, based on mime-type
         $type = $object->guessExtension();
-        if ($type === null)
-            return false;
+        if ($type === null) {
+                    return false;
+        }
 
         return ($all ? true : Arr::in($type, $value));
     }
@@ -321,21 +343,24 @@ 
     public static function sizeFile($object, $value): bool
     {
         // allow empty field, validate on filter 'required'
-        if ($object === null)
-            return true;
+        if ($object === null) {
+                    return true;
+        }
 
         if (!Any::isArray($value)) {
             $value = [0, $value];
         }
 
         // input file is not object?
-        if ($object === null || !Any::isObj($object))
-            return false;
+        if ($object === null || !Any::isObj($object)) {
+                    return false;
+        }
 
         // get file upload size in bytes
         $realSize = $object->getClientSize();
-        if ($realSize === null)
-            return false;
+        if ($realSize === null) {
+                    return false;
+        }
 
         return $realSize > $value[0] && $realSize <= $value[1];
     }

src/Ffcms/Core/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

@@ -54,7 +54,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;
@@ -69,7 +69,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));
@@ -85,7 +85,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 [];
 
@@ -121,7 +121,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, '/');
         }

Braces +18 added lines, -12 removed lines

@@ -48,8 +48,9 @@ 
                 $this->indexes[] = $index;
             }
 
-            if ($this->cached && Any::isStr($text) && isset($this->cached[$text]))
-                $text = $this->cached[$text];
+            if ($this->cached && Any::isStr($text) && isset($this->cached[$text])) {
+                            $text = $this->cached[$text];
+            }
         }
 
         if (Any::isArray($params) && count($params) > 0) {
@@ -86,8 +87,9 @@ 
     protected function load(string $index): ?array
     {
         $file = root . '/I18n/' . env_name . '/' . App::$Request->getLanguage() . '/' . $index . '.php';
-        if (!File::exist($file))
-            return [];
+        if (!File::exist($file)) {
+                    return [];
+        }
 
         return require($file);
     }
@@ -101,13 +103,15 @@ 
     {
         $path = Normalize::diskFullPath($path);
         // check if file exist
-        if (!File::exist($path))
-            return false;
+        if (!File::exist($path)) {
+                    return false;
+        }
 
         // load file translations
         $addTranslation = require($path);
-        if (!Any::isArray($addTranslation))
-            return false;
+        if (!Any::isArray($addTranslation)) {
+                    return false;
+        }
 
         // merge data
         $this->cached = Arr::merge($this->cached, $addTranslation);
@@ -142,11 +146,13 @@ 
             $lang = App::$Request->getLanguage();
         }
         // unserialize from string to array
-        if (Any::isStr($input))
-            $input = Serialize::decode($input);
+        if (Any::isStr($input)) {
+                    $input = Serialize::decode($input);
+        }
 
-        if (Any::isArray($input) && array_key_exists($lang, $input))
-            return $input[$lang];
+        if (Any::isArray($input) && array_key_exists($lang, $input)) {
+                    return $input[$lang];
+        }
 
         return $default;
     }

src/Ffcms/Core/Managers/EventManager.php

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 
 use Ffcms\Core\App;
 use Ffcms\Core\Helper\Type\Any;
-use Ffcms\Core\Helper\Type\Obj;
 
 /**
  * Class EventManager. Control and run events.

Braces +12 added lines, -8 removed lines

@@ -33,8 +33,9 @@ 
     public function on($event, \Closure $callback): void
     {
         // check if event is a single string and parse it to array single item
-        if (!Any::isArray($event))
-            $event = [$event];
+        if (!Any::isArray($event)) {
+                    $event = [$event];
+        }
         
         foreach ($event as $item) {
             $this->events[$item][] = $callback;
@@ -50,13 +51,15 @@ 
     public function listen($event, \Closure $callback)
     {
         // check if $event is a single string and set it as array with one item
-        if (!Any::isArray($event))
-            $event = [$event];
+        if (!Any::isArray($event)) {
+                    $event = [$event];
+        }
         
         // each every one event in array
         foreach ($event as $item) {
-            if (Any::isArray($this->runned) && array_key_exists($item, $this->runned))
-                return call_user_func_array($callback, $this->runned[$item]);
+            if (Any::isArray($this->runned) && array_key_exists($item, $this->runned)) {
+                            return call_user_func_array($callback, $this->runned[$item]);
+            }
         }
         
         return false;
@@ -71,8 +74,9 @@ 
         // dynamicly parse input params
         $args = func_get_args();
         
-        if (count($args) < 1)
-            return false;
+        if (count($args) < 1) {
+                    return false;
+        }
         
         // get event name
         $eventName = array_shift($args);

src/Ffcms/Core/Managers/MigrationsManager.php

Unused Use Statements -1 removed lines

@@ -8,7 +8,6 @@
 use Ffcms\Core\Helper\FileSystem\Normalize;
 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 +6 added lines, -6 removed lines

@@ -70,7 +70,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;
                 }
@@ -100,16 +100,16 @@ 
         }
 
         // 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, '-');
 
@@ -144,10 +144,10 @@ 
         }
 
         // 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, '-');
 

Braces +21 added lines, -14 removed lines

@@ -30,14 +30,16 @@ 
      */
     public function __construct(?string $dir = null, ?string $connectionName = null)
     {
-        if ($dir === null)
-            $dir = static::DEFAULT_DIR;
+        if ($dir === null) {
+                    $dir = static::DEFAULT_DIR;
+        }
 
         $this->dir = rtrim($dir, '/');
         $this->connection = $connectionName;
 
-        if ($this->dir !== static::DEFAULT_DIR)
-            $this->customDir = true;
+        if ($this->dir !== static::DEFAULT_DIR) {
+                    $this->customDir = true;
+        }
     }
 
     /**
@@ -50,16 +52,18 @@ 
     {
         // initialize db migration record
         $records = new Migration();
-        if ($this->connection !== null)
-            $records->setConnection($this->connection);
+        if ($this->connection !== null) {
+                    $records->setConnection($this->connection);
+        }
 
         // get installed migrations
         $dbmigrations = Arr::pluck('migration', $records->get()->toArray());
 
         // list migrations
         $migrations = File::listFiles($this->dir, ['.php'], true);
-        if (!Any::isArray($migrations) || count($migrations) < 1)
-            return false;
+        if (!Any::isArray($migrations) || count($migrations) < 1) {
+                    return false;
+        }
 
         $found = false;
         foreach ($migrations as $migration) {
@@ -100,8 +104,9 @@ 
         }
 
         // check if migration file is exists
-        if (!File::exist($this->dir . '/' . $file))
-            return false;
+        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)) {
@@ -114,8 +119,9 @@ 
         $class = Str::firstIn($fullName, '-');
 
         // check if class is instance of migration interface
-        if (!class_exists($class) || !is_a($class, 'Ffcms\Core\Migrations\MigrationInterface', true))
-            return false;
+        if (!class_exists($class) || !is_a($class, 'Ffcms\Core\Migrations\MigrationInterface', true)) {
+                    return false;
+        }
 
         // implement migration
         $init = new $class($fullName, $this->connection);
@@ -144,8 +150,9 @@ 
         }
 
         // check if exists
-        if (!File::exist($this->dir . '/' . $file))
-            return false;
+        if (!File::exist($this->dir . '/' . $file)) {
+                    return false;
+        }
 
         File::inc($this->dir . '/' . $file, false, false);
         $fullName = Str::cleanExtension($file);

src/Ffcms/Core/Traits/ModelValidator.php

Unused Use Statements -1 removed lines

@@ -8,7 +8,6 @@
 use Ffcms\Core\Exception\SyntaxException;
 use Ffcms\Core\Helper\ModelFilters;
 use Ffcms\Core\Helper\Type\Any;
-use Ffcms\Core\Helper\Type\Obj;
 use Ffcms\Core\Helper\Type\Str;
 
 /**

Spacing +4 added lines, -4 removed lines

@@ -137,7 +137,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);
                     }
                 }
             }
@@ -146,7 +146,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) {
@@ -155,14 +155,14 @@ 
                 $check = ModelFilters::$filterName($fieldValue);
             }
         } else {
-            throw new SyntaxException('Filter "' . $filterName . '" is not exist');
+            throw new SyntaxException('Filter "'.$filterName.'" is not exist');
         }
 
         // if one from all validation tests is fail - mark as incorrect attribute
         if ($check !== true) {
             $this->_badAttr[] = $propertyName;
             if (App::$Debug)
-                App::$Debug->addMessage('Validation failed. Property: ' . $propertyName . ', filter: ' . $filterName, 'warning');
+                App::$Debug->addMessage('Validation failed. Property: '.$propertyName.', filter: '.$filterName, 'warning');
         } else {
             $field_set_name = $propertyName;
             // prevent array-type setting

Braces +27 added lines, -18 removed lines

@@ -59,21 +59,24 @@ 
     public function runValidate(array $rules = null)
     {
         // skip validation on empty rules
-        if ($rules === null)
-            return true;
+        if ($rules === null) {
+                    return true;
+        }
 
         $success = true;
         // list each rule as single one
         foreach ($rules as $rule) {
             // 0 = field (property) name, 1 = filter name, 2 = filter value
-            if ($rule[0] === null || $rule[1] === null)
-                continue;
+            if ($rule[0] === null || $rule[1] === null) {
+                            continue;
+            }
 
             $propertyName = $rule[0];
             $validationRule = $rule[1];
             $validationValue = null;
-            if (isset($rule[2]))
-                $validationValue = $rule[2];
+            if (isset($rule[2])) {
+                            $validationValue = $rule[2];
+            }
 
             // check if target field defined as array and make recursive validation
             if (Any::isArray($propertyName)) {
@@ -161,8 +164,9 @@ 
         // if one from all validation tests is fail - mark as incorrect attribute
         if ($check !== true) {
             $this->_badAttr[] = $propertyName;
-            if (App::$Debug)
-                App::$Debug->addMessage('Validation failed. Property: ' . $propertyName . ', filter: ' . $filterName, 'warning');
+            if (App::$Debug) {
+                            App::$Debug->addMessage('Validation failed. Property: ' . $propertyName . ', filter: ' . $filterName, 'warning');
+            }
         } else {
             $field_set_name = $propertyName;
             // prevent array-type setting
@@ -173,8 +177,9 @@ 
                 if ($propertyName !== $field_set_name) { // array-based property
                     $dot_path = trim(strstr($propertyName, '.'), '.');
                     // prevent throws any exceptions for null and false objects
-                    if (!Any::isArray($this->{$field_set_name}))
-                        $this->{$field_set_name} = [];
+                    if (!Any::isArray($this->{$field_set_name})) {
+                                            $this->{$field_set_name} = [];
+                    }
 
                     // use dot-data provider to compile output array
                     $dotData = new DotData($this->{$field_set_name});
@@ -204,14 +209,16 @@ 
         $sources = $this->sources();
         $types = $this->types();
         // validate sources for current field
-        if (array_key_exists($propertyName, $sources))
-            $inputType = Str::lowerCase($sources[$propertyName]);
+        if (array_key_exists($propertyName, $sources)) {
+                    $inputType = Str::lowerCase($sources[$propertyName]);
+        }
 
 
         // check if field is array-nested element by dots and use first element as general
         $filterField = $propertyName;
-        if (array_key_exists($filterField, $types))
-            $filterType = Str::lowerCase($types[$filterField]);
+        if (array_key_exists($filterField, $types)) {
+                    $filterType = Str::lowerCase($types[$filterField]);
+        }
 
         // get clear field value
         $propertyValue = $this->getRequest($propertyName, $inputType);
@@ -313,8 +320,9 @@ 
      */
     public function getRequest($param, $method = null)
     {
-        if ($method === null)
-            $method = $this->_sendMethod;
+        if ($method === null) {
+                    $method = $this->_sendMethod;
+        }
 
         $method = Str::lowerCase($method);
         // get root request as array or string
@@ -338,8 +346,9 @@ 
         if (Str::contains('.', $param)) {
             $response = $request;
             foreach (explode('.', $param) as $path) {
-                if (!array_key_exists($path, $response))
-                    return null;
+                if (!array_key_exists($path, $response)) {
+                                    return null;
+                }
                 // find deep array nesting offset
                 $response = $response[$path];
             }

src/Ffcms/Core/Debug/Manager.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
         $this->render = $this->bar->getJavascriptRenderer();
         try {
             $this->bar->addCollector(new ConfigCollector());
-        } catch (\Exception $oe){}
+        } catch (\Exception $oe) {}
     }
 
     /**

Braces +3 added lines, -2 removed lines

@@ -80,8 +80,9 @@
      */
     public function addMessage($m, $type = 'info')
     {
-        if (!Any::isStr($m) || !Any::isStr($type))
-            return;
+        if (!Any::isStr($m) || !Any::isStr($type)) {
+                    return;
+        }
 
         $m = App::$Security->secureHtml($m);
         try {

Unused Use Statements -2 removed lines

@@ -5,10 +5,8 @@
 use Cassandra\Exception;
 use DebugBar\DataCollector\ConfigCollector;
 use DebugBar\DebugBarException;
-use DebugBar\StandardDebugBar;
 use Ffcms\Core\App;
 use Ffcms\Core\Helper\Type\Any;
-use Ffcms\Core\Helper\Type\Obj;
 
 /**
  * Class Debug. Provide methods of display information about debug and collected data in debug bar