flake8 cleanup

collective · Feb 1, 2016 · f6989fd · f6989fd
1 parent ba77b3b
commit f6989fd
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 13 deletions.
diff --git a/src/pas/plugins/sqlalchemy/__init__.py b/src/pas/plugins/sqlalchemy/__init__.py
@@ -13,11 +13,11 @@ def initialize(context):
     if plugin.Plugin.meta_type not in MultiPlugins:
         registerMultiPlugin(plugin.Plugin.meta_type)
         context.registerClass(
-                plugin.Plugin,
-                permission=add_user_folders,
-                icon=os.path.join(os.path.dirname(os.path.realpath(__file__)),
-                                                  'www', 'sql.png'),
-                constructors=(plugin.manage_addSqlalchemyPluginForm,
-                              plugin.addSqlalchemyPlugin),
-                visibility=None
-            )
+            plugin.Plugin,
+            permission=add_user_folders,
+            icon=os.path.join(os.path.dirname(os.path.realpath(__file__)),
+                              'www', 'sql.png'),
+            constructors=(plugin.manage_addSqlalchemyPluginForm,
+                          plugin.addSqlalchemyPlugin),
+            visibility=None
+        )
diff --git a/src/pas/plugins/sqlalchemy/plugin.py b/src/pas/plugins/sqlalchemy/plugin.py
@@ -844,8 +844,6 @@ def addPrincipalToGroup(self, principal_id, group_id):
             if not principal:
                 return False
             principal = self._add_principal(zope_id=principal.getId())
-            #principal = session.query(self.principal_class)\
-            #    .filter_by(zope_id=principal_id).first()
 
         group.members.append(principal)
         return True
@@ -886,11 +884,12 @@ def removePrincipalFromGroup(self, principal_id, group_id):
 
         group.members.remove(user)
 
-        # If user doesn't belongs to this PAS plugin, garbage can be left behind
+        # If user doesn't belongs to this plugin, garbage can be left behind
         # in that case we can have orphan items in the principals table
-        full_user = session.query(self.user_class).filter_by(id=user.id).first()
+        full_user = session.query(self.user_class).filter_by(id=user.id)\
+            .first()
         relations_left = session.query(model.group_member_table).filter(
-            model.group_member_table.c.principal_id==user.id
+            model.group_member_table.c.principal_id == user.id
             ).count()
         if not full_user and relations_left == 0:
             session.query(self.principal_class).filter_by(id=user.id)\