phpffcms / ffcms-core

src/Ffcms/Core/Arch/Model.php

Doc Comments +2 added lines, -2 removed lines

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

src/Ffcms/Core/Arch/View.php

Doc Comments +1 added lines, -1 removed lines

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

Spacing +16 added lines, -16 removed lines

@@ -54,16 +54,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
@@ -106,7 +106,7 @@ 
         
         // path still not defined?
         if ($path === null) {
-            throw new SyntaxException('Viewer is not founded: ' . App::$Security->strip_tags($path));
+            throw new SyntaxException('Viewer is not founded: '.App::$Security->strip_tags($path));
         }
 
         // cleanup from slashes on start/end
@@ -135,7 +135,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
@@ -151,14 +151,14 @@ 
 
             // depended controller is not founded? Let finish
             if ($calledController === null) {
-                throw new NativeException('View render is failed: callback controller not founded! Call with relative path: ' . $path);
+                throw new NativeException('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
@@ -168,18 +168,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 !== null) {
-                    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 !== null) {
-            App::$Debug->addMessage('Viewer not founded on rendering: ' . $path, 'warning');
+            App::$Debug->addMessage('Viewer not founded on rendering: '.$path, 'warning');
         }
 
         return null;
@@ -274,7 +274,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;
         }
@@ -283,9 +283,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";
             }
         }
 
@@ -308,7 +308,7 @@ 
 
         $code = null;
         foreach (App::$Alias->getPlainCode($type) as $row) {
-            $code .= $row . "\n";
+            $code .= $row."\n";
         }
 
         return $code;

src/Ffcms/Core/Debug/Manager.php

Doc Comments +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
 
     /**
      * Add exception into debug bar and stop execute
-     * @param $e
+     * @param \Ffcms\Core\Exception\TemplateException $e
      * @throws \DebugBar\DebugBarException
      */
     public function addException($e)

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

Doc Comments +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
     /**
      * Construct table based on passed elements as array: properties, thead, tbody, rows, items etc
      * @param array $elements
-     * @return null
+     * @return null|string
      */
     public static function display($elements)
     {

Spacing +14 added lines, -14 removed lines

@@ -40,13 +40,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']) && Obj::isArray($elements['thead']) && count($elements['thead']) > 0 && Obj::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;
@@ -61,12 +61,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']);
                         }
@@ -85,14 +85,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 (!Obj::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) {
@@ -106,9 +106,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)])) . ' ' . $text;
+                                        }, Arr::merge($selectOptions['selector'], ['value' => htmlentities($text)])).' '.$text;
                                     }
                                     return $text;
                                 }, $item['property']);
@@ -131,14 +131,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 (!Obj::isArray($btn)) {
                         continue;
                     }
-                    $table .= $dom->input(function(){
+                    $table .= $dom->input(function() {
                         return null;
-                    }, $btn) . ' ';
+                    }, $btn).' ';
                 }
                 return $table;
             }, $selectOptions['form']);
@@ -154,7 +154,7 @@ 
     private static function buildSelectorHtml(array $opt)
     {
         $js = '$(function () {
-            var targetSwitchbox = $(\'input[name="' . $opt['selector']['name'] . '"]\');
+            var targetSwitchbox = $(\'input[name="' . $opt['selector']['name'].'"]\');
             $(\'input[name="selectAll"]\').change(function() {
                 $(targetSwitchbox).each(function () {
                     $(this).prop(\'checked\', !$(this).is(\':checked\'));

src/Ffcms/Core/Helper/Text.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
                 $breakerPos = mb_strpos($text, '<br', null, 'UTF-8');
             } else {
                 // add length('</p>')
-                $breakerPos+= 4;
+                $breakerPos += 4;
             }
             // cut text from position caret before </p> (+4 symbols to save item as valid)
             if ($breakerPos !== false) {

src/Ffcms/Core/Debug/LaravelDatabaseCollector.php

Spacing +3 added lines, -3 removed lines

@@ -160,7 +160,7 @@ 
             $hints[] = '<code>LIMIT</code> without <code>ORDER BY</code> causes non-deterministic results, depending on the query execution plan';
         }
         if (preg_match('/LIKE\\s[\'"](%.*?)[\'"]/i', $query, $matches)) {
-            $hints[] = 'An argument has a leading wildcard character: <code>' . $matches[1] . '</code>.
+            $hints[] = 'An argument has a leading wildcard character: <code>'.$matches[1].'</code>.
 								The predicate with this argument is not sargable and cannot use an index if one exists.';
         }
         return implode("<br />", $hints);
@@ -175,7 +175,7 @@ 
         foreach ($traces as $trace) {
             if (isset($trace['class']) && isset($trace['file']) && strpos(
                     $trace['file'],
-                    DIRECTORY_SEPARATOR . 'vendor' . DIRECTORY_SEPARATOR
+                    DIRECTORY_SEPARATOR.'vendor'.DIRECTORY_SEPARATOR
                 ) === false
             ) {
                 if (isset($trace['object']) && is_a($trace['object'], 'Twig_Template')) {
@@ -187,7 +187,7 @@ 
                     $line = isset($trace['line']) ? $trace['line'] : '?';
                 }
 
-                return $this->normalizeFilename($file) . ':' . $line;
+                return $this->normalizeFilename($file).':'.$line;
             } elseif (isset($trace['function']) && $trace['function'] == 'Illuminate\Routing\{closure}') {
                 return 'Route binding';
             }

src/Ffcms/Core/Helper/FileSystem/Normalize.php

Spacing +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
     {
         $path = self::diskPath($path);
         if (!Str::startsWith(root, $path)) {
-            $path = root . DIRECTORY_SEPARATOR . ltrim($path, '\\/');
+            $path = root.DIRECTORY_SEPARATOR.ltrim($path, '\\/');
         }
         return $path;
     }

src/Ffcms/Core/Helper/FileSystem/Directory.php

Spacing +2 added lines, -2 removed lines

@@ -91,7 +91,7 @@ 
             return false;
         }
 
-        $pattern = rtrim($path, '/') . '/*';
+        $pattern = rtrim($path, '/').'/*';
         $entry = glob($pattern, $mod);
 
         if ($returnRelative === true) {
@@ -122,7 +122,7 @@ 
         array_pop($separatedPath);
         $clearPath = implode(DIRECTORY_SEPARATOR, $separatedPath);
 
-        @rename($path, $clearPath . DIRECTORY_SEPARATOR . $newDirName);
+        @rename($path, $clearPath.DIRECTORY_SEPARATOR.$newDirName);
 
         return true;
     }

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

Spacing +5 added lines, -5 removed lines

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