Prefer one_or_none() to first() for safety

iambibhas · Jun 2, 2016 · 9b43af9 · 9b43af9
1 parent 31f3b07
commit 9b43af9
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/hasjob/models/user.py b/hasjob/models/user.py
@@ -152,7 +152,7 @@ class EventSession(EventSessionBase, BaseMixin, db.Model):
 
     @classmethod
     def get_session(cls, uuid, user=None, anon_user=None):
-        ues = cls.query.filter_by(uuid=uuid).first() if uuid else None
+        ues = cls.query.filter_by(uuid=uuid).one_or_none() if uuid else None
 
         # We no longer hard-link sessions to users, so this is commented out:
         # if (not not user) + (not not anon_user) != 1: