Silence pychecker and pylint warnings
authorMichael Spang <mspang@uwaterloo.ca>
Mon, 10 Dec 2007 08:04:43 +0000 (03:04 -0500)
committerMichael Spang <mspang@csclub.uwaterloo.ca>
Mon, 10 Dec 2007 08:15:45 +0000 (03:15 -0500)
pylib/csc/adm/accounts.py
pylib/csc/adm/members.py
pylib/csc/apps/urwid/main.py
pylib/csc/apps/urwid/newmember.py
pylib/csc/apps/urwid/renew.py
pylib/csc/backends/ldapi.py

index feffa5d..780accf 100644 (file)
@@ -4,7 +4,7 @@ UNIX Accounts Administration
 This module contains functions for creating, deleting, and manipulating
 UNIX user accounts and account groups in the CSC LDAP directory.
 """
 This module contains functions for creating, deleting, and manipulating
 UNIX user accounts and account groups in the CSC LDAP directory.
 """
-import re, pwd, grp, os, pwd, subprocess
+import re, pwd, grp, os, subprocess
 from csc.common import conf
 from csc.common.excep import InvalidArgument
 from csc.backends import ldapi, krb
 from csc.common import conf
 from csc.common.excep import InvalidArgument
 from csc.backends import ldapi, krb
index be95c7a..73e9796 100644 (file)
@@ -9,7 +9,7 @@ Transactions are used in each method that modifies the database.
 Future changes to the members database that need to be atomic
 must also be moved into this module.
 """
 Future changes to the members database that need to be atomic
 must also be moved into this module.
 """
-import re, ldap, os, pwd
+import re, ldap
 from csc.adm import terms
 from csc.backends import ldapi
 from csc.common import conf
 from csc.adm import terms
 from csc.backends import ldapi
 from csc.common import conf
@@ -234,9 +234,6 @@ def list_positions():
 
     ceo_ldap = ldap_connection.ldap
     user_base = ldap_connection.user_base
 
     ceo_ldap = ldap_connection.ldap
     user_base = ldap_connection.user_base
-    escape = ldap_connection.escape
-
-    if not ldap_connection.connected(): ldap_connection.connect()
 
     members = ceo_ldap.search_s(user_base, ldap.SCOPE_SUBTREE, '(position=*)')
     positions = {}
 
     members = ceo_ldap.search_s(user_base, ldap.SCOPE_SUBTREE, '(position=*)')
     positions = {}
index 70f499a..43f7ca7 100644 (file)
@@ -1,5 +1,5 @@
-import random, time, re
-import urwid, urwid.curses_display
+import random
+import urwid.curses_display
 
 from csc.apps.urwid.widgets import *
 from csc.apps.urwid.window import *
 
 from csc.apps.urwid.widgets import *
 from csc.apps.urwid.window import *
index ecb3f17..c966cd6 100644 (file)
@@ -1,7 +1,6 @@
 import urwid
 from csc.apps.urwid.widgets import *
 from csc.apps.urwid.window import *
 import urwid
 from csc.apps.urwid.widgets import *
 from csc.apps.urwid.window import *
-from csc.apps.urwid.ldapfilter import LdapFilter
 
 from csc.adm import accounts, members, terms
 from csc.common.excep import InvalidArgument
 
 from csc.adm import accounts, members, terms
 from csc.common.excep import InvalidArgument
index 5615eae..9217004 100644 (file)
@@ -56,7 +56,7 @@ class TermPage(WizardPanel):
     def check(self):
         try:
             self.state['terms'] = terms.interval( self.start.get_edit_text(), self.count.value() )
     def check(self):
         try:
             self.state['terms'] = terms.interval( self.start.get_edit_text(), self.count.value() )
-        except e:
+        except Exception, e:
             self.focus_widget( self.start )
             set_status( "Invalid start term" )
             return True
             self.focus_widget( self.start )
             set_status( "Invalid start term" )
             return True
index 65651f0..6fa426f 100644 (file)
@@ -385,8 +385,6 @@ class LDAPConnection(object):
             raise LDAPException("group search failed: %s" % e)
 
         # list for groups found
             raise LDAPException("group search failed: %s" % e)
 
         # list for groups found
-        group_cns = []
-
         results = {}
         for match in matches:
             dn, attrs = match
         results = {}
         for match in matches:
             dn, attrs = match