refs #134 -- using django.contrib.messages in callbacks

Author: brunobord

demo/demo/__init__.py

@@ -1,5 +1,6 @@
 import logging
 
+from django.contrib import messages
 
 logger = logging.getLogger(__name__)
 
@@ -31,9 +32,11 @@ def callback_success_message(request):
     This function will be called a form post-save/create.
 
     It adds a logging message
+
     """
-    logger.info('Sucessfully recorded form :)')
-    return True
+    msg = 'Sucessfully recorded form :)'
+    logger.info(msg)
+    messages.info(request._request, msg)
 
 
 def callback_fail_message(request):
@@ -42,5 +45,6 @@ def callback_fail_message(request):
 
     It adds a logging message (error)
     """
-    logger.error('Form storing has failed :(')
-    return True
+    msg = 'Form storing has failed :('
+    logger.error(msg)
+    messages.error(request._request, msg)

demo/demo/builder/templates/formidable/formidable_list.html

@@ -4,6 +4,15 @@
 <div class="title">
   <h1>May the Form be with you!</h1>
 </div>
+
+{% if messages %}
+<ul class="messages">
+    {% for message in messages %}
+    <li{% if message.tags %} class="{{ message.tags }}"{% endif %}>{{ message }}</li>
+    {% endfor %}
+</ul>
+{% endif %}
+
 <div>
   <table>
     <thead>

formidable/views.py

@@ -73,6 +73,12 @@ class CallbackMixin(object):
     def _call_callback(self, callback):
         """
         Tool to simply call the callback function and handle edge-cases.
+
+        **WARNING!** the DRF request is not inherited from django core,
+        `HTTPRequest`, and you should not assume they'll behave the same way.
+
+        If you need the "true" HTTPRequest object,
+        use ``self.request._request`` in your callback functions.
         """
         # If there's no callback value, it's pointless to try to extract it
         if not callback:
@@ -83,6 +89,11 @@ def _call_callback(self, callback):
         if not func:
             return
         try:
+            # WARNING! the DRF request is not inherited from django core,
+            # `HTTPRequest`, and you should not assume they'll behave the same
+            # way.
+            # If you need the "true" HTTPRequest object, use
+            # ``self.request._request``
             func(self.request)
         except Exception:
             logger.error(