Browse Source

Make sure override_acl makes a deep copy of user permissions dict before
modifying it in place to avoid any side effects

NeKit 8 years ago
parent
commit
6197c8166a
1 changed files with 2 additions and 1 deletions
  1. 2 1
      misago/acl/testutils.py

+ 2 - 1
misago/acl/testutils.py

@@ -1,4 +1,5 @@
 # pylint: disable=protected-access
+from copy import deepcopy
 from hashlib import md5
 
 from misago.core import threadstore
@@ -25,7 +26,7 @@ def fake_post_data(target, data_dict):
 
 def override_acl(user, new_acl):
     """overrides user permissions with specified ones"""
-    final_cache = user.acl
+    final_cache = deepcopy(user.acl)
     final_cache.update(new_acl)
 
     if user.is_authenticated():