add some more docs here in there as I try and remember what I wrote

Author: crccheck

django_object_actions/utils.py

@@ -59,9 +59,18 @@ def to_dict(tool_name):
     ##################
 
     def get_object_actions(self, request, context, **kwargs):
+        """Override this to customize what actions get sent."""
         return self.objectactions
 
     def get_djoa_button_attrs(self, tool):
+        """
+        Get the HTML attributes associated with a tool.
+
+        There are some standard attributes (class and title) that the template
+        will always want. Any number of additional attributes can be specified
+        and passed on. This is kinda awkward and due for a refactor for
+        readability.
+        """
         attrs = getattr(tool, 'attrs', {})
         # href is not allowed to be set. should an exception be raised instead?
         if 'href' in attrs:
@@ -101,7 +110,7 @@ def get(self, request, **kwargs):
         try:
             ret = self.tools[kwargs['tool']](request, obj)
         except KeyError:
-            raise Http404
+            raise Http404(u'Tool does not exist')
         if isinstance(ret, HttpResponse):
             return ret
         back = request.path.rsplit('/', 3)[0] + '/'
@@ -117,7 +126,13 @@ def message_user(self, request, message):
 
 
 class QuerySetIsh(QuerySet):
-    """Takes an instance and mimics it coming from a QuerySet."""
+    """
+    Takes an instance and mimics it coming from a QuerySet.
+
+    This is a hack to support the `takes_instance_or_queryset` decorator so
+    that you can re-use functions written for standard Django admin actions and
+    use them for Object Tools too.
+    """
     def __init__(self, instance=None, *args, **kwargs):
         try:
             model = instance._meta.model
@@ -141,7 +156,7 @@ def get(self, *args, **kwargs):
 
 
 def takes_instance_or_queryset(func):
-    """Decorator that makes standard actions compatible."""
+    """Decorator that makes standard Django admin actions compatible."""
     @wraps(func)
     def decorated_function(self, request, queryset):
         # func follows the prototype documented at: