jenkins-bot has submitted this change and it was merged.
Change subject: 'Main Page' might not exist, use localized name
......................................................................
'Main Page' might not exist, use localized name
Change-Id: I6b034b3ebb9978749e91f5f3bd6e862bdd2acf91
---
M includes/CirrusSearchUpdatePagesJob.php
1 file changed, 3 insertions(+), 4 deletions(-)
Approvals:
Bartosz Dziewoński: Looks good to me, approved
jenkins-bot: Verified
diff --git a/includes/CirrusSearchUpdatePagesJob.php
b/includes/CirrusSearchUpdatePagesJob.php
index 393c281..356d365 100644
--- a/includes/CirrusSearchUpdatePagesJob.php
+++ b/includes/CirrusSearchUpdatePagesJob.php
@@ -20,15 +20,14 @@
*/
class CirrusSearchUpdatePagesJob extends Job {
public static function build( $revisions, $checkFreshness ) {
- // We don't have a "title" for this job so we use the Main Page
because it exists.
- $title = Title::newFromText( 'Main Page' );
-
// Strip $revisions down to page ids so we don't put a ton of
stuff in the job queue.
$pageIds = array();
foreach ( $revisions as $rev ) {
$pageIds[] = $rev[ 'id' ];
}
- return new CirrusSearchUpdatePagesJob( $title, array(
+
+ // We don't have a "title" for this job so we use the Main Page
because it exists.
+ return new CirrusSearchUpdatePagesJob( Title::newMainPage(),
array(
'pageIds' => $pageIds,
'checkFreshness' => $checkFreshness,
) );
--
To view, visit https://gerrit.wikimedia.org/r/94281
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I6b034b3ebb9978749e91f5f3bd6e862bdd2acf91
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/CirrusSearch
Gerrit-Branch: master
Gerrit-Owner: Chad <[email protected]>
Gerrit-Reviewer: Bartosz Dziewoński <[email protected]>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
