Passed
Push — master ( 4c7da5...a55b86 )
by Ilia
09:52
created
mod/pleio/classes/ModPleio/Helpers.php 1 patch
Doc Comments   +4 added lines patch added patch discarded remove patch
@@ -2,6 +2,10 @@
 block discarded – undo
2 2
 namespace ModPleio;
3 3
 
4 4
 class Helpers {
5
+
6
+    /**
7
+     * @return string
8
+     */
5 9
     public static function generateUsername($username) {
6 10
         $hidden = access_show_hidden_entities(true);
7 11
 
Please login to merge, or discard this patch.
mod/gc_mobile_api/models/wire.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -109,6 +109,9 @@
 block discarded – undo
109 109
 	false
110 110
 );
111 111
 
112
+/**
113
+ * @param ElggUser $user_entity
114
+ */
112 115
 function wires_foreach($wire_posts, $user_entity)
113 116
 {
114 117
 	foreach ($wire_posts as $wire_post) {
Please login to merge, or discard this patch.
engine/lib/input.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@
 block discarded – undo
48 48
  * @param mixed $var Anything that does not include an object (strings, ints, arrays)
49 49
  *					 This includes multi-dimensional arrays.
50 50
  *
51
- * @return mixed The filtered result - everything will be strings
51
+ * @return string The filtered result - everything will be strings
52 52
  */
53 53
 function filter_tags($var) {
54 54
 	return elgg_trigger_plugin_hook('validate', 'input', null, $var);
Please login to merge, or discard this patch.
mod/thewire/start.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -258,7 +258,7 @@  discard block
 block discarded – undo
258 258
  * @param int    $access_id   Public/private etc
259 259
  * @param int    $parent_guid Parent post guid (if any)
260 260
  * @param string $method      The method (default: 'site')
261
- * @return guid or false if failure
261
+ * @return integer or false if failure
262 262
  */
263 263
 function thewire_save_post($text, $userid, $access_id, $parent_guid = 0, $method = "site") {
264 264
 	$post = new ElggObject();
@@ -473,7 +473,7 @@  discard block
 block discarded – undo
473 473
 /**
474 474
  * Runs unit tests for the wire
475 475
  *
476
- * @return array
476
+ * @return string[]
477 477
  */
478 478
 function thewire_test($hook, $type, $value, $params) {
479 479
 	global $CONFIG;
Please login to merge, or discard this patch.
mod/phpmailer/start.php 1 patch
Doc Comments   +6 added lines, -4 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
  * @param string $subject The subject of the message.
30 30
  * @param string $message The message body
31 31
  * @param array $params Optional parameters (not used)
32
- * @return bool
32
+ * @return null|boolean
33 33
  */
34 34
 function phpmailer_notify_handler(ElggEntity $from, ElggUser $to, $subject, $message, array $params = NULL) {
35 35
 
@@ -72,6 +72,7 @@  discard block
 block discarded – undo
72 72
  * This is a stupid function pulled from original Elgg code
73 73
  *
74 74
  * @param  ElggEntity The entity sending the message
75
+ * @param ElggEntity $from
75 76
  * @return string with email address
76 77
  */
77 78
 function phpmailer_extract_from_email($from) {
@@ -98,8 +99,6 @@  discard block
 block discarded – undo
98 99
 /**
99 100
  * Send an email using phpmailer
100 101
  *
101
- * @param string $from       From address
102
- * @param string $from_name  From name
103 102
  * @param string $to         To address
104 103
  * @param string $to_name    To name
105 104
  * @param string $subject    The subject of the message.
@@ -110,7 +109,7 @@  discard block
 block discarded – undo
110 109
  * @param array  $files      Array of file descriptor arrays, each file array
111 110
  *                           consists of full path and name
112 111
  * @param array  $params     Additional parameters
113
- * @return bool
112
+ * @return null|boolean
114 113
  */
115 114
 function phpmailer_send($to, $to_name, $subject, $body, array $bcc = NULL, $html = true, array $files = NULL, array $params = NULL, $image = NULL) {
116 115
 
@@ -255,6 +254,9 @@  discard block
 block discarded – undo
255 254
 
256 255
 
257 256
 
257
+/**
258
+ * @param string $type
259
+ */
258 260
 function phpmailer_logging($errMess, $errStack, $type, $errType) {
259 261
 	// logging mechanism
260 262
 	if (elgg_is_active_plugin('wet4')) {
Please login to merge, or discard this patch.
mod/gc_mobile_api/models/blog.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -85,6 +85,9 @@
 block discarded – undo
85 85
  false
86 86
 );
87 87
 
88
+/**
89
+ * @param ElggUser $user_entity
90
+ */
88 91
 function foreach_blogs($blogs, $user_entity, $lang)
89 92
 {
90 93
 	foreach ($blogs as $blog_post) {
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,9 @@  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 $lang
15
+ */
13 16
 function mm_pack_language($lwc, $lwe, $lop, $lang)
14 17
 {
15 18
     $returner = '';
@@ -41,6 +44,9 @@  discard block
 block discarded – undo
41 44
 /*
42 45
  * Unpacks a language string into an array of its component variable.
43 46
  */
47
+/**
48
+ * @param string|null $lang
49
+ */
44 50
 function mm_unpack_language($data_string, $lang)
45 51
 {
46 52
     $returner = array();
@@ -71,6 +77,11 @@  discard block
 block discarded – undo
71 77
 /*
72 78
  * Packs the time (hour and minute) into a single string.
73 79
  */
80
+/**
81
+ * @param string|null $hour
82
+ * @param string|null $min
83
+ * @param string|null $day
84
+ */
74 85
 function mm_pack_time($hour, $min, $day)
75 86
 {
76 87
     $returner = '';
@@ -95,6 +106,9 @@  discard block
 block discarded – undo
95 106
 /*
96 107
  * Unpacks the time into an array of hour and minute.
97 108
  */
109
+/**
110
+ * @param string|null $day
111
+ */
98 112
 function mm_unpack_time($data_string, $day)
99 113
 {
100 114
     $returner = array();
@@ -118,6 +132,9 @@  discard block
 block discarded – undo
118 132
 /*
119 133
  * Unpacks every unpackable item within a given entity.
120 134
  */
135
+/**
136
+ * @param ElggEntity $entity
137
+ */
121 138
 function mm_unpack_mission($entity)
122 139
 {
123 140
     $returner = array();
@@ -429,6 +446,10 @@  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 $title_en
451
+ * @param string $title_fr
452
+ */
432 453
 function mm_notify_user($recipient, $sender, $subject, $title_en,$title_fr,$body_en, $body_fr) {
433 454
 	//notify_user($recipient, $sender, $subject, $body, array(), 'email');
434 455
 
@@ -622,6 +643,9 @@  discard block
 block discarded – undo
622 643
 /*
623 644
  * Returns an accept button to a user who has received an offer if the mission is not full.
624 645
  */
646
+/**
647
+ * @param ElggUser|null $applicant
648
+ */
625 649
 function mm_finalize_button($mission, $applicant) {
626 650
 	$relationship_count = elgg_get_entities_from_relationship(array(
627 651
 			'relationship' => 'mission_accepted',
@@ -787,6 +811,9 @@  discard block
 block discarded – undo
787 811
 /*
788 812
  * Check if the user is opted in to any of the opt-in options.
789 813
  */
814
+/**
815
+ * @param ElggEntity|null $current_user
816
+ */
790 817
 function check_if_opted_in($current_user) {
791 818
     //Nick - adding additional checks for other opportunity types
792 819
 	if($current_user->opt_in_missions == 'gcconnex_profile:opt:yes') {
@@ -864,6 +891,9 @@  discard block
 block discarded – undo
864 891
 	return false;
865 892
 }
866 893
 
894
+/**
895
+ * @param ElggEntity $mission
896
+ */
867 897
 function mm_complete_mission_inprogress_reports($mission, $if_no_applicants = false) {
868 898
 		if ($if_no_applicants) {
869 899
 				// Find out how many accepted applicants are on this mission.
Please login to merge, or discard this patch.
mod/missions_organization/lib/missions-organization.php 1 patch
Doc Comments   +9 added lines patch added patch discarded remove patch
@@ -46,6 +46,9 @@  discard block
 block discarded – undo
46 46
  * Function to format the dropdown menu input.
47 47
  * If the input is not an entity then it returns false.
48 48
  */
49
+/**
50
+ * @param integer $input
51
+ */
49 52
 function mo_format_input_node($input) {
50 53
 	if(elgg_entity_exists($input)) {
51 54
 		return 'MOrg:' . $input;
@@ -118,6 +121,9 @@  discard block
 block discarded – undo
118 121
 /*
119 122
  * Function which gets all the ancestors and then creates a string out of their names.
120 123
  */
124
+/**
125
+ * @param string|false $node_string
126
+ */
121 127
 function mo_string_all_ancestors($node_string) {
122 128
 	$ancestor_guids = mo_get_all_ancestors($node_string);
123 129
 	$ancestor_names_english = array();
@@ -145,6 +151,9 @@  discard block
 block discarded – undo
145 151
 /*
146 152
  * Extracts node guid from the stored data.
147 153
  */
154
+/**
155
+ * @return integer
156
+ */
148 157
 function mo_extract_node_guid($node_string) {
149 158
 	$node_array = explode(':', $node_string);
150 159
 	return $node_array[1];
Please login to merge, or discard this patch.
mod/gc_lang_url_handler/start.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -14,7 +14,7 @@
 block discarded – undo
14 14
  * @param $type
15 15
  * @param $returnvalue
16 16
  * @param $params
17
- * @return False to stop processing the response, True will let elgg handle the response
17
+ * @return null|boolean to stop processing the response, True will let elgg handle the response
18 18
  */
19 19
 function global_url_handler($hook, $type, $returnvalue, $params) {
20 20
 
Please login to merge, or discard this patch.