diff --git a/modules/bnporc/deprecated/pro.py b/modules/bnporc/deprecated/pro.py index 1b456740..d76f92c0 100644 --- a/modules/bnporc/deprecated/pro.py +++ b/modules/bnporc/deprecated/pro.py @@ -101,7 +101,7 @@ class ProAccountHistory(Page): except IndexError: return - if not 'checked' in checkbox.attrib: + if 'checked' not in checkbox.attrib: self.browser.select_form(name='formtri') self.browser['szTriDate'] = ['date'] self.browser['szTriRub'] = [] diff --git a/modules/bnporc/pp/browser.py b/modules/bnporc/pp/browser.py index 7a8afc99..d9893baa 100644 --- a/modules/bnporc/pp/browser.py +++ b/modules/bnporc/pp/browser.py @@ -51,7 +51,7 @@ class JsonBrowserMixin(object): def open(self, *args, **kwargs): if isJSON(kwargs.get('data')): kwargs['data'] = json.dumps(kwargs['data'][1]) - if not 'headers' in kwargs: + if 'headers' not in kwargs: kwargs['headers'] = {} kwargs['headers']['Content-Type'] = 'application/json' diff --git a/modules/boursorama/pages/accounts_list.py b/modules/boursorama/pages/accounts_list.py index ad9778fc..1f0c6d20 100644 --- a/modules/boursorama/pages/accounts_list.py +++ b/modules/boursorama/pages/accounts_list.py @@ -116,5 +116,5 @@ class AccountsList(Page): account.balance = Decimal(0) else: # because of some weird useless - if account.id is not None and (not account._link_id or not 'moneycenter' in account._link_id): + if account.id is not None and (not account._link_id or 'moneycenter' not in account._link_id): yield account diff --git a/modules/bred/bred/browser.py b/modules/bred/bred/browser.py index 5dea79a5..d28fde62 100644 --- a/modules/bred/bred/browser.py +++ b/modules/bred/bred/browser.py @@ -89,7 +89,7 @@ class BredBrowser(DomainBrowser): a.currency = poste['montantTitres']['monnaie']['code'].strip() yield a - if not 'libelle' in poste: + if 'libelle' not in poste: continue a.label = ' '.join([content['intitule'].strip(), poste['libelle'].strip()]) diff --git a/modules/lcl/pages.py b/modules/lcl/pages.py index 474089e1..00f51f54 100644 --- a/modules/lcl/pages.py +++ b/modules/lcl/pages.py @@ -233,7 +233,7 @@ class AccountHistoryPage(LoggedPage, HTMLPage): return self.parent.get_colnum('date') is not None and \ len(self.el.findall('td')) >= 3 and \ self.el.get('class') and \ - not 'tableTr' in self.el.get('class') + 'tableTr' not in self.el.get('class') def validate(self, obj): if obj.category == 'RELEVE CB': diff --git a/modules/tinder/module.py b/modules/tinder/module.py index 8df11e9e..059f8aa9 100644 --- a/modules/tinder/module.py +++ b/modules/tinder/module.py @@ -98,7 +98,7 @@ class TinderModule(Module, CapMessages, CapMessagesPost, CapDating): def iter_threads(self): for thread in self.browser.get_threads(): - if not 'person' in thread: + if 'person' not in thread: # The account has been removed, probably because it was a # spammer. continue diff --git a/weboob/browser/browsers.py b/weboob/browser/browsers.py index f4254880..0892091e 100644 --- a/weboob/browser/browsers.py +++ b/weboob/browser/browsers.py @@ -794,7 +794,7 @@ class APIBrowser(DomainBrowser): def open(self, *args, **kwargs): if 'data' in kwargs: kwargs['data'] = json.dumps(kwargs['data']) - if not 'headers' in kwargs: + if 'headers' not in kwargs: kwargs['headers'] = {} kwargs['headers']['Content-Type'] = 'application/json' diff --git a/weboob/browser/elements.py b/weboob/browser/elements.py index 7d3f0fb9..9b263509 100644 --- a/weboob/browser/elements.py +++ b/weboob/browser/elements.py @@ -298,7 +298,7 @@ class TableElement(ListElement): for el in self.el.xpath(self.head_xpath): title = self.cleaner.clean(el).lower() for name, titles in columns.iteritems(): - if title in titles and not name in self._cols: + if title in titles and name not in self._cols: self._cols[name] = colnum try: colnum += int(el.attrib.get('colspan', 1))