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

Change subject: Avoid exception in ApiHardMessages
......................................................................


Avoid exception in ApiHardMessages

Bug: 46227
Change-Id: If71bfd3af65673e325dfc912ff416e59d936e407
---
M api/ApiHardMessages.php
1 file changed, 4 insertions(+), 0 deletions(-)

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



diff --git a/api/ApiHardMessages.php b/api/ApiHardMessages.php
index 20b2ea6..9a80d84 100644
--- a/api/ApiHardMessages.php
+++ b/api/ApiHardMessages.php
@@ -31,6 +31,10 @@
                }
 
                $handle = new MessageHandle( $title );
+               if ( !$handle->isValid() ) {
+                       $this->dieUsage( 'Invalid title', 'invalidtitle' );
+               }
+
                $baseTitle = Title::makeTitle( $title->getNamespace(),
                        $handle->getKey() . '/' . 
$handle->getGroup()->getSourceLanguage() );
                $revision = Revision::newFromTitle( $baseTitle );

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

Gerrit-MessageType: merged
Gerrit-Change-Id: If71bfd3af65673e325dfc912ff416e59d936e407
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Translate
Gerrit-Branch: master
Gerrit-Owner: Nikerabbit <[email protected]>
Gerrit-Reviewer: Siebrand <[email protected]>
Gerrit-Reviewer: jenkins-bot

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

Reply via email to