Completed
Pull Request — gcconnex (#1464)
by Ilia
26:45
created
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/gcRegistration/actions/registerAJAX.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -141,7 +141,7 @@  discard block
 block discarded – undo
141 141
  * Checks for invalid / external domains
142 142
  * 
143 143
  * @param  $domain <string> domain part of the user's email address
144
- * @return true if domain is invalid, false otherwise
144
+ * @return boolean if domain is invalid, false otherwise
145 145
  * 
146 146
  **/
147 147
 function checkInvalidDomain($dom) 
@@ -195,7 +195,7 @@  discard block
 block discarded – undo
195 195
  * Array key is the character to remove and the associated item is what will replace it.
196 196
  *
197 197
  * @param  $emailInput <string> users email address
198
- * @return always returns "" because we use the response text directly
198
+ * @return string returns "" because we use the response text directly
199 199
  * @author Matthew April <[email protected]>
200 200
  */
201 201
 
Please login to merge, or discard this patch.
mod/gcRegistration_collab/actions/registerAJAX.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -194,7 +194,7 @@  discard block
 block discarded – undo
194 194
  * 
195 195
  * @param  $domain <string> domain part of the user's email address
196 196
  * @param  $email <string> the user's full email address
197
- * @return true if domain is invalid, false otherwise
197
+ * @return boolean if domain is invalid, false otherwise
198 198
  * 
199 199
  **/
200 200
 function checkInvalidDomain($dom, $email){
@@ -251,7 +251,7 @@  discard block
 block discarded – undo
251 251
  * Array key is the character to remove and the associated item is what will replace it.
252 252
  *
253 253
  * @param  $emailInput <string> users email address
254
- * @return always returns "" because we use the response text directly
254
+ * @return string returns "" because we use the response text directly
255 255
  * @author Matthew April <[email protected]>
256 256
  */
257 257
 
Please login to merge, or discard this patch.
mod/group_tools/lib/functions.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -446,7 +446,7 @@
 block discarded – undo
446 446
  * @param int    $group_guid the group GUID
447 447
  * @param string $email      the email address
448 448
  *
449
- * @return boolean|string the invite code, or false on failure
449
+ * @return string|false the invite code, or false on failure
450 450
  */
451 451
 function group_tools_generate_email_invite_code($group_guid, $email) {
452 452
 	$result = false;
Please login to merge, or discard this patch.
mod/group_tools/lib/hooks.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -685,7 +685,7 @@
 block discarded – undo
685 685
  * @param bool   $return_value true, return false to stop the action
686 686
  * @param null   $params       passed on params
687 687
  *
688
- * @return bool
688
+ * @return boolean|null
689 689
  */
690 690
 function group_tools_join_group_action_handler($hook, $type, $return_value, $params) {
691 691
 	// hacky way around a short comming of Elgg core to allow users to join a group
Please login to merge, or discard this patch.
mod/groups/start.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -939,7 +939,7 @@  discard block
 block discarded – undo
939 939
  * @param string $type
940 940
  * @param string $return
941 941
  * @param array  $params
942
- * @return bool
942
+ * @return false|null
943 943
  */
944 944
 function discussion_comment_override($hook, $type, $return, $params) {
945 945
 	if (elgg_instanceof($params['entity'], 'object', 'groupforumtopic')) {
@@ -1290,7 +1290,7 @@  discard block
 block discarded – undo
1290 1290
 /**
1291 1291
  * Runs unit tests for groups
1292 1292
  *
1293
- * @return array
1293
+ * @return string[]
1294 1294
  */
1295 1295
 function groups_test($hook, $type, $value, $params) {
1296 1296
 	global $CONFIG;
Please login to merge, or discard this patch.
mod/htmlawed/start.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -191,7 +191,7 @@
 block discarded – undo
191 191
 /**
192 192
  * Runs unit tests for htmlawed
193 193
  *
194
- * @return array
194
+ * @return string[]
195 195
  */
196 196
 function htmlawed_test($hook, $type, $value, $params) {
197 197
     $value[] = dirname(__FILE__) . '/tests/tags.php';
Please login to merge, or discard this patch.
mod/mentions/start.php 1 patch
Doc Comments   -2 removed lines patch added patch discarded remove patch
@@ -51,8 +51,6 @@
 block discarded – undo
51 51
  * Rewrites a view for @username mentions.
52 52
  *
53 53
  * @param string $hook    The name of the hook
54
- * @param string $type    The type of the hook
55
- * @param string $content The content of the page
56 54
  * @return string
57 55
  */
58 56
 function mentions_rewrite($hook, $entity_type, $returnvalue, $params) {
Please login to merge, or discard this patch.
mod/messages/start.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -630,7 +630,7 @@
 block discarded – undo
630 630
  * @param string $return_value
631 631
  * @param array  $params
632 632
  *
633
- * @return array
633
+ * @return string
634 634
  */
635 635
 function messages_ecml_views_hook($hook, $entity_type, $return_value, $params) {
636 636
 	$return_value['messages/messages'] = elgg_echo('messages');
Please login to merge, or discard this patch.