Opened 15 years ago
Closed 15 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 , 15 years ago
| Attachment: | 0002-Add-sort-option-to-deluge-console-s-info-command.patch added |
|---|
comment:1 by , 15 years ago
comment:2 by , 15 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 , 15 years ago
| Attachment: | 0001-Add-sort-option-to-deluge-console-s-info-command.patch added |
|---|
comment:3 by , 15 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 , 15 years ago
| Owner: | set to |
|---|---|
| Status: | new → accepted |
comment:5 by , 15 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?