#24621 closed Bug (fixed)
request.session.pop's second param is not documented
Reported by: | Gábor Farkas | Owned by: | Adam Zapletal |
---|---|---|---|
Component: | Documentation | Version: | 1.8 |
Severity: | Normal | Keywords: | |
Cc: | Triage Stage: | Accepted | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | yes | UI/UX: | no |
Description
python dictionaries support the pop method's second parameter, for example {}.pop('key','default')
.
the django session seems to support this second parameter too, but the documentation does not mention it, so i'm not sure if i can use it or not.
Change History (8)
comment:1 by , 10 years ago
Easy pickings: | set |
---|---|
Triage Stage: | Unreviewed → Accepted |
Type: | Uncategorized → Bug |
comment:2 by , 10 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
comment:3 by , 10 years ago
Has patch: | set |
---|
comment:4 by , 10 years ago
Owner: | changed from | to
---|
comment:5 by , 10 years ago
Owner: | changed from | to
---|
I've got this one, priidukull. I'll remove myself as owner if I'm not able to close it. Thanks!
Discussion is happening here: https://github.com/django/django/pull/4480
Note:
See TracTickets
for help on using tickets.
I think it's just an oversite in the docs as the behavior is tested.