Opened 10 years ago

Closed 10 years ago

Last modified 8 years ago

#22897 closed Cleanup/optimization (fixed)

Make query_string argument to QueryDict optional

Reported by: duncan@… Owned by: nobody
Component: HTTP handling Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Previously, a blank QueryDict could be created by calling

QueryDict(str('')) or QueryDict(None)

It seems rather nicer to just allow the query_string argument to
default to None so that we can just write QueryDict().

It's a commit in ​https://github.com/django/django/pull/2778

Change History (3)

comment:1 by Tim Graham, 10 years ago

Summary: Made query_string argument to QueryDict optionalMake query_string argument to QueryDict optional
Triage Stage: UnreviewedAccepted

comment:2 by Tim Graham <timograham@…>, 10 years ago

Resolution: fixed
Status: newclosed

In d68987ae25f0e0fb30727473803758af53d75671:

Fixed #22897 -- Made QueryDict query_string argument optional.

Now QueryDict() is equivalent to QueryDict() or QueryDict(None).

comment:3 by Tim Graham <timograham@…>, 8 years ago

In ead21a1:

Refs #22897 -- Removed unneeded empty string QueryDict argument.

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