Passed
Push — Mobile-API---Comments ( 439939 )
by
unknown
19:32
created
mod/thewire_images/start.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -82,7 +82,7 @@
 block discarded – undo
82 82
  *	Download an attachment: thewire_image/download/<guid>/<title>
83 83
  *
84 84
  * @param array $page From the page_handler function
85
- * @return bool
85
+ * @return boolean|null
86 86
  */
87 87
 function thewire_image_page_handler($page) {
88 88
 	
Please login to merge, or discard this patch.
mod/gcforums/start.php 1 patch
Doc Comments   +4 added lines patch added patch discarded remove patch
@@ -606,6 +606,10 @@
 block discarded – undo
606 606
 
607 607
 
608 608
 /// recursively go through the nested forums to create the breadcrumb
609
+/**
610
+ * @param integer $forum_guid
611
+ * @param integer $recurse_forum_guid
612
+ */
609 613
 function assemble_nested_forums($breadcrumb, $forum_guid, $recurse_forum_guid) {
610 614
 	$lang = get_current_language();
611 615
 	$entity = get_entity($recurse_forum_guid);
Please login to merge, or discard this patch.
mod/missions/lib/missions.php 1 patch
Doc Comments   +30 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();
@@ -118,6 +135,9 @@  discard block
 block discarded – undo
118 135
 /*
119 136
  * Unpacks every unpackable item within a given entity.
120 137
  */
138
+/**
139
+ * @param ElggEntity $entity
140
+ */
121 141
 function mm_unpack_mission($entity)
122 142
 {
123 143
     $returner = array();
@@ -429,6 +449,10 @@  discard block
 block discarded – undo
429 449
  * Notify user function which uses messages_send() to send on site notifications and mail() to send e-mail.
430 450
  * This is here because notify_user() is currently not working.
431 451
  */
452
+/**
453
+ * @param string $title_en
454
+ * @param string $title_fr
455
+ */
432 456
 function mm_notify_user($recipient, $sender, $subject, $title_en,$title_fr,$body_en, $body_fr) {
433 457
 	//notify_user($recipient, $sender, $subject, $body, array(), 'email');
434 458
 
@@ -622,6 +646,9 @@  discard block
 block discarded – undo
622 646
 /*
623 647
  * Returns an accept button to a user who has received an offer if the mission is not full.
624 648
  */
649
+/**
650
+ * @param ElggUser|null $applicant
651
+ */
625 652
 function mm_finalize_button($mission, $applicant) {
626 653
 	$relationship_count = elgg_get_entities_from_relationship(array(
627 654
 			'relationship' => 'mission_accepted',
@@ -787,6 +814,9 @@  discard block
 block discarded – undo
787 814
 /*
788 815
  * Check if the user is opted in to any of the opt-in options.
789 816
  */
817
+/**
818
+ * @param ElggEntity|null $current_user
819
+ */
790 820
 function check_if_opted_in($current_user) {
791 821
     //Nick - adding additional checks for other opportunity types
792 822
 	if($current_user->opt_in_missions == 'gcconnex_profile:opt:yes') {
Please login to merge, or discard this patch.