luyadev / luya

core/base/Boot.php

Doc Comments +2 added lines, -2 removed lines

@@ -100,7 +100,7 @@ 
     /**
      * Run the application based on the Sapi Name.
      *
-     * @return luya\web\Application|luya\cli\Application Application objected based on the sapi name.
+     * @return null|integer Application objected based on the sapi name.
      */
     public function run()
     {
@@ -133,7 +133,7 @@ 
     /**
      * Run Web-Application based on the provided config file.
      * 
-     * @return string|void Returns the Yii Application run() method if mock is disabled. Otherwise returns void
+     * @return integer|null Returns the Yii Application run() method if mock is disabled. Otherwise returns void
      */
     public function applicationWeb()
     {

Indentation +15 added lines, -15 removed lines

@@ -91,7 +91,7 @@ 
      */
     public function setConfigArray(array $config)
     {
-    	$this->_configArray = $config;
+        $this->_configArray = $config;
     }
     
     /**
@@ -103,24 +103,24 @@ 
      */
     public function getConfigArray()
     {
-    	if ($this->_configArray === null) {
-	        if (!file_exists($this->configFile)) {
-	            throw new Exception("Unable to load the config file '".$this->configFile."'.");
-	        }
+        if ($this->_configArray === null) {
+            if (!file_exists($this->configFile)) {
+                throw new Exception("Unable to load the config file '".$this->configFile."'.");
+            }
 	
-	        $config = require $this->configFile;
+            $config = require $this->configFile;
 	
-	        if (!is_array($config)) {
-	            throw new Exception("config file '".$this->configFile."' found but no array returning.");
-	        }
+            if (!is_array($config)) {
+                throw new Exception("config file '".$this->configFile."' found but no array returning.");
+            }
 	
-	        // adding default configuration timezone if not set
-	        if (!array_key_exists('timezone', $config)) {
-	            $config['timezone'] = 'Europe/Berlin';
-	        }
+            // adding default configuration timezone if not set
+            if (!array_key_exists('timezone', $config)) {
+                $config['timezone'] = 'Europe/Berlin';
+            }
 	     
-	        $this->_configArray = $config;
-    	}
+            $this->_configArray = $config;
+        }
 
         return $this->_configArray;
     }

core/console/commands/BlockController.php

Doc Comments +16 added lines, -2 removed lines

@@ -56,6 +56,11 @@ 
         ];
     }
     
+    /**
+     * @param string $type
+     * @param string $varName
+     * @param string $func
+     */
     private function getExtraVarDef($type, $varName, $func)
     {
         $info = [
@@ -72,6 +77,9 @@ 
         return false;
     }
 
+    /**
+     * @param string $type
+     */
     private function getVariableTypeOption($type)
     {
         $types = $this->getVariableTypesOptions();
@@ -79,6 +87,9 @@ 
         return $types[$type];
     }
 
+    /**
+     * @param string $type
+     */
     private function hasVariableTypeOption($type)
     {
         return array_key_exists($type, $this->getVariableTypesOptions());
@@ -292,6 +303,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);
@@ -310,8 +324,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)

core/console/commands/MigrateController.php

Doc Comments +5 added lines

@@ -42,6 +42,11 @@
         }
     }
 
