Skip to content

Commit

Permalink
fix notification adding error
Browse files Browse the repository at this point in the history
The test on whether the user notification queue existed or not was just wrong.
  • Loading branch information
kakwa committed Nov 8, 2015
1 parent af01f9b commit e981451
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 2 additions & 0 deletions ChangeLog.rst
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
Dev
***

* [fix ] notifications missing in case of multiple notification waiting to be displayed

Version 0.2.2
*************

Expand Down
2 changes: 1 addition & 1 deletion ldapcherry/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -431,7 +431,7 @@ def _add_notification(self, message):
"""
sess = cherrypy.session
username = sess.get(SESSION_KEY, None)
if username is not self.notifications:
if username not in self.notifications:
self.notifications[username] = []
self.notifications[username].append(message)

Expand Down

0 comments on commit e981451

Please sign in to comment.