phpffcms / ffcms-core

src/Ffcms/Core/Traits/ModelValidator.php

Spacing +3 added lines, -3 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) { // if one from all validation tests is fail - mark as incorrect attribute
             $this->_badAttr[] = $propertyName;

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;
 
 /**

Braces +24 added lines, -16 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)) {
@@ -170,8 +173,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});
@@ -201,14 +205,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);
@@ -310,8 +316,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
@@ -335,8 +342,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/DebugMeasure.php

Braces +6 added lines, -4 removed lines

@@ -16,8 +16,9 @@ 
      */
     public function startMeasure(string $name): void
     {
-        if (App::$Debug)
-            App::$Debug->startMeasure($name);
+        if (App::$Debug) {
+                    App::$Debug->startMeasure($name);
+        }
     }
 
     /**
@@ -26,7 +27,8 @@ 
      */
     public function stopMeasure(string $name): void
     {
-        if (App::$Debug)
-            App::$Debug->stopMeasure($name);
+        if (App::$Debug) {
+                    App::$Debug->stopMeasure($name);
+        }
     }
 }
\ No newline at end of file

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

Spacing +5 added lines, -5 removed lines

@@ -93,9 +93,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
@@ -122,9 +122,9 @@ 
             }
             // sounds like single standalone property, ex required, selected etc
             if ($value === null || $value === false) {
-                $build .= ' ' . htmlentities($property, ENT_QUOTES, "UTF-8");
+                $build .= ' '.htmlentities($property, ENT_QUOTES, "UTF-8");
             } else { // sounds like a classic key="value" property
-                $build .= ' ' . htmlentities($property, ENT_QUOTES, "UTF-8") . '="' . htmlentities($value, ENT_QUOTES, "UTF-8") . '"';
+                $build .= ' '.htmlentities($property, ENT_QUOTES, "UTF-8").'="'.htmlentities($value, ENT_QUOTES, "UTF-8").'"';
             }
         }
         return $build;

src/Ffcms/Core/Network/Request.php

Spacing +10 added lines, -10 removed lines

@@ -60,11 +60,11 @@ 
 
         $basePath = trim(App::$Properties->get('basePath'), '/');
         if ($basePath !== null && Str::length($basePath) > 0) {
-            $basePath = '/' . $basePath;
+            $basePath = '/'.$basePath;
         }
 
         if (!defined('env_no_uri') || env_no_uri === false) {
-            $basePath .= '/' . strtolower(env_name);
+            $basePath .= '/'.strtolower(env_name);
         }
 
         // we never try to use path's without friendly url's
