diff options
author | Denis Cornehl <denis.cornehl@gmail.com> | 2013-11-23 15:35:24 +0100 |
---|---|---|
committer | Baptiste Mispelon <bmispelon@gmail.com> | 2013-11-23 17:11:22 +0100 |
commit | 1e970584178c44d561d54dc8bf0c3837b89bad5a (patch) | |
tree | b72778185ef8b90d16c39420d3731f17f7aa8937 /django/contrib/messages/api.py | |
parent | a480f8320a5b36501bfd0e8cd70b8dc04adf2d08 (diff) | |
download | django-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.py | 5 |
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: |