Declare visibility for class properties of LBFactory and LBFactorySimple
authorSiebrand Mazeland <s.mazeland@xs4all.nl>
Thu, 26 Dec 2013 22:57:07 +0000 (23:57 +0100)
committerSiebrand <siebrand@wikimedia.org>
Fri, 27 Dec 2013 09:20:56 +0000 (09:20 +0000)
Change-Id: Ia4ffc4e99006384a097ea03f6ee982376a06cde8

includes/db/LBFactory.php

index c4e7976..1289bdc 100644 (file)
  * @ingroup Database
  */
 abstract class LBFactory {
-       /**
-        * @var LBFactory
-        */
-       static $instance;
+       /** @var LBFactory */
+       protected static $instance;
 
        /**
         * Disables all access to the load balancer, will cause all database access
@@ -198,15 +196,14 @@ abstract class LBFactory {
  * A simple single-master LBFactory that gets its configuration from the b/c globals
  */
 class LBFactorySimple extends LBFactory {
+       /** @var LoadBalancer */
+       protected $mainLB;
 
-       /**
-        * @var LoadBalancer
-        */
-       var $mainLB;
-       var $extLBs = array();
+       /** @var LoadBalancer[] */
+       protected $extLBs = array();
 
-       # Chronology protector
-       var $chronProt;
+       /** @var ChronologyProtector */
+       protected $chronProt;
 
        function __construct( $conf ) {
                $this->chronProt = new ChronologyProtector;