luyadev / luya

core/base/Module.php

Doc Comments +1 added lines, -1 removed lines

@@ -198,7 +198,7 @@
      * Define a last of importer class with an array or run code directily with the import() method.
      * 
      * @param ImportControllerInterface $importer
-     * @return boolean|array
+     * @return boolean
      */
     public function import(ImportControllerInterface $importer)
     {

core/console/Importer.php

Doc Comments -1 removed lines

@@ -71,7 +71,6 @@
      * $this->addLog('block', 'new block <ID> have been found and added to database');
      * ```
      * 
-     * @param string $section
      * @param string $value
 	 * @todo trigger deprecated section call
      */

core/console/commands/ImportController.php

Spacing +6 added lines, -6 removed lines

@@ -128,7 +128,7 @@ 
             $this->verbosePrint('Run import index', __METHOD__);
             foreach (Yii::$app->getModules() as $id => $module) {
                 if ($module instanceof \luya\base\Module) {
-                    $this->verbosePrint('collect module importers from module: ' . $id, __METHOD__);
+                    $this->verbosePrint('collect module importers from module: '.$id, __METHOD__);
                     $response = $module->import($this);
                     if (is_array($response)) { // importer returns an array with class names
                         foreach ($response as $class) {
@@ -150,9 +150,9 @@ 
             ksort($queue);
     
             foreach ($queue as $pos => $object) {
-                $this->verbosePrint("run object '" .$object->className() . " on pos $pos.", __METHOD__);
+                $this->verbosePrint("run object '".$object->className()." on pos $pos.", __METHOD__);
                 $objectResponse = $object->run();
-                $this->verbosePrint("run object response: " . var_export($objectResponse, true), __METHOD__);
+                $this->verbosePrint("run object response: ".var_export($objectResponse, true), __METHOD__);
             }
     
             if (Yii::$app->hasModule('admin')) {
@@ -160,15 +160,15 @@ 
                 Yii::$app->db->createCommand()->update('admin_user', ['force_reload' => 1])->execute();
             }
     
-            $this->verbosePrint('importer finished, get log output: ' . var_export($this->getLog(), true), __METHOD__);
+            $this->verbosePrint('importer finished, get log output: '.var_export($this->getLog(), true), __METHOD__);
             
             foreach ($this->getLog() as $section => $value) {
-                $this->outputInfo(PHP_EOL . $section . ":");
+                $this->outputInfo(PHP_EOL.$section.":");
                 foreach ($value as $k => $v) {
                     if (is_array($v)) {
                         $this->output(print_r($v, true));
                     } else {
-                        $this->output(" - " . $v);
+                        $this->output(" - ".$v);
                     }
                 }
             }

core/helpers/StringHelper.php

Spacing +2 added lines, -2 removed lines

@@ -20,9 +20,9 @@
     {
         if (is_numeric($string)) {
             if (is_float($string)) {
-                return (float)$string;
+                return (float) $string;
             } else {
-                return (int)$string;
+                return (int) $string;
             }
         } elseif (is_array($string)) {
             return ArrayHelper::typeCast($string);

luyacheck.php

Indentation +5 added lines, -5 removed lines

@@ -1,10 +1,10 @@
 <?php
 $tests = [
-	"in_array('mod_rewrite', apache_get_modules());",
-	"ini_get('short_open_tag');",
-	"ini_get('error_reporting');",
-	"phpversion()",
-	"php_ini_loaded_file()",
+    "in_array('mod_rewrite', apache_get_modules());",
+    "ini_get('short_open_tag');",
+    "ini_get('error_reporting');",
+    "phpversion()",
+    "php_ini_loaded_file()",
 ];
 foreach ($tests as $i => $test) {
     $result = eval('return ' . $test . ';');

Spacing +1 added lines, -1 removed lines

@@ -7,6 +7,6 @@
 	"php_ini_loaded_file()",
 ];
 foreach ($tests as $i => $test) {
-    $result = eval('return ' . $test . ';');
+    $result = eval('return '.$test.';');
     printf("%2d: [%s] %s<br />", $i + 1, $test, var_export($result, true));
 }
\ No newline at end of file

core/web/Element.php

Spacing +2 added lines, -2 removed lines

@@ -217,12 +217,12 @@
     {
         if ($this->renderEngine == 'php') {
             $view = new View();
-            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);
         } elseif ($this->renderEngine == 'twig') {
             $twig = Yii::$app->twig->env(new Twig_Loader_Filesystem($this->getFolder()));
             return $twig->render(FileHelper::ensureExtension($file, 'twig'), $args);
         }
         
-        throw new Exception('Not supported render engine: ' . $this->renderEngine);
+        throw new Exception('Not supported render engine: '.$this->renderEngine);
     }
 }

core/console/Application.php

Spacing +2 added lines, -2 removed lines

@@ -76,12 +76,12 @@
             if (isset($partial[0]) && (count($partial) > 1) && ($module = Yii::$app->getModule($partial[0]))) {
                 try {
                     // change the controller namespace of this module to make usage of `commands`.
-                    $module->controllerNamespace = $module->namespace . '\commands';
+                    $module->controllerNamespace = $module->namespace.'\commands';
                     unset($partial[0]);
                     // action response
                     return $module->runAction(implode("/", $partial), $params);
                 } catch (\Exception $e) {
-                    return Console::output("Command Exception: '" . $e->getMessage() . "' in file '" . $e->getFile() . "' on line '" . $e->getLine() . "'.");
+                    return Console::output("Command Exception: '".$e->getMessage()."' in file '".$e->getFile()."' on line '".$e->getLine()."'.");
                 }
             }
         }

core/console/commands/BlockController.php

Doc Comments +17 added lines, -3 removed lines

@@ -61,6 +61,11 @@ 
         ];
     }
     
+    /**
+     * @param string $type
+     * @param string $varName
+     * @param string $func
+     */
     private function getExtraVarDef($type, $varName, $func)
     {
         $info = [
@@ -77,6 +82,9 @@ 
         return false;
     }
 
+    /**
+     * @param string $type
+     */
     private function getVariableTypeOption($type)
     {
         $types = $this->getVariableTypesOptions();
@@ -84,6 +92,9 @@ 
         return $types[$type];
     }
 
+    /**
+     * @param string $type
+     */
     private function hasVariableTypeOption($type)
     {
         return array_key_exists($type, $this->getVariableTypesOptions());
@@ -92,7 +103,7 @@ 
     /**
      * Wizzard to create a new CMS block.
      * 
-     * @return number
+     * @return integer
      */
     public function actionCreate()
     {
@@ -302,6 +313,9 @@ 
         return $this->outputError("Error while creating file '$file'");
     }
 
+    /**
+     * @param string $prefix
+     */
     private function placeholderCreator($prefix)
     {
         $this->output(PHP_EOL.'-> Create new '.$prefix, Console::FG_YELLOW);
@@ -320,8 +334,8 @@ 
 
     /**
      * 
-     * @param unknown $prefix
-     * @param string $type 'var', 'cfg'
+     * @param string $prefix
+     * @param string $typeCast
      * @return multitype:string Ambigous <string, array>
      */
     private function varCreator($prefix, $typeCast)

Spacing +8 added lines, -8 removed lines

@@ -64,11 +64,11 @@ 
     private function getExtraVarDef($type, $varName, $func)
     {
         $info = [
-            'image-upload' => function ($varName) use ($func) { return '$this->zaaImageUpload($this->'.$func.'(\''.$varName.'\')),'; },
-            'image-array-upload' => function ($varName) use ($func) { return '$this->zaaImageArrayUpload($this->'.$func.'(\''.$varName.'\')),'; },
-            'file-upload' => function ($varName) use ($func) { return '$this->zaaFileUpload($this->'.$func.'(\''.$varName.'\')),'; },
-            'file-array-upload' => function ($varName) use ($func) { return '$this->zaaFileArrayUpload($this->'.$func.'(\''.$varName.'\')),'; },
-            'cms-page' => function ($varName) use ($func) { return 'Yii::$app->menu->findOne([\'nav_id\' => $this->'.$func.'(\''.$varName.'\', 0)]),'; },
+            'image-upload' => function($varName) use ($func) { return '$this->zaaImageUpload($this->'.$func.'(\''.$varName.'\')),'; },
+            'image-array-upload' => function($varName) use ($func) { return '$this->zaaImageArrayUpload($this->'.$func.'(\''.$varName.'\')),'; },
+            'file-upload' => function($varName) use ($func) { return '$this->zaaFileUpload($this->'.$func.'(\''.$varName.'\')),'; },
+            'file-array-upload' => function($varName) use ($func) { return '$this->zaaFileArrayUpload($this->'.$func.'(\''.$varName.'\')),'; },
+            'cms-page' => function($varName) use ($func) { return 'Yii::$app->menu->findOne([\'nav_id\' => $this->'.$func.'(\''.$varName.'\', 0)]),'; },
         ];
         
         if (array_key_exists($type, $info)) {
@@ -186,15 +186,15 @@ 
         $content .= '     * @var bool Choose whether block is a layout/container/segmnet/section block or not, Container elements will be optically displayed'.PHP_EOL;
         $content .= '     * in a different way for a better user experience. Container block will not display isDirty colorizing.'.PHP_EOL;
         $content .= '     */'.PHP_EOL;
-        $content .= '    public $isContainer = false;'.PHP_EOL . PHP_EOL;
+        $content .= '    public $isContainer = false;'.PHP_EOL.PHP_EOL;
         $content .= '    /**'.PHP_EOL;
         $content .= '     * @var bool Choose whether a block can be cached trough the caching component. Be carefull with caching container blocks.'.PHP_EOL;
         $content .= '     */'.PHP_EOL;
-        $content .= '    public $cacheEnabled = false;'.PHP_EOL . PHP_EOL;
+        $content .= '    public $cacheEnabled = false;'.PHP_EOL.PHP_EOL;
         $content .= '    /**'.PHP_EOL;
         $content .= '     * @var int The cache lifetime for this block in seconds (3600 = 1 hour), only affects when cacheEnabled is true'.PHP_EOL;
         $content .= '     */'.PHP_EOL;
-        $content .= '    public $cacheExpiration = 3600;'.PHP_EOL . PHP_EOL;
+        $content .= '    public $cacheExpiration = 3600;'.PHP_EOL.PHP_EOL;
         
         // method name
         $content .= '    public function name()'.PHP_EOL;

core/console/commands/ModuleController.php

Doc Comments +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
     /**
      * Create a new frontend/admin module.
      * 
-     * @return number
+     * @return integer
      */
     public function actionCreate()
     {