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

Change subject: Fix exception on null parser output
......................................................................


Fix exception on null parser output

Bug: 49842
Change-Id: I45d1a94905e7908e19b1fe0aa73e32c46069386a
---
M GeoDataHooks.php
1 file changed, 5 insertions(+), 2 deletions(-)

Approvals:
  MaxSem: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/GeoDataHooks.php b/GeoDataHooks.php
index 29dae54..3bac0c3 100644
--- a/GeoDataHooks.php
+++ b/GeoDataHooks.php
@@ -237,8 +237,11 @@
                $wp = WikiPage::factory( $file->getTitle() );
                $po = new ParserOptions();
                $pout = $wp->getParserOutput( $po );
-               $lu = new LinksUpdate( $file->getTitle(), $pout );
-               self::onLinksUpdate( $lu );
+               if ( $pout ) {
+                       wfDebugLog( 'mobile', __METHOD__ . "(): no parser 
output returned for file {$file->getName()}" );
+                       $lu = new LinksUpdate( $file->getTitle(), $pout );
+                       self::onLinksUpdate( $lu );
+               }
                wfProfileOut( __METHOD__ );
                return true;
        }

-- 
To view, visit https://gerrit.wikimedia.org/r/86724
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I45d1a94905e7908e19b1fe0aa73e32c46069386a
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/GeoData
Gerrit-Branch: master
Gerrit-Owner: MaxSem <[email protected]>
Gerrit-Reviewer: MaxSem <[email protected]>
Gerrit-Reviewer: jenkins-bot

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to