[MediaWiki-commits] [Gerrit] mediawiki...Newsletter[master]: Change Return-To URL in Special:Newsletter/xx/announce to 'R...

2016-11-29 Thread jenkins-bot (Code Review) 
jenkins-bot has submitted this change and it was merged.

Change subject: Change Return-To URL in Special:Newsletter/xx/announce to 
'Return to Newsletter:Foo' instead of 'Return to Special:Newsletter/1'- 
Part of migration to ContentHandler
..


Change Return-To URL in Special:Newsletter/xx/announce to 'Return to 
Newsletter:Foo' instead of 'Return to Special:Newsletter/1'
- Part of migration to ContentHandler

Bug: T150096
Change-Id: I82b494dd243250f906090dbaa85df8d5b1421481
---
M includes/specials/SpecialNewsletter.php
1 file changed, 1 insertion(+), 1 deletion(-)

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



diff --git a/includes/specials/SpecialNewsletter.php 
b/includes/specials/SpecialNewsletter.php
index fe6312c..90638d9 100644
--- a/includes/specials/SpecialNewsletter.php
+++ b/includes/specials/SpecialNewsletter.php
@@ -527,7 +527,7 @@
->numParams( 
$this->newsletter->getSubscriberCount() )
->parseAsBlock()
);
-   $out->addReturnTo( $this->getPageTitle( 
$this->newsletter->getId() ) );
+   $out->addReturnTo( Title::makeTitleSafe( NS_NEWSLETTER, 
$this->newsletter->getName() ) );
}
}
 

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I82b494dd243250f906090dbaa85df8d5b1421481
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Newsletter
Gerrit-Branch: master
Gerrit-Owner: MtDu 
Gerrit-Reviewer: 01tonythomas <01tonytho...@gmail.com>
Gerrit-Reviewer: Addshore 
Gerrit-Reviewer: jenkins-bot <>

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

[MediaWiki-commits] [Gerrit] mediawiki...Newsletter[master]: Change Return-To URL in Special:Newsletter/xx/announce to 'R...

2016-11-29 Thread MtDu (Code Review) 
MtDu has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/324361

Change subject: Change Return-To URL in Special:Newsletter/xx/announce to 
'Return to Newsletter:Foo' instead of 'Return to Special:Newsletter/1'- 
Part of migration to ContentHandler
..

Change Return-To URL in Special:Newsletter/xx/announce to 'Return to 
Newsletter:Foo' instead of 'Return to Special:Newsletter/1'
- Part of migration to ContentHandler

Bug: T150096
Change-Id: I82b494dd243250f906090dbaa85df8d5b1421481
---
M includes/specials/SpecialNewsletter.php
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Newsletter 
refs/changes/61/324361/1

diff --git a/includes/specials/SpecialNewsletter.php 
b/includes/specials/SpecialNewsletter.php
index fe6312c..90638d9 100644
--- a/includes/specials/SpecialNewsletter.php
+++ b/includes/specials/SpecialNewsletter.php
@@ -527,7 +527,7 @@
->numParams( 
$this->newsletter->getSubscriberCount() )
->parseAsBlock()
);
-   $out->addReturnTo( $this->getPageTitle( 
$this->newsletter->getId() ) );
+   $out->addReturnTo( Title::makeTitleSafe( NS_NEWSLETTER, 
$this->newsletter->getName() ) );
}
}
 

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I82b494dd243250f906090dbaa85df8d5b1421481
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Newsletter
Gerrit-Branch: master
Gerrit-Owner: MtDu 

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits