aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMuriithi Frederick Muriuki2018-02-21 11:46:52 +0300
committerMuriithi Frederick Muriuki2018-02-21 11:46:52 +0300
commitf8970931932ed9b32c078cf3f2a1203f50f73ab0 (patch)
tree796163f70e9b21bbd473fc925a32686c1cea4ac2
parent2035387823ffc87bc2b6a817a06cb3a47aa006a0 (diff)
downloadgenenetwork2-f8970931932ed9b32c078cf3f2a1203f50f73ab0.tar.gz
Fix assumption of existing collection
* When logging in, if a user selects "Import existing collections", the system would throw an exception, since Redis would return a NoneType, which would then be accessed by json.loads(), which doesn't seem to know how to process that. This fixes that, by providing a string representing an empty json array ("[]"), in place of the NoneType.
-rw-r--r--wqflask/wqflask/user_manager.py3
1 files changed, 2 insertions, 1 deletions
diff --git a/wqflask/wqflask/user_manager.py b/wqflask/wqflask/user_manager.py
index fd1d56ff..e1920f4e 100644
--- a/wqflask/wqflask/user_manager.py
+++ b/wqflask/wqflask/user_manager.py
@@ -122,7 +122,8 @@ class AnonUser(object):
return collections
def import_traits_to_user(self):
- collections_list = json.loads(Redis.get(self.key))
+ result = Redis.get(self.key)
+ collections_list = json.loads(result if result else "[]")
for collection in collections_list:
uc = model.UserCollection()
uc.name = collection['name']