diff options
author | Adam Zapletal <adamzap@gmail.com> | 2015-04-13 08:48:16 -0500 |
---|---|---|
committer | Tim Graham <timograham@gmail.com> | 2015-04-13 10:32:03 -0400 |
commit | 872eb26f542caead338d6456b345ca2f5c0684e2 (patch) | |
tree | b667a057813bc0b441c0d29d307f11e04f9016d3 | |
parent | b295fcd19c00706d7f60db73741c1311ee141409 (diff) | |
download | django-872eb26f542caead338d6456b345ca2f5c0684e2.tar.gz |
Fixed #24621 -- Fixed and documented SessionBase.pop's second argument
Changed SessionBase.pop's second argument to explicitly be default=None
rather than *args since _session is always a dict. Thanks gabor for the
report and Tim Graham for the review.
-rw-r--r-- | django/contrib/sessions/backends/base.py | 4 | ||||
-rw-r--r-- | docs/topics/http/sessions.txt | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/django/contrib/sessions/backends/base.py b/django/contrib/sessions/backends/base.py index 3b24b8d3d7..113f658a48 100644 --- a/django/contrib/sessions/backends/base.py +++ b/django/contrib/sessions/backends/base.py @@ -58,9 +58,9 @@ class SessionBase(object): def get(self, key, default=None): return self._session.get(key, default) - def pop(self, key, *args): + def pop(self, key, default=None): self.modified = self.modified or key in self._session - return self._session.pop(key, *args) + return self._session.pop(key, default) def setdefault(self, key, value): if key in self._session: diff --git a/docs/topics/http/sessions.txt b/docs/topics/http/sessions.txt index 40faf1870a..a7d444a4e6 100644 --- a/docs/topics/http/sessions.txt +++ b/docs/topics/http/sessions.txt @@ -205,9 +205,9 @@ You can edit it multiple times. Example: ``fav_color = request.session.get('fav_color', 'red')`` - .. method:: pop(key) + .. method:: pop(key, default=None) - Example: ``fav_color = request.session.pop('fav_color')`` + Example: ``fav_color = request.session.pop('fav_color', 'blue')`` .. method:: keys() |