Quentin Defenouillere authored Aug 22, 2018 and Romain Bignon committed Sep 16, 2018

Some connections are in wrongpass for the BECM module but the website
does not return an invalid login/password message: it seems that these
credentials are actually Crédit Mutuel customers who picked the wrong
bank. I added an ActionNeeded to suggest them to try with creditmutuel
instead. The "real" wrongpass cases are caught in the on_load() method
of LoginPage.

flake8 --select E713, semi-manual fixing

flake8 --select W391,E302,E301,E304

autopep8 can't fix W391 even though it claims it can.
Fixed using a simple custom script.

weboob.tools.browser -> weboob.deprecated.browser
weboob.tools.parsers -> weboob.deprecated.browser.parsers
weboob.tools.mech -> weboob.deprecated.mech
weboob.browser2 -> weboob.browser
weboob.core.exceptions -> weboob.exceptions

Also, the new tree for browser2 is:

weboob.browser: import weboob.browser.browsers.* and weboob.browser.url.*
weboob.browser.browsers: all browsers (including PagesBrowser and LoginBrowser)
weboob.browser.url: the URL class
weboob.browser.profiles: all Profile classes
weboob.browser.sessions: WeboobSession and FuturesSession
weboob.browser.cookies: that's a cookies thing
weboob.browser.pages: all Page and derivated classes, and Form class
weboob.browser.exceptions: specific browser exceptions
weboob.browser.elements: AbstractElement classes, and 'method' decorator
weboob.browser.filters.*: all filters

Remove useless trailing \
Remove trailing spaces
Add missing empty lines

autopep8 -ir -j2 --select=E301,E302,E502,W291,W293,W391 .

Diff quickly checked.

mechanize does not support embeded optgroups, so it crashes when parsing
another form is the page