Last modified: 2012-04-10 20:25:00 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T34384, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 32384 - Allow descending order for list=watchlistraw
Allow descending order for list=watchlistraw
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
API (Other open bugs)
1.20.x
All All
: Normal enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
: patch
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-11-12 17:22 UTC by db [inactive,noenotif]
Modified: 2012-04-10 20:25 UTC (History)
7 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
Patch against trunk (1.69 KB, patch)
2011-12-07 23:44 UTC, MrBlueSky
Details
Patch against trunk (1.69 KB, patch)
2011-12-17 21:20 UTC, MrBlueSky
Details

Description db [inactive,noenotif] 2011-11-12 17:22:05 UTC
It can be nice to get the raw watchlist items in descending order. Please add a
wrdir param with 'ascending' and 'descending'.

Thanks.
Comment 1 MrBlueSky 2011-12-07 23:44:33 UTC
Created attachment 9636 [details]
Patch against trunk
Comment 2 Brion Vibber 2011-12-16 21:14:11 UTC
I think this needs to reverse the order on the namespaces too, or else you won't get entirely reversed results.

Labeling on the param looks correct, looks like it matches with other things in the parameters used etc.
Comment 3 Roan Kattouw 2011-12-17 16:09:09 UTC
(In reply to comment #2)
> I think this needs to reverse the order on the namespaces too, or else you
> won't get entirely reversed results.
> 
And, don't forget, queries using a mix of ASC/DESC will filesort. So you need to use the same operator (< or >) for both namespace and title, and use ORDER BY wl_namespace DESC, wl_title DESC in the descending case.
Comment 4 MrBlueSky 2011-12-17 21:20:37 UTC
Created attachment 9732 [details]
Patch against trunk

This time also reversing the namespaces. I don't know why I thought it would be a good idea to not do that..
Comment 5 MrBlueSky 2012-04-10 20:24:34 UTC
Done with Gerrit change #4026

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links