Opened 8 years ago

Closed 8 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


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 Changed 8 years ago by Daniel Hillier

Owner: changed from nobody to Daniel Hillier
Status: newassigned

comment:2 Changed 8 years ago by Daniel Hillier

Has patch: set

The changes for this ticket can be found on the following commit:

Let me know of any improvements and I'll get it into shape.

comment:3 Changed 8 years ago by Tim Graham

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