Return True on success

This should make checking for success clearer, rather than needing to differentiate between two falsey values (False and None).
mXaln · Jul 8, 2017 · 1e3bbd9 · 1e3bbd9
1 parent 3e1f618
commit 1e3bbd9
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/organizations/backends/defaults.py b/organizations/backends/defaults.py
@@ -289,6 +289,7 @@ def send_invitation(self, user, sender=None, **kwargs):
         token = self.get_token(user)
         kwargs.update({'token': token})
         self._send_email(user, self.invitation_subject, self.invitation_body, sender, **kwargs)
+        return True
 
     def send_notification(self, user, sender=None, **kwargs):
         """
@@ -299,3 +300,4 @@ def send_notification(self, user, sender=None, **kwargs):
         if not user.is_active:
             return False
         self._send_email(user, self.notification_subject, self.notification_body, sender, **kwargs)
+        return True