From: Tony Mack <tmack@cs.princeton.edu>
Date: Tue, 11 Aug 2009 17:03:11 +0000 (+0000)
Subject: move sfa package import inside of __init__()
X-Git-Tag: PLCAPI-4.3-23~17
X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=d48149e2b753c4ae146dc391883b2389b52b4f73;p=plcapi.git

move sfa package import inside of __init__()
---

diff --git a/PLC/SFA.py b/PLC/SFA.py
index 30ef40b6..fbb490ee 100644
--- a/PLC/SFA.py
+++ b/PLC/SFA.py
@@ -1,11 +1,5 @@
 from types import StringTypes
 import traceback
-try:
-    from sfa.plc.sfaImport import sfaImport, cleanup_string
-    from sfa.plc.api import GeniAPI
-    from sfa.util.debug import log
-    packages_imported = True  
-except:
     packages_imported = False
     traceback.print_exc()
 
@@ -21,6 +15,16 @@ class SFA:
     
     @wrap_exception
     def __init__(self):
+        try:
+            from sfa.plc.sfaImport import sfaImport, cleanup_string
+            from sfa.plc.api import GeniAPI
+            from sfa.util.debug import log
+            packages_imported = True
+        except:
+            packages_imported = False
+            traceback.print_exc()        
+        
+        self.cleanup_string = cleanup_string
         self.log = log
         self.sfa = sfaImport()
         geniapi = GeniAPI()
@@ -68,7 +72,7 @@ class SFA:
                 login_bases = [login_bases]
 
             for login_base in login_bases:
-                login_base = cleanup_string(login_base)
+                login_base = self.cleanup_string(login_base)
                 parent_hrn = self.authority + "." + login_base
                 if type in ['person']:
                     self.sfa.import_person(parent_hrn, object)
@@ -97,7 +101,7 @@ class SFA:
             login_bases = [login_base]
 
         for login_base in login_bases:
-            login_base = cleanup_string(login_base)
+            login_base = self.cleanup_string(login_base)
             parent_hrn = self.authority + "." + login_base
             self.sfa.delete_record(parent_hrn, object, type)