From f47a95ba1df58ccf9784c47beeaa0702c469b3e1 Mon Sep 17 00:00:00 2001 From: Simo Sorce Date: Tue, 20 May 2014 15:25:29 -0400 Subject: [PATCH] Fix E713 with stricter pep8 error checker Signed-off-by: Simo Sorce --- ipsilon/helpers/ipa.py | 2 +- ipsilon/login/authkrb.py | 2 +- ipsilon/root.py | 2 +- ipsilon/util/page.py | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/ipsilon/helpers/ipa.py b/ipsilon/helpers/ipa.py index 4948624..722b404 100755 --- a/ipsilon/helpers/ipa.py +++ b/ipsilon/helpers/ipa.py @@ -176,7 +176,7 @@ class Installer(object): self.get_keytab(opts) # Forcibly use krb then pam modules - if not 'lm_order' in opts: + if 'lm_order' not in opts: opts['lm_order'] = [] opts['krb'] = 'yes' if 'krb' not in opts['lm_order']: diff --git a/ipsilon/login/authkrb.py b/ipsilon/login/authkrb.py index 7132913..af659e7 100755 --- a/ipsilon/login/authkrb.py +++ b/ipsilon/login/authkrb.py @@ -52,7 +52,7 @@ class KrbError(LoginPageBase): # If we have no negotiate header return whatever mod_auth_kerb # generated and wait for the next request - if not 'WWW-Authenticate' in cherrypy.request.headers: + if 'WWW-Authenticate' not in cherrypy.request.headers: cherrypy.response.status = 401 if self.lm.next_login: diff --git a/ipsilon/root.py b/ipsilon/root.py index 9451d22..c5fc5ae 100755 --- a/ipsilon/root.py +++ b/ipsilon/root.py @@ -33,7 +33,7 @@ sites = dict() class Root(Page): def __init__(self, site, template_env): - if not site in sites: + if site not in sites: sites[site] = dict() if template_env: sites[site]['template_env'] = template_env diff --git a/ipsilon/util/page.py b/ipsilon/util/page.py index 3a01811..1968009 100755 --- a/ipsilon/util/page.py +++ b/ipsilon/util/page.py @@ -38,7 +38,7 @@ def protect(): class Page(object): def __init__(self, site, form=False): - if not 'template_env' in site: + if 'template_env' not in site: raise ValueError('Missing template environment') self._site = site self.basepath = cherrypy.config.get('base.mount', "") -- 2.20.1