[Erp5-report] r6879 - /erp5/trunk/products/ERP5Subversion/Tool/SubversionTool.py

[nobody at svn.erp5.org]

Author: chris
Date: Mon Apr 24 11:42:31 2006
New Revision: 6879

URL: http://svn.erp5.org?rev=6879&view=rev
Log:
- don't use strings for some exceptions anymore (reported by Jerome)

Modified:
    erp5/trunk/products/ERP5Subversion/Tool/SubversionTool.py

Modified: erp5/trunk/products/ERP5Subversion/Tool/SubversionTool.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Subversion/Tool/SubversionTool.py?rev=6879&r1=6878&r2=6879&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Subversion/Tool/SubversionTool.py (original)
+++ erp5/trunk/products/ERP5Subversion/Tool/SubversionTool.py Mon Apr 24 11:42:31 2006
@@ -52,6 +52,16 @@
 
 class Error(exceptions.EnvironmentError):
     pass
+
+class SubversionPreferencesError(Exception):
+  """The base exception class for the Subversion preferences.
+  """
+  pass
+  
+class SubversionUnknownBusinessTemplateError(Exception):
+  """The base exception class when business template is unknown.
+  """
+  pass
 
 def removeAll(entry):
   '''
@@ -603,10 +613,10 @@
     if not wc_list:
       wc_list = self.getPortalObject().portal_preferences.default_site_preference.getPreferredSubversionWorkingCopyList()
       if not wc_list:
-        raise 'Preferences Error', 'Please set at least one Subversion Working Copy in preferences first.'
+        raise SubversionPreferencesError, 'Please set at least one Subversion Working Copy in preferences first.'
     bt_name = bt.getTitle()
     if len(wc_list) == 0 :
-      raise 'Preferences Error', 'Please set at least one Subversion Working Copy in preferences first.'
+      raise SubversionPreferencesError, 'Please set at least one Subversion Working Copy in preferences first.'
     for wc in wc_list:
       if bt_name in os.listdir(wc) :
         wc_path = os.path.join(wc, bt_name)
@@ -615,7 +625,7 @@
             return wc_path
           else:
             return os.sep.join(wc_path.split(os.sep)[:-1])
-    raise 'Unknown Business Template', "Could not find '"+bt_name+"' at first level of working copies."
+    raise SubversionUnknownBusinessTemplateError, "Could not find '"+bt_name+"' at first level of working copies."
     
   security.declareProtected('Import/Export objects', 'update')
   def update(self, path):