Completed
Pull Request — gcconnex (#1529)
by
unknown
16:45
created
mod/missions_collab/lib/missions.php 1 patch
Doc Comments   +26 added lines patch added patch discarded remove patch
@@ -10,6 +10,12 @@  discard block
 block discarded – undo
10 10
 /*
11 11
  * Packs the language variables for a single language into a single string.
12 12
  */
13
+/**
14
+ * @param string|null $lwc
15
+ * @param string|null $lwe
16
+ * @param string|null $lop
17
+ * @param string|null $lang
18
+ */
13 19
 function mm_pack_language($lwc, $lwe, $lop, $lang)
14 20
 {
15 21
     $returner = '';
@@ -41,6 +47,9 @@  discard block
 block discarded – undo
41 47
 /*
42 48
  * Unpacks a language string into an array of its component variable.
43 49
  */
50
+/**
51
+ * @param string|null $lang
52
+ */
44 53
 function mm_unpack_language($data_string, $lang)
45 54
 {
46 55
     $returner = array();
@@ -71,6 +80,11 @@  discard block
 block discarded – undo
71 80
 /*
72 81
  * Packs the time (hour and minute) into a single string.
73 82
  */
83
+/**
84
+ * @param string|null $hour
85
+ * @param string|null $min
86
+ * @param string|null $day
87
+ */
74 88
 function mm_pack_time($hour, $min, $day)
75 89
 {
76 90
     $returner = '';
@@ -95,6 +109,9 @@  discard block
 block discarded – undo
95 109
 /*
96 110
  * Unpacks the time into an array of hour and minute.
97 111
  */
112
+/**
113
+ * @param string|null $day
114
+ */
98 115
 function mm_unpack_time($data_string, $day)
99 116
 {
100 117
     $returner = array();
@@ -429,6 +446,9 @@  discard block
 block discarded – undo
429 446
  * Notify user function which uses messages_send() to send on site notifications and mail() to send e-mail.
430 447
  * This is here because notify_user() is currently not working.
431 448
  */
449
+/**
450
+ * @param string $body
451
+ */
432 452
 function mm_notify_user($recipient, $sender, $subject, $body) {
433 453
 	//notify_user($recipient, $sender, $subject, $body, array(), 'email');
434 454
 
@@ -558,6 +578,9 @@  discard block
 block discarded – undo
558 578
 /*
559 579
  * Returns an accept button to a user who has received an offer if the mission is not full.
560 580
  */
581
+/**
582
+ * @param ElggUser|null $applicant
583
+ */
561 584
 function mm_finalize_button($mission, $applicant) {
562 585
 	$relationship_count = elgg_get_entities_from_relationship(array(
563 586
 			'relationship' => 'mission_accepted',
@@ -723,6 +746,9 @@  discard block
 block discarded – undo
723 746
 /*
724 747
  * Check if the user is opted in to any of the opt-in options.
725 748
  */
749
+/**
750
+ * @param ElggEntity|null $current_user
751
+ */
726 752
 function check_if_opted_in($current_user) {
727 753
     //Nick - adding additional checks for other opportunity types
728 754
 	if($current_user->opt_in_missions == 'gcconnex_profile:opt:yes') {
Please login to merge, or discard this patch.
mod/uservalidationbyemail/start.php 1 patch
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -62,7 +62,7 @@  discard block
 block discarded – undo
62 62
  * @param string $type
63 63
  * @param bool   $value
64 64
  * @param array  $params
65
- * @return bool
65
+ * @return null|boolean
66 66
  */
67 67
 function uservalidationbyemail_disable_new_user($hook, $type, $value, $params) {
68 68
 	$user = elgg_extract('user', $params);
@@ -155,7 +155,7 @@  discard block
 block discarded – undo
155 155
  * Checks if an account is validated
156 156
  *
157 157
  * @params array $credentials The username and password
158
- * @return bool
158
+ * @return boolean|null
159 159
  */
160 160
 function uservalidationbyemail_check_auth_attempt($credentials) {
161 161
 
@@ -234,7 +234,7 @@  discard block
 block discarded – undo
234 234
  * @param string   $event
235 235
  * @param string   $type
236 236
  * @param ElggUser $user
237
- * @return bool
237
+ * @return false|null
238 238
  *
239 239
  * @throws LoginException
240 240
  */
Please login to merge, or discard this patch.