alexcoe / remote-torrent-adder

Automatically exported from code.google.com/p/remote-torrent-adder
0 stars 2 forks source link

Opening settings page resets directory and label lists. #123

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
There seems to be an issue with the logic in your add-on. If you go to the 
settings page and change something, you lose the directory and label list 
drop-down values.

Doing some debugging led me to this (content_rta.js: 84-85):
var dirlist = server["dirlist"] && JSON.parse(server["dirlist"]);
var labellist = server["labellist"] && JSON.parse(server["labellist"]);

What I noticed is that if you change "server" to "settings" here it will work 
fine. Once you add the torrent, the next time the pop-up opens it works fine as 
well. I think the config page doesn't add the current dirlist or labellist to 
the 'server' field when the settings are saved, so the next time around the 
dirlist will be empty. 

Original issue reported on code.google.com by wscottjo...@gmail.com on 20 Nov 2014 at 9:47

GoogleCodeExporter commented 9 years ago
yes, that's a bug, but unfortunately your solution doesn't represent expected 
behaviour. i'll have to think of something else for this.

Original comment by jul...@gmail.com on 21 Nov 2014 at 2:11

GoogleCodeExporter commented 9 years ago
Issue 126 has been merged into this issue.

Original comment by jul...@gmail.com on 27 Nov 2014 at 2:37