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

Change subject: Check that we are not trying to construct a title from null
......................................................................


Check that we are not trying to construct a title from null

Change-Id: I66de96029c4c0cc8a223745ed7000d2fd5d3be00
---
M specials/SpecialTranslationStash.php
1 file changed, 1 insertion(+), 1 deletion(-)

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



diff --git a/specials/SpecialTranslationStash.php 
b/specials/SpecialTranslationStash.php
index 11ed062..0c3e82f 100644
--- a/specials/SpecialTranslationStash.php
+++ b/specials/SpecialTranslationStash.php
@@ -29,7 +29,7 @@
                $this->stash = new TranslationStashStorage( wfGetDB( DB_MASTER 
) );
 
                if ( !$this->hasPermissionToUse() ) {
-                       if ( $this->getUser()->isLoggedIn() ) {
+                       if ( $this->getUser()->isLoggedIn() && 
$wgTranslateSecondaryPermissionUrl ) {
                                $out->redirect(
                                        Title::newFromText( 
$wgTranslateSecondaryPermissionUrl )->getLocalUrl()
                                );

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

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

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

Reply via email to