Fix ApiQuerySiteInfo with siprop=rightsinfo to not die if your wgRightsPage is null
authorAlex Monk <krenair@gmail.com>
Mon, 6 Apr 2015 17:24:15 +0000 (18:24 +0100)
committerAlex Monk <krenair@gmail.com>
Mon, 6 Apr 2015 17:31:51 +0000 (18:31 +0100)
Follows-up I2b36b7a3

Change-Id: I7d76cd48021a0097ce5e11f4ad4100cb85ee5ad0

includes/api/ApiQuerySiteinfo.php

index 5ac1036..d4f7e6a 100644 (file)
@@ -674,8 +674,14 @@ class ApiQuerySiteinfo extends ApiQueryBase {
 
        protected function appendRightsInfo( $property ) {
                $config = $this->getConfig();
-               $title = Title::newFromText( $config->get( 'RightsPage' ) );
-               $url = $title ? wfExpandUrl( $title->getFullURL(), PROTO_CURRENT ) : $config->get( 'RightsUrl' );
+               $rightsPage = $config->get( 'RightsPage' );
+               if ( is_string( $rightsPage ) ) {
+                       $title = Title::newFromText( $rightsPage );
+                       $url = wfExpandUrl( $title, PROTO_CURRENT );
+               } else {
+                       $title = false;
+                       $url = $config->get( 'RightsUrl' );
+               }
                $text = $config->get( 'RightsText' );
                if ( !$text && $title ) {
                        $text = $title->getPrefixedText();