@@ -137,7 +137,7 @@ |
||
137 | 137 | /** |
138 | 138 | * Get a list of users of which the given conditions match with a LIKE '%cond%' |
139 | 139 | * @param array $conditions a list of condition (exemple : status=>STUDENT) |
140 | - * @param array $order_by a list of fields on which sort |
|
140 | + * @param string[] $order_by a list of fields on which sort |
|
141 | 141 | * @return array An array with all users of the platform. |
142 | 142 | * @todo optional course code parameter, optional sorting parameters... |
143 | 143 | *@todo Use the UserManager class |
@@ -370,7 +370,7 @@ discard block |
||
370 | 370 | * Register a user into a queue for a session |
371 | 371 | * @param $userId |
372 | 372 | * @param $sessionId |
373 | - * @return bool|int |
|
373 | + * @return false|string |
|
374 | 374 | */ |
375 | 375 | public function addToQueue($userId, $sessionId) |
376 | 376 | { |
@@ -396,7 +396,7 @@ discard block |
||
396 | 396 | * Register message with type and status |
397 | 397 | * @param $mailId |
398 | 398 | * @param $userId |
399 | - * @param $sessionId |
|
399 | + * @param integer $sessionId |
|
400 | 400 | * @return bool|int |
401 | 401 | */ |
402 | 402 | public function saveLastMessage($mailId, $userId, $sessionId) |
@@ -509,7 +509,7 @@ discard block |
||
509 | 509 | |
510 | 510 | /** |
511 | 511 | * Check if session is open for subscription |
512 | - * @param $sessionId |
|
512 | + * @param integer $sessionId |
|
513 | 513 | * @param string $fieldVariable |
514 | 514 | * @return bool |
515 | 515 | */ |
@@ -531,8 +531,8 @@ discard block |
||
531 | 531 | |
532 | 532 | /** |
533 | 533 | * Check if user is in the session's target group based on its area |
534 | - * @param $userId |
|
535 | - * @param $sessionId |
|
534 | + * @param integer $userId |
|
535 | + * @param integer $sessionId |
|
536 | 536 | * @param string $userFieldVariable |
537 | 537 | * @param string $sessionFieldVariable |
538 | 538 | * @return bool |
@@ -1002,7 +1002,7 @@ discard block |
||
1002 | 1002 | /** |
1003 | 1003 | * Return the session details data from a session ID (including the extra |
1004 | 1004 | * fields used for the advanced subscription mechanism) |
1005 | - * @param $sessionId |
|
1005 | + * @param integer $sessionId |
|
1006 | 1006 | * @return bool|mixed |
1007 | 1007 | */ |
1008 | 1008 | public function getSessionDetails($sessionId) |
@@ -1102,7 +1102,7 @@ discard block |
||
1102 | 1102 | |
1103 | 1103 | /** |
1104 | 1104 | * Return the url to go to session |
1105 | - * @param $sessionId |
|
1105 | + * @param integer $sessionId |
|
1106 | 1106 | * |
1107 | 1107 | * @return string |
1108 | 1108 | */ |
@@ -213,7 +213,8 @@ |
||
213 | 213 | /** |
214 | 214 | * Returns a meeting "join" URL |
215 | 215 | * @param string The name of the meeting (usually the course code) |
216 | - * @return mixed The URL to join the meeting, or false on error |
|
216 | + * @param false|string $meetingId |
|
217 | + * @return false|null The URL to join the meeting, or false on error |
|
217 | 218 | * @todo implement moderator pass |
218 | 219 | * @assert ('') === false |
219 | 220 | * @assert ('abcdefghijklmnopqrstuvwxyzabcdefghijklmno') === false |
@@ -36,6 +36,11 @@ |
||
36 | 36 | private $_pass; |
37 | 37 | private $_url; |
38 | 38 | |
39 | + /** |
|
40 | + * @param string $host |
|
41 | + * @param string $user |
|
42 | + * @param string $pass |
|
43 | + */ |
|
39 | 44 | public function __construct($host, $user, $pass) |
40 | 45 | { |
41 | 46 | $this->_user = urlencode($user); |
@@ -27,6 +27,9 @@ discard block |
||
27 | 27 | */ |
28 | 28 | class OpenMeetingsRestService |
29 | 29 | { |
30 | + /** |
|
31 | + * @param string $request |
|
32 | + */ |
|
30 | 33 | function call($request, $returnAttribute = "return") |
31 | 34 | { |
32 | 35 | // This will allow you to view errors in the browser |
@@ -121,6 +124,9 @@ discard block |
||
121 | 124 | |
122 | 125 | } |
123 | 126 | |
127 | + /** |
|
128 | + * @param DOMDocument $node |
|
129 | + */ |
|
124 | 130 | function getArray($node) |
125 | 131 | { |
126 | 132 | if (is_null($node) || !is_object($node)) { |
@@ -99,7 +99,7 @@ |
||
99 | 99 | /** |
100 | 100 | * Gets a string from a boolean attribute |
101 | 101 | * @param string $attribute Name of the attribute |
102 | - * @param mixed $voidReturn What to return if the value is not defined |
|
102 | + * @param string $voidReturn What to return if the value is not defined |
|
103 | 103 | * @return string The boolean value expressed as string ('true' or 'false') |
104 | 104 | */ |
105 | 105 | public function getString($attribute, $voidReturn = false) |
@@ -40,11 +40,19 @@ |
||
40 | 40 | return isset($_GET[$key]) ? $_GET[$key] : $default; |
41 | 41 | } |
42 | 42 | |
43 | + /** |
|
44 | + * @param string $key |
|
45 | + * |
|
46 | + * @return string |
|
47 | + */ |
|
43 | 48 | public static function server($key, $default = '') |
44 | 49 | { |
45 | 50 | return isset($_SERVER[$key]) ? $_SERVER[$key] : $default; |
46 | 51 | } |
47 | 52 | |
53 | + /** |
|
54 | + * @param string $name |
|
55 | + */ |
|
48 | 56 | public static function get_lang($name) |
49 | 57 | { |
50 | 58 | return SearchCoursePlugin::create()->get_lang($name); |
@@ -94,7 +94,7 @@ |
||
94 | 94 | /** |
95 | 95 | * @param array $templates |
96 | 96 | * |
97 | - * @return null |
|
97 | + * @return null|string |
|
98 | 98 | */ |
99 | 99 | public function formatTemplates($templates) |
100 | 100 | { |
@@ -84,7 +84,7 @@ |
||
84 | 84 | |
85 | 85 | /** |
86 | 86 | * Available driver list. |
87 | - * @return array |
|
87 | + * @return string[] |
|
88 | 88 | */ |
89 | 89 | private function getDefaultDriverList() |
90 | 90 | { |