From 72981cdc309793a8d5ef07bf05078e04f188ad2a Mon Sep 17 00:00:00 2001 From: Romain Bignon Date: Tue, 18 May 2010 12:53:52 +0200 Subject: [PATCH] do not use retry() with submit() and follow_link(), because the state is lost --- weboob/tools/browser.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/weboob/tools/browser.py b/weboob/tools/browser.py index e27b8c30..05b3ad54 100644 --- a/weboob/tools/browser.py +++ b/weboob/tools/browser.py @@ -208,11 +208,10 @@ class BaseBrowser(mechanize.Browser): return mechanize.Browser.open(self, *args, **kwargs) except (mechanize.response_seek_wrapper, urllib2.HTTPError, urllib2.URLError), e: raise BrowserUnavailable('%s (url="%s")' % (e, args and args[0] or 'None')) - except mechanize.BrowserStateError: + except (mechanize.BrowserStateError, BrowserRetry): self.home() return mechanize.Browser.open(self, *args, **kwargs) - @retry(BrowserUnavailable, tries=3) def submit(self, *args, **kwargs): """ Submit the selected form. @@ -229,7 +228,6 @@ class BaseBrowser(mechanize.Browser): def is_on_page(self, pageCls): return isinstance(self.page, pageCls) - @retry(BrowserUnavailable, tries=3) def follow_link(self, *args, **kwargs): try: self._change_location(mechanize.Browser.follow_link(self, *args, **kwargs))