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

Raise exception instances, not classes

Browse source
This commit is contained in:
Laurent Bachelier 2012-02-11 01:04:12 +01:00
commit 32b0895ea6
3 changed files with 6 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

2
modules/aum/contact.py
View file

@ -32,7 +32,7 @@ class FieldBase:
self.key2 = key2 self.key2 = key2
def get_value(self, value, consts): def get_value(self, value, consts):
raise NotImplementedError raise NotImplementedError()
class FieldStr(FieldBase): class FieldStr(FieldBase):
def get_value(self, profile, consts): def get_value(self, profile, consts):

8
modules/cmb/backend.py
View file

@ -107,9 +107,9 @@ class CmbBackend(BaseBackend, ICapBank):
tree = etree.parse(StringIO(data), parser) tree = etree.parse(StringIO(data), parser)
table = tree.xpath('/html/body/table') table = tree.xpath('/html/body/table')
if len(table) == 0: if len(table) == 0:
raise BrokenPageError raise BrokenPageError()
else: else:
raise BrokenPageError raise BrokenPageError()
for tr in table[1].getiterator('tr'): for tr in table[1].getiterator('tr'):
if tr.get('class') != 'LnTit' and tr.get('class') != 'LnTot': if tr.get('class') != 'LnTit' and tr.get('class') != 'LnTot':
@ -189,9 +189,9 @@ class CmbBackend(BaseBackend, ICapBank):
tree = etree.parse(StringIO(data), parser) tree = etree.parse(StringIO(data), parser)
tables = tree.xpath('/html/body/table') tables = tree.xpath('/html/body/table')
if len(tables) == 0: if len(tables) == 0:
raise BrokenPageError raise BrokenPageError()
else: else:
raise BrokenPageError raise BrokenPageError()
i = 0 i = 0

2
modules/pastebin/browser.py
View file

@ -119,7 +119,7 @@ class PastebinBrowser(BaseBrowser):
self._validate_api_response(res) self._validate_api_response(res)
except BadAPIRequest, e: except BadAPIRequest, e:
if str(e) == 'invalid login': if str(e) == 'invalid login':
raise BrowserIncorrectPassword raise BrowserIncorrectPassword()
else: else:
raise e raise e
self.user_key = res self.user_key = res