Page MenuHomePhabricator

Allow descending order for list=watchlistraw
Closed, ResolvedPublic

Description

It can be nice to get the raw watchlist items in descending order. Please add a
wrdir param with 'ascending' and 'descending'.

Thanks.


Version: 1.20.x
Severity: enhancement

Details

Reference
bz32384
TitleReferenceAuthorSource BranchDest Branch
Make some strlen() calls in OAuth code safe for PHP 8.0repos/phabricator/extensions!16aklapperT343923strlenPhp81wmf/stable
Customize query in GitLab

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 12:02 AM
bzimport set Reference to bz32384.
bzimport added a subscriber: Unknown Object (MLST).

Created attachment 9636
Patch against trunk

attachment watchlistraw_dir.patch ignored as obsolete

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.

(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.

Created attachment 9732
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..

Attached: