jenkins-bot has submitted this change and it was merged. Change subject: Check latest revision in mobileview ......................................................................
Check latest revision in mobileview Bug: 53378 Change-Id: I9862785a26e0ceab5de3bea9b335a81a006acc46 --- M includes/api/ApiMobileView.php 1 file changed, 5 insertions(+), 0 deletions(-) Approvals: Jdlrobson: Looks good to me, but someone else must approve Awjrichards: Looks good to me, approved jenkins-bot: Verified diff --git a/includes/api/ApiMobileView.php b/includes/api/ApiMobileView.php index f3c3eea..ac67659 100644 --- a/includes/api/ApiMobileView.php +++ b/includes/api/ApiMobileView.php @@ -191,6 +191,11 @@ } } $latest = $wp->getLatest(); + if ( !$latest ) { + // https://bugzilla.wikimedia.org/show_bug.cgi?id=53378 + // Title::exists() above doesn't seem to always catch recently deleted pages + $this->dieUsageMsg( array( 'notanarticle', $title->getPrefixedText() ) ); + } if ( $this->file ) { $key = wfMemcKey( 'mf', 'mobileview', self::CACHE_VERSION, $noImages, $latest, $this->noTransform, $this->file->getSha1(), $this->variant ); -- To view, visit https://gerrit.wikimedia.org/r/85980 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9862785a26e0ceab5de3bea9b335a81a006acc46 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/MobileFrontend Gerrit-Branch: master Gerrit-Owner: MaxSem <maxsem.w...@gmail.com> Gerrit-Reviewer: Awjrichards <aricha...@wikimedia.org> Gerrit-Reviewer: Jdlrobson <jrob...@wikimedia.org> Gerrit-Reviewer: jenkins-bot _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits