nojhan/weboob-devel
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

do not use retry() with submit() and follow_link(), because the state is lost

Browse source
This commit is contained in:
Romain Bignon 2010-05-18 12:53:52 +02:00
commit 72981cdc30
1 changed files with 1 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

4
weboob/tools/browser.py
View file

@ -208,11 +208,10 @@ class BaseBrowser(mechanize.Browser):
return mechanize.Browser.open(self, *args, **kwargs) return mechanize.Browser.open(self, *args, **kwargs)
except (mechanize.response_seek_wrapper, urllib2.HTTPError, urllib2.URLError), e: except (mechanize.response_seek_wrapper, urllib2.HTTPError, urllib2.URLError), e:
raise BrowserUnavailable('%s (url="%s")' % (e, args and args[0] or 'None')) raise BrowserUnavailable('%s (url="%s")' % (e, args and args[0] or 'None'))
except mechanize.BrowserStateError: except (mechanize.BrowserStateError, BrowserRetry):
self.home() self.home()
return mechanize.Browser.open(self, *args, **kwargs) return mechanize.Browser.open(self, *args, **kwargs)
@retry(BrowserUnavailable, tries=3)
def submit(self, *args, **kwargs): def submit(self, *args, **kwargs):
""" """
Submit the selected form. Submit the selected form.
@ -229,7 +228,6 @@ class BaseBrowser(mechanize.Browser):
def is_on_page(self, pageCls): def is_on_page(self, pageCls):
return isinstance(self.page, pageCls) return isinstance(self.page, pageCls)
@retry(BrowserUnavailable, tries=3)
def follow_link(self, *args, **kwargs): def follow_link(self, *args, **kwargs):
try: try:
self._change_location(mechanize.Browser.follow_link(self, *args, **kwargs)) self._change_location(mechanize.Browser.follow_link(self, *args, **kwargs))