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 Daniel Hillier, 9 years ago

Owner: changed from nobody to Daniel Hillier
Status: newassigned

comment:2 by Daniel Hillier, 9 years ago

Has patch: set

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.

comment:3 by Tim Graham, 9 years ago

Resolution: wontfix
Status: assignedclosed

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.

Note: See TracTickets for help on using tickets.
Back to Top