jenkins-bot has submitted this change and it was merged.

Change subject: Page images return non-free images by default

Page images return non-free images by default

Page images was updated to have a split between the 'best' page
image, and the best free page image. Unfortunately the deployment
plan didn't take into account that the default 'free' would be
pointing to an unpopulated page prop, which will not be populated
until LinksUpdate has run for every page on every wiki which could
take weeks or months.

To restore some semblance of order, make the default point at the
currently populated field. A followup will need to be done to
populate the appropriate field.

Bug: T152155
Change-Id: I1d35e965dc37c8c4ecdcc43313b3198e951e1978
M includes/ApiQueryPageImages.php
M tests/phpunit/ApiQueryPageImagesTest.php
2 files changed, 2 insertions(+), 2 deletions(-)

  EBernhardson: Looks good to me, approved
  jenkins-bot: Verified

diff --git a/includes/ApiQueryPageImages.php b/includes/ApiQueryPageImages.php
index 4a207d8..78f5165 100644
--- a/includes/ApiQueryPageImages.php
+++ b/includes/ApiQueryPageImages.php
@@ -249,7 +249,7 @@
                        'license' => [
                                ApiBase::PARAM_TYPE => [ 
                                ApiBase::PARAM_ISMULTI => false,
-                               ApiBase::PARAM_DFLT => self::PARAM_LICENSE_FREE,
+                               ApiBase::PARAM_DFLT => self::PARAM_LICENSE_ANY,
                        'continue' => [
                                ApiBase::PARAM_TYPE => 'integer',
diff --git a/tests/phpunit/ApiQueryPageImagesTest.php 
index 9eab2fb..1359279 100644
--- a/tests/phpunit/ApiQueryPageImagesTest.php
+++ b/tests/phpunit/ApiQueryPageImagesTest.php
@@ -105,7 +105,7 @@
                $this->assertContainsOnly( 'array', $params );
                $this->assertArrayHasKey( 'license', $params );
                $this->assertEquals( $params['license'][\ApiBase::PARAM_TYPE], 
['free', 'any'] );
-               $this->assertEquals( $params['license'][\ApiBase::PARAM_DFLT], 
'free' );
+               $this->assertEquals( $params['license'][\ApiBase::PARAM_DFLT], 
'any' );
$params['license'][\ApiBase::PARAM_ISMULTI], false );

To view, visit
To unsubscribe, visit

Gerrit-MessageType: merged
Gerrit-Change-Id: I1d35e965dc37c8c4ecdcc43313b3198e951e1978
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/PageImages
Gerrit-Branch: wmf/1.29.0-wmf.4
Gerrit-Owner: EBernhardson <>
Gerrit-Reviewer: EBernhardson <>
Gerrit-Reviewer: jenkins-bot <>

MediaWiki-commits mailing list

Reply via email to