From 72aaed43fb8f9b989b8ade0436e4c06a3e1605e8 Mon Sep 17 00:00:00 2001 From: Florent Date: Wed, 18 Mar 2015 20:35:26 +0100 Subject: [PATCH] Fix exception in some cases When self.loggedpage.is_here is false, we try to raise a browserIncorrectPassword(error). Sadly, error is not set. If we are not on the login page, we are in an unknown state (the site is perhaps down?) --- modules/creditcooperatif/perso/browser.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/modules/creditcooperatif/perso/browser.py b/modules/creditcooperatif/perso/browser.py index 4d8e7122..2924f8e7 100644 --- a/modules/creditcooperatif/perso/browser.py +++ b/modules/creditcooperatif/perso/browser.py @@ -18,7 +18,7 @@ # along with weboob. If not, see . from weboob.browser import LoginBrowser, URL, need_login -from weboob.exceptions import BrowserIncorrectPassword +from weboob.exceptions import BrowserIncorrectPassword, BrowserUnavailable from .pages import LoginPage, CreditLoggedPage, AccountsPage, TransactionsPage, TransactionsJSONPage, ComingTransactionsPage @@ -51,6 +51,8 @@ class CreditCooperatif(LoginBrowser): error = self.page.get_error() if error is None: return + else: + raise BrowserUnavailable("not on the login page") raise BrowserIncorrectPassword(error)