#189 Web Interface / Multiple Priority Change

open
nobody
None
5
2005-11-05
2005-11-05
Felix
No

Hi,
I'm experiencing a bug with ABC's Web Interface.
Following this document:
http://pingpong-abc.sourceforge.net/protocol_v3.txt
A query to update priorities fo several torrents should
look like this:
PRIORITY|<hash>,<priority>|<hash>,<priority>

This however returns: Error=Command should end with |
(This is also returns if the above query is submitted
with ending |)

Looking throgh your code i found this message to come
from here:
try:
cmd, info = cmdline.split("|")
except:
#Bad Command
conn.send("Feedback\nError=Command should end with |")
conn.close()
continue

I'm not experienced in Python at all, but this looks to
me, as if the split would always fail if there were
more or less than exactly one "|". (It DOES work if you
only update ONE priority)

I guess the easiest solution would be to change the
"priority-delimiters" to something else.. like ";", so
it would look like this:
PRIORITY|<hash>,<priority>;<hash>,<priority>

Thank you.

Discussion