phpffcms / ffcms-core

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

Braces +9 added lines, -6 removed lines

@@ -24,8 +24,9 @@ 
     public static function display($elements)
     {
         // check input elements
-        if (!Arr::in($elements['type'], ['ul', 'ol']) || count($elements['items']) < 1)
-            return null;
+        if (!Arr::in($elements['type'], ['ul', 'ol']) || count($elements['items']) < 1) {
+                    return null;
+        }
 
         // initialize new DOM model
         $dom = new Dom();
@@ -62,11 +63,13 @@ 
      */
     private static function buildDropdown($dom, $item)
     {
-        if (!Any::isArray($item['items']))
-            return null;
+        if (!Any::isArray($item['items'])) {
+                    return null;
+        }
 
-        if (!isset($item['menuProperty']['class']))
-            $item['menuProperty']['class'] = 'dropdown-menu';
+        if (!isset($item['menuProperty']['class'])) {
+                    $item['menuProperty']['class'] = 'dropdown-menu';
+        }
 
         return $dom->li(function() use($dom, $item){
             $dropdownLink = $dom->a(function() use ($dom, $item){

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

Braces +27 added lines, -18 removed lines

@@ -19,8 +19,9 @@ 
     public static function in(?string $needle, ?array $haystack = null, bool $strict = true): bool
     {
         // prevent errors
-        if (!Any::isArray($haystack) || $needle === null)
-            return false;
+        if (!Any::isArray($haystack) || $needle === null) {
+                    return false;
+        }
 
         return in_array($needle, $haystack, $strict);
     }
@@ -33,8 +34,9 @@ 
     {
         $arguments = [];
         foreach (func_get_args() as $key => $val) {
-            if (!Any::isArray($val))
-                $val = [];
+            if (!Any::isArray($val)) {
+                            $val = [];
+            }
 
             $arguments[$key] = $val;
         }
@@ -49,8 +51,9 @@ 
     {
         $arguments = [];
         foreach (func_get_args() as $key => $val) {
-            if (!Any::isArray($val))
-                $val = [];
+            if (!Any::isArray($val)) {
+                            $val = [];
+            }
 
             $arguments[$key] = $val;
         }
@@ -67,18 +70,21 @@ 
     public static function getByPath(string $path, ?array $array = null, $delimiter = '.')
     {
         // path of nothing? interest
-        if (!Any::isArray($array) || count($array) < 1 || !Any::isStr($path) || Str::likeEmpty($path))
-            return null;
+        if (!Any::isArray($array) || count($array) < 1 || !Any::isStr($path) || Str::likeEmpty($path)) {
+                    return null;
+        }
 
         // c'mon man, what the f*ck are you doing? ))
-        if (!Str::contains($delimiter, $path))
-            return $array[$path];
+        if (!Str::contains($delimiter, $path)) {
+                    return $array[$path];
+        }
 
         $output = $array;
         $pathArray = explode($delimiter, $path);
         foreach ($pathArray as $key) {
-            if (!Any::isArray($output) || !array_key_exists($key, $output))
-                return null;
+            if (!Any::isArray($output) || !array_key_exists($key, $output)) {
+                            return null;
+            }
 
             $output = $output[$key];
         }
@@ -93,14 +99,16 @@ 
      */
     public static function pluck(?string $key = null, ?array $array = null): array
     {
-        if (!Any::isArray($array) || !Any::isStr($key))
-            return [];
+        if (!Any::isArray($array) || !Any::isStr($key)) {
+                    return [];
+        }
 
         $output = [];
         foreach ($array as $item) {
             $object = $item[$key];
-            if (!self::in($object, $output))
-                $output[] = $object;
+            if (!self::in($object, $output)) {
+                            $output[] = $object;
+            }
         }
         return $output;
     }
@@ -148,8 +156,9 @@ 
      */
     public static function onlyNumericValues(?array $array = null)
     {
-        if (!Any::isArray($array))
-            return false;
+        if (!Any::isArray($array)) {
+                    return false;
+        }
 
         return is_numeric(implode('', $array));
     }

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

Braces +15 added lines, -10 removed lines

@@ -17,12 +17,14 @@ 
     public static function isEmpty($var = null): bool
     {
         // var is array type?
-        if (is_array($var))
-            return count($var) < 1;
+        if (is_array($var)) {
+                    return count($var) < 1;
+        }
 
         // var seems to be object?
-        if (is_object($var))
-            return count(get_object_vars($var)) < 1;
+        if (is_object($var)) {
+                    return count(get_object_vars($var)) < 1;
+        }
 
         // float,int,string,bool and null left. Check if not empty. Int and float will never equal to null.
         return ($var === null || $var === '' || $var === false);
@@ -123,14 +125,17 @@ 
     public static function guessValueType($var = null)
     {
         // pass var by reference and guess var type
-        if (self::isInt($var))
-            return $var;
+        if (self::isInt($var)) {
+                    return $var;
+        }
 
-        if (self::isFloat($var))
-            return $var;
+        if (self::isFloat($var)) {
+                    return $var;
+        }
 
-        if (self::isBool($var))
-            return $var;
+        if (self::isBool($var)) {
+                    return $var;
+        }
 
 
         return $var;

src/Ffcms/Core/Helper/Url.php

Spacing +14 added lines, -14 removed lines

@@ -25,7 +25,7 @@ 
     public static function to($controller_action, $id = null, $add = null, array $params = null, $encode = true)
     {
         $pathway = self::buildPathway([$controller_action, $id, $add, $params], $encode);
-        return App::$Alias->baseUrl . '/' . $pathway;
+        return App::$Alias->baseUrl.'/'.$pathway;
     }
 
     /**
@@ -61,7 +61,7 @@ 
                 $controller = Str::lastIn($controller, '\\', true);
             }
 
-            $response = $controller . '/' . $action;
+            $response = $controller.'/'.$action;
         }
 
         // check if controller and action is defined
@@ -71,17 +71,17 @@ 
 
         // id is defined?
         if (isset($to[1]) && !Str::likeEmpty($to[1])) {
-            $response .= '/' . self::safeUri($to[1], $encode);
+            $response .= '/'.self::safeUri($to[1], $encode);
         }
 
         // add param is defined?
         if (isset($to[2]) && !Str::likeEmpty($to[2])) {
-            $response .= '/' . self::safeUri($to[2], $encode);
+            $response .= '/'.self::safeUri($to[2], $encode);
         }
 
         // try to find static alias
-        if (isset($routing['Alias'][env_name]) && Arr::in('/' . $response, $routing['Alias'][env_name])) {
-            $pathAlias = array_search('/' . $response, $routing['Alias'][env_name]);
+        if (isset($routing['Alias'][env_name]) && Arr::in('/'.$response, $routing['Alias'][env_name])) {
+            $pathAlias = array_search('/'.$response, $routing['Alias'][env_name]);
             if ($pathAlias !== false) {
                 $response = Str::lowerCase(trim($pathAlias, '/'));
             }
@@ -97,7 +97,7 @@ 
             }
             $queryString = http_build_query($to[3]);
             if (Str::length($queryString) > 0) {
-                $response .= '?' . http_build_query($to[3]);
+                $response .= '?'.http_build_query($to[3]);
             }
             if ($anchor !== false) {
                 $response .= $anchor;
@@ -119,7 +119,7 @@ 
     public static function buildPathwayFromRequest()
     {
         return self::buildPathway([
-            Str::lowerCase(App::$Request->getController()) . '/' . Str::lowerCase(App::$Request->getAction()),
+            Str::lowerCase(App::$Request->getController()).'/'.Str::lowerCase(App::$Request->getAction()),
             App::$Request->getID(),
             App::$Request->getAdd(),
             App::$Request->query->all()
@@ -140,9 +140,9 @@ 
             $to = [$to];
 
         // call Url::to(args)
-        $callbackTo = call_user_func_array([__NAMESPACE__ . '\Url', 'to'], $to);
+        $callbackTo = call_user_func_array([__NAMESPACE__.'\Url', 'to'], $to);
 
-        return '<a href="' . $callbackTo . '"' . $compile_property . '>' . $name . '</a>';
+        return '<a href="'.$callbackTo.'"'.$compile_property.'>'.$name.'</a>';
     }
 
     /**
@@ -155,19 +155,19 @@ 
     {
         /** @var array $configs */
         $configs = \App::$Properties->getAll('default');
-        $httpHost = $configs['baseProto'] . '://' . $configs['baseDomain'];
+        $httpHost = $configs['baseProto'].'://'.$configs['baseDomain'];
         if ($configs['basePath'] !== '/') {
-            $httpHost .= $configs['basePath'] . '/';
+            $httpHost .= $configs['basePath'].'/';
         }
 
         // check if is this is URI not URL
         if (!Str::startsWith($httpHost, $uri)) {
             // check if lang is defined in URI or define it
             if ($lang !== null && $configs['multiLanguage'] && !Str::startsWith($lang, $uri)) {
-                $uri = $lang . '/' . ltrim($uri, '/');
+                $uri = $lang.'/'.ltrim($uri, '/');
             }
             // add basic httpHost data
-            $uri = rtrim($httpHost, '/') . '/' . ltrim($uri, '/');
+            $uri = rtrim($httpHost, '/').'/'.ltrim($uri, '/');
         }
 
         return $uri;

Braces +3 added lines, -2 removed lines

@@ -136,8 +136,9 @@
     public static function link($to, $name, array $property = null)
     {
         $compile_property = self::applyProperty($property);
-        if (!Any::isArray($to))
-            $to = [$to];
+        if (!Any::isArray($to)) {
+                    $to = [$to];
+        }
 
         // call Url::to(args)
         $callbackTo = call_user_func_array([__NAMESPACE__ . '\Url', 'to'], $to);

src/Ffcms/Core/Helper/Simplify.php

Braces +9 added lines, -6 removed lines

@@ -21,13 +21,15 @@ 
      */
     public static function parseUserNick($userId = null, $onEmpty = 'guest'): ?string
     {
-        if (!Any::isInt($userId))
-            return \App::$Security->strip_tags($onEmpty);
+        if (!Any::isInt($userId)) {
+                    return \App::$Security->strip_tags($onEmpty);
+        }
 
         // try to find user active record as object
         $identity = App::$User->identity($userId);
-        if (!$identity)
-            return \App::$Security->strip_tags($onEmpty);
+        if (!$identity) {
+                    return \App::$Security->strip_tags($onEmpty);
+        }
 
         // return user nickname from profile
         return $identity->profile->getNickname();
@@ -44,8 +46,9 @@ 
     {
         $nick = self::parseUserNick($userId, $onEmpty);
         // new name is not found, lets return default
-        if ($nick === $onEmpty || (int)$userId < 1)
-            return $nick;
+        if ($nick === $onEmpty || (int)$userId < 1) {
+                    return $nick;
+        }
 
         return Url::link([$controllerAction, (int)$userId], $nick);
     }