From 245fbb2d8258cddaf55662b00c2f7c2c9daa69b4 Mon Sep 17 00:00:00 2001 From: nathan Date: Thu, 5 May 2022 14:08:26 -0600 Subject: [PATCH] Infolog: Make sure to use the current user as set in Tracking, rather than $GLOBALS[egw_info']['user']['ccount_id'] when getting reply_to because we change the GLOBALS user when generating the notification. --- infolog/inc/class.infolog_tracking.inc.php | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/infolog/inc/class.infolog_tracking.inc.php b/infolog/inc/class.infolog_tracking.inc.php index 8dcc83006d..c4c7f56e45 100644 --- a/infolog/inc/class.infolog_tracking.inc.php +++ b/infolog/inc/class.infolog_tracking.inc.php @@ -328,8 +328,10 @@ class infolog_tracking extends Api\Storage\Tracking switch($name) { case 'reply_to': // for async notification use owner of infolog, otherwise the user causing the change - if (($reply_to = Api\Accounts::getInstance()->id2name(!empty($GLOBALS['egw_info']['flags']['async-service']) ? - $data['info_owner'] : $GLOBALS['egw_info']['user']['account_id'], 'account_email'))) + if(($reply_to = Api\Accounts::getInstance() + ->id2name(!empty($GLOBALS['egw_info']['flags']['async-service']) ? + $data['info_owner'] : $this->user, 'account_email' + ))) { $config = $reply_to; }