#1858: deluge-console: Add command to show and set per-torrent options
---------------------+------------------------------------------------------
Reporter: eirikba | Owner: Cas
Type: patch | Status: assigned
Priority: minor | Milestone: 1.3.x
Component: console | Version: git 1.3-stable
Keywords: |
---------------------+------------------------------------------------------
Comment(by Cas):
The torrent names rather than ids would be better in success output for
the user:
{{{
- self.console.write('Setting %s to %s for torrents %s...' % (key,
val, torrent_ids))
+ self.console.write('Setting %s to %s for torrent(s): %s' % (key,
val,
+ ', '.join(self.console.get_torrent_name(ids) for ids in
torrent_ids)))
}}}
I think nogare is correct in that using * to explicitly set all torrents
makes more sense but it may not be a crucially needed addition so could
maybe just leave it.
I discussed with nogare in irc about the distinction between config and
manage setting all torrents.
Using * to list all torrents does make sense but it would need to
documented and the same change should be applied to other commands such as
info.
--
Ticket URL: <http://dev.deluge-torrent.org/ticket/1858#comment:9>
Deluge <http://deluge-torrent.org/>
Deluge project
--
You received this message because you are subscribed to the Google Groups
"Deluge Dev" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/deluge-dev?hl=en.
