Dvogel hallowelt has uploaded a new change for review.

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

Change subject: Readers: Fixed grid sorting
......................................................................

Readers: Fixed grid sorting

Sorting was not implemented.

Change-Id: I8eef2ecd42d572f3de4ce1c4283669b6363808a2
---
M Readers/Readers.class.php
1 file changed, 32 insertions(+), 13 deletions(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/BlueSpiceExtensions 
refs/changes/43/237343/1

diff --git a/Readers/Readers.class.php b/Readers/Readers.class.php
index f65c7fd..9acd553 100644
--- a/Readers/Readers.class.php
+++ b/Readers/Readers.class.php
@@ -265,7 +265,15 @@
                $sSort = $oStoreParams->getSort( 'MAX(readers_ts)' );
                $sDirection = $oStoreParams->getDirection();
 
-               if ( $sSort == 'user_page' ) $sSort = 'readers_user_name';
+               if ( $sSort == 'user_name' ) {
+                       $sSort = 'readers_user_name';
+               }
+               if ( $sSort == 'user_ts' ) {
+                       $sSort = 'readers_ts';
+               }
+               if ( $sSort == 'user_readers' ) {
+                       $sSort = 'readers_user_name';
+               }
 
                $oDbr = wfGetDB( DB_SLAVE );
                $res = $oDbr->select(
@@ -365,21 +373,32 @@
                $iLimit = $oStoreParams->getLimit();
                $iStart = $oStoreParams->getStart();
                $sSort = $oStoreParams->getSort( 'MAX(readers_ts)' );
+               $sDirection = $oStoreParams->getDirection();
 
-               if ( $sSort == 'user_page' ) $sSort = 'readers_user_name';
+               if ( $sSort == 'pv_page' ) {
+                       $sSort = 'page_title';
+               }
+               if ( $sSort == 'pv_ts' ) {
+                       $sSort = 'MAX( readers_ts )';
+               }
 
                $res = $oDbr->select(
-                               array( 'bs_readers', 'page' ),
-                               array( 'readers_page_id', 'MAX(readers_ts) as 
readers_ts' ),
-                               array( 'readers_user_id' => $iUserID ),
-                               __METHOD__,
-                               array(
-                                       'GROUP BY' => 'readers_page_id',
-                                       'ORDER BY' => 'MAX(readers_ts) DESC',
-                                       'LIMIT' => $iLimit,
-                                       'OFFSET' => $iStart
-                               ),
-                               array( 'page' => array( 'INNER JOIN', 
'readers_page_id = page_id' ) )
+                       array( 'page', 'bs_readers' ),
+                       array(
+                               'page_title', 'readers_page_id', 
'readers_user_name',
+                               'MAX( readers_ts ) as readers_ts'
+                       ),
+                       array(
+                               'readers_page_id = page_id',
+                               'readers_user_id' => $iUserID
+                       ),
+                       __Method__,
+                       array(
+                               'GROUP BY' => 'readers_page_id',
+                               'ORDER BY' => $sSort . " " . $sDirection,
+                               'LIMIT' => $iLimit,
+                               'OFFSET' => $iStart
+                       )
                );
 
                $aPages = array();

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8eef2ecd42d572f3de4ce1c4283669b6363808a2
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/BlueSpiceExtensions
Gerrit-Branch: REL1_23
Gerrit-Owner: Dvogel hallowelt <daniel.vo...@hallowelt.com>

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

Reply via email to