From 3c736f711a7b7fc407654c21b26bfac2979dacd3 Mon Sep 17 00:00:00 2001 From: Siebrand Mazeland Date: Fri, 27 Dec 2013 00:45:36 +0100 Subject: [PATCH] Declare visibility for class properties in DatabaseSqlite $mName appears to be unused. Change-Id: Id9ef59cf070e379413d16a8d4456fd93df296442 --- includes/db/DatabaseSqlite.php | 19 ++++++++++++++----- 1 file changed, 14 insertions(+), 5 deletions(-) diff --git a/includes/db/DatabaseSqlite.php b/includes/db/DatabaseSqlite.php index 89f0818e5e..ea35391d21 100644 --- a/includes/db/DatabaseSqlite.php +++ b/includes/db/DatabaseSqlite.php @@ -26,16 +26,25 @@ * @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 ) { -- 2.20.1