SimpleMachines / SMF2.1

Sources/ManageBans.php

Doc Comments +2 added lines, -2 removed lines

@@ -929,7 +929,7 @@ 
  * Doesn't clean the inputs
  *
  * @param array $items_ids The items to remove
- * @param bool|int $group_id The ID of the group these triggers are associated with or false if deleting them from all groups
+ * @param integer $group_id The ID of the group these triggers are associated with or false if deleting them from all groups
  * @return bool Always returns true
  */
 function removeBanTriggers($items_ids = array(), $group_id = false)
@@ -1123,7 +1123,7 @@ 
  * Errors in $context['ban_errors']
  *
  * @param array $triggers The triggers to validate
- * @return array An array of riggers and log info ready to be used
+ * @return integer An array of riggers and log info ready to be used
  */
 function validateTriggers(&$triggers)
 {

Sources/ManagePaid.php

Doc Comments +1 added lines, -1 removed lines

@@ -1452,7 +1452,7 @@
  *
  * @param int $id_subscribe The subscription ID
  * @param int $id_member The ID of the member
- * @param int|string $renewal 0 if we're forcing start/end time, otherwise a string indicating how long to renew the subscription for ('D', 'W', 'M' or 'Y')
+ * @param integer $renewal 0 if we're forcing start/end time, otherwise a string indicating how long to renew the subscription for ('D', 'W', 'M' or 'Y')
  * @param int $forceStartTime If set, forces the subscription to start at the specified time
  * @param int $forceEndTime If set, forces the subscription to end at the specified time
  */

Sources/PersonalMessage.php

Doc Comments +1 added lines, -1 removed lines

@@ -4070,7 +4070,7 @@
  *
  * @param int $pmID The ID of the PM
  * @param string $validFor Which folders this is valud for - can be 'inbox', 'outbox' or 'in_or_outbox'
- * @return boolean Whether the PM is accessible in that folder for the current user
+ * @return boolean|null Whether the PM is accessible in that folder for the current user
  */
 function isAccessiblePM($pmID, $validFor = 'in_or_outbox')
 {

Sources/Profile-Modify.php

Doc Comments +6 added lines, -6 removed lines

@@ -2183,7 +2183,7 @@ 
  * Deletes a single or a group of alerts by ID
  *
  * @param int|array The ID of a single alert to delete or an array containing the IDs of multiple alerts. The function will convert integers into an array for better handling.
- * @param bool|int $memID The user ID. Used to update the user unread alerts count.
+ * @param integer $memID The user ID. Used to update the user unread alerts count.
  * @return void|int If the $memID param is set, returns the new amount of unread alerts.
  */
 function alert_delete($toDelete, $memID = false)
@@ -2839,7 +2839,7 @@ 
 /**
  * Handles the "manage groups" section of the profile
  *
- * @return true Always returns true
+ * @return boolean Always returns true
  */
 function profileLoadGroups()
 {
@@ -2896,7 +2896,7 @@ 
 /**
  * Load key signature context data.
  *
- * @return true Always returns true
+ * @return boolean Always returns true
  */
 function profileLoadSignatureData()
 {
@@ -2960,7 +2960,7 @@ 
 /**
  * Load avatar context data.
  *
- * @return true Always returns true
+ * @return boolean Always returns true
  */
 function profileLoadAvatarData()
 {
@@ -3033,7 +3033,7 @@ 
  * Save a members group.
  *
  * @param int &$value The ID of the (new) primary group
- * @return true Always returns true
+ * @return boolean Always returns true
  */
 function profileSaveGroups(&$value)
 {
@@ -3138,7 +3138,7 @@ 
  * @todo argh, the avatar here. Take this out of here!
  *
  * @param string &$value What kind of avatar we're expecting. Can be 'none', 'server_stored', 'gravatar', 'external' or 'upload'
- * @return bool|string False if success (or if memID is empty and password authentication failed), otherwise a string indicating what error occurred
+ * @return false|string False if success (or if memID is empty and password authentication failed), otherwise a string indicating what error occurred
  */
 function profileSaveAvatarData(&$value)
 {

Sources/Profile-View.php

Doc Comments +1 added lines, -1 removed lines

@@ -2605,7 +2605,7 @@
  * @param int $start Which item to start with (for pagination purposes)
  * @param int $items_per_page How many items to show on each page
  * @param string $sort A string indicating how to sort the results
- * @param int $memID The ID of the member
+ * @param string $memID The ID of the member
  * @return array An array of information about the user's group requests
  */
 function list_getGroupRequests($start, $items_per_page, $sort, $memID)

Sources/QueryString.php

Doc Comments +1 added lines, -1 removed lines

@@ -325,7 +325,7 @@
  * Validates a IPv6 address. returns true if it is ipv6.
  *
  * @param string $ip The ip address to be validated
- * @return boolean Whether the specified IP is a valid IPv6 address
+ * @return false|string Whether the specified IP is a valid IPv6 address
  */
 function isValidIPv6($ip)
 {

Sources/RemoveTopic.php

Doc Comments +1 added lines, -1 removed lines

@@ -1491,7 +1491,7 @@
 
 /**
  * Try to determine if the topic has already been deleted by another user.
- * @return bool False if it can't be deleted (recycling not enabled or no recycling board set), true if we've confirmed it can be deleted. Dies with an error if it's already been deleted.
+ * @return boolean|null False if it can't be deleted (recycling not enabled or no recycling board set), true if we've confirmed it can be deleted. Dies with an error if it's already been deleted.
  */
 function removeDeleteConcurrence()
 {

Sources/Reports.php

Doc Comments +1 added lines, -1 removed lines

@@ -1013,7 +1013,7 @@
  * @param string $title The title of the separator
  * @param null|string $custom_table The ID of the custom table
  *
- * @return void|bool Returns false if there are no tables
+ * @return null|false Returns false if there are no tables
  */
 function addSeparator($title = '', $custom_table = null)
 {

Sources/ScheduledTasks.php

Doc Comments +1 added lines, -1 removed lines

@@ -807,7 +807,7 @@
 /**
  * Send a group of emails from the mail queue.
  *
- * @param bool|int $number The number to send each loop through or false to use the standard limits
+ * @param integer $number The number to send each loop through or false to use the standard limits
  * @param bool $override_limit Whether to bypass the limit
  * @param bool $force_send Whether to forcibly send the messages now (useful when using cron jobs)
  * @return bool Whether things were sent