Fix E256 with stricter pep8 error checker

Signed-off-by: Simo Sorce <simo@redhat.com>

diff --git a/ipsilon/login/common.py b/ipsilon/login/common.py
index f0efebd..a576345 100755 (executable)
--- a/ipsilon/login/common.py
+++ b/ipsilon/login/common.py
@@ -108,7 +108,7 @@ class LoginManagerBase(PluginObject):
         if self not in plugins['enabled']:
             return
 
         if self not in plugins['enabled']:
             return
 

-        #remove self from chain
+        # remove self from chain

         root = plugins['root']
         if root.first_login == self:
             root.first_login = self.next_login
         root = plugins['root']
         if root.first_login == self:
             root.first_login = self.next_login

diff --git a/ipsilon/providers/saml2/admin.py b/ipsilon/providers/saml2/admin.py
index 4f33e1b..a9fb9e0 100755 (executable)
--- a/ipsilon/providers/saml2/admin.py
+++ b/ipsilon/providers/saml2/admin.py
@@ -51,8 +51,8 @@ class NewSPAdminPage(Page):
     def POST(self, *args, **kwargs):
 
         if self.user.is_admin:
     def POST(self, *args, **kwargs):
 
         if self.user.is_admin:

-            #TODO: allow authenticated user to create SPs on their own
-            #      set the owner in that case
+            # TODO: allow authenticated user to create SPs on their own
+            #       set the owner in that case

             name = None
             meta = None
             if 'content-type' not in cherrypy.request.headers:
             name = None
             meta = None
             if 'content-type' not in cherrypy.request.headers:

diff --git a/ipsilon/providers/saml2/auth.py b/ipsilon/providers/saml2/auth.py
index 65d795d..036ed5e 100755 (executable)
--- a/ipsilon/providers/saml2/auth.py
+++ b/ipsilon/providers/saml2/auth.py
@@ -174,10 +174,10 @@ class AuthenticateRequest(ProviderPageBase):
 
         nameid = None
         if nameidfmt == lasso.SAML2_NAME_IDENTIFIER_FORMAT_PERSISTENT:
 
         nameid = None
         if nameidfmt == lasso.SAML2_NAME_IDENTIFIER_FORMAT_PERSISTENT:

-            ## TODO map to something else ?
+            # TODO map to something else ?

             nameid = provider.normalize_username(user.name)
         elif nameidfmt == lasso.SAML2_NAME_IDENTIFIER_FORMAT_TRANSIENT:
             nameid = provider.normalize_username(user.name)
         elif nameidfmt == lasso.SAML2_NAME_IDENTIFIER_FORMAT_TRANSIENT:

-            ## TODO map to something else ?
+            # TODO map to something else ?

             nameid = provider.normalize_username(user.name)
         elif nameidfmt == lasso.SAML2_NAME_IDENTIFIER_FORMAT_KERBEROS:
             nameid = us.get_data('user', 'krb_principal_name')
             nameid = provider.normalize_username(user.name)
         elif nameidfmt == lasso.SAML2_NAME_IDENTIFIER_FORMAT_KERBEROS:
             nameid = us.get_data('user', 'krb_principal_name')

diff --git a/ipsilon/tools/certs.py b/ipsilon/tools/certs.py
index 3a60488..2ff63f1 100755 (executable)
--- a/ipsilon/tools/certs.py
+++ b/ipsilon/tools/certs.py
@@ -50,7 +50,7 @@ class Certificate(object):
         with open(self.cert, 'r') as f:
             cert = f.readlines()
 
         with open(self.cert, 'r') as f:
             cert = f.readlines()
 

-        #poor man stripping of BEGIN/END lines
+        # poor man stripping of BEGIN/END lines

         if cert[0] == '-----BEGIN CERTIFICATE-----\n':
             cert = cert[1:]
         if cert[-1] == '-----END CERTIFICATE-----\n':
         if cert[0] == '-----BEGIN CERTIFICATE-----\n':
             cert = cert[1:]
         if cert[-1] == '-----END CERTIFICATE-----\n':

diff --git a/ipsilon/util/plugin.py b/ipsilon/util/plugin.py
index c57cb29..730ce6c 100755 (executable)
--- a/ipsilon/util/plugin.py
+++ b/ipsilon/util/plugin.py
@@ -36,7 +36,7 @@ class Plugins(object):
         try:
             if ext.lower() == '.py':
                 mod = imp.load_source(name, file_name)
         try:
             if ext.lower() == '.py':
                 mod = imp.load_source(name, file_name)

-            #elif ext.lower() == '.pyc':
+            # elif ext.lower() == '.pyc':

             #    mod = imp.load_compiled(name, file_name)
             else:
                 return
             #    mod = imp.load_compiled(name, file_name)
             else:
                 return

diff --git a/ipsilon/util/user.py b/ipsilon/util/user.py
index ea0b974..f47c072 100755 (executable)
--- a/ipsilon/util/user.py
+++ b/ipsilon/util/user.py
@@ -94,7 +94,7 @@ class User(object):
 
     @sites.setter
     def sites(self):
 
     @sites.setter
     def sites(self):

-        #TODO: implement setting sites via the user object ?
+        # TODO: implement setting sites via the user object ?

         raise AttributeError
 
 
         raise AttributeError