Merge "Declare visibility for class properties in DatabaseMssql"

diff --combined includes/db/DatabaseMssql.php
index 49b0121,96b4c15..3a89cd5
--- 1/includes/db/DatabaseMssql.php
--- 2/includes/db/DatabaseMssql.php
+++ b/includes/db/DatabaseMssql.php
@@@ -28,11 -28,20 +28,20 @@@
   * @ingroup Database
   */
  class DatabaseMssql extends DatabaseBase {
-       var $mInsertId = null;
-       var $mLastResult = null;
-       var $mAffectedRows = null;
+       /** @var resource */
+       protected $mLastResult = null;
  
-       var $mPort;
+       /** @var int The number of rows affected as an integer */
+       protected $mAffectedRows = null;
+ 
+       /**
+        * @var int Post number for database
+        * @todo Unused and can be removed?
+        */
+       protected $mPort;
+ 
+       /** @var int */
+       private $mInsertId = null;
  
        function cascadingDeletes() {
                return true;
@@@ -88,7 -97,7 +97,7 @@@
  
                # e.g. the class is being loaded
                if ( !strlen( $user ) ) {
 -                      return;
 +                      return null;
                }
  
                $this->close();
@@@ -271,7 -280,7 +280,7 @@@
  
        /**
         * This must be called after nextSequenceVal
-        * @return null
+        * @return int|null
         */
        function insertId() {
                return $this->mInsertId;