Opened 9 years ago
Closed 9 years ago
#25792 closed Cleanup/optimization (wontfix)
Rename JsonResponse to JSONResponse
Reported by: | Daniel Hillier | Owned by: | Daniel Hillier |
---|---|---|---|
Component: | HTTP handling | Version: | dev |
Severity: | Normal | Keywords: | |
Cc: | Daniel Hillier | Triage Stage: | Unreviewed |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
Rename django.http.JsonResponse to JSONResponse. This change brings the capitalization of 'JSON' in line with the rest of the project where it is spelt with either all capitals or all lower case (eg. django.contrib.sessions.serializers.JSONSerializer, django.contrib.postgres.fields.JSONField, json_dumps_params, etc.)
Change History (3)
comment:1 by , 9 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
comment:2 by , 9 years ago
Has patch: | set |
---|
comment:3 by , 9 years ago
Resolution: | → wontfix |
---|---|
Status: | assigned → closed |
The naming was discussed during the original implementation (#17942), and I don't see a benefit to changing it at this point. Feel free to raise the issue on the DevelopersMailingList if you feel strongly about it.
The changes for this ticket can be found on the following commit:
https://github.com/danifus/django/commit/9381fda7b86b34f21d1656df60c60b8225734773
Let me know of any improvements and I'll get it into shape.