Yoshi2889 / SMF2.1

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)
 {

Sources/Subs-Db-postgresql.php

Doc Comments +1 added lines, -2 removed lines

@@ -447,7 +447,6 @@ 
 
 /**
  * affected_rows
- * @param resource $connection
  */
 function smf_db_affected_rows($result = null)
 {
@@ -790,7 +789,7 @@ 
  *
  * @param string $db_name The database name
  * @param resource $db_connection The database connection
- * @return true Always returns true
+ * @return boolean Always returns true
  */
 function smf_db_select_db($db_name, $db_connection)
 {

Sources/Subs-Graphics.php

Doc Comments +1 added lines, -1 removed lines

@@ -170,7 +170,7 @@
  *
  * @param string $fileName The path to the file
  * @param int $preferred_format The preferred format - 0 to automatically determine, 1 for gif, 2 for jpg, 3 for png, 6 for bmp and 15 for wbmp
- * @return boolean Whether the reencoding was successful
+ * @return false|null Whether the reencoding was successful
  */
 function reencodeImage($fileName, $preferred_format = 0)
 {

Sources/Subs-Membergroups.php

Doc Comments +1 added lines

@@ -243,6 +243,7 @@
  * @param null|array The groups to remove the member(s) from. If null, the specified members are stripped from all their membergroups.
  * @param bool $permissionCheckDone Whether we've already checked permissions prior to calling this function
  * @param bool $ignoreProtected Whether to ignore protected groups
+ * @param integer $groups
  * @return bool Whether the operation was successful
  */
 function removeMembersFromGroups($members, $groups = null, $permissionCheckDone = false, $ignoreProtected = false)

Sources/Subs-Members.php

Doc Comments +1 added lines, -1 removed lines

@@ -842,7 +842,7 @@
  * @param int $current_ID_MEMBER The ID of the current member (to avoid false positives with the current member)
  * @param bool $is_name Whether we're checking against reserved names or just usernames
  * @param bool $fatal Whether to die with a fatal error if the name is reserved
- * @return bool|void False if name is not reserved, otherwise true if $fatal is false or dies with a fatal_lang_error if $fatal is true
+ * @return boolean False if name is not reserved, otherwise true if $fatal is false or dies with a fatal_lang_error if $fatal is true
  */
 function isReservedName($name, $current_ID_MEMBER = 0, $is_name = true, $fatal = true)
 {

Sources/Subs-Menu.php

Doc Comments +1 added lines, -1 removed lines

@@ -286,7 +286,7 @@
 /**
  * Delete a menu.
  * @param string $menu_id The ID of the menu to destroy or 'last' for the most recent one
- * @return bool|void False if the menu doesn't exist, nothing otherwise
+ * @return false|null False if the menu doesn't exist, nothing otherwise
  */
 function destroyMenu($menu_id = 'last')
 {

Sources/Subs-Package.php

Doc Comments +3 added lines, -3 removed lines

@@ -28,7 +28,7 @@ 
  * @param bool $single_file If true returns the contents of the file specified by destination if it exists
  * @param bool $overwrite Whether to overwrite existing files
  * @param null|array $files_to_extract Specific files to extract
- * @return array|false An array of information about extracted files or false on failure
+ * @return string An array of information about extracted files or false on failure
  */
 function read_tgz_file($gzfilename, $destination, $single_file = false, $overwrite = false, $files_to_extract = null)
 {
@@ -59,7 +59,7 @@ 
  * @param bool $single_file Whether to only extract a single file
  * @param bool $overwrite Whether to overwrite existing data
  * @param null|array $files_to_extract If set, only extracts the specified files
- * @return array|false An array of information about the extracted files or false on failure
+ * @return string An array of information about the extracted files or false on failure
  */
 function read_tgz_data($gzfilename, $destination, $single_file = false, $overwrite = false, $files_to_extract = null)
 {
@@ -3244,7 +3244,7 @@ 
 	 * http://www.php.net/crc32#79567
 	 *
 	 * @param string $number
-	 * @return string The crc32
+	 * @return integer The crc32
 	 */
 	function smf_crc32($number)
 	{