fixes

[wolnelektury.git] / src / ajaxable / utils.py

diff --git a/src/ajaxable/utils.py b/src/ajaxable/utils.py
old mode 100755 (executable)
new mode 100644 (file)
index 80d398f..f8e99ce
--- a/src/ajaxable/utils.py
+++ b/src/ajaxable/utils.py
@@ -1,26 +1,24 @@
-# -*- coding: utf-8 -*-

 # This file is part of Wolnelektury, licensed under GNU Affero GPLv3 or later.
 # Copyright © Fundacja Nowoczesna Polska. See NOTICE for more information.
 #
 from functools import wraps
 # This file is part of Wolnelektury, licensed under GNU Affero GPLv3 or later.
 # Copyright © Fundacja Nowoczesna Polska. See NOTICE for more information.
 #
 from functools import wraps

+import json

 
 from django.http import HttpResponse, HttpResponseRedirect, HttpResponseForbidden
 
 from django.http import HttpResponse, HttpResponseRedirect, HttpResponseForbidden

-from django.shortcuts import render_to_response
-from django.template import RequestContext
-from django.utils.encoding import force_unicode
+from django.shortcuts import render
+from django.utils.encoding import force_text

 from django.utils.functional import Promise
 from django.utils.http import urlquote_plus
 from django.utils.functional import Promise
 from django.utils.http import urlquote_plus

-import json

 from django.utils.translation import ugettext_lazy as _
 from django.views.decorators.vary import vary_on_headers
 from honeypot.decorators import verify_honeypot_value
 
 
 class LazyEncoder(json.JSONEncoder):
 from django.utils.translation import ugettext_lazy as _
 from django.views.decorators.vary import vary_on_headers
 from honeypot.decorators import verify_honeypot_value
 
 
 class LazyEncoder(json.JSONEncoder):

-    def default(self, obj):
-        if isinstance(obj, Promise):
-            return force_unicode(obj)
-        return obj
+    def default(self, o):
+        if isinstance(o, Promise):
+            return force_text(o)
+        return o

 
 
 def method_decorator(function_decorator):
 
 
 def method_decorator(function_decorator):


@@ -42,8 +40,7 @@ def require_login(request):
     """Return 403 if request is AJAX. Redirect to login page if not."""
     if request.is_ajax():
         return HttpResponseForbidden('Not logged in')
     """Return 403 if request is AJAX. Redirect to login page if not."""
     if request.is_ajax():
         return HttpResponseForbidden('Not logged in')

-    else:
-        return HttpResponseRedirect('/uzytkownicy/zaloguj')  # next?=request.build_full_path())
+    return HttpResponseRedirect('/uzytkownicy/zaloguj')  # next?=request.build_full_path())

 
 
 def placeholdized(form):
 
 
 def placeholdized(form):


@@ -52,7 +49,7 @@ def placeholdized(form):
     return form
 
 
     return form
 
 

-class AjaxableFormView(object):
+class AjaxableFormView:

     """Subclass this to create an ajaxable view for any form.
 
     In the subclass, provide at least form_class.
     """Subclass this to create an ajaxable view for any form.
 
     In the subclass, provide at least form_class.


@@ -77,6 +74,11 @@ class AjaxableFormView(object):
     def __call__(self, request, *args, **kwargs):
         """A view displaying a form, or JSON if request is AJAX."""
         obj = self.get_object(request, *args, **kwargs)
     def __call__(self, request, *args, **kwargs):
         """A view displaying a form, or JSON if request is AJAX."""
         obj = self.get_object(request, *args, **kwargs)

+
+        response = self.validate_object(obj, request)
+        if response:
+            return response
+

         form_args, form_kwargs = self.form_args(request, obj)
         if self.form_prefix:
             form_kwargs['prefix'] = self.form_prefix
         form_args, form_kwargs = self.form_args(request, obj)
         if self.form_prefix:
             form_kwargs['prefix'] = self.form_prefix


@@ -88,7 +90,7 @@ class AjaxableFormView(object):
                     return response
 
             # do I need to be logged in?
                     return response
 
             # do I need to be logged in?

-            if self.POST_login and not request.user.is_authenticated():
+            if self.POST_login and not request.user.is_authenticated:

                 return require_login(request)
 
             form_kwargs['data'] = request.POST
                 return require_login(request)
 
             form_kwargs['data'] = request.POST


@@ -104,7 +106,7 @@ class AjaxableFormView(object):
                     response_data.update(add_args)
                 if not request.is_ajax() and response_data['redirect']:
                     return HttpResponseRedirect(urlquote_plus(
                     response_data.update(add_args)
                 if not request.is_ajax() and response_data['redirect']:
                     return HttpResponseRedirect(urlquote_plus(

-                            response_data['redirect'], safe='/?=&'))
+                        response_data['redirect'], safe='/?=&'))

             elif request.is_ajax():
                 # Form was sent with errors. Send them back.
                 if self.form_prefix:
             elif request.is_ajax():
                 # Form was sent with errors. Send them back.
                 if self.form_prefix:


@@ -119,7 +121,7 @@ class AjaxableFormView(object):
             if request.is_ajax():
                 return HttpResponse(LazyEncoder(ensure_ascii=False).encode(response_data))
         else:
             if request.is_ajax():
                 return HttpResponse(LazyEncoder(ensure_ascii=False).encode(response_data))
         else:

-            if self.POST_login and not request.user.is_authenticated() and not request.is_ajax():
+            if self.POST_login and not request.user.is_authenticated and not request.is_ajax():

                 return require_login(request)
 
             form = self.form_class(*form_args, **form_kwargs)
                 return require_login(request)
 
             form = self.form_class(*form_args, **form_kwargs)


@@ -136,19 +138,22 @@ class AjaxableFormView(object):
         if self.placeholdize:
             form = placeholdized(form)
         context = {
         if self.placeholdize:
             form = placeholdized(form)
         context = {

-                self.formname: form,
-                "title": title,
-                "honeypot": self.honeypot,
-                "placeholdize": self.placeholdize,
-                "submit": self.submit,
-                "action": self.action,
-                "response_data": response_data,
-                "ajax_template": self.template,
-                "view_args": args,
-                "view_kwargs": kwargs,
-            }
+            self.formname: form,
+            "title": title,
+            "honeypot": self.honeypot,
+            "placeholdize": self.placeholdize,
+            "submit": self.submit,
+            "action": self.action,
+            "response_data": response_data,
+            "ajax_template": self.template,
+            "view_args": args,
+            "view_kwargs": kwargs,
+        }

         context.update(self.extra_context(request, obj))
         context.update(self.extra_context(request, obj))

-        return render_to_response(template, context, context_instance=RequestContext(request))
+        return render(request, template, context)
+
+    def validate_object(self, obj, request):
+        return None

 
     def redirect_or_refresh(self, request, path, message=None):
         """If the form is AJAX, refresh the page. If not, go to `path`."""
 
     def redirect_or_refresh(self, request, path, message=None):
         """If the form is AJAX, refresh the page. If not, go to `path`."""


@@ -157,8 +162,7 @@ class AjaxableFormView(object):
             if message:
                 output = "<div class='normal-text'>" + message + "</div>" + output
             return HttpResponse(output)
             if message:
                 output = "<div class='normal-text'>" + message + "</div>" + output
             return HttpResponse(output)

-        else:
-            return HttpResponseRedirect(path)
+        return HttpResponseRedirect(path)

 
     def get_object(self, request, *args, **kwargs):
         """Override to parse view args and get some associated data."""
 
     def get_object(self, request, *args, **kwargs):
         """Override to parse view args and get some associated data."""