From 90d2135893ebca422b221b0b01342b7c39534816 Mon Sep 17 00:00:00 2001 From: Erik Bernhardson Date: Mon, 18 Jul 2016 10:26:03 -0700 Subject: [PATCH] Update BatchRowIterator constructor to allow multiple tables This already supported multiple tables, but the annotations claimed it didn't. This $table value gets passed on directly to IDatabase::select which takes a string or array, so mark it as such here as well. Change-Id: I28fa61429544e592f90c0855ea59279af897283f --- includes/utils/BatchRowIterator.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/includes/utils/BatchRowIterator.php b/includes/utils/BatchRowIterator.php index 419ee4730d..c7bd395b5d 100644 --- a/includes/utils/BatchRowIterator.php +++ b/includes/utils/BatchRowIterator.php @@ -31,7 +31,7 @@ class BatchRowIterator implements RecursiveIterator { protected $db; /** - * @var string $table The name of the table to read from + * @var string|array $table The name or names of the table to read from */ protected $table; @@ -79,7 +79,7 @@ class BatchRowIterator implements RecursiveIterator { /** * @param IDatabase $db The database to read from - * @param string $table The name of the table to read from + * @param string|array $table The name or names of the table to read from * @param string|array $primaryKey The name or names of the primary key columns * @param integer $batchSize The number of rows to fetch per iteration * @throws MWException -- 2.20.1