luyadev / luya

core/helpers/ArrayHelper.php

Doc Comments +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
      * ```
      *
      * @param array $data The input data to cover given sensitive key values. `['username' => 'foo', 'password' => 'bar']`.
-     * @param array $key The keys which can contain sensitive data inside the $data array. `['password', 'pwd', 'pass']` if no keys provided the {{luya\helpers\ArrayHelper::$sensitiveDefaultKeys}} is used.
+     * @param array $keys The keys which can contain sensitive data inside the $data array. `['password', 'pwd', 'pass']` if no keys provided the {{luya\helpers\ArrayHelper::$sensitiveDefaultKeys}} is used.
      * @since 1.0.6
      */
     public static function coverSensitiveValues(array $data, array $keys = [])

Spacing +4 added lines, -4 removed lines

@@ -158,7 +158,7 @@ 
     public static function search(array $array, $searchText, $sensitive = false, array $keys = [])
     {
         $function = $sensitive ? 'strpos' : 'stripos';
-        return array_filter($array, function ($item) use ($searchText, $function, $keys) {
+        return array_filter($array, function($item) use ($searchText, $function, $keys) {
             $response = false;
             foreach ($item as $key => $value) {
                 if ($response) {
@@ -206,7 +206,7 @@ 
         $array = array_values($array); // align array keys
         $columns = array_column($array, $column);
         $key = array_search($search, $columns);
-        return ($key !== false) ?  $array[$key] : false;
+        return ($key !== false) ? $array[$key] : false;
     }
     
     /**
@@ -238,7 +238,7 @@ 
      */
     public static function searchColumns(array $array, $column, $search)
     {
-        $keys = array_filter($array, function ($var) use ($column, $search) {
+        $keys = array_filter($array, function($var) use ($column, $search) {
             return strcasecmp($search, $var[$column]) == 0 ? true : false;
         });
         
@@ -278,7 +278,7 @@ 
         $array = array_combine($range, $range);
         
         if ($text) {
-            array_walk($array, function (&$item, $key) use ($text) {
+            array_walk($array, function(&$item, $key) use ($text) {
                 if (is_array($text)) {
                     list($singular, $plural) = $text;
                     if ($key == 1) {

core/web/Bootstrap.php

Spacing +1 added lines, -1 removed lines

@@ -88,7 +88,7 @@
         
         // generate the rules from apiRules defintions as they are own entries:
         foreach ($rules as $api => $rule) {
-            $rule['controller'] = 'admin/' . $api;
+            $rule['controller'] = 'admin/'.$api;
             $result[] = $rule;
         }
         

Unused Use Statements -2 removed lines

@@ -2,8 +2,6 @@
 
 namespace luya\web;
 
-use Yii;
-use yii\base\InvalidConfigException;
 use yii\helpers\ArrayHelper;
 use luya\base\AdminModuleInterface;
 use luya\TagParser;

core/messages/pl/luya.php

Indentation +1 added lines, -1 removed lines

@@ -1,5 +1,5 @@
 <?php
- return [
+    return [
     '{attribute} must have a minimum length of {length} characters.' => '{attribute} musi mieć co najmniej {length} znaków.',
     '{attribute} must include at least one special character.' => '{attribute} musi zawierać przynajmniej jeden znak specjalny.',
     '{attribute} must include at least one digit.' => '{attribute} musi zawierać co najmniej jedną cyfrę.',

dev/TranslationController.php

Spacing +2 added lines, -2 removed lines

@@ -41,7 +41,7 @@ 
         $messageFiles = glob("$repoPath/src/**/messages/$language/$filename.php") ?: glob("$repoPath/src/messages/$language/$filename.php");
 
         $this->outputInfo('Following files will be affected:');
-        $this->output(implode("\n", $messageFiles) . "\n");
+        $this->output(implode("\n", $messageFiles)."\n");
 
         $key = $this->prompt('Insert translation key:');
         $text = $this->prompt('Insert translation text:');
@@ -59,7 +59,7 @@ 
 
         if (!$this->dry) {
             if (exec("[ -d $repoPath/.git ] && command -v git")) {
-                $diffCommand = "git --git-dir=$repoPath/.git --work-tree=$repoPath diff -- " . str_replace($repoPath . '/', '', implode(" ", $messageFiles));
+                $diffCommand = "git --git-dir=$repoPath/.git --work-tree=$repoPath diff -- ".str_replace($repoPath.'/', '', implode(" ", $messageFiles));
                 exec($diffCommand, $diff);
                 $this->output(implode("\n", $diff));
             }

core/web/Controller.php

Spacing +3 added lines, -3 removed lines

@@ -33,14 +33,14 @@
     {
         if ($this->module instanceof Module) {
             if ($this->module->useAppViewPath) {
-                return '@app/views/' . $this->module->id . '/' . $this->id;
+                return '@app/views/'.$this->module->id.'/'.$this->id;
             } elseif (is_array($this->module->viewMap)) {
-                $currentAction = $this->id . '/' . ($this->action ? $this->action->id : $this->defaultAction);
+                $currentAction = $this->id.'/'.($this->action ? $this->action->id : $this->defaultAction);
                 foreach ($this->module->viewMap as $action => $viewPath) {
 
                     // Special case for map all views of controller
                     if ($action === '*') {
-                        return $viewPath . '/' . $this->id;
+                        return $viewPath.'/'.$this->id;
                     } elseif (fnmatch($action, $currentAction)) {
                         return $viewPath;
                     }

core/web/Svg.php

Spacing +7 added lines, -7 removed lines

@@ -100,25 +100,25 @@ 
     public function run()
     {
         // Cache generated code
-        return $this->getOrSetHasCache(['svg', $this->folder, $this->file, $this->cssClass], function () {
+        return $this->getOrSetHasCache(['svg', $this->folder, $this->file, $this->cssClass], function() {
 
             // Check if file ends with .svg, if not add the extension
-            $svgFile = StringHelper::endsWith($this->file, '.svg') ? $this->file : $this->file . '.svg';
+            $svgFile = StringHelper::endsWith($this->file, '.svg') ? $this->file : $this->file.'.svg';
 
             if ($this->symbolMode) {
-                $svgPath = Yii::$app->view->publicHtml . '/' . $svgFile;
+                $svgPath = Yii::$app->view->publicHtml.'/'.$svgFile;
 
-                return '<svg class="symbol symbol--' . $this->symbolName . '' . ($this->cssClass ? ' ' . $this->cssClass : '') . '"><use class="symbol__use" xlink:href="' . $svgPath . '#' . $this->symbolName . '" /></svg>';
+                return '<svg class="symbol symbol--'.$this->symbolName.''.($this->cssClass ? ' '.$this->cssClass : '').'"><use class="symbol__use" xlink:href="'.$svgPath.'#'.$this->symbolName.'" /></svg>';
             } else {
                 // Build the full svg file path
-                $svgPath = $this->folder . DIRECTORY_SEPARATOR . $svgFile;
+                $svgPath = $this->folder.DIRECTORY_SEPARATOR.$svgFile;
 
                 // Get the svg contents
                 $content = FileHelper::getFileContent($svgPath);
 
                 // If a cssClass string is given, add it to the <svg> tag
                 if ($this->cssClass && is_string($this->cssClass)) {
-                    $content = preg_replace('/<svg/', '<svg class="' . $this->cssClass . '"', $content);
+                    $content = preg_replace('/<svg/', '<svg class="'.$this->cssClass.'"', $content);
                 }
 
                 if ($content) {
@@ -126,7 +126,7 @@ 
                 }
             }
 
-            throw new Exception('Unable to access SVG File: ' . $svgPath);
+            throw new Exception('Unable to access SVG File: '.$svgPath);
         });
     }
 }

core/web/Element.php

Spacing +1 added lines, -1 removed lines

@@ -255,6 +255,6 @@
     {
         $view = new View();
         $view->autoRegisterCsrf = false;
-        return $view->renderPhpFile(rtrim($this->getFolder(), DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR . FileHelper::ensureExtension($file, 'php'), $args);
+        return $view->renderPhpFile(rtrim($this->getFolder(), DIRECTORY_SEPARATOR).DIRECTORY_SEPARATOR.FileHelper::ensureExtension($file, 'php'), $args);
     }
 }

core/helpers/StringHelper.php

Spacing +4 added lines, -4 removed lines

@@ -96,7 +96,7 @@ 
             return true;
         }
         
-        return ($value == (string)(float) $value);
+        return ($value == (string) (float) $value);
     }
     
     /**
@@ -205,7 +205,7 @@ 
         // we could not find any match, therefore use casual truncate method.
         if ($first === false) {
             // as the length value in truncate middle stands for to the left and to the right, we multiple this value with 2
-            return self::truncate($content, ($length*2), $affix);
+            return self::truncate($content, ($length * 2), $affix);
         }
 
         $last = $first + mb_strlen($word);
@@ -217,9 +217,9 @@ 
 
         // string before
         $before = (count($left) > $length) ? $affix.implode("", array_slice($left, -$length)) : implode("", $left);
-        $after = (count($right) > $length) ? implode("", array_slice($right, 0, $length)) . $affix : implode("", $right);
+        $after = (count($right) > $length) ? implode("", array_slice($right, 0, $length)).$affix : implode("", $right);
 
-        return $before . implode("", $middle) . $after;
+        return $before.implode("", $middle).$after;
     }
 
     /**

Doc Comments +3 added lines, -3 removed lines

@@ -111,7 +111,7 @@ 
      * @param string $search Search string to look for.
      * @param string $replace Replacement value for the first found occurrence.
      * @param string $subject The string you want to look up to replace the first element.
-     * @return mixed Replaced string
+     * @return string Replaced string
      */
     public static function replaceFirst($search, $replace, $subject)
     {
@@ -141,7 +141,7 @@ 
      * }
      * ```
      *
