Completed
Pull Request — master (#825)
by Martin
03:03
created
core/base/Boot.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -100,7 +100,7 @@  discard block
 block discarded – undo
100 100
     /**
101 101
      * Run the application based on the Sapi Name.
102 102
      *
103
-     * @return luya\web\Application|luya\cli\Application Application objected based on the sapi name.
103
+     * @return null|integer Application objected based on the sapi name.
104 104
      */
105 105
     public function run()
106 106
     {
@@ -133,7 +133,7 @@  discard block
 block discarded – undo
133 133
     /**
134 134
      * Run Web-Application based on the provided config file.
135 135
      * 
136
-     * @return string|void Returns the Yii Application run() method if mock is disabled. Otherwise returns void
136
+     * @return integer|null Returns the Yii Application run() method if mock is disabled. Otherwise returns void
137 137
      */
138 138
     public function applicationWeb()
139 139
     {
Please login to merge, or discard this patch.
core/base/Module.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -196,7 +196,7 @@
 block discarded – undo
196 196
     /**
197 197
      * The import method will be called from exec/import command.
198 198
      *
199
-     * @return void|string
199
+     * @return boolean
200 200
      */
201 201
     public function import(\luya\console\interfaces\ImportController $exec)
202 202
     {
Please login to merge, or discard this patch.
core/console/commands/BlockController.php 1 patch
Doc Comments   +16 added lines, -2 removed lines patch added patch discarded remove patch
@@ -56,6 +56,11 @@  discard block
 block discarded – undo
56 56
         ];
57 57
     }
58 58
     
59
+    /**
60
+     * @param string $type
61
+     * @param string $varName
62
+     * @param string $func
63
+     */
59 64
     private function getExtraVarDef($type, $varName, $func)
60 65
     {
61 66
         $info = [
@@ -72,6 +77,9 @@  discard block
 block discarded – undo
72 77
         return false;
73 78
     }
74 79
 
80
+    /**
81
+     * @param string $type
82
+     */
75 83
     private function getVariableTypeOption($type)
76 84
     {
77 85
         $types = $this->getVariableTypesOptions();
@@ -79,6 +87,9 @@  discard block
 block discarded – undo
79 87
         return $types[$type];
80 88
     }
81 89
 
90
+    /**
91
+     * @param string $type
92
+     */
82 93
     private function hasVariableTypeOption($type)
83 94
     {
84 95
         return array_key_exists($type, $this->getVariableTypesOptions());
@@ -292,6 +303,9 @@  discard block
 block discarded – undo
292 303
         return $this->outputError("Error while creating file '$file'");
293 304
     }
294 305
 
306
+    /**
307
+     * @param string $prefix
308
+     */
295 309
     private function placeholderCreator($prefix)
296 310
     {
297 311
         $this->output(PHP_EOL.'-> Create new '.$prefix, Console::FG_YELLOW);
@@ -310,8 +324,8 @@  discard block
 block discarded – undo
310 324
 
311 325
     /**
312 326
      * 
313
-     * @param unknown $prefix
314
-     * @param string $type 'var', 'cfg'
327
+     * @param string $prefix
328
+     * @param string $typeCast
315 329
      * @return multitype:string Ambigous <string, array>
316 330
      */
317 331
     private function varCreator($prefix, $typeCast)
Please login to merge, or discard this patch.
core/console/commands/HealthController.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -79,7 +79,7 @@
 block discarded – undo
79 79
     /**
80 80
      * Use --verbose=1 to enable smtp debug output
81 81
      * 
82
-     * @return bool|null
82
+     * @return integer|null
83 83
      * @throws Exception On smtp failure
84 84
      */
85 85
     public function actionMailer()
Please login to merge, or discard this patch.
core/console/commands/MigrateController.php 1 patch
Doc Comments   +5 added lines patch added patch discarded remove patch
@@ -42,6 +42,11 @@
 block discarded – undo
42 42
         }
43 43
     }
44 44
 
45
+    /**
46
+     * @param string|boolean $module
47
+     *
48
+     * @return string
49
+     */
45 50
     private function getModuleMigrationDirectorie($module)
46 51
     {
47 52
         if (!array_key_exists($module, $this->moduleMigrationDirectories)) {
Please login to merge, or discard this patch.
core/console/commands/SetupController.php 1 patch
Doc Comments   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -9,6 +9,9 @@  discard block
 block discarded – undo
9 9
 
10 10
 class SetupController extends \luya\console\Command
11 11
 {
12
+    /**
13
+     * @param string $table
14
+     */
12 15
     private function insert($table, $fields)
13 16
     {
14 17
         return Yii::$app->db->createCommand()->insert($table, $fields)->execute();
@@ -84,8 +87,6 @@  discard block
 block discarded – undo
84 87
      * @todo use options instead, override options()
85 88
      * @todo see if admin is availoable
86 89
      *
87
-     * @param string $email
88
-     * @param string $password
89 90
      */
90 91
     public function actionIndex()
91 92
     {
Please login to merge, or discard this patch.
core/helpers/Url.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -37,7 +37,7 @@
 block discarded – undo
37 37
      *
38 38
      * @param string $route
39 39
      * @param array  $params
40
-     * @param boolean $sheme Whether to return static url or not
40
+     * @param boolean $scheme Whether to return static url or not
41 41
      * @todo we have to remove this method as it provides no additinal functions to the yii\helpers\url to method
42 42
      */
43 43
     public static function toManager($route, array $params = [], $scheme = false)
Please login to merge, or discard this patch.
core/web/Composition.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -234,7 +234,7 @@
 block discarded – undo
234 234
     /**
235 235
      * Return a path like string with all composition with trailing slash e.g. us/e.
236 236
      *
237
-     * @return void|string
237
+     * @return string
238 238
      */
239 239
     public function getFull()
240 240
     {
Please login to merge, or discard this patch.
core/web/UrlManager.php 1 patch
Doc Comments   +17 added lines patch added patch discarded remove patch
@@ -28,6 +28,9 @@  discard block
 block discarded – undo
28 28
 
29 29
     private $_composition = null;
30 30
 
31
+    /**
32
+     * @param Request $request
33
+     */
31 34
     public function parseRequest($request)
32 35
     {
33 36
         // extra data from request to composition, which changes the pathInfo of the Request-Object.
@@ -91,6 +94,9 @@  discard block
 block discarded – undo
91 94
         return $this->_menu;
92 95
     }
93 96
 
97
+    /**
98
+     * @return Composition
99
+     */
94 100
     public function getComposition()
95 101
     {
96 102
         if ($this->_composition === null) {
@@ -105,6 +111,9 @@  discard block
 block discarded – undo
105 111
         return rtrim($this->baseUrl, '/').'/'.ltrim($route, '/');
106 112
     }
107 113
 
114
+    /**
115
+     * @return string
116
+     */
108 117
     public function removeBaseUrl($route)
109 118
     {
110 119
         return preg_replace('#'.preg_quote($this->baseUrl).'#', '', $route, 1);
@@ -133,6 +142,11 @@  discard block
 block discarded – undo
133 142
         return $this->urlReplaceModule($url, $navItemId, $composition);
134 143
     }
135 144
 
145
+    /**
146
+     * @param string $route
147
+     *
148
+     * @return string
149
+     */
136 150
     private function findModuleInRoute($route)
137 151
     {
138 152
         $parts = array_values(array_filter(explode('/', $route)));
@@ -146,6 +160,9 @@  discard block
 block discarded – undo
146 160
         return false;
147 161
     }
148 162
 
163
+    /**
164
+     * @param string $url
165
+     */
149 166
     private function urlReplaceModule($url, $navItemId, \luya\web\Composition $composition)
150 167
     {
151 168
         $route = $this->removeBaseUrl($url);
Please login to merge, or discard this patch.