Declare visibility for class properties in DatabaseSqlite
authorSiebrand Mazeland <s.mazeland@xs4all.nl>
Thu, 26 Dec 2013 23:45:36 +0000 (00:45 +0100)
committerSiebrand <siebrand@wikimedia.org>
Fri, 27 Dec 2013 09:23:29 +0000 (09:23 +0000)
$mName appears to be unused.

Change-Id: Id9ef59cf070e379413d16a8d4456fd93df296442

includes/db/DatabaseSqlite.php

index 89f0818..ea35391 100644 (file)
  * @ingroup Database
  */
 class DatabaseSqlite extends DatabaseBase {
+       /** @var bool Whether full text is enabled */
        private static $fulltextEnabled = null;
 
-       var $mAffectedRows;
-       var $mLastResult;
-       var $mDatabaseFile;
-       var $mName;
+       /** @var string File name for SQLite database file */
+       public $mDatabaseFile;
+
+       /** @var int The number of rows affected as an integer */
+       protected $mAffectedRows;
+
+       /** @var resource */
+       protected $mLastResult;
 
        /**
-        * @var PDO
+        * @var string
+        * @todo Seems to serve no purpose. Remove?
         */
+       private $mName;
+
+       /** @var PDO */
        protected $mConn;
 
        function __construct( $p = null ) {