Opened 14 years ago
Closed 14 years ago
#1856 closed patch (Fixed)
deluge-console: Add option to sort items in "info" command
Reported by: | eirikba | Owned by: | Calum |
---|---|---|---|
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)
by , 14 years ago
Attachment: | 0002-Add-sort-option-to-deluge-console-s-info-command.patch added |
---|
comment:1 by , 14 years ago
comment:2 by , 14 years ago
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.
by , 14 years ago
Attachment: | 0001-Add-sort-option-to-deluge-console-s-info-command.patch added |
---|
comment:3 by , 14 years ago
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 by , 14 years ago
Owner: | set to |
---|---|
Status: | new → accepted |
comment:5 by , 14 years ago
Component: | other → console |
---|---|
Milestone: | Future → 1.3.2 |
Resolution: | → fixed |
Status: | accepted → 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?