From 320b86b57817ee15ac01a831ec493982a925f504 Mon Sep 17 00:00:00 2001 From: Jerome Berthier Date: Wed, 15 May 2019 10:00:31 +0200 Subject: [PATCH] [banquepopulaire] raise BrowserIncorrectPassword instead of BrowserPasswordExpired When there is a virtual keyboard the password must be digits only. It is more clear to return a wrong pass than a password expired. --- modules/banquepopulaire/pages.py | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/modules/banquepopulaire/pages.py b/modules/banquepopulaire/pages.py index ea16731a3d..c061dabc83 100644 --- a/modules/banquepopulaire/pages.py +++ b/modules/banquepopulaire/pages.py @@ -31,7 +31,7 @@ from weboob.browser.filters.standard import CleanText, CleanDecimal, Regexp, Eval, Date, Field from weboob.browser.filters.html import Attr, Link, AttributeNotFound from weboob.browser.filters.json import Dict -from weboob.exceptions import BrowserUnavailable, BrowserIncorrectPassword, ActionNeeded, BrowserPasswordExpired +from weboob.exceptions import BrowserUnavailable, BrowserIncorrectPassword, ActionNeeded from weboob.browser.pages import HTMLPage, LoggedPage, FormNotFound, JsonPage, RawPage, XMLPage @@ -387,9 +387,8 @@ def login(self, login, password): if v[0].get('virtualKeyboard'): if not password.isdigit(): - # Users who get virtualkeyboard must change their passwords - # If there are letters in the password it means they did not do it. - raise BrowserPasswordExpired() + # Users who get the virtual keyboard needs a password with digits only + raise BrowserIncorrectPassword() password = self.virtualkeyboard(vk_obj=v[0]['virtualKeyboard'], password=password) -- GitLab