Opened 2 years ago

Closed 2 years ago

Last modified 6 months ago

#22897 closed Cleanup/optimization (fixed)

Make query_string argument to QueryDict optional

Reported by: duncan@… Owned by: nobody
Component: HTTP handling Version: master
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


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 ​

Change History (3)

comment:1 Changed 2 years ago by Tim Graham

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

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

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 Changed 6 months ago by Tim Graham <timograham@…>

In ead21a1:

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

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