Completed
Pull Request — gcconnex (#1529)
by
unknown
15:28
created
mod/cp_notifications/lib/functions.php 1 patch
Doc Comments   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -6,6 +6,7 @@  discard block
 block discarded – undo
6 6
  * @param integer 	$entity_guid_static
7 7
  * @param integer 	$entity_guid
8 8
  *
9
+ * @return integer
9 10
  */
10 11
 function get_forum_in_group($entity_guid_static, $entity_guid) {
11 12
 	$entity = get_entity($entity_guid);
@@ -159,7 +160,7 @@  discard block
 block discarded – undo
159 160
  * @param ElggEntity 	$entity
160 161
  * @param ElggUser 		$send_to
161 162
  * @param string 		$entity_url (default value empty)
162
- * @return Success 		true/false
163
+ * @return boolean 		true/false
163 164
  */
164 165
 function create_digest($invoked_by, $subtype, $entity, $send_to, $entity_url = '') {
165 166
 
Please login to merge, or discard this patch.
mod/cp_notifications/start.php 1 patch
Doc Comments   +4 added lines, -6 removed lines patch added patch discarded remove patch
@@ -132,7 +132,7 @@  discard block
 block discarded – undo
132 132
  * @param mixed  $value   The current value of the plugin hook
133 133
  * @param mixed  $params  Data passed from the trigger
134 134
  *
135
- * @return mixed if not null, this will be the new value of the plugin hook
135
+ * @return boolean if not null, this will be the new value of the plugin hook
136 136
  */
137 137
 function minor_save_hook_handler($hook, $type, $value, $params) {
138 138
 
@@ -542,7 +542,6 @@  discard block
 block discarded – undo
542 542
 /**
543 543
  * returns the headers for ical
544 544
  *
545
- * @param string 		$type_event
546 545
  * @param ElggObject 	$event
547 546
  * @param string 		$start_date
548 547
  * @param string 		$end_date
@@ -1357,6 +1356,9 @@  discard block
 block discarded – undo
1357 1356
 }
1358 1357
 
1359 1358
 
1359
+/**
1360
+ * @param string $error_message
1361
+ */
1360 1362
 function notification_logging($error_message) {
1361 1363
 	// logging mechanism
1362 1364
 	if (elgg_is_active_plugin('wet4')) {
@@ -1407,8 +1409,6 @@  discard block
 block discarded – undo
1407 1409
  * run crontab, retrieve users, send digest, reset timer (update timestamp)
1408 1410
  *
1409 1411
  * @param string $hook    The name of the plugin hook
1410
- * @param string $type    The type of the plugin hook
1411
- * @param mixed  $value   The current value of the plugin hook
1412 1412
  * @param mixed  $params  Data passed from the trigger
1413 1413
  */
1414 1414
 function cp_digest_weekly_cron_handler($hook, $entity_type, $return_value, $params) {
@@ -1674,7 +1674,6 @@  discard block
 block discarded – undo
1674 1674
  *
1675 1675
  * @param string $hook    The name of the plugin hook
1676 1676
  * @param string $type    The type of the plugin hook
1677
- * @param mixed  $value   The current value of the plugin hook
1678 1677
  * @param mixed  $params  Data passed from the trigger
1679 1678
  */
1680 1679
 function cpn_email_handler_hook($hook, $type, $notification, $params) {
@@ -1688,7 +1687,6 @@  discard block
 block discarded – undo
1688 1687
  *
1689 1688
  * @param string $hook    The name of the plugin hook
1690 1689
  * @param string $type    The type of the plugin hook
1691
- * @param mixed  $value   The current value of the plugin hook
1692 1690
  * @param mixed  $params  Data passed from the trigger
1693 1691
  *
1694 1692
  * @return mixed if not null, this will be the new value of the plugin hook
Please login to merge, or discard this patch.
mod/gcforums/start.php 1 patch
Doc Comments   +11 added lines patch added patch discarded remove patch
@@ -126,6 +126,9 @@  discard block
 block discarded – undo
126 126
 }
127 127
 
128 128
 /// recursively go through the forums and return group entity
129
+/**
130
+ * @return integer
131
+ */
129 132
 function gcforums_get_forum_in_group($entity_guid_static, $entity_guid) {
130 133
 	$entity = get_entity($entity_guid);
131 134
 	// (base) stop recursing when we reach group guid
@@ -220,6 +223,9 @@  discard block
 block discarded – undo
220 223
 
221 224
 
222 225
 
226
+/**
227
+ * @param ElggEntity $entity
228
+ */
223 229
 function assemble_forum_breadcrumb($entity) {
224 230
 	$forum_guid = $entity->guid;
225 231
 	if ($entity instanceof ElggGroup) {
@@ -511,6 +517,7 @@  discard block
 block discarded – undo
511 517
  * TOTAL_POST : 1
512 518
  * TOTAL_TOPICS : 2
513 519
  * RECENT_POST : 3
520
+ * @param integer $type
514 521
  */
515 522
 function get_forums_statistics_information($container_guid, $type) {
516 523
 	$dbprefix = elgg_get_config('dbprefix');
@@ -662,6 +669,10 @@  discard block
 block discarded – undo
662 669
 
663 670
 
664 671
 
672
+/**
673
+ * @param integer|null $forum_guid
674
+ * @param integer|null $group_guid
675
+ */
665 676
 function gcforums_menu_buttons($forum_guid, $group_guid, $is_topic=false) { 
666 677
 	// main page if forum_guid is not present
667 678
 	elgg_load_css('gcforums-css');
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.