phpffcms / ffcms-core

src/Ffcms/Core/Arch/Model.php

Doc Comments +2 added lines, -2 removed lines

@@ -41,7 +41,7 @@ 
     /**
      * Get label value by variable name
      * @param string $param
-     * @return mixed
+     * @return string
      */
     final public function getLabel($param)
     {
@@ -65,7 +65,7 @@ 
 
     /**
      * Set attribute labels for model variables
-     * @return array
+     * @return string
      */
     public function labels(): array
     {

Spacing +1 added lines, -1 removed lines

@@ -156,7 +156,7 @@
      * Get all properties for current model in key=>value array
      * @return array|null
      */
-    public function getAllProperties(): ?array
+    public function getAllProperties(): ? array
     {
         $properties = null;
         // list all properties here, array_walk sucks on performance!

src/Ffcms/Core/Managers/BootManager.php

Spacing +5 added lines, -5 removed lines

@@ -42,7 +42,7 @@ 
         // check if cache is enabled
         if (App::$Cache !== null) {
             // try to get bootable class map from cache, or initialize parsing
-            $cache = App::$Cache->getItem('boot.' . env_name . '.class.map');
+            $cache = App::$Cache->getItem('boot.'.env_name.'.class.map');
             if (!$cache->isHit()) {
                 $this->compileBootableClasses();
                 $cache->set($this->objects)->expiresAfter(static::CACHE_TREE_TIME);
@@ -97,11 +97,11 @@ 
     {
         // list app root's
         foreach ($this->appRoots as $app) {
-            $app .= '/Apps/Controller/' . env_name;
+            $app .= '/Apps/Controller/'.env_name;
             $files = File::listFiles($app, ['.php'], true);
             foreach ($files as $file) {
                 // define full class name with namespace
-                $class = 'Apps\Controller\\' . env_name . '\\' . Str::cleanExtension($file);
+                $class = 'Apps\Controller\\'.env_name.'\\'.Str::cleanExtension($file);
                 // check if class exists (must be loaded over autoloader), boot method exist and this is controller instanceof
                 if (class_exists($class) && method_exists($class, 'boot') && is_a($class, 'Ffcms\Core\Arch\Controller', true)) {
                     $this->objects[] = $class;
@@ -111,14 +111,14 @@ 
 
         // list widget root's
         foreach ($this->widgetRoots as $widget) {
-            $widget .= '/Widgets/' . env_name;
+            $widget .= '/Widgets/'.env_name;
             // widgets are packed in directory, classname should be the same with root directory name
             $dirs = Directory::scan($widget, GLOB_ONLYDIR, true);
             if (!Obj::isArray($dirs)) {
                 continue;
             }
             foreach ($dirs as $instance) {
-                $class = 'Widgets\\' . env_name . '\\' . $instance . '\\' . $instance;
+                $class = 'Widgets\\'.env_name.'\\'.$instance.'\\'.$instance;
                 if (class_exists($class) && method_exists($class, 'boot') && is_a($class, 'Ffcms\Core\Arch\Widget', true)) {
                     $this->objects[] = $class;
                 }

src/Ffcms/Core/Arch/Controller.php

Spacing +4 added lines, -4 removed lines

@@ -67,9 +67,9 @@ 
         } else {
             $this->startMeasure(__METHOD__);
 
-            $layoutPath = App::$Alias->currentViewPath . '/layout/' . $this->layout . '.php';
+            $layoutPath = App::$Alias->currentViewPath.'/layout/'.$this->layout.'.php';
             if (!File::exist($layoutPath)) {
-                throw new NativeException('Layout not founded: ' . $layoutPath);
+                throw new NativeException('Layout not founded: '.$layoutPath);
             }
 
             $body = $this->output;
@@ -88,7 +88,7 @@ 
             // set custom css library's not included on static call
             $cssIncludeCode = App::$View->showCodeLink('css');
             if (!Str::likeEmpty($cssIncludeCode)) {
-                $content = Str::replace('</head>', $cssIncludeCode . '</head>', $content);
+                $content = Str::replace('</head>', $cssIncludeCode.'</head>', $content);
             }
 
 
@@ -98,7 +98,7 @@ 
             if (App::$Debug) {
                 $content = Str::replace(
                     ['</body>', '</head>'],
-                    [App::$Debug->renderOut() . '</body>', App::$Debug->renderHead() . '</head>'],
+                    [App::$Debug->renderOut().'</body>', App::$Debug->renderHead().'</head>'],
                     $content);
             }
 

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;

Braces +6 added lines, -4 removed lines

@@ -317,8 +317,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
@@ -342,8 +343,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/App.php

Spacing +7 added lines, -7 removed lines

@@ -188,7 +188,7 @@ 
         try {
             /** @var \Ffcms\Core\Arch\Controller $callClass */
             $callClass = null;
-            $callMethod = 'action' . self::$Request->getAction();
+            $callMethod = 'action'.self::$Request->getAction();
 
             // founded callback injection alias
             if (self::$Request->getCallbackAlias() !== false) {
@@ -196,16 +196,16 @@ 
                 if (class_exists($cName)) {
                     $callClass = new $cName;
                 } else {
-                    throw new NotFoundException('Callback alias of class "' . App::$Security->strip_tags($cName) . '" is not founded');
+                    throw new NotFoundException('Callback alias of class "'.App::$Security->strip_tags($cName).'" is not founded');
                 }
             } else { // typical parsing of native apps
-                $cName = '\Apps\Controller\\' . env_name . '\\' . self::$Request->getController();
+                $cName = '\Apps\Controller\\'.env_name.'\\'.self::$Request->getController();
 
                 // try to initialize class object
                 if (class_exists($cName)) {
                     $callClass = new $cName;
                 } else {
-                    throw new NotFoundException('Application can not be runned. Initialized class not founded: ' . App::$Security->strip_tags($cName));
+                    throw new NotFoundException('Application can not be runned. Initialized class not founded: '.App::$Security->strip_tags($cName));
                 }
             }
 
@@ -238,10 +238,10 @@ 
                     ]));
                 }
 
-                $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], $actionQuery);
-                $this->stopMeasure($cName . '::' . $callMethod);
+                $this->stopMeasure($cName.'::'.$callMethod);
 
                 if ($actionResponse !== null && !Str::likeEmpty($actionResponse)) {
                     // set response to controller property object
@@ -251,7 +251,7 @@ 
                 // build full compiled output html data
                 $html = $callClass->buildOutput();
             } else {
-                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).'"');
             }
         } catch (NotFoundException $e) { // catch exceptions and set output
             $html = $e->display();