X-Git-Url: https://git.mdrn.pl/wolnelektury.git/blobdiff_plain/ae60b2a3949e96357477cc04f90fd0873cee8a92..15cae7d86e624d8afdb7eac794b58990ded9e64b:/src/ajaxable/utils.py?ds=sidebyside diff --git a/src/ajaxable/utils.py b/src/ajaxable/utils.py old mode 100755 new mode 100644 index c2c368840..f8e99cece --- 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 +import json 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 -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): - 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): @@ -42,17 +40,16 @@ 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') - else: - return HttpResponseRedirect('/uzytkownicy/zaloguj') # next?=request.build_full_path()) + return HttpResponseRedirect('/uzytkownicy/zaloguj') # next?=request.build_full_path()) def placeholdized(form): for field in form.fields.values(): - field.widget.attrs['placeholder'] = field.label + field.widget.attrs['placeholder'] = field.label + ('*' if field.required else '') 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. @@ -63,6 +60,7 @@ class AjaxableFormView(object): # override to customize form look template = "ajaxable/form.html" submit = _('Send') + action = '' title = '' success_message = '' @@ -76,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) + + 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 @@ -87,7 +90,7 @@ class AjaxableFormView(object): 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 @@ -103,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['redirect'], safe='/?=&')) + response_data['redirect'], safe='/?=&')) elif request.is_ajax(): # Form was sent with errors. Send them back. if self.form_prefix: @@ -118,7 +121,7 @@ class AjaxableFormView(object): 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) @@ -135,18 +138,22 @@ class AjaxableFormView(object): if self.placeholdize: form = placeholdized(form) context = { - self.formname: form, - "title": title, - "honeypot": self.honeypot, - "placeholdize": self.placeholdize, - "submit": self.submit, - "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)) - 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`.""" @@ -155,8 +162,7 @@ class AjaxableFormView(object): if message: output = "
" + message + "
" + 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."""