Avoid PHP 7.2 warnings in DBConRefTest about count() on non-Countable
authorAaron Schulz <aschulz@wikimedia.org>
Sat, 26 May 2018 00:29:17 +0000 (17:29 -0700)
committerAaron Schulz <aschulz@wikimedia.org>
Sat, 26 May 2018 00:29:17 +0000 (17:29 -0700)
Change-Id: Ida81bf998b462f2f6bb2b708df1f15bbc1933db1

includes/libs/rdbms/database/DBConnRef.php

index 9de16c4..7fed3e1 100644 (file)
@@ -33,7 +33,7 @@ class DBConnRef implements IDatabase {
                $this->lb = $lb;
                if ( $conn instanceof Database ) {
                        $this->conn = $conn; // live handle
                $this->lb = $lb;
                if ( $conn instanceof Database ) {
                        $this->conn = $conn; // live handle
-               } elseif ( count( $conn ) >= 4 && $conn[self::FLD_DOMAIN] !== false ) {
+               } elseif ( is_array( $conn ) && count( $conn ) >= 4 && $conn[self::FLD_DOMAIN] !== false ) {
                        $this->params = $conn;
                } else {
                        throw new InvalidArgumentException( "Missing lazy connection arguments." );
                        $this->params = $conn;
                } else {
                        throw new InvalidArgumentException( "Missing lazy connection arguments." );