-     * @param string|array $needle The char or word to find in the $haystack. Can be an array to multi find words or char in the string.
+     * @param string $needle The char or word to find in the $haystack. Can be an array to multi find words or char in the string.
      * @param string $haystack The haystack where the $needle string should be looked up. A string or phrase with words.
      * @param boolean $strict If an array of needles is provided the $strict parameter defines whether all keys must be found ($strict = true) or just one result must be found ($strict = false).
      * @return boolean If an array of values is provided the response may change depending on $findAll.
@@ -178,7 +178,7 @@ 
      * @param string $content The content to minify.
      * @param array $options Optional arguments to provide for minification:
      * - comments: boolean, where html comments should be removed or not. defaults to false
-     * @return mixed Returns the minified content.
+     * @return string Returns the minified content.
      * @since 1.0.7
      */
     public static function minify($content, array $options = [])

core/lazyload/LazyLoad.php

Spacing +5 added lines, -5 removed lines

@@ -150,10 +150,10 @@ 
      */
     public function run()
     {
-        $class = ($this->placeholderSrc ? 'lazyimage-wrapper' : 'lazy-image') . ' ' . $this->extraClass;
+        $class = ($this->placeholderSrc ? 'lazyimage-wrapper' : 'lazy-image').' '.$this->extraClass;
 
         if ($this->placeholderSrc && $this->placeholderAsBase64) {
-            $this->placeholderSrc = 'data:image/jpg;base64,' . base64_encode(file_get_contents($this->placeholderSrc));
+            $this->placeholderSrc = 'data:image/jpg;base64,'.base64_encode(file_get_contents($this->placeholderSrc));
         }
 
         if ($this->attributesOnly && !$this->placeholderSrc) {
@@ -161,15 +161,15 @@ 
         }
 
         if ($this->placeholderSrc) {
-            $tag = '<div class="' . $class . '">';
+            $tag = '<div class="'.$class.'">';
             $tag .= Html::tag('img', '', ['class' => 'lazy-image lazyimage', 'data-src' => $this->src]);
             $tag .= Html::tag('img', '', ['class' => 'lazyimage-placeholder-image', 'src' => $this->placeholderSrc]);
-            $tag .= '<noscript><img class="lazyimage-image" src="' . $this->src . '" /></noscript>';
+            $tag .= '<noscript><img class="lazyimage-image" src="'.$this->src.'" /></noscript>';
             $tag .= '</div>';
         } else {
             $tag = Html::tag('img', '', ['class' => $class, 'data-src' => $this->src, 'data-width' => $this->width, 'data-height' => $this->height]);
             if ($this->width && $this->height) {
-                $tag .= '<div class="lazy-placeholder ' . $class .'"><div style="display: block; height: 0px; padding-bottom: ' . ($this->height / $this->width) * 100 . '%;"></div><div class="loader"></div></div>';
+                $tag .= '<div class="lazy-placeholder '.$class.'"><div style="display: block; height: 0px; padding-bottom: '.($this->height / $this->width) * 100.'%;"></div><div class="loader"></div></div>';
             }
             $tag .= '<noscript><img class="'.$class.'" src="'.$this->src.'" /></noscript>';
         }