luyadev / luya

core/helpers/ArrayHelper.php

Spacing +3 added lines, -3 removed lines

@@ -50,9 +50,9 @@ 
         foreach ($array as $k => $v) {
             if (is_numeric($v)) {
                 if (is_float($v)) {
-                    $return[$k] = (float)$v;
+                    $return[$k] = (float) $v;
                 } else {
-                    $return[$k] = (int)$v;
+                    $return[$k] = (int) $v;
                 }
             } elseif (is_array($v)) {
                 $return[$k] = self::typeCast($v);
@@ -92,7 +92,7 @@ 
     public static function search($array, $searchText, $sensitive = false)
     {
         $function = ($sensitive) ? 'strpos' : 'stripos';
-        return array_filter($array, function ($item) use ($searchText, $function) {
+        return array_filter($array, function($item) use ($searchText, $function) {
             $response = false;
             foreach ($item as $key => $value) {
                 if ($response) {

core/console/commands/BaseCrudController.php

Spacing +7 added lines, -7 removed lines

@@ -74,10 +74,10 @@ 
         }
         $rules = [];
         foreach ($types as $type => $columns) {
-            $rules[] = "[['" . implode("', '", $columns) . "'], '$type']";
+            $rules[] = "[['".implode("', '", $columns)."'], '$type']";
         }
         foreach ($lengths as $length => $columns) {
-            $rules[] = "[['" . implode("', '", $columns) . "'], 'string', 'max' => $length]";
+            $rules[] = "[['".implode("', '", $columns)."'], 'string', 'max' => $length]";
         }
         $db = $this->getDbConnection();
         // Unique indexes rules
@@ -88,12 +88,12 @@ 
                 if (!$this->isColumnAutoIncremental($table, $uniqueColumns)) {
                     $attributesCount = count($uniqueColumns);
                     if ($attributesCount === 1) {
-                        $rules[] = "[['" . $uniqueColumns[0] . "'], 'unique']";
+                        $rules[] = "[['".$uniqueColumns[0]."'], 'unique']";
                     } elseif ($attributesCount > 1) {
                         $labels = array_intersect_key($this->generateLabels($table), array_flip($uniqueColumns));
                         $lastLabel = array_pop($labels);
                         $columnsList = implode("', '", $uniqueColumns);
-                        $rules[] = "[['$columnsList'], 'unique', 'targetAttribute' => ['$columnsList'], 'message' => 'The combination of " . implode(', ', $labels) . " and $lastLabel has already been taken.']";
+                        $rules[] = "[['$columnsList'], 'unique', 'targetAttribute' => ['$columnsList'], 'message' => 'The combination of ".implode(', ', $labels)." and $lastLabel has already been taken.']";
                     }
                 }
             }
@@ -136,7 +136,7 @@ 
         $fullTableName = $tableName;
         if (($pos = strrpos($tableName, '.')) !== false) {
             if (($useSchemaName === null && $this->useSchemaName) || $useSchemaName) {
-                $schemaName = substr($tableName, 0, $pos) . '_';
+                $schemaName = substr($tableName, 0, $pos).'_';
             }
             $tableName = substr($tableName, $pos + 1);
         }
@@ -149,7 +149,7 @@ 
             if (($pos = strrpos($pattern, '.')) !== false) {
                 $pattern = substr($pattern, $pos + 1);
             }
-            $patterns[] = '/^' . str_replace('*', '(\w+)', $pattern) . '$/';
+            $patterns[] = '/^'.str_replace('*', '(\w+)', $pattern).'$/';
         }
         $className = $tableName;
         foreach ($patterns as $pattern) {
@@ -193,7 +193,7 @@ 
             } else {
                 $label = Inflector::camel2words($column->name);
                 if (!empty($label) && substr_compare($label, ' id', -3, 3, true) === 0) {
-                    $label = substr($label, 0, -3) . ' ID';
+                    $label = substr($label, 0, -3).' ID';
                 }
                 $labels[$column->name] = $label;
             }

core/console/commands/views/block/create_block.php

Spacing +2 added lines, -2 removed lines

@@ -88,8 +88,8 @@
     {
         return [
 <?php foreach ($extras as $extra):?>
-            <?= $extra;?>
-<?php endforeach;?>
+            <?= $extra; ?>
+<?php endforeach; ?>
 
         ];
     }

core/console/commands/CrudController.php

Indentation +1 added lines, -1 removed lines

@@ -279,7 +279,7 @@
                 $this->getModelNameCamlized(),
                 $this->apiEndpoint,
                 $this->getDbTableShema()
-             ),
+                ),
         ];
         
         foreach ($files as $file) {

Doc Comments +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
     /**
      * Get the $moduleName without admin suffix (if any).
      *
-     * @return mixed Return the module name without admin suffix.
+     * @return string Return the module name without admin suffix.
      */
     public function getModuleNameWithoutAdminSuffix()
     {

Unused Use Statements -1 removed lines

@@ -6,7 +6,6 @@
 use yii\helpers\Console;
 use yii\helpers\Inflector;
 use yii\db\TableSchema;
-use luya\Boot;
 use luya\helpers\FileHelper;
 
 /**

Spacing +14 added lines, -14 removed lines

@@ -79,7 +79,7 @@ 
      */
     public function getAbsoluteModelNamespace()
     {
-        return $this->getModelNamespace() . '\\models\\' . $this->getModelNameCamlized();
+        return $this->getModelNamespace().'\\models\\'.$this->getModelNameCamlized();
     }
     
     /**
@@ -375,7 +375,7 @@ 
                 $sqlTable = $this->prompt('Database Table name for the Model:', ['required' => true, 'default' => $this->getDatabaseNameSuggestion()]);
                 if ($sqlTable == '?') {
                     foreach ($this->getSqlTablesArray() as $table) {
-                        $this->outputInfo("- " . $table);
+                        $this->outputInfo("- ".$table);
                     }
                 }
                 if (isset($this->getSqlTablesArray()[$sqlTable])) {
@@ -398,26 +398,26 @@ 
         // api content
 
         $files['api'] = [
-            'path' => $this->getBasePath() . DIRECTORY_SEPARATOR . 'apis',
-            'fileName' => $this->getModelNameCamlized() . 'Controller.php',
-            'content' => $this->generateApiContent($this->getNamespace() . '\\apis', $this->getModelNameCamlized() . 'Controller', $this->getAbsoluteModelNamespace()),
+            'path' => $this->getBasePath().DIRECTORY_SEPARATOR.'apis',
+            'fileName' => $this->getModelNameCamlized().'Controller.php',
+            'content' => $this->generateApiContent($this->getNamespace().'\\apis', $this->getModelNameCamlized().'Controller', $this->getAbsoluteModelNamespace()),
         ];
         
         // controller
 
         $files['controller'] = [
-            'path' =>  $this->getBasePath() . DIRECTORY_SEPARATOR . 'controllers',
-            'fileName' => $this->getModelNameCamlized() . 'Controller.php',
-            'content' => $this->generateControllerContent($this->getNamespace() . '\\controllers', $this->getModelNameCamlized() . 'Controller', $this->getAbsoluteModelNamespace()),
+            'path' =>  $this->getBasePath().DIRECTORY_SEPARATOR.'controllers',
+            'fileName' => $this->getModelNameCamlized().'Controller.php',
+            'content' => $this->generateControllerContent($this->getNamespace().'\\controllers', $this->getModelNameCamlized().'Controller', $this->getAbsoluteModelNamespace()),
         ];
         
         // model
 
         $files['model'] = [
-            'path' =>  $this->getModelBasePath() . DIRECTORY_SEPARATOR . 'models',
-            'fileName' => $this->getModelNameCamlized() . '.php',
+            'path' =>  $this->getModelBasePath().DIRECTORY_SEPARATOR.'models',
+            'fileName' => $this->getModelNameCamlized().'.php',
             'content' => $this->generateModelContent(
-                $this->getModelNamespace() . '\\models',
+                $this->getModelNamespace().'\\models',
                 $this->getModelNameCamlized(),
                 $this->apiEndpoint,
                 $this->getDbTableShema(),
@@ -427,19 +427,19 @@ 
         
         foreach ($files as $file) {
             FileHelper::createDirectory($file['path']);
-            if (file_exists($file['path'] . DIRECTORY_SEPARATOR . $file['fileName'])) {
+            if (file_exists($file['path'].DIRECTORY_SEPARATOR.$file['fileName'])) {
                 if (!$this->confirm("The File '{$file['fileName']}' already exists, do you want to override the existing file?")) {
                     continue;
                 }
             }
             
-            if (FileHelper::writeFile($file['path'] . DIRECTORY_SEPARATOR . $file['fileName'], $file['content'])) {
+            if (FileHelper::writeFile($file['path'].DIRECTORY_SEPARATOR.$file['fileName'], $file['content'])) {
                 $this->outputSuccess("Wrote file '{$file['fileName']}'.");
             } else {
                 $this->outputError("Error while writing file '{$file['fileName']}'.");
             }
         }
         
-        return $this->outputSuccess($this->generateBuildSummery($this->apiEndpoint, $this->getNamespace() . '\\apis\\' . $this->getModelNameCamlized() . 'Controller', $this->getModelNameCamlized(), $this->getSummaryControllerRoute()));
+        return $this->outputSuccess($this->generateBuildSummery($this->apiEndpoint, $this->getNamespace().'\\apis\\'.$this->getModelNameCamlized().'Controller', $this->getModelNameCamlized(), $this->getSummaryControllerRoute()));
     }
 }

core/console/commands/views/crud/create_controller.php

Spacing +1 added lines, -1 removed lines

@@ -19,5 +19,5 @@
     /**
      * @var string $modelClass The path to the model which is the provider for the rules and fields.
      */
-    public $modelClass = '<?= $modelClass;?>';
+    public $modelClass = '<?= $modelClass; ?>';
 }
\ No newline at end of file

core/console/commands/views/crud/create_api.php

Spacing +1 added lines, -1 removed lines

@@ -19,5 +19,5 @@
     /**
      * @var string $modelClass The path to the model which is the provider for the rules and fields.
      */
-    public $modelClass = '<?= $modelClass;?>';
+    public $modelClass = '<?= $modelClass; ?>';
 }
\ No newline at end of file

core/console/commands/views/crud/create_model.php

Spacing +4 added lines, -4 removed lines

@@ -10,8 +10,8 @@ 
 /**
  * NgRest Model created with LUYA Version <?php echo $luyaVersion; ?>.
  *
-<?php foreach ($properties as $name => $type): ?> * @property <?= $type; ?> $<?= $name . PHP_EOL; ?>
-<?php endforeach;?>
+<?php foreach ($properties as $name => $type): ?> * @property <?= $type; ?> $<?= $name.PHP_EOL; ?>
+<?php endforeach; ?>
  */
 class <?= $className; ?> extends NgRestModel
 {
@@ -51,7 +51,7 @@ 
      */
     public $i18n = ['<?= implode("', '", $textFields); ?>'];
 
-<?php endif;?>
+<?php endif; ?>
     /**
      * @inheritdoc
      */
@@ -86,7 +86,7 @@ 
     {
         return [
         <?php foreach ($ngrestFieldConfig as $name => $type): ?>
-    '<?=$name; ?>' => '<?= $type;?>',
+    '<?=$name; ?>' => '<?= $type; ?>',
         <?php endforeach; ?>];
     }
     

core/console/commands/BlockController.php

Spacing +12 added lines, -12 removed lines

@@ -146,19 +146,19 @@ 
     private function getExtraVarDef($type, $varName, $func)
     {
         $info = [
-            'image-upload' => function ($varName) use ($func) {
+            'image-upload' => function($varName) use ($func) {
                 return 'BlockHelper::imageUpload($this->'.$func.'(\''.$varName.'\'), false, true),';
             },
-            'image-array-upload' => function ($varName) use ($func) {
+            'image-array-upload' => function($varName) use ($func) {
                 return 'BlockHelper::imageArrayUpload($this->'.$func.'(\''.$varName.'\'), false, true),';
             },
-            'file-upload' => function ($varName) use ($func) {
+            'file-upload' => function($varName) use ($func) {
                 return 'BlockHelper::fileUpload($this->'.$func.'(\''.$varName.'\'), true),';
             },
-            'file-array-upload' => function ($varName) use ($func) {
+            'file-array-upload' => function($varName) use ($func) {
                 return 'BlockHelper::fileArrayUpload($this->'.$func.'(\''.$varName.'\'), true),';
             },
-            'cms-page' => function ($varName) use ($func) {
+            'cms-page' => function($varName) use ($func) {
                 return 'Yii::$app->menu->findOne([\'nav_id\' => $this->'.$func.'(\''.$varName.'\', 0)]),';
             },
         ];
@@ -246,7 +246,7 @@ 
             return 'app\\blocks';
         }
 
-        return Yii::$app->getModule($this->moduleName)->getNamespace()  . '\\blocks';
+        return Yii::$app->getModule($this->moduleName)->getNamespace().'\\blocks';
     }
 
     protected function getFileBasePath()
@@ -343,8 +343,8 @@ 
             }
         }
         
-        $folder = $this->getFileBasePath() . DIRECTORY_SEPARATOR . 'blocks';
-        $filePath = $folder . DIRECTORY_SEPARATOR . $this->blockName . '.php';
+        $folder = $this->getFileBasePath().DIRECTORY_SEPARATOR.'blocks';
+        $filePath = $folder.DIRECTORY_SEPARATOR.$this->blockName.'.php';
         
         sort($this->phpdoc);
         
@@ -368,14 +368,14 @@ 
         if (FileHelper::createDirectory($folder) && FileHelper::writeFile($filePath, $content)) {
             
             // generate view file based on block object view context
-            $object = Yii::createObject(['class' => $this->getFileNamespace() . '\\' . $this->blockName]);
+            $object = Yii::createObject(['class' => $this->getFileNamespace().'\\'.$this->blockName]);
             $viewsFolder = Yii::getAlias($object->getViewPath());
-            $viewFilePath = $viewsFolder . DIRECTORY_SEPARATOR . $object->getViewFileName('php');
+            $viewFilePath = $viewsFolder.DIRECTORY_SEPARATOR.$object->getViewFileName('php');
             if (FileHelper::createDirectory($viewsFolder) && FileHelper::writeFile($viewFilePath, $this->generateViewFile($this->blockName))) {
-                $this->outputInfo('View file for the block has been created: ' . $viewFilePath);
+                $this->outputInfo('View file for the block has been created: '.$viewFilePath);
             }
             
-            return $this->outputSuccess("Block {$this->blockName} has been created: " . $filePath);
+            return $this->outputSuccess("Block {$this->blockName} has been created: ".$filePath);
         }
         
         return $this->outputError("Error while creating block '$filePath'");

Doc Comments +14 added lines

@@ -146,6 +146,11 @@ 
         ];
     }
     
+    /**
+     * @param string $type
+     * @param string $varName
+     * @param string $func
+     */
     private function getExtraVarDef($type, $varName, $func)
     {
         $info = [
@@ -173,6 +178,9 @@ 
         return false;
     }
 
+    /**
+     * @param string $type
+     */
     private function getVariableTypeOption($type)
     {
         $types = $this->getVariableTypesOptions();
@@ -180,11 +188,17 @@ 
         return $types[$type];
     }
 
+    /**
+     * @param string $type
+     */
     private function hasVariableTypeOption($type)
     {
         return array_key_exists($type, $this->getVariableTypesOptions());
     }
 
+    /**
+     * @param string $prefix
+     */
     private function placeholderCreator($prefix)
     {
         $this->output(PHP_EOL.'-> Create new '.$prefix, Console::FG_YELLOW);

core/web/Composition.php

Doc Comments +1 added lines, -1 removed lines

@@ -397,7 +397,7 @@
      * 
      * @see ArrayAccess::offsetGet()
      * @param string $offset The key to get from the array.
-     * @return mixed The value for the offset key from the array.
+     * @return string|boolean The value for the offset key from the array.
      */
     public function offsetGet($offset)
     {