@@ -129,7 +129,7 @@ |
||
129 | 129 | /** |
130 | 130 | * Assigns a Smarty variable |
131 | 131 | * |
132 | - * @param array|string $name the template variable name(s) |
|
132 | + * @param string $name the template variable name(s) |
|
133 | 133 | * @param mixed $value the value to assign |
134 | 134 | */ |
135 | 135 | abstract protected function assign($name, $value); |
@@ -47,7 +47,7 @@ |
||
47 | 47 | /** |
48 | 48 | * Gets the default route if no explicit route is requested. |
49 | 49 | * |
50 | - * @return callable |
|
50 | + * @return string[] |
|
51 | 51 | */ |
52 | 52 | protected function getDefaultRoute() |
53 | 53 | { |
@@ -435,7 +435,7 @@ |
||
435 | 435 | } |
436 | 436 | |
437 | 437 | /** |
438 | - * @return callable |
|
438 | + * @return string[] |
|
439 | 439 | */ |
440 | 440 | protected function getDefaultRoute() |
441 | 441 | { |
@@ -24,6 +24,10 @@ discard block |
||
24 | 24 | |
25 | 25 | trait NavigationMenuAccessControl |
26 | 26 | { |
27 | + /** |
|
28 | + * @param string $name |
|
29 | + * @param boolean $value |
|
30 | + */ |
|
27 | 31 | protected abstract function assign($name, $value); |
28 | 32 | |
29 | 33 | /** |
@@ -32,7 +36,7 @@ discard block |
||
32 | 36 | protected abstract function getSecurityManager(); |
33 | 37 | |
34 | 38 | /** |
35 | - * @param $currentUser |
|
39 | + * @param \Waca\DataObjects\User $currentUser |
|
36 | 40 | */ |
37 | 41 | protected function setupNavMenuAccess($currentUser) |
38 | 42 | { |
@@ -44,6 +44,9 @@ discard block |
||
44 | 44 | } |
45 | 45 | } |
46 | 46 | |
47 | + /** |
|
48 | + * @return string |
|
49 | + */ |
|
47 | 50 | protected abstract function getRegistrationTemplate(); |
48 | 51 | |
49 | 52 | protected function isProtectedPage() |
@@ -70,12 +73,12 @@ discard block |
||
70 | 73 | } |
71 | 74 | |
72 | 75 | /** |
73 | - * @param $emailAddress |
|
74 | - * @param $password |
|
75 | - * @param $username |
|
76 | + * @param null|string $emailAddress |
|
77 | + * @param null|string $password |
|
78 | + * @param null|string $username |
|
76 | 79 | * @param $useOAuthSignup |
77 | - * @param $confirmationId |
|
78 | - * @param $onwikiUsername |
|
80 | + * @param null|integer $confirmationId |
|
81 | + * @param null|string $onwikiUsername |
|
79 | 82 | * |
80 | 83 | * @throws ApplicationLogicException |
81 | 84 | */ |
@@ -233,6 +233,9 @@ |
||
233 | 233 | Logger::backgroundJobComplete($this->getDatabase(), $this->getJob()); |
234 | 234 | } |
235 | 235 | |
236 | + /** |
|
237 | + * @param string $reason |
|
238 | + */ |
|
236 | 239 | protected function markCancelled($reason = null) |
237 | 240 | { |
238 | 241 | $this->job->setStatus(JobQueue::STATUS_CANCELLED); |
@@ -105,6 +105,9 @@ discard block |
||
105 | 105 | return $this->getMediaWikiHelper()->checkAccountExists($name); |
106 | 106 | } |
107 | 107 | |
108 | + /** |
|
109 | + * @param string $reason |
|
110 | + */ |
|
108 | 111 | protected function markFailed($reason = null) |
109 | 112 | { |
110 | 113 | $this->request->setStatus(RequestStatus::HOSPITAL); |
@@ -116,7 +119,7 @@ discard block |
||
116 | 119 | } |
117 | 120 | |
118 | 121 | /** |
119 | - * @param $user |
|
122 | + * @param User $user |
|
120 | 123 | * |
121 | 124 | * @throws ApplicationLogicException |
122 | 125 | */ |
@@ -225,7 +225,7 @@ |
||
225 | 225 | } |
226 | 226 | |
227 | 227 | /** |
228 | - * @return mixed |
|
228 | + * @return string |
|
229 | 229 | */ |
230 | 230 | public function getError() |
231 | 231 | { |
@@ -83,7 +83,7 @@ discard block |
||
83 | 83 | #region properties |
84 | 84 | |
85 | 85 | /** |
86 | - * @return mixed |
|
86 | + * @return integer |
|
87 | 87 | */ |
88 | 88 | public function getUserId() |
89 | 89 | { |
@@ -99,7 +99,7 @@ discard block |
||
99 | 99 | } |
100 | 100 | |
101 | 101 | /** |
102 | - * @return mixed |
|
102 | + * @return string |
|
103 | 103 | */ |
104 | 104 | public function getToken() |
105 | 105 | { |
@@ -115,7 +115,7 @@ discard block |
||
115 | 115 | } |
116 | 116 | |
117 | 117 | /** |
118 | - * @return mixed |
|
118 | + * @return string |
|
119 | 119 | */ |
120 | 120 | public function getSecret() |
121 | 121 | { |
@@ -131,7 +131,7 @@ discard block |
||
131 | 131 | } |
132 | 132 | |
133 | 133 | /** |
134 | - * @return mixed |
|
134 | + * @return string |
|
135 | 135 | */ |
136 | 136 | public function getType() |
137 | 137 | { |
@@ -139,7 +139,7 @@ discard block |
||
139 | 139 | } |
140 | 140 | |
141 | 141 | /** |
142 | - * @param mixed $type |
|
142 | + * @param string $type |
|
143 | 143 | */ |
144 | 144 | public function setType($type) |
145 | 145 | { |