Failed Conditions
Branch master (404fd5)
by Rick
07:45
created
Sources/QueryString.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -333,8 +333,8 @@
 block discarded – undo
333 333
 
334 334
 /**
335 335
  * Expands a IPv6 address to its full form.
336
- * @param string $ip ip address to be converted
337
- * @return bool/string expanded ipv6 address.
336
+ * @param string $addr
337
+ * @return string expanded ipv6 address.
338 338
  */
339 339
 function expandIPv6($addr, $strict_check = true)
340 340
 {
Please login to merge, or discard this patch.
Sources/RemoveTopic.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -562,7 +562,7 @@
 block discarded – undo
562 562
  *
563 563
  * @param int $message The message id
564 564
  * @param bool $decreasePostCount if true users' post count will be reduced
565
- * @return array an array to set the cookie on with domain and path in it, in that order
565
+ * @return boolean an array to set the cookie on with domain and path in it, in that order
566 566
  */
567 567
 function removeMessage($message, $decreasePostCount = true)
568 568
 {
Please login to merge, or discard this patch.
Sources/Reports.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -906,7 +906,7 @@
 block discarded – undo
906 906
  * @param string $title = ''
907 907
  * @param string $custom_table = null
908 908
  *
909
- * @return bool returns false if there are no tables
909
+ * @return null|false returns false if there are no tables
910 910
  */
911 911
 function addSeparator($title = '', $custom_table = null)
912 912
 {
Please login to merge, or discard this patch.
Sources/Security.php 1 patch
Doc Comments   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -737,7 +737,7 @@  discard block
 block discarded – undo
737 737
  *
738 738
  * @param string $action
739 739
  * @param string $type = 'post'
740
- * @return array
740
+ * @return string[]
741 741
  */
742 742
 function createToken($action, $type = 'post')
743 743
 {
@@ -848,7 +848,7 @@  discard block
 block discarded – undo
848 848
  *
849 849
  * @param string $action
850 850
  * @param bool $is_fatal = true
851
- * @return bool
851
+ * @return null|boolean
852 852
  */
853 853
 function checkSubmitOnce($action, $is_fatal = true)
854 854
 {
@@ -1152,6 +1152,7 @@  discard block
 block discarded – undo
1152 1152
  * The time taken depends on error_type - generally uses the modSetting.
1153 1153
  *
1154 1154
  * @param string $error_type, used also as a $txt index. (not an actual string.)
1155
+ * @param string $error_type
1155 1156
  * @return boolean
1156 1157
  */
1157 1158
 function spamProtection($error_type)
Please login to merge, or discard this patch.
Sources/Subs-Admin.php 2 patches
Doc Comments   +1 added lines, -3 removed lines patch added patch discarded remove patch
@@ -512,6 +512,4 @@
 block discarded – undo
512 512
 			$emaildata = loadEmailTemplate($template, $replacements, empty($recipient['lang']) || empty($modSettings['userLanguage']) ? $language : $recipient['lang']);
513 513
 
514 514
 			// Send off the email.
515
-			sendmail($recipient['email'], $emaildata['subject'], $emaildata['body'], null, null, false, 1);
516
-		}
517
-}
515
+			sendmail($recipient['email'], $emaildata['subject'], $emaildata[
518 516
\ No newline at end of file
Please login to merge, or discard this patch.
Unused Use Statements   +1 added lines, -3 removed lines patch added patch discarded remove patch
@@ -512,6 +512,4 @@
 block discarded – undo
512 512
 			$emaildata = loadEmailTemplate($template, $replacements, empty($recipient['lang']) || empty($modSettings['userLanguage']) ? $language : $recipient['lang']);
513 513
 
514 514
 			// Send off the email.
515
-			sendmail($recipient['email'], $emaildata['subject'], $emaildata['body'], null, null, false, 1);
516
-		}
517
-}
515
+			sendmail($recipient['email'], $emaildata['subject'], $emaildata[
518 516
\ No newline at end of file
Please login to merge, or discard this patch.
Sources/Subs-Auth.php 1 patch
Doc Comments   +3 added lines, -4 removed lines patch added patch discarded remove patch
@@ -112,7 +112,7 @@  discard block
 block discarded – undo
112 112
  *
113 113
  * @param bool $local,
114 114
  * @param bool $global
115
- * @return array an array to set the cookie on with domain and path in it, in that order
115
+ * @return string an array to set the cookie on with domain and path in it, in that order
116 116
  */
117 117
 function url_parts($local, $global)
118 118
 {
@@ -250,8 +250,6 @@  discard block
 block discarded – undo
250 250
  * Used by the adminLogin() function.
251 251
  * if 'value' is an array, the function is called recursively.
252 252
  *
253
- * @param string $key
254
- * @param string $value
255 253
  * @return string 'hidden' HTML form fields, containing key-value-pairs
256 254
  */
257 255
 function adminLogin_outputPostVars($k, $v)
@@ -276,7 +274,7 @@  discard block
 block discarded – undo
276 274
  *
277 275
  * @global type $scripturl
278 276
  * @param type $get
279
- * @return our query string
277
+ * @return string query string
280 278
  */
281 279
 function construct_query_string($get)
282 280
 {
@@ -602,6 +600,7 @@  discard block
 block discarded – undo
602 600
  *
603 601
  * @param int $memID,
604 602
  * @param string $username
603
+ * @param integer $memID
605 604
  * @return string Returns null if fine
606 605
  */
607 606
 function validateUsername($memID, $username, $return_error = false, $check_reserved_name = true)
Please login to merge, or discard this patch.
Sources/Subs-Calendar.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -109,9 +109,9 @@
 block discarded – undo
109 109
  * - censors the posted event titles.
110 110
  * - uses the current user's permissions if use_permissions is true, otherwise it does nothing "permission specific"
111 111
  *
112
- * @param string $earliest_date
113
- * @param string $latest_date
114 112
  * @param bool $use_permissions = true
113
+ * @param string $low_date
114
+ * @param string $high_date
115 115
  * @return array contextual information if use_permissions is true, and an array of the data needed to build that otherwise
116 116
  */
117 117
 function getEventRange($low_date, $high_date, $use_permissions = true)
Please login to merge, or discard this patch.
Sources/Subs-Categories.php 1 patch
Doc Comments   +2 added lines, -4 removed lines patch added patch discarded remove patch
@@ -122,7 +122,6 @@  discard block
 block discarded – undo
122 122
  * allows (almost) the same options as the modifyCat() function.
123 123
  * returns the ID of the newly created category.
124 124
  *
125
- * @param int $createCategory
126 125
  * @param array $catOptions
127 126
  */
128 127
 function createCategory($catOptions)
@@ -178,8 +177,7 @@  discard block
 block discarded – undo
178 177
  * deletes all information that's associated with the given categories.
179 178
  * updates the statistics to reflect the new situation.
180 179
  *
181
- * @param array $categories_to_remove
182
- * @param int $moveChildrenTo = null
180
+ * @param integer $moveBoardsTo
183 181
  */
184 182
 function deleteCategories($categories, $moveBoardsTo = null)
185 183
 {
@@ -259,7 +257,7 @@  discard block
 block discarded – undo
259 257
  * allows three changes to the status: 'expand', 'collapse' and 'toggle'.
260 258
  * if check_collapsable is set, only category allowed to be collapsed, will be collapsed.
261 259
  *
262
- * @param array $categories
260
+ * @param integer[] $categories
263 261
  * @param string $new_status
264 262
  * @param array $members = null
265 263
  * @param bool $check_collapsable = true
Please login to merge, or discard this patch.
Sources/Subs-Compat.php 1 patch
Doc Comments   +16 added lines patch added patch discarded remove patch
@@ -45,6 +45,9 @@  discard block
 block discarded – undo
45 45
  */
46 46
 if (!function_exists('md5_file'))
47 47
 {
48
+	/**
49
+	 * @param string $filename
50
+	 */
48 51
 	function md5_file($filename)
49 52
 	{
50 53
 		// This isn't the most efficient way in the world, but then we don't have MD5_CTX do we?
@@ -140,6 +143,7 @@  discard block
 block discarded – undo
140 143
 
141 144
 /**
142 145
  * This is the core SHA-1 calculation routine, used by sha1().
146
+ * @param integer $len
143 147
  */
144 148
 function sha1_core($x, $len)
145 149
 {
@@ -189,6 +193,12 @@  discard block
 block discarded – undo
189 193
 /*
190 194
  * Helper function for the core SHA-1 calculation
191 195
  */
196
+/**
197
+ * @param integer $t
198
+ * @param integer $b
199
+ * @param integer $c
200
+ * @param integer $d
201
+ */
192 202
 function sha1_ft($t, $b, $c, $d)
193 203
 {
194 204
 	if ($t < 20)
@@ -204,6 +214,9 @@  discard block
 block discarded – undo
204 214
 /*
205 215
  * Helper function for the core SHA-1 calculation
206 216
  */
217
+/**
218
+ * @param integer $t
219
+ */
207 220
 function sha1_kt($t)
208 221
 {
209 222
 	return $t < 20 ? 1518500249 : ($t < 40 ? 1859775393 : ($t < 60 ? -1894007588 : -899497514));
@@ -212,6 +225,9 @@  discard block
 block discarded – undo
212 225
 /*
213 226
  * Helper function for the core SHA-1 calculation
214 227
  */
228
+/**
229
+ * @param integer $cnt
230
+ */
215 231
 function sha1_rol($num, $cnt)
216 232
 {
217 233
 	// Unfortunately, PHP uses unsigned 32-bit longs only.  So we have to kludge it a bit.
Please login to merge, or discard this patch.