summaryrefslogtreecommitdiff
path: root/django/contrib/messages/api.py
diff options
context:
space:
mode:
authorDenis Cornehl <denis.cornehl@gmail.com>2013-11-23 15:35:24 +0100
committerBaptiste Mispelon <bmispelon@gmail.com>2013-11-23 17:11:22 +0100
commit1e970584178c44d561d54dc8bf0c3837b89bad5a (patch)
treeb72778185ef8b90d16c39420d3731f17f7aa8937 /django/contrib/messages/api.py
parenta480f8320a5b36501bfd0e8cd70b8dc04adf2d08 (diff)
downloaddjango-1e970584178c44d561d54dc8bf0c3837b89bad5a.tar.gz
Fixes #21412 -- Better error message for messages.add_message
Problem were users calling messages.debug/info/* with a wrong argument and getting the error "You cannot add messages without installing MessageMiddleware" Thanks to trac-user merb for the report.
Diffstat (limited to 'django/contrib/messages/api.py')
-rw-r--r--django/contrib/messages/api.py5
1 files changed, 5 insertions, 0 deletions
diff --git a/django/contrib/messages/api.py b/django/contrib/messages/api.py
index 8e08655aa0..248e0f0e64 100644
--- a/django/contrib/messages/api.py
+++ b/django/contrib/messages/api.py
@@ -1,10 +1,12 @@
from django.contrib.messages import constants
from django.contrib.messages.storage import default_storage
+from django.http import HttpRequest
__all__ = (
'add_message', 'get_messages',
'get_level', 'set_level',
'debug', 'info', 'success', 'warning', 'error',
+ 'MessageFailure',
)
@@ -16,6 +18,9 @@ def add_message(request, level, message, extra_tags='', fail_silently=False):
"""
Attempts to add a message to the request using the 'messages' app.
"""
+ if not isinstance(request, HttpRequest):
+ raise TypeError("add_message() argument must be an HttpRequest object, "
+ "not '%s'." % request.__class__.__name__)
if hasattr(request, '_messages'):
return request._messages.add(level, message, extra_tags)
if not fail_silently: