albertlast / 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

@@ -559,7 +559,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)
 {
@@ -754,7 +754,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

@@ -457,7 +457,6 @@ 
 
 /**
  * affected_rows
- * @param resource $connection
  */
 function smf_db_affected_rows($result = null)
 {
@@ -800,7 +799,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-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)
 {
@@ -3251,7 +3251,7 @@ 
 	 * http://www.php.net/crc32#79567
 	 *
 	 * @param string $number
-	 * @return string The crc32
+	 * @return integer The crc32
 	 */
 	function smf_crc32($number)
 	{

Sources/Subs-Post.php

Doc Comments +4 added lines, -4 removed lines

@@ -1445,7 +1445,7 @@ 
  * @param resource $socket Socket to send on
  * @param string $code The expected response code
  * @param string $response The response from the SMTP server
- * @return bool Whether it responded as such.
+ * @return string|boolean Whether it responded as such.
  */
 function server_parse($message, $socket, $code, &$response = null)
 {
@@ -2259,7 +2259,7 @@ 
  * @param array $msgs Array of message ids
  * @param bool $approve Whether to approve the posts (if false, posts are unapproved)
  * @param bool $notify Whether to notify users
- * @return bool Whether the operation was successful
+ * @return null|boolean Whether the operation was successful
  */
 function approvePosts($msgs, $approve = true, $notify = true)
 {
@@ -2512,7 +2512,7 @@ 
  *
  * @param array $topics Array of topic ids
  * @param bool $approve Whether to approve the topics. If false, unapproves them instead
- * @return bool Whether the operation was successful
+ * @return null|boolean Whether the operation was successful
  */
 function approveTopics($topics, $approve = true)
 {
@@ -2902,7 +2902,7 @@ 
  *
  * @param resource $dict An enchant or pspell dictionary resource set up by {@link spell_init()}
  * @param string $word A word to check the spelling of
- * @return bool Whether or not the specified word is spelled properly
+ * @return boolean|null Whether or not the specified word is spelled properly
  */
 function spell_check($dict, $word)
 {