Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#1099 closed Bug (fixed)

transmission-remote confuses -ph, -pn and -pl options with the -p option

Reported by: Murmex Owned by: charles
Priority: Normal Milestone: 1.30
Component: Daemon Version: 1.22+
Severity: Normal Keywords:
Cc:

Description

When using the short options -ph, -pn and -pl options to set the files priorities, transmission-remote confuses them with the -p options and returns

Not a number: "h"

As the long “--priority-(high|normal|low)” options works, this is clearly an issue caused by the -p option being treated first.

Furthermore, there is a mistake in the transmission-remote help with the following line

-pl --priority-normal <files> Set the files' priorities as low

which should be

-pl --priority-low <files> Set the files' priorities as low

Change History (6)

comment:1 in reply to: ↑ description Changed 13 years ago by Murmex

Furthermore, there is a mistake in the transmission-remote help with the following line

-pl --priority-normal <files> Set the files' priorities as low

which should be

-pl --priority-low <files> Set the files' priorities as low

In fact, it seems that this prevents from setting a low priority as the help content is probably generated automatically and thus, the --priority-low option is not recognized.

comment:2 Changed 13 years ago by charles

  • Milestone changed from None Set to 1.30
  • Status changed from new to assigned

comment:3 Changed 13 years ago by charles

  • Resolution set to fixed
  • Status changed from assigned to closed

Fixed in r6379.

comment:4 Changed 13 years ago by Murmex

  • Resolution fixed deleted
  • Status changed from closed to reopened

in root/trunk/daemon/remote.c, line 74 (as of rev 6380) should be:

    { 902, "priority-low", "Set the files' priorities as low", "pl", 1, "<files>" },

instead of:

    { 902, "priority-normal", "Set the files' priorities as low", "pl", 1, "<files>" },

This is a bit off-topic for this bug but as I had already mentioned it earlier and it is kinda small.

comment:5 Changed 13 years ago by charles

  • Resolution set to fixed
  • Status changed from reopened to closed

fixed in r6383, thanks for spotting this. :)

comment:6 Changed 13 years ago by charles

  • Summary changed from transmission-remote confuses -ph, -pn and -pl (priorities) options with the -p (port) option to transmission-remote confuses -ph, -pn and -pl options with the -p option
Note: See TracTickets for help on using tickets.