RESOLVED [0.7.3b1] can't save changed server retention time

[teracow]

Hi all,

using 0.7.3Beta1

I am unable to change the server retention time.

I enter ANY number in the [Retention time] field, then press [Save Changes], the button then changes to [Failed]. If I exit this screen, then navigate to this field again, it shows the old value.

here's /var/logs/sabnzbd.log

Code: Select all

2012-07-28 10:24:05,434::DEBUG::[_cplogging:55] [28/Jul/2012:10:24:05] HTTP Traceback (most recent call last):
  File "/usr/share/sabnzbdplus/cherrypy/_cprequest.py", line 618, in respond
    cherrypy.response.body = self.handler()
  File "/usr/share/sabnzbdplus/cherrypy/_cpdispatch.py", line 25, in __call__
    return self.callable(*self.args, **self.kwargs)
  File "/usr/share/sabnzbdplus/sabnzbd/interface.py", line 1484, in saveServer
    return handle_server(kwargs, self.__root)
  File "/usr/share/sabnzbdplus/sabnzbd/interface.py", line 1583, in handle_server
    svr.set_dict(kwargs)
  File "/usr/share/sabnzbdplus/sabnzbd/config.py", line 360, in set_dict
    exec 'self.%s.set(value)' % kw
  File "<string>", line 1, in <module>
  File "/usr/share/sabnzbdplus/sabnzbd/config.py", line 164, in set
    self._Option__set(int(value))
TypeError: int() argument must be a string or a number, not 'list'

Anyone else seen this?

[sab12397]

I was going to create a new thread for my error that I had not had chance to post yet but as it is in the same area I will add to this one.

The error was displayed when I first updated to SABnzbd 073b1 and I tried to save server settings.

I ended up shutting down SABnabd and editing the SABnzbd.ini file directly.

Code: Select all

2012-07-25 18:07:46,472::DEBUG::[_cplogging:55] [25/Jul/2012:20:00:46] HTTP Traceback (most recent call last):
  File "/home/user/sabnzbd/cherrypy/_cprequest.py", line 618, in respond
    cherrypy.response.body = self.handler()
  File "/home/user/sabnzbd/cherrypy/_cpdispatch.py", line 25, in __call__
    return self.callable(*self.args, **self.kwargs)
  File "/home/user/sabnzbd/sabnzbd/interface.py", line 1484, in saveServer
    return handle_server(kwargs, self.__root)
  File "/home/user/sabnzbd/sabnzbd/interface.py", line 1583, in handle_server
    svr.set_dict(kwargs)
  File "/home/user/sabnzbd/sabnzbd/config.py", line 360, in set_dict
    exec 'self.%s.set(value)' % kw
  File "<string>", line 1, in <module>
  File "/home/user/sabnzbd/sabnzbd/config.py", line 164, in set
    self._Option__set(int(value))
TypeError: int() argument must be a string or a number, not 'list'

[shypike]

I cannot reproduce your problem.
Which skin (Plush, smpl, Classic) are you using and are you using
the new "Config" pages?

[teracow]

I'm using "Plush - gold".

I guess it's the new config page - it looks different to the previous version. I now have to click a [Show details] button to see my server info.

[shypike]

OK, so we're talking about the same test situation.
Nevertheless, I cannot reproduce the problem. I tried Firefox 14, Safari 5, Chrome-current and IE9.
What's you're browser and OS?
Have you tried clearing the browser cache?

[sab12397]

I have just tried saving my server settings again and the "Save Changes" button thinks for a couple of seconds and then the button text changes to "Failed". FF14.0.1, Ubuntu, clearing cache makes no difference.

Code: Select all

File "/home/user/sabnzbd/cherrypy/_cprequest.py", line 618, in respond
    cherrypy.response.body = self.handler()
  File "/home/user/sabnzbd/cherrypy/_cpdispatch.py", line 25, in __call__
    return self.callable(*self.args, **self.kwargs)
  File "/home/user/sabnzbd/sabnzbd/interface.py", line 1484, in saveServer
    return handle_server(kwargs, self.__root)
  File "/home/user/sabnzbd/sabnzbd/interface.py", line 1583, in handle_server
    svr.set_dict(kwargs)
  File "/home/user/sabnzbd/sabnzbd/config.py", line 360, in set_dict
    exec 'self.%s.set(value)' % kw
  File "<string>", line 1, in <module>
  File "/home/user/sabnzbd/sabnzbd/config.py", line 164, in set
    self._Option__set(int(value))
TypeError: int() argument must be a string or a number, not 'list'

[teracow]

same deal as sab12397

• Lubuntu v12.04
• Firefox v14.0.1
• sabnzbdplus v0.7.3~beta1-0ubuntu1~jcfp1~precise
• sabnzbdplus-theme-smpl (same)
• sabnzbdplus-theme-mobile (same)
• sabnzbdplus-theme-classic (same)
• sabnzbdplus-theme-plush (same)

also tried access from other *buntus with different browsers prior to reporting this. First noticed this bug when accessing sab from my android tablet.

tried clearing cache but no difference.

[shypike]

It's not the retention time at all, but another error.
When you change any parameter of an already enabled server, the bug triggers.
The fact that there are now two enable fields will send a list of "enable" fields to SABnzbd
instead of a single value. That is what is causing the bug.

Work-around for now:
Disable the server, change the parameters, enable the server.

If you're into some hacking.
Find the file interfaces/Config/templates/config_server.tmpl
and change the first part of line 91 from

Code: Select all

<td><input type="checkbox" class="toggleServerCheckbox" name="enable" value="1"

to

Code: Select all

<td><input type="checkbox" class="toggleServerCheckbox" name="q_enable" value="1"

[teracow]

Yep, that's fixed it. Did your suggested mod to [config_server.tmpl] and now it saves that field without error.

Thanks!

btw: should I mark this as resolved?

[shypike]

btw: should I mark this as resolved?

That's up to you.