Tinaj1234 has uploaded a new change for review.

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

Change subject: Correct sorting of newsletters in Special: Newsletters
......................................................................

Correct sorting of newsletters in Special: Newsletters

Newsletters the user has subscribed come first, followed by
the rest.

Bug: T107557
Change-Id: I267cf468392678cd9f0739e9db1f4fe818ef98f4
---
M includes/specials/pagers/NewsletterTablePager.php
1 file changed, 2 insertions(+), 1 deletion(-)


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

diff --git a/includes/specials/pagers/NewsletterTablePager.php 
b/includes/specials/pagers/NewsletterTablePager.php
index de23adb..fab47ff 100644
--- a/includes/specials/pagers/NewsletterTablePager.php
+++ b/includes/specials/pagers/NewsletterTablePager.php
@@ -98,7 +98,8 @@
        }
 
        public function getDefaultSort() {
-               return 'nl_name';
+               $this->mDefaultDirection = IndexPager::DIR_DESCENDING;
+               return 'current_user_subscribed';
        }
 
        public function isFieldSortable( $field ) {

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I267cf468392678cd9f0739e9db1f4fe818ef98f4
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Newsletter
Gerrit-Branch: master
Gerrit-Owner: Tinaj1234 <[email protected]>

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

Reply via email to