From b20d64a55d0b304d7bcad405a910e8dd32112b1a Mon Sep 17 00:00:00 2001 From: Hadi Nategh Date: Fri, 26 Jan 2018 16:24:21 +0100 Subject: [PATCH] * Mail: Fix mail with S/MIME signature fails if user has no rights to notifications --- notifications/inc/class.notifications_push.inc.php | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/notifications/inc/class.notifications_push.inc.php b/notifications/inc/class.notifications_push.inc.php index 75ba1f38f0..fd3f2bbcff 100644 --- a/notifications/inc/class.notifications_push.inc.php +++ b/notifications/inc/class.notifications_push.inc.php @@ -81,15 +81,13 @@ class notifications_push implements Json\PushBackend * @param int $account_id account_id to push message too * @param string $key * @param mixed $data - * @throws egw_json_push_exception_not_online if $account_id is not online + * + * This function doesn't throw exception regarding if user availability anymore, + * if you need to check if user is online or not please use this function + * Api\Session::notifications_active($account_id) in a clause. */ public function addGeneric($account_id, $key, $data) { - // todo: check $account_id is online - if ($account_id > 0 && !Api\Session::notifications_active($account_id)) - { - throw new Json\Exception\NotOnline(); - } self::$db->insert(self::TABLE, array( 'account_id' => $account_id, 'notify_type' => self::TYPE,