Switch BaseProcessingFail.exception_path's separator from period to colon
authorChristopher Allan Webber <cwebber@dustycloud.org>
Sun, 14 Aug 2011 12:53:24 +0000 (07:53 -0500)
committerChristopher Allan Webber <cwebber@dustycloud.org>
Sun, 14 Aug 2011 12:53:24 +0000 (07:53 -0500)
Also removing .generator_error_message() which doesn't make sense
really... we need to get the message when we don't have an instance of
the exception, and this method requires an instance.

mediagoblin/process_media/errors.py

index f2ae87ffa0744eb1495a86ffd8d3efe6a7d24b80..f8ae9ab2da92945c7656cc7b2298306b9264c57a 100644 (file)
@@ -29,22 +29,12 @@ class BaseProcessingFail(Exception):
   
     @property
     def exception_path(self):
-        return u"%s.%s" % (
+        return u"%s:%s" % (
             self.__class__.__module__, self.__class__.__name__)
 
     def __init__(self, **metadata):
         self.metadata = metadata or {}
   
-    def generate_error_message(self):
-        """
-        Generate an error to display to users in the panel.
-  
-        Uses this class's general_message, possibly interpolated
-        with any metadata in self.metadata['error_message_vars'],
-        if appropriate.
-        """
-        return self.general_message % self.metadata.get('error_message_vars', {})
-  
   
 class BadMediaFail(BaseProcessingFail):
     """