Ticket #4729: sessionwrapper_pop_modified.3.patch

django/contrib/sessions/middleware.py

@@ -37 +37 @@
         return self._session.get(key, default)
 
     def pop(self, key, *args):
+        self.modified = self.modified or key in self._session 
         return self._session.pop(key, *args)
 
     def set_test_cookie(self):

django/contrib/sessions/tests.py

@@ -5 +5 @@
 >>> s._session_cache = {}
 >>> s._session_cache['some key'] = 'exists'
 
+>>> s.accessed
+False
+>>> s.modified
+False
+
+>>> s.pop('non existant key', 'does not exist')
+'does not exist'
+>>> s.accessed
+True
+>>> s.modified
+False
+
 >>> s.pop('some key')
 'exists'
+>>> s.accessed
+True
+>>> s.modified
+True
 
 >>> s.pop('some key', 'does not exist')
 'does not exist'