+    /**
+     * @param string|boolean $module
+     *
+     * @return string
+     */
     private function getModuleMigrationDirectorie($module)
     {
         if (!array_key_exists($module, $this->moduleMigrationDirectories)) {

core/helpers/Url.php

Doc Comments +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
      *
      * @param string $route
      * @param array  $params
-     * @param boolean $sheme Whether to return static url or not
+     * @param boolean $scheme Whether to return static url or not
      * @todo we have to remove this method as it provides no additinal functions to the yii\helpers\url to method
      */
     public static function toManager($route, array $params = [], $scheme = false)

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/traits/CacheableTrait.php

Doc Comments +1 added lines

@@ -73,6 +73,7 @@
      * @param string $key The identifier key
      * @param mixed $value The value to store in the cache component.
      * @param \yii\caching\Dependency $dependency Dependency of the cached item. If the dependency changes, the corresponding value in the cache will be invalidated when it is fetched via get(). This parameter is ignored if $serializer is false.
+     * @param integer $cacheExpiration
      * @return void
      */
     public function setHasCache($key, $value, $dependency = null, $cacheExpiration = null)

core/web/UrlManager.php

Doc Comments +7 added lines, -3 removed lines

@@ -37,6 +37,7 @@ 
      *
      * {@inheritDoc}
      * @see \yii\web\UrlManager::parseRequest()
+     * @param Request $request
      */
     public function parseRequest($request)
     {
@@ -141,7 +142,7 @@ 
      * Remove the base url from a route
      *
      * @param string $route The route where the baseUrl should be removed from.
-     * @return mixed
+     * @return string
      */
     public function removeBaseUrl($route)
     {
@@ -189,7 +190,7 @@ 
      * Yii2 createUrl base implementation extends the prepand of the comosition
      *
      * @param string|array $params An array with params or not (e.g. `['module/controller/action', 'param1' => 'value1']`)
-     * @param null|object $composition Composition instance to change the route behavior
+     * @param Composition $composition Composition instance to change the route behavior
      * @return string
      */
     public function internalCreateUrl($params, $composition = null)
@@ -241,7 +242,7 @@ 
      * 
      * The module to test must be an instance of `luya\base\Module`.
      * 
-     * @param unknown $route
+     * @param string $route
      * @return boolean|string
      */
     private function findModuleInRoute($route)
@@ -255,6 +256,9 @@ 
         return false;
     }
     
+    /**
+     * @param string $url
+     */
     private function urlReplaceModule($url, $navItemId, Composition $composition)
     {
         $route = $composition->removeFrom($this->removeBaseUrl($url));

core/helpers/StringHelper.php

Doc Comments +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
      * @param string $from
      * @param string $to
      * @param string $subject
-     * @return mixed
+     * @return string
      * @since 1.0.0-rc1
      */
     public static function replaceFirst($from, $to, $subject)

core/console/Application.php

Indentation +20 added lines, -20 removed lines

@@ -25,26 +25,26 @@
 
     public function coreCommands()
     {
-    	return [
-    		// yii default commands
-    		'asset' => 'yii\console\controllers\AssetController',
-    		'cache' => 'yii\console\controllers\CacheController',
-    		'fixture' => 'yii\console\controllers\FixtureController',
-    		'help' => 'yii\console\controllers\HelpController',
-    		'message' => 'yii\console\controllers\MessageController',
-    		'serve' => 'yii\console\controllers\ServeController',
-    		// luya default commands
-    		'migrate' => 'luya\console\commands\MigrateController',
-	        'crud' => 'luya\console\commands\CrudController',
-	        'module' => 'luya\console\commands\ModuleController',
-	        'command' => 'luya\console\commands\CommandController',
-	        'import' => 'luya\console\commands\ImportController',
-	        'setup' => 'luya\console\commands\SetupController',
-	        'health' => 'luya\console\commands\HealthController',
-	        'block' => 'luya\console\commands\BlockController',
-	        'storage' => 'luya\console\commands\StorageController',
-	        'aw' => 'luya\console\commands\ActiveWindowController',
-    	];
+        return [
+            // yii default commands
+            'asset' => 'yii\console\controllers\AssetController',
+            'cache' => 'yii\console\controllers\CacheController',
+            'fixture' => 'yii\console\controllers\FixtureController',
+            'help' => 'yii\console\controllers\HelpController',
+            'message' => 'yii\console\controllers\MessageController',
+            'serve' => 'yii\console\controllers\ServeController',
+            // luya default commands
+            'migrate' => 'luya\console\commands\MigrateController',
+            'crud' => 'luya\console\commands\CrudController',
+            'module' => 'luya\console\commands\ModuleController',
+            'command' => 'luya\console\commands\CommandController',
+            'import' => 'luya\console\commands\ImportController',
+            'setup' => 'luya\console\commands\SetupController',
+            'health' => 'luya\console\commands\HealthController',
+            'block' => 'luya\console\commands\BlockController',
+            'storage' => 'luya\console\commands\StorageController',
+            'aw' => 'luya\console\commands\ActiveWindowController',
+        ];
     }
     
     /**