Yoshi2889 / SMF2.1

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

@@ -2610,7 +2610,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

@@ -321,7 +321,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

Sources/Subs-Admin.php

Doc Comments +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
 
 /**
  * Get a list of versions that are currently installed on the server.
- * @param array $checkFor An array of what to check versions for - can contain one or more of 'gd', 'imagemagick', 'db_server', 'phpa', 'memcache', 'xcache', 'apc', 'php' or 'server'
+ * @param string[] $checkFor An array of what to check versions for - can contain one or more of 'gd', 'imagemagick', 'db_server', 'phpa', 'memcache', 'xcache', 'apc', 'php' or 'server'
  * @return array An array of versions (keys are same as what was in $checkFor, values are the versions)
  */
 function getServerVersions($checkFor)

Sources/Subs-Auth.php

Doc Comments +1 added lines, -1 removed lines

@@ -150,7 +150,7 @@
  *
  * @param bool $local Whether we want local cookies
  * @param bool $global Whether we want global cookies
- * @return array An array to set the cookie on with domain and path in it, in that order
+ * @return string An array to set the cookie on with domain and path in it, in that order
  */
 function url_parts($local, $global)
 {

Sources/Subs-Db-mysqli.php

Doc Comments +2 added lines, -2 removed lines

@@ -530,7 +530,7 @@ 
  * Backtrace, log, try to fix.
  *
  * @param string $db_string The DB string
- * @param object $connection The connection to use (if null, $db_connection is used)
+ * @param resource $connection The connection to use (if null, $db_connection is used)
  */
 function smf_db_error($db_string, $connection = null)
 {
@@ -725,7 +725,7 @@ 
  * @param array $data The data to insert
  * @param array $keys The keys for the table
  * @param bool $disable_trans Whether to disable transactions
- * @param object $connection The connection to use (if null, $db_connection is used)
+ * @param resource $connection The connection to use (if null, $db_connection is used)
  */
 function smf_db_insert($method = 'replace', $table, $columns, $data, $keys, $disable_trans = false, $connection = null)
 {