Opened 11 days ago

Last modified 11 days ago

#36278 closed New feature

MultiValueDict should use sentinel for default values — at Version 1

Reported by: Peter Larsen - Puzzel Solutions Owned by:
Component: Utilities Version: dev
Severity: Normal Keywords: MultiValueDict getlist default
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no
Pull Requests:How to create a pull request

Description (last modified by Peter Larsen - Puzzel Solutions)

MultiValueDict has many methods that has a 'default' parameter, but sets the default to None. fx:
​https://github.com/django/django/blob/main/django/utils/datastructures.py#L129

This makes
mvdict.getlist('key', None)
return [], instead of the expected None.

Change History (1)

comment:1 by Peter Larsen - Puzzel Solutions, 11 days ago

Description: modified (diff)
Note: See TracTickets for help on using tickets.
Back to Top