luyadev / luya

core/base/ModuleReflection.php

Spacing +4 added lines, -4 removed lines

@@ -216,7 +216,7 @@ 
         
         return [
             'module' => $this->module->id,
-            'route' => $this->module->id . '/' . $request['route'],
+            'route' => $this->module->id.'/'.$request['route'],
             'params' => $request['originalArgs'],
         ];
     }
@@ -237,7 +237,7 @@ 
             throw new NotFoundHttpException(sprintf("Unable to create controller '%s' for module '%s'.", $requestRoute['route'], $this->module->id));
         }
         
-        Yii::debug('LUYA module run module "'.$this->module->id.'" route ' . $requestRoute['route'], __METHOD__);
+        Yii::debug('LUYA module run module "'.$this->module->id.'" route '.$requestRoute['route'], __METHOD__);
         
         $this->controller = $controller[0];
         $originalController = Yii::$app->controller;
@@ -248,7 +248,7 @@ 
          *
          * @see https://github.com/luyadev/luya/issues/1730
          */
-        $this->controller->on(Controller::EVENT_BEFORE_ACTION, function ($event) {
+        $this->controller->on(Controller::EVENT_BEFORE_ACTION, function($event) {
             Yii::$app->controller = $this->controller;
         });
         /**
@@ -256,7 +256,7 @@ 
          *
          * @see https://github.com/luyadev/luya/issues/1768
          */
-        $this->controller->on(Controller::EVENT_AFTER_ACTION, function ($event) use ($originalController) {
+        $this->controller->on(Controller::EVENT_AFTER_ACTION, function($event) use ($originalController) {
             Yii::$app->controller = $originalController;
         });
         

core/web/EmailLink.php

Doc Comments +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
     /**
      * Getter method for the e-mail.
      *
-     * @return string|boolean Returns the e-mail from the setter method, if mail is not valid false is returned.
+     * @return string|false Returns the e-mail from the setter method, if mail is not valid false is returned.
      */
     public function getEmail()
     {

Spacing +1 added lines, -1 removed lines

@@ -61,7 +61,7 @@
      */
     public function getHref()
     {
-        return empty($this->getEmail()) ? null : 'mailto:' . $this->getEmail();
+        return empty($this->getEmail()) ? null : 'mailto:'.$this->getEmail();
     }
 
     /**

core/console/commands/ThemeController.php

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 use luya\theme\ThemeConfig;
 use Yii;
 use yii\helpers\Console;
-use yii\helpers\Inflector;
 
 /**
  * Command to create a new LUYA theme.

Spacing +7 added lines, -7 removed lines

@@ -47,7 +47,7 @@ 
     
         $availableModules = implode(', ', array_column(Yii::$app->getFrontendModules(), 'id'));
         $themeLocation = $this->prompt("Enter the theme location where to generate (as path alias e.g. app, $availableModules):", ['default' => 'app']);
-        $themeLocation = '@' . ltrim($themeLocation, '@');
+        $themeLocation = '@'.ltrim($themeLocation, '@');
     
         preg_match("#^@[A-z]+#", $themeLocation, $newThemeLocation);
     
@@ -59,15 +59,15 @@ 
             }
         }
         
-        $basePath = $themeLocation . DIRECTORY_SEPARATOR . 'themes' . DIRECTORY_SEPARATOR . $themeName;
+        $basePath = $themeLocation.DIRECTORY_SEPARATOR.'themes'.DIRECTORY_SEPARATOR.$themeName;
         $themeFolder = Yii::getAlias($basePath);
     
         if (file_exists($themeFolder)) {
-            return $this->outputError("The folder " . $themeFolder . " exists already.");
+            return $this->outputError("The folder ".$themeFolder." exists already.");
         }
     
-        $this->outputInfo("Theme path alias: " . $basePath);
-        $this->outputInfo("Theme real path: " . $themeFolder);
+        $this->outputInfo("Theme path alias: ".$basePath);
+        $this->outputInfo("Theme real path: ".$themeFolder);
         if (!$this->confirm("Do you want continue?")) {
             return $this->outputError('Abort by user.');
         }
@@ -81,11 +81,11 @@ 
         ];
 
         foreach ($folders as $folder) {
-            FileHelper::createDirectory($themeFolder . DIRECTORY_SEPARATOR . $folder);
+            FileHelper::createDirectory($themeFolder.DIRECTORY_SEPARATOR.$folder);
         }
         
         $contents = [
-            $themeFolder. DIRECTORY_SEPARATOR . 'theme.json' => $this->renderJson($basePath, $themeName),
+            $themeFolder.DIRECTORY_SEPARATOR.'theme.json' => $this->renderJson($basePath, $themeName),
         ];
         
         foreach ($contents as $fileName => $content) {

core/traits/ErrorHandlerTrait.php

Doc Comments +2 added lines, -1 removed lines

@@ -67,7 +67,7 @@ 
      *
      * @param string $message The message you want to send to the error api server.
      * @param string $file The you are currently send the message (use __FILE__)
-     * @param string $line The line you want to submit (use __LINE__)
+     * @param integer $line The line you want to submit (use __LINE__)
      * @return bool|null
      */
     public function transferMessage($message, $file = __FILE__, $line = __LINE__)
@@ -105,6 +105,7 @@ 
     
     /**
      * @inheritdoc
+     * @param \Exception $exception
      */
     public function renderException($exception)
     {

Spacing +4 added lines, -4 removed lines

@@ -176,9 +176,9 @@ 
             $_file = $exception->getFile();
             $_line = $exception->getLine();
         } elseif (is_string($exception)) {
-            $_message = 'exception string: ' . $exception;
+            $_message = 'exception string: '.$exception;
         } elseif (is_array($exception)) {
-            $_message = isset($exception['message']) ? $exception['message'] : 'exception array dump: ' . print_r($exception, true);
+            $_message = isset($exception['message']) ? $exception['message'] : 'exception array dump: '.print_r($exception, true);
             $_file = isset($exception['file']) ? $exception['file'] : __FILE__;
             $_line = isset($exception['line']) ? $exception['line'] : __LINE__;
         }
@@ -249,7 +249,7 @@ 
 
         if (!empty($file)) {
             try {
-                $lineInArray = $line -1;
+                $lineInArray = $line - 1;
                 // load file from path
                 $fileInfo = file($file, FILE_IGNORE_NEW_LINES);
                 // load file if false from real path
@@ -289,7 +289,7 @@ 
             'class' => isset($item['class']) ? $item['class'] : null,
             // currently arguments wont be transmited due to large amount of informations based on base object
             //'args' => isset($item['args']) ? ArrayHelper::coverSensitiveValues($item['args'], $this->sensitiveKeys) : [],
-        ], function ($value) {
+        ], function($value) {
             return !empty($value);
         });
     }

core/helpers/ExportHelper.php

Spacing +3 added lines, -3 removed lines

@@ -139,7 +139,7 @@ 
     {
         $output = null;
         foreach ($input as $row) {
-            $output.= self::generateRow($row, $delimiter, '"');
+            $output .= self::generateRow($row, $delimiter, '"');
         }
 
         return $output;
@@ -156,7 +156,7 @@ 
      */
     protected static function generateRow(array $row, $delimiter, $enclose)
     {
-        array_walk($row, function (&$item) use ($enclose) {
+        array_walk($row, function(&$item) use ($enclose) {
             if (is_bool($item)) {
                 $item = (int) $item;
             } elseif (is_null($item)) {
@@ -171,6 +171,6 @@ 
             ], $item).$enclose;
         });
 
-        return implode($delimiter, $row) . PHP_EOL;
+        return implode($delimiter, $row).PHP_EOL;
     }
 }

Doc Comments +1 added lines, -2 removed lines

@@ -37,7 +37,7 @@ 
      * @param array|QueryInterface $input
      * @param array $keys Defines which keys should be packed into the generated xlsx. The defined keys does not change the sort behavior of the generated xls.
      * @param bool $header
-     * @return mixed
+     * @return string
      * @throws Exception
      * @since 1.0.4
      */
@@ -73,7 +73,6 @@ 
      * Generate content by rows.
      *
      * @param array $contentRows
-     * @param string $delimiter
      * @param array $keys
      * @param bool $generateHeader
      * @return array

core/theme/ThemeManager.php

Spacing +6 added lines, -6 removed lines

@@ -53,18 +53,18 @@ 
         if (strpos($basePath, '@') === 0) {
             $dir = Yii::getAlias($basePath);
         } elseif (strpos($basePath, '/') !== 0) {
-            $dir = $basePath = Yii::$app->basePath . DIRECTORY_SEPARATOR . $basePath;
+            $dir = $basePath = Yii::$app->basePath.DIRECTORY_SEPARATOR.$basePath;
         } else {
             $dir = $basePath;
         }
         
         if (!is_dir($dir) || !is_readable($dir)) {
-            throw new Exception('Theme directory not exists or readable: ' . $dir);
+            throw new Exception('Theme directory not exists or readable: '.$dir);
         }
         
-        $themeFile = $dir . '/theme.json';
+        $themeFile = $dir.'/theme.json';
         if (!file_exists($themeFile)) {
-            throw new InvalidConfigException('Theme config file missing at: ' . $themeFile);
+            throw new InvalidConfigException('Theme config file missing at: '.$themeFile);
         }
         
         $config = Json::decode(file_get_contents($themeFile)) ?: [];
@@ -159,7 +159,7 @@ 
         
         if (file_exists(Yii::getAlias('@app/themes'))) {
             foreach (glob(Yii::getAlias('@app/themes/*')) as $dirPath) {
-                $themeDefinitions[] = "@app/themes/" . basename($dirPath);
+                $themeDefinitions[] = "@app/themes/".basename($dirPath);
             }
         }
         
@@ -205,7 +205,7 @@ 
         
         $this->_themes[$basePath] = $themeConfig;
         
-        Yii::setAlias('@' . basename($basePath) . 'Theme', $basePath);
+        Yii::setAlias('@'.basename($basePath).'Theme', $basePath);
     }
     
     /**

core/theme/Theme.php

Spacing +2 added lines, -2 removed lines

@@ -74,7 +74,7 @@ 
         }
     
         $pos = strpos($viewPath, '/');
-        $rootPath = $pos === false ? $viewPath : (substr($viewPath, 0, $pos) . '/views');
+        $rootPath = $pos === false ? $viewPath : (substr($viewPath, 0, $pos).'/views');
         $this->pathMap[$rootPath] = $pathMap;
     
         $this->pathMap['@app/views'] = $pathMap;
@@ -109,7 +109,7 @@ 
      */
     public function getLayoutPath()
     {
-        return $this->viewPath . '/' . self::LAYOUTS_PATH;
+        return $this->viewPath.'/'.self::LAYOUTS_PATH;
     }
     
     /**

core/web/filters/RobotsFilter.php

Spacing +1 added lines, -1 removed lines

@@ -123,7 +123,7 @@
     {
         if (Yii::$app->request->isPost) {
             if ($this->getElapsedProcessTime() < $this->delay) {
-                throw new WhitelistedException("Robots Filter has detected an invalid Request: " . VarDumper::export(ArrayHelper::coverSensitiveValues(Yii::$app->request->post())));
+                throw new WhitelistedException("Robots Filter has detected an invalid Request: ".VarDumper::export(ArrayHelper::coverSensitiveValues(Yii::$app->request->post())));
             }
         }
         

core/theme/ThemeConfig.php

Spacing +1 added lines, -1 removed lines

@@ -140,6 +140,6 @@
      */
     public function getViewPath()
     {
-        return $this->getBasePath() . '/views';
+        return $this->getBasePath().'/views';
     }
 }