@@ -90,7 +90,7 @@ 
             } else {
                 // try to find language in pathway
                 foreach (App::$Properties->get('languages') as $lang) {
-                    if (Str::startsWith('/' . $lang, $this->getPathInfo())) {
+                    if (Str::startsWith('/'.$lang, $this->getPathInfo())) {
                         $this->language = $lang;
                         $this->languageInPath = true;
                     }
@@ -117,11 +117,11 @@ 
                     // parse query string
                     $queryString = null;
                     if (count($this->query->all()) > 0) {
-                        $queryString = '?' . http_build_query($this->query->all());
+                        $queryString = '?'.http_build_query($this->query->all());
                     }
 
                     // build response with redirect to language-based path
-                    $response = new Redirect($this->getSchemeAndHttpHost() . $this->basePath . '/' . $userLang . $this->getPathInfo() . $queryString);
+                    $response = new Redirect($this->getSchemeAndHttpHost().$this->basePath.'/'.$userLang.$this->getPathInfo().$queryString);
                     $response->send();
                     exit();
                 }
@@ -180,7 +180,7 @@ 
         if (array_key_exists($pathway, $routing['Redirect'])) {
             $target = $this->getSchemeAndHttpHost(); // . $this->getBasePath() . '/' . rtrim($routing['Redirect'][$pathway], '/');
             if ($this->getBasePath() !== null && !Str::likeEmpty($this->getBasePath())) {
-                $target .= '/' . $this->getBasePath();
+                $target .= '/'.$this->getBasePath();
             }
             $target .= rtrim($routing['Redirect'][$pathway], '/');
             $redirect = new Redirect($target);
@@ -206,9 +206,9 @@ 
             // find "new path" as binding uri slug
             $binding = array_search($pathway, $map, true);
             // build url to redirection
-            $url = $this->getSchemeAndHttpHost() . $this->getBasePath() . '/';
+            $url = $this->getSchemeAndHttpHost().$this->getBasePath().'/';
             if (App::$Properties->get('multiLanguage')) {
-                $url .= $this->language . '/';
+                $url .= $this->language.'/';
             }
             $url .= ltrim($binding, '/');
 
@@ -302,7 +302,7 @@ 
     {
         $route = $this->languageInPath ? Str::sub(parent::getPathInfo(), Str::length($this->language) + 1) : parent::getPathInfo();
         if (!Str::startsWith('/', $route))
-            $route = '/' . $route;
+            $route = '/'.$route;
         return $route;
     }
 
@@ -466,7 +466,7 @@ 
      */
     public function getFullUrl(): string
     {
-        return $this->getSchemeAndHttpHost() . $this->getRequestUri();
+        return $this->getSchemeAndHttpHost().$this->getRequestUri();
     }
 
     /**

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;
 use Symfony\Component\HttpFoundation\RedirectResponse as Redirect;
 use Symfony\Component\HttpFoundation\Request as FoundationRequest;

Braces +27 added lines, -18 removed lines

@@ -141,18 +141,21 @@ 
         $routing = App::$Properties->getAll('Routing');
 
         // try to work with static aliases
-        if (Any::isArray($routing) && isset($routing['Alias'], $routing['Alias'][env_name]))
-            $pathway = $this->findStaticAliases($routing['Alias'][env_name], $pathway);
+        if (Any::isArray($routing) && isset($routing['Alias'], $routing['Alias'][env_name])) {
+                    $pathway = $this->findStaticAliases($routing['Alias'][env_name], $pathway);
+        }
 
         $this->setPathdata(explode('/', trim($pathway, '/')));
 
         // set default controller and action for undefined data
-        if (!$this->action)
-            $this->action = 'Index';
+        if (!$this->action) {
+                    $this->action = 'Index';
+        }
 
         // empty or contains backslashes? set to main
-        if (!$this->controller || Str::contains('\\', $this->controller))
-            $this->controller = 'Main';
+        if (!$this->controller || Str::contains('\\', $this->controller)) {
+                    $this->controller = 'Main';
+        }
 
         // find callback injection in routing configs (calculated in App::run())
         if (Any::isArray($routing) && isset($routing['Callback'], $routing['Callback'][env_name])) {
@@ -170,8 +173,9 @@ 
         /** @var array $routing */
         $routing = App::$Properties->getAll('Routing');
 
-        if (!Any::isArray($routing) || !isset($routing['Redirect']) || !Any::isArray($routing['Redirect']))
-            return;
+        if (!Any::isArray($routing) || !isset($routing['Redirect']) || !Any::isArray($routing['Redirect'])) {
+                    return;
+        }
 
         // check if source uri is key in redirect target map
         if (array_key_exists($pathway, $routing['Redirect'])) {
@@ -230,8 +234,9 @@ 
      */
     private function findDynamicCallbacks(array $map = null, ?string $controller = null)
     {
-        if ($map === null)
-            return;
+        if ($map === null) {
+                    return;
+        }
 
         // try to find global callback for this controller slug
         if (array_key_exists($controller, $map)) {
@@ -248,8 +253,9 @@ 
     private function loadTrustedProxies()
     {
         $proxies = App::$Properties->get('trustedProxy');
-        if ($proxies === null || Str::likeEmpty($proxies))
-            return;
+        if ($proxies === null || Str::likeEmpty($proxies)) {
+                    return;
+        }
 
         $pList = explode(',', $proxies);
         $resultList = [];
@@ -265,8 +271,9 @@ 
      */
     private function setPathdata(?array $pathArray = null)
     {
-        if (!Any::isArray($pathArray) || count($pathArray) < 1)
-            return;
+        if (!Any::isArray($pathArray) || count($pathArray) < 1) {
+                    return;
+        }
 
         // check if array length is more then 4 basic elements and slice it recursive
         if (count($pathArray) > 4) {
@@ -295,8 +302,9 @@ 
     public function getPathInfo()
     {
         $route = $this->languageInPath ? Str::sub(parent::getPathInfo(), Str::length($this->language) + 1) : parent::getPathInfo();
-        if (!Str::startsWith('/', $route))
-            $route = '/' . $route;
+        if (!Str::startsWith('/', $route)) {
+                    $route = '/' . $route;
+        }
         return $route;
     }
 
@@ -439,8 +447,9 @@ 
     public function getPathWithoutControllerAction(): ?string
     {
         $path = trim($this->getPathInfo(), '/');
-        if ($this->aliasPathTarget !== null)
-            $path = trim($this->aliasPathTarget, '/');
+        if ($this->aliasPathTarget !== null) {
+                    $path = trim($this->aliasPathTarget, '/');
+        }
 
         $pathArray = explode('/', $path);
         if ($pathArray[0] === Str::lowerCase($this->getController())) {

src/Ffcms/Core/App.php

Unused Use Statements -4 removed lines

@@ -6,15 +6,11 @@
 use Ffcms\Core\Cache\MemoryObject;
 use Ffcms\Core\Debug\DebugMeasure;
 use Ffcms\Core\Debug\Manager as Debug;
-use Ffcms\Core\Exception\ForbiddenException;
-use Ffcms\Core\Exception\JsonException;
 use Ffcms\Core\Exception\NativeException;
 use Ffcms\Core\Exception\NotFoundException;
-use Ffcms\Core\Exception\SyntaxException;
 use Ffcms\Core\Exception\TemplateException;
 use Ffcms\Core\Helper\Security;
 use Ffcms\Core\Helper\Type\Any;
-use Ffcms\Core\Helper\Type\Obj;
 use Ffcms\Core\Helper\Type\Str;
 use Ffcms\Core\I18n\Translate;
 use Ffcms\Core\Managers\BootManager;

Spacing +9 added lines, -9 removed lines

@@ -185,22 +185,22 @@ 
         $html = null;
         // lets try to get html full content to page render
         try {
-            $this->startMeasure(__METHOD__ . '::callback');
+            $this->startMeasure(__METHOD__.'::callback');
             /** @var \Ffcms\Core\Arch\Controller $callClass */
             $callClass = null;
-            $callMethod = 'action' . self::$Request->getAction();
+            $callMethod = 'action'.self::$Request->getAction();
 
             // define callback class namespace/name full path
-            $cName = (self::$Request->getCallbackAlias() ?? '\Apps\Controller\\' . env_name . '\\' . self::$Request->getController());
+            $cName = (self::$Request->getCallbackAlias() ?? '\Apps\Controller\\'.env_name.'\\'.self::$Request->getController());
             if (!class_exists($cName))
-                throw new NotFoundException('Callback class not found: ' . App::$Security->strip_tags($cName));
+                throw new NotFoundException('Callback class not found: '.App::$Security->strip_tags($cName));
 
             $callClass = new $cName;
             // check if callback method (action) is exist in class object
             if (!method_exists($callClass, $callMethod))
-                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).'"');
 
-            $this->stopMeasure(__METHOD__ . '::callback');
+            $this->stopMeasure(__METHOD__.'::callback');
             $params = [];
             if (!Str::likeEmpty(self::$Request->getID())) {
                 $params[] = self::$Request->getID();
@@ -224,10 +224,10 @@ 
                     'current' => count($params)
                 ]));
 
-            $this->startMeasure($cName . '::' . $callMethod);
+            $this->startMeasure($cName.'::'.$callMethod);
             // make callback call to action in controller and get response
             $actionResponse = call_user_func_array([$callClass, $callMethod], $params);
-            $this->stopMeasure($cName . '::' . $callMethod);
+            $this->stopMeasure($cName.'::'.$callMethod);
 
             // set response to controller attribute
             if (!Str::likeEmpty($actionResponse))
@@ -241,7 +241,7 @@ 
                 $html = $e->display();
             } else { // or hook exception to system based :)))
                 if (App::$Debug) {
-                    $msg = $e->getMessage() . $e->getTraceAsString();
+                    $msg = $e->getMessage().$e->getTraceAsString();
                     $html = (new NativeException($msg))->display();
                 } else {
                     $html = (new NativeException($e->getMessage()))->display();

Braces +21 added lines, -14 removed lines

@@ -156,8 +156,9 @@ 
 
         /** @var array $objects */
         $objects = App::$Properties->getAll('object');
-        if (!Any::isArray($objects))
-            throw new NativeException('Object configurations is not loaded: /Private/Config/Object.php');
+        if (!Any::isArray($objects)) {
+                    throw new NativeException('Object configurations is not loaded: /Private/Config/Object.php');
+        }
 
         // each all objects as service_name => service_instance()
         foreach ($objects as $name => $instance) {
@@ -192,20 +193,23 @@ 
 
             // define callback class namespace/name full path
             $cName = (self::$Request->getCallbackAlias() ?? '\Apps\Controller\\' . env_name . '\\' . self::$Request->getController());
-            if (!class_exists($cName))
-                throw new NotFoundException('Callback class not found: ' . App::$Security->strip_tags($cName));
+            if (!class_exists($cName)) {
+                            throw new NotFoundException('Callback class not found: ' . App::$Security->strip_tags($cName));
+            }
 
             $callClass = new $cName;
             // check if callback method (action) is exist in class object
-            if (!method_exists($callClass, $callMethod))
-                throw new NotFoundException('Method "' . App::$Security->strip_tags($callMethod) . '()" not founded in "' . get_class($callClass) . '"');
+            if (!method_exists($callClass, $callMethod)) {
+                            throw new NotFoundException('Method "' . App::$Security->strip_tags($callMethod) . '()" not founded in "' . get_class($callClass) . '"');
+            }
 
             $this->stopMeasure(__METHOD__ . '::callback');
             $params = [];
             if (!Str::likeEmpty(self::$Request->getID())) {
                 $params[] = self::$Request->getID();
-                if (!Str::likeEmpty(self::$Request->getAdd()))
-                    $params[] = self::$Request->getAdd();
+                if (!Str::likeEmpty(self::$Request->getAdd())) {
+                                    $params[] = self::$Request->getAdd();
+                }
             }
 
             // get instance of callback object (class::method) as reflection
@@ -213,16 +217,18 @@ 
             $argCount = 0;
             // calculate method defined arguments count
             foreach ($instance->getParameters() as $arg) {
-                if (!$arg->isOptional())
-                    $argCount++;
+                if (!$arg->isOptional()) {
+                                    $argCount++;
+                }
             }
             // compare method arg count with passed
-            if (count($params) < $argCount)
-                throw new NotFoundException(__('Arguments for method %method% is not enough. Expected: %required%, got: %current%.', [
+            if (count($params) < $argCount) {
+                            throw new NotFoundException(__('Arguments for method %method% is not enough. Expected: %required%, got: %current%.', [
                     'method' => $callMethod,
                     'required' => $argCount,
                     'current' => count($params)
                 ]));
+            }
 
             $this->startMeasure($cName . '::' . $callMethod);
             // make callback call to action in controller and get response
@@ -230,8 +236,9 @@ 
             $this->stopMeasure($cName . '::' . $callMethod);
 
             // set response to controller attribute
-            if (!Str::likeEmpty($actionResponse))
-                $callClass->setOutput($actionResponse);
+            if (!Str::likeEmpty($actionResponse)) {
+                            $callClass->setOutput($actionResponse);
+            }
 
             // build full compiled output html data with default layout and widgets
             $html = $callClass->buildOutput();

src/Ffcms/Core/Arch/View.php

Doc Comments +1 added lines, -1 removed lines

@@ -249,7 +249,7 @@
     /**
      * Show custom code library link
      * @param string $type - js or css allowed
-     * @return array|null|string
+     * @return null|string
      */
     public function showCodeLink(string $type)
     {

Unused Use Statements -1 removed lines

@@ -9,7 +9,6 @@
 use Ffcms\Core\Helper\FileSystem\File;
 use Ffcms\Core\Helper\FileSystem\Normalize;
 use Ffcms\Core\Helper\Type\Any;
-use Ffcms\Core\Helper\Type\Obj;
 use Ffcms\Core\Helper\Type\Str;
 use Ffcms\Core\Template\Variables;
 use Ffcms\Core\Traits\DynamicGlobal;

Spacing +16 added lines, -16 removed lines

@@ -55,16 +55,16 @@ 
         $this->lang = App::$Request->getLanguage();
         // get theme config and build full path
         $themeConfig = App::$Properties->get('theme');
-        $this->themePath = root . DIRECTORY_SEPARATOR . 'Apps' . DIRECTORY_SEPARATOR . 'View' . DIRECTORY_SEPARATOR . env_name;
+        $this->themePath = root.DIRECTORY_SEPARATOR.'Apps'.DIRECTORY_SEPARATOR.'View'.DIRECTORY_SEPARATOR.env_name;
         if (isset($themeConfig[env_name]) && Str::length($themeConfig[env_name]) > 0) {
-            $this->themePath .=  DIRECTORY_SEPARATOR . $themeConfig[env_name];
+            $this->themePath .= DIRECTORY_SEPARATOR.$themeConfig[env_name];
         } else {
-            $this->themePath .= DIRECTORY_SEPARATOR . 'default';
+            $this->themePath .= DIRECTORY_SEPARATOR.'default';
         }
 
         // check if theme is available
         if (!Directory::exist($this->themePath))
-            throw new NativeException('Apps theme is not founded: ' . Str::replace(root, null, $this->themePath));
+            throw new NativeException('Apps theme is not founded: '.Str::replace(root, null, $this->themePath));
 
         // get input args and build class properties
         $args = func_get_args();
@@ -102,7 +102,7 @@ 
         
         // path still not defined?
         if (!$path)
-            throw new SyntaxException('View not found: ' . App::$Security->strip_tags($path));
+            throw new SyntaxException('View not found: '.App::$Security->strip_tags($path));
 
         // cleanup from slashes on start/end
         $path = trim($path, '/\\');
@@ -130,7 +130,7 @@ 
             // lets try to get full path for current theme
             $tmpPath = $path;
             if (!Str::startsWith($this->themePath, $path))
-                $tmpPath = Normalize::diskPath($this->themePath . '/' . $path . '.php');
+                $tmpPath = Normalize::diskPath($this->themePath.'/'.$path.'.php');
         } else { // sounds like a object-depended view call from controller or etc
             // get stack trace of callbacks
             $calledLog = debug_backtrace();
@@ -144,13 +144,13 @@ 
 
             // depended controller is not founded? Let finish
             if (!$calledController)
-                throw new SyntaxException('View render is failed: callback controller not founded! Call with relative path: ' . $path);
+                throw new SyntaxException('View render is failed: callback controller not founded! Call with relative path: '.$path);
 
             // get controller name
-            $controllerName = Str::sub($calledController, Str::length('Apps\Controller\\' . env_name . '\\'));
+            $controllerName = Str::sub($calledController, Str::length('Apps\Controller\\'.env_name.'\\'));
             $controllerName = Str::lowerCase($controllerName);
             // get full path
-            $tmpPath = $this->themePath . DIRECTORY_SEPARATOR . $controllerName . DIRECTORY_SEPARATOR . $path . '.php';
+            $tmpPath = $this->themePath.DIRECTORY_SEPARATOR.$controllerName.DIRECTORY_SEPARATOR.$path.'.php';
         }
 
         // check if builded view full path is exist
@@ -159,18 +159,18 @@ 
 
         // hmm, not founded. Lets try to find in caller directory (for widgets, apps packages and other)
         if ($source !== null) {
-            $tmpPath = Normalize::diskPath($source . DIRECTORY_SEPARATOR . $path . '.php');
+            $tmpPath = Normalize::diskPath($source.DIRECTORY_SEPARATOR.$path.'.php');
             if (File::exist($tmpPath)) {
                 // add notify for native views
                 if (App::$Debug)
-                    App::$Debug->addMessage('Render native viewer: ' . Str::replace(root, null, $tmpPath), 'info');
+                    App::$Debug->addMessage('Render native viewer: '.Str::replace(root, null, $tmpPath), 'info');
 
                 return $tmpPath;
             }
         }
 
         if (App::$Debug)
-            App::$Debug->addMessage('Viewer not founded on rendering: ' . $path, 'warning');
+            App::$Debug->addMessage('Viewer not founded on rendering: '.$path, 'warning');
 
         return null;
     }
@@ -262,7 +262,7 @@ 
         foreach ($items as $item) {
             $item = trim($item, '/');
             if (!Str::startsWith(App::$Alias->scriptUrl, $item) && !Str::startsWith('http', $item)) { // is local without proto and domain
-                $item = App::$Alias->scriptUrl . '/' . $item;
+                $item = App::$Alias->scriptUrl.'/'.$item;
             }
             $output[] = $item;
         }
@@ -271,9 +271,9 @@ 
         $output = null;
         foreach ($clear as $row) {
             if ($type === 'css') {
-                $output .= '<link rel="stylesheet" type="text/css" href="' . $row . '">' . "\n";
+                $output .= '<link rel="stylesheet" type="text/css" href="'.$row.'">'."\n";
             } elseif ($type === 'js') {
-                $output .= '<script src="' . $row . '"></script>' . "\n";
+                $output .= '<script src="'.$row.'"></script>'."\n";
             }
         }
 
@@ -295,7 +295,7 @@ 
 
         $code = null;
         foreach (App::$Alias->getPlainCode($type) as $row) {
-            $code .= $row . "\n";
+            $code .= $row."\n";
         }
 
         return $code;

Braces +45 added lines, -30 removed lines

@@ -63,8 +63,9 @@ 
         }
 
         // check if theme is available
-        if (!Directory::exist($this->themePath))
-            throw new NativeException('Apps theme is not founded: ' . Str::replace(root, null, $this->themePath));
+        if (!Directory::exist($this->themePath)) {
+                    throw new NativeException('Apps theme is not founded: ' . Str::replace(root, null, $this->themePath));
+        }
 
         // get input args and build class properties
         $args = func_get_args();
@@ -89,20 +90,24 @@ 
         $source = $this->sourcePath;
 
         // if path is not defined - try to find it in arguments
-        if (!$path)
-            $path = array_shift($arguments);
+        if (!$path) {
+                    $path = array_shift($arguments);
+        }
 
         // if arguments is not define - try to find in arguments
-        if (!$params)
-            $params = array_shift($arguments);
+        if (!$params) {
+                    $params = array_shift($arguments);
+        }
 
         // if directory of caller is not defiend - lets find in argument
-        if (!$source)
-            $source = array_shift($arguments);
+        if (!$source) {
+                    $source = array_shift($arguments);
+        }
         
         // path still not defined?
-        if (!$path)
-            throw new SyntaxException('View not found: ' . App::$Security->strip_tags($path));
+        if (!$path) {
+                    throw new SyntaxException('View not found: ' . App::$Security->strip_tags($path));
+        }
 
         // cleanup from slashes on start/end
         $path = trim($path, '/\\');
@@ -129,8 +134,9 @@ 
         if (Str::contains('/', $path)) {
             // lets try to get full path for current theme
             $tmpPath = $path;
-            if (!Str::startsWith($this->themePath, $path))
-                $tmpPath = Normalize::diskPath($this->themePath . '/' . $path . '.php');
+            if (!Str::startsWith($this->themePath, $path)) {
+                            $tmpPath = Normalize::diskPath($this->themePath . '/' . $path . '.php');
+            }
         } else { // sounds like a object-depended view call from controller or etc
             // get stack trace of callbacks
             $calledLog = debug_backtrace();
@@ -138,13 +144,15 @@ 
 
             // lets try to find controller in backtrace
             foreach ($calledLog as $caller) {
-                if (isset($caller['class']) && Str::startsWith('Apps\Controller\\', $caller['class']))
-                    $calledController = (string)$caller['class'];
+                if (isset($caller['class']) && Str::startsWith('Apps\Controller\\', $caller['class'])) {
+                                    $calledController = (string)$caller['class'];
+                }
             }
 
             // depended controller is not founded? Let finish
-            if (!$calledController)
-                throw new SyntaxException('View render is failed: callback controller not founded! Call with relative path: ' . $path);
+            if (!$calledController) {
+                            throw new SyntaxException('View render is failed: callback controller not founded! Call with relative path: ' . $path);
+            }
 
             // get controller name
             $controllerName = Str::sub($calledController, Str::length('Apps\Controller\\' . env_name . '\\'));
@@ -154,23 +162,26 @@ 
         }
 
         // check if builded view full path is exist
-        if (File::exist($tmpPath))
-            return $tmpPath;
+        if (File::exist($tmpPath)) {
+                    return $tmpPath;
+        }
 
         // hmm, not founded. Lets try to find in caller directory (for widgets, apps packages and other)
         if ($source !== null) {
             $tmpPath = Normalize::diskPath($source . DIRECTORY_SEPARATOR . $path . '.php');
             if (File::exist($tmpPath)) {
                 // add notify for native views
-                if (App::$Debug)
-                    App::$Debug->addMessage('Render native viewer: ' . Str::replace(root, null, $tmpPath), 'info');
+                if (App::$Debug) {
+                                    App::$Debug->addMessage('Render native viewer: ' . Str::replace(root, null, $tmpPath), 'info');
+                }
 
                 return $tmpPath;
             }
         }
 
-        if (App::$Debug)
-            App::$Debug->addMessage('Viewer not founded on rendering: ' . $path, 'warning');
+        if (App::$Debug) {
+                    App::$Debug->addMessage('Viewer not founded on rendering: ' . $path, 'warning');
+        }
 
         return null;
     }
@@ -193,8 +204,9 @@ 
      */
     protected function renderSandbox($path, $params = null)
     {
-        if (!$path || !File::exist($path))
-            return null;
+        if (!$path || !File::exist($path)) {
+                    return null;
+        }
 
         // render defaults params as variables
         if (Any::isArray($params) && count($params) > 0) {
@@ -218,8 +230,9 @@ 
             ob_end_clean();
             // prepare output message
             $msg = $e->getMessage();
-            if (!Str::likeEmpty($msg))
-                $msg .= '. ';
+            if (!Str::likeEmpty($msg)) {
+                            $msg .= '. ';
+            }
 
             $msg .= __('Native exception catched in view: %path% in line %line%', ['path' => $path, 'line' => $e->getLine()]);
             exit($e->display($msg));
@@ -255,8 +268,9 @@ 
     {
         $items = App::$Alias->getCustomLibraryArray($type);
         // check if custom library available
-        if (!$items || !Any::isArray($items) || count($items) < 1)
-            return null;
+        if (!$items || !Any::isArray($items) || count($items) < 1) {
+                    return null;
+        }
 
         $output = [];
         foreach ($items as $item) {
@@ -290,8 +304,9 @@ 
      */
     public function showPlainCode(?string $type = null): ?string
     {
-        if (!App::$Alias->getPlainCode($type) || !Any::isArray(App::$Alias->getPlainCode($type)))
-            return null;
+        if (!App::$Alias->getPlainCode($type) || !Any::isArray(App::$Alias->getPlainCode($type))) {
+                    return null;
+        }
 
         $code = null;
         foreach (App::$Alias->getPlainCode($type) as $row) {

src/Ffcms/Core/Debug/Manager.php

Unused Use Statements -1 removed lines

@@ -8,7 +8,6 @@
 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

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 {

src/Ffcms/Core/Helper/HTML/Bootstrap/Nav.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;
 use Ffcms\Core\Helper\Type\Str;
 
 class Nav extends NativeGenerator

Spacing +6 added lines, -6 removed lines

@@ -59,7 +59,7 @@ 
             } elseif ($item['type'] === 'tab') {
                 $activeObject = false;
                 $item['type'] = 'link'; // fix for global Listing builder
-                $item['link'] = '#' . $elements['tabAnchor'] . $tabIdx;
+                $item['link'] = '#'.$elements['tabAnchor'].$tabIdx;
 
                 $item['property']['role'] = 'presentation';
 
@@ -74,10 +74,10 @@ 
 
                 // mark active tab
                 if ($activeObject === true)
-                    $item['property']['class'] .= (Str::length($item['property']['class']) > 0 ? ' ' : null) . 'active';
+                    $item['property']['class'] .= (Str::length($item['property']['class']) > 0 ? ' ' : null).'active';
 
                 // tab special properties for bootstrap
-                $item['linkProperty']['aria-controls'] = $elements['tabAnchor'] . $tabIdx;
+                $item['linkProperty']['aria-controls'] = $elements['tabAnchor'].$tabIdx;
                 $item['linkProperty']['role'] = 'tab';
                 $item['linkProperty']['data-toggle'] = 'tab';
 
@@ -96,7 +96,7 @@ 
                     } else {
                         return self::nohtml($itemContent);
                     }
-                }, ['role' => 'tabpanel', 'class' => 'tab-pane fade' . ($activeObject === true ? ' in active' : null), 'id' => $elements['tabAnchor'] . $tabIdx]);
+                }, ['role' => 'tabpanel', 'class' => 'tab-pane fade'.($activeObject === true ? ' in active' : null), 'id' => $elements['tabAnchor'].$tabIdx]);
                 $tabIdx++;
             }
         }
@@ -104,7 +104,7 @@ 
         // check if global class "nav" isset
         if ($elements['property']['class'] !== null) {
             if (!Str::contains('nav ', $elements['property']['class'])) {
-                $elements['property']['class'] = 'nav ' . $elements['property']['class'];
+                $elements['property']['class'] = 'nav '.$elements['property']['class'];
             }
         } else {
             $elements['property']['class'] = 'nav';
@@ -128,7 +128,7 @@ 
                     return $tabContent;
                 }, $elements['tabProperty']);
             }
-            return $listing . $tabContent;
+            return $listing.$tabContent;
         }, $blockProperty);
     }
 }
\ No newline at end of file

Braces +12 added lines, -8 removed lines

@@ -20,18 +20,21 @@ 
     public static function display(array $elements): ?string
     {
         // check if elements isn't empty and contains rows
-        if (!isset($elements['items']) || count($elements['items']) < 1)
-            return null;
+        if (!isset($elements['items']) || count($elements['items']) < 1) {
+                    return null;
+        }
 
         // prepare tab order
-        if ($elements['tabAnchor'] === null)
-            $elements['tabAnchor'] = Str::randomLatin(mt_rand(6, 12));
+        if ($elements['tabAnchor'] === null) {
+                    $elements['tabAnchor'] = Str::randomLatin(mt_rand(6, 12));
+        }
 
         // set global element properties
         $blockProperty = [];
         if ($elements['blockProperty'] !== null) {
-            if (Any::isArray($elements['blockProperty']))
-                $blockProperty = $elements['blockProperty'];
+            if (Any::isArray($elements['blockProperty'])) {
+                            $blockProperty = $elements['blockProperty'];
+            }
             unset($elements['blockProperty']);
         }
 
@@ -73,8 +76,9 @@ 
                 }
 
                 // mark active tab
-                if ($activeObject === true)
-                    $item['property']['class'] .= (Str::length($item['property']['class']) > 0 ? ' ' : null) . 'active';
+                if ($activeObject === true) {
+                                    $item['property']['class'] .= (Str::length($item['property']['class']) > 0 ? ' ' : null) . 'active';
+                }
 
                 // tab special properties for bootstrap
                 $item['linkProperty']['aria-controls'] = $elements['tabAnchor'] . $tabIdx;

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

Unused Use Statements -2 removed lines

@@ -4,14 +4,12 @@
 
 use Ffcms\Core\App;
 use Ffcms\Core\Arch\Model;
-use Ffcms\Core\Exception\NativeException;
 use Ffcms\Core\Exception\SyntaxException;
 use Ffcms\Core\Helper\FileSystem\File;
 use Ffcms\Core\Helper\HTML\Form\Constructor;
 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;
 use Ffcms\Core\Helper\Type\Str;
 
 /**

Spacing +9 added lines, -9 removed lines

@@ -79,7 +79,7 @@ 
     {
         $form = self::buildSingleTag('form', $this->formProperty, false);
         if ($this->model->_tokenRequired) {
-            $form .= PHP_EOL . $this->field('_csrf_token', 'hidden', ['value' => $this->model->_csrf_token]);
+            $form .= PHP_EOL.$this->field('_csrf_token', 'hidden', ['value' => $this->model->_csrf_token]);
         }
         return $form;
     }
@@ -97,7 +97,7 @@ 
     {
         if ($this->model === null) {
             if (App::$Debug)
-                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;
         }
@@ -110,7 +110,7 @@ 
         // check if model contains current tag name as property
         if (!property_exists($this->model, $propertyName)) {
             if (App::$Debug)
-                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;
         }
@@ -131,13 +131,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);
         }
 
@@ -178,7 +178,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);
     }
@@ -192,10 +192,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);
                 }
@@ -206,7 +206,7 @@ 
                 $formName = $this->model->getFormName();
                 if (Any::isArray($badAttr) && count($badAttr) > 0) {
                     foreach ($badAttr as $attr) {
-                        $itemId = $formName . '-' . $attr;
+                        $itemId = $formName.'-'.$attr;
                         try {
                             $render = App::$View->render(static::$structLayer['jsnotify'], ['itemId' => $itemId]);
                             App::$Alias->addPlainCode('js', $render);

Braces +24 added lines, -16 removed lines

@@ -46,8 +46,9 @@ 
     public function __construct($model, array $property = null, array $layerFiles = null)
     {
         // prevent white-screen locks when model is not passed or passed wrong
-        if (!$model instanceof Model)
-            throw new SyntaxException('Bad model type passed in form builder. Check for init: new Form()');
+        if (!$model instanceof Model) {
+                    throw new SyntaxException('Bad model type passed in form builder. Check for init: new Form()');
+        }
 
         $this->model = $model;
         $this->name = $model->getFormName();
@@ -55,8 +56,9 @@ 
         // check if passed custom layer file
         if (Any::isArray($layerFiles) && count($layerFiles) > 0) {
             foreach (array_keys(static::$structLayer) as $type) {
-                if (isset($layerFiles[$type]) && Any::isStr($layerFiles[$type]))
-                    static::$structLayer[$type] = $layerFiles[$type];
+                if (isset($layerFiles[$type]) && Any::isStr($layerFiles[$type])) {
+                                    static::$structLayer[$type] = $layerFiles[$type];
+                }
             }
         }
         // set model submit method
@@ -64,8 +66,9 @@ 
 
         $property['id'] = $this->name; // define form id
         // if action is not defined - define it
-        if (!$property['action'])
-            $property['action'] = App::$Request->getFullUrl();
+        if (!$property['action']) {
+                    $property['action'] = App::$Request->getFullUrl();
+        }
 
         // set property global for this form
         $this->formProperty = $property;
@@ -96,21 +99,24 @@ 
     public function field(string $object, string $type, ?array $property = null, ?string $helper = null, ?string $layerFile = null)
     {
         if ($this->model === null) {
-            if (App::$Debug)
-                App::$Debug->addMessage('Form model is not defined for field name: [' . strip_tags($object) . ']');
+            if (App::$Debug) {
+                            App::$Debug->addMessage('Form model is not defined for field name: [' . strip_tags($object) . ']');
+            }
 
             return null;
         }
 
         // can be dots separated object
         $propertyName = $object;
-        if (Str::contains('.', $propertyName))
-            $propertyName = strstr($propertyName, '.', true);
+        if (Str::contains('.', $propertyName)) {
+                    $propertyName = strstr($propertyName, '.', true);
+        }
 
         // check if model contains current tag name as property
         if (!property_exists($this->model, $propertyName)) {
-            if (App::$Debug)
-                App::$Debug->addMessage('Form field ["' . $object . '"] is not defined in model: [' . get_class($this->model) . ']', 'error');
+            if (App::$Debug) {
+                            App::$Debug->addMessage('Form field ["' . $object . '"] is not defined in model: [' . get_class($this->model) . ']', 'error');
+            }
 
             return null;
         }
@@ -159,8 +165,9 @@ 
                 'help' => self::nohtml($helper)
             ]);
         } catch (SyntaxException $e) {
-            if (App::$Debug)
-                App::$Debug->addException($e);
+            if (App::$Debug) {
+                            App::$Debug->addException($e);
+            }
 
             $response = null;
         }
@@ -211,8 +218,9 @@ 
                             $render = App::$View->render(static::$structLayer['jsnotify'], ['itemId' => $itemId]);
                             App::$Alias->addPlainCode('js', $render);
                         } catch (SyntaxException $e) {
-                            if (App::$Debug)
-                                App::$Debug->addException($e);
+                            if (App::$Debug) {
+                                                            App::$Debug->addException($e);
+                            }
                         }
                     }
                 }