Opened 13 years ago
Closed 13 years ago
#1856 closed patch (Fixed)
deluge-console: Add option to sort items in "info" command
Reported by: | eirikba | Owned by: | Cas |
---|---|---|---|
Priority: | minor | Milestone: | 1.3.2 |
Component: | Console UI | Version: | 1.3-stable |
Keywords: | Cc: |
Description
This patch adds a --sort option to the "info" command of deluge-console. This option decides which order to display the torrents.
Attachments (2)
Change History (7)
Changed 13 years ago by eirikba
comment:1 Changed 13 years ago by andar
comment:2 Changed 13 years ago by eirikba
Yes, --sort-reverse sounds better. Will do.
The 'keys' array is what I use for sorting. But I guess it is better to use the 'sorted' function (on status.items()). I'll try that.
Changed 13 years ago by eirikba
comment:3 Changed 13 years ago by eirikba
A side effect of this updated patch is that the list is now always sorted (by default on name). I think that may be a feature :)
comment:4 Changed 13 years ago by Cas
- Owner set to Cas
- Status changed from new to accepted
comment:5 Changed 13 years ago by Cas
- Component changed from other to console
- Milestone changed from Future to 1.3.2
- Resolution set to fixed
- Status changed from accepted to closed
Added to 1.3-stable a549eac
Note: See
TracTickets for help on using
tickets.
Instead of using the _rev suffix, I would suggest you change this to include a --sort-reverse option instead and factor the code accordingly.
Also, you can remove the line:
Unless I'm missing some sort of point to this?