Removed incorrect JSON exception subclasses

Author: mitsuhiko

CHANGES

@@ -54,6 +54,9 @@ Release date to be decided.
 - Added `message_flashed` signal that simplifies flashing testing.
 - Added support for copying of request contexts for better working with
   greenlets.
+- Removed custom JSON HTTP exception subclasses.  If you were relying on them
+  you can reintroduce them again yourself trivially.  Using them however is
+  strongly discouraged as the interface was flawed.
 
 Version 0.9
 -----------

flask/exceptions.py

@@ -35,23 +35,11 @@ def test_json_bad_requests(self):
         app = flask.Flask(__name__)
         @app.route('/json', methods=['POST'])
         def return_json():
-            return unicode(flask.request.json)
+            return flask.jsonify(foo=unicode(flask.request.json))
         c = app.test_client()
         rv = c.post('/json', data='malformed', content_type='application/json')
         self.assert_equal(rv.status_code, 400)
 
-    def test_json_bad_requests_content_type(self):
-        app = flask.Flask(__name__)
-        @app.route('/json', methods=['POST'])
-        def return_json():
-            return unicode(flask.request.json)
-        c = app.test_client()
-        rv = c.post('/json', data='malformed', content_type='application/json')
-        self.assert_equal(rv.status_code, 400)
-        self.assert_equal(rv.mimetype, 'application/json')
-        self.assert_('description' in flask.json.loads(rv.data))
-        self.assert_('<p>' not in flask.json.loads(rv.data)['description'])
-
     def test_json_body_encoding(self):
         app = flask.Flask(__name__)
         app.testing = True

flask/testsuite/helpers.py

@@ -11,8 +11,8 @@
 
 from werkzeug.wrappers import Request as RequestBase, Response as ResponseBase
 from werkzeug.utils import cached_property
+from werkzeug.exceptions import BadRequest
 
-from .exceptions import JSONBadRequest
 from .debughelpers import attach_enctype_error_multidict
 from . import json
 from .globals import _request_ctx_stack
@@ -107,21 +107,16 @@ def json(self):
     def on_json_loading_failed(self, e):
         """Called if decoding of the JSON data failed.  The return value of
         this method is used by :attr:`json` when an error occurred.  The default
-        implementation raises a :class:`JSONBadRequest`, which is a subclass of
-        :class:`~werkzeug.exceptions.BadRequest` which sets the
-        ``Content-Type`` to ``application/json`` and provides a JSON-formatted
-        error description::
+        implementation just raises a :class:`BadRequest` exception.
 
-            {"description": "The browser (or proxy) sent a request that \
-                             this server could not understand."}
-
-        .. versionchanged:: 0.9
-           Return a :class:`JSONBadRequest` instead of a
-           :class:`~werkzeug.exceptions.BadRequest` by default.
+        .. versionchanged:: 0.10
+           Removed buggy previous behavior of generating a random JSON
+           response.  If you want that behavior back you can trivially
+           add it by subclassing.
 
         .. versionadded:: 0.8
         """
-        raise JSONBadRequest()
+        raise BadRequest()
 
     def _load_form_data(self):
         RequestBase._load_form_data(self)