osqa-23

[osqa.git] / forum / views / decorators.py

diff --git a/forum/views/decorators.py b/forum/views/decorators.py
index cf5f90373288f5b5a7e45dca546a1bb88e0b4e67..141887a3e0f7cfdc8a4402c04205ec1ca35ed276 100644 (file)
--- a/forum/views/decorators.py
+++ b/forum/views/decorators.py
@@ -3,6 +3,8 @@ from django.utils import simplejson
 from django.core.paginator import Paginator\r
 from django.shortcuts import render_to_response\r
 from django.template import RequestContext\r
+from django.utils.translation import ungettext, ugettext as _\r
+import logging\r
 \r
 def render(template=None, tab=None):\r
     def decorator(func):\r
@@ -28,7 +30,7 @@ def list(paginate, default_page_size):
             paginator = Paginator(big_list, pagesize)\r
 \r
             page_obj = paginator.page(page)\r
-            context[paginate] = page_obj.object_list\r
+            context[paginate] = page_obj.object_list.lazy()\r
 \r
             base_path = context.get('base_path', None) or request.path\r
             sort = request.utils.sort_method('')\r
@@ -55,19 +57,34 @@ def list(paginate, default_page_size):
     return decorator\r
 \r
 \r
+class CommandException(Exception):\r
+    pass\r
+\r
+\r
 def command(func):\r
     def decorated(request, *args, **kwargs):\r
         try:\r
             response = func(request, *args, **kwargs)\r
+\r
+            if isinstance(response, HttpResponse):\r
+                return response\r
+\r
             response['success'] = True\r
         except Exception, e:\r
             #import sys, traceback\r
             #traceback.print_exc(file=sys.stdout)\r
 \r
-            response = {\r
-                'success': False,\r
-                'error_message': str(e)\r
-            }\r
+            if isinstance(e, CommandException):\r
+                response = {\r
+                    'success': False,\r
+                    'error_message': str(e)\r
+                }\r
+            else:\r
+                logging.error("%s: %s" % (func.__name__, str(e)))\r
+                response = {\r
+                    'success': False,\r
+                    'error_message': _("We're sorry, but an unknown error ocurred.<br />Please try again in a while.")\r
+                }\r
 \r
         if request.is_ajax():\r
             return HttpResponse(simplejson.dumps(response), mimetype="application/json")\r