Summary | Bcc field not saved in sent messages |
Queue | IMP |
Queue Version | FRAMEWORK_5_2 |
Type | Bug |
State | Resolved |
Priority | 2. Medium |
Owners | jan (at) horde (dot) org |
Requester | wahnes (at) uni-koeln (dot) de |
Created | 07/06/2020 (1835 days ago) |
Due | |
Updated | 01/19/2021 (1638 days ago) |
Assigned | 07/14/2020 (1827 days ago) |
Resolved | 01/19/2021 (1638 days ago) |
Github Issue Link | |
Github Pull Request | |
Milestone | |
Patch | Yes |
Assigned to Jan Schneider
Taken from Michael Rubinsky
commit db9d2e1ec4d23701c7c02fa03dd951de5e81203c
Author: Jan Schneider <jan@horde.org>
Date: Fri, 17 Jul 2020 11:52:05 +0200
Headers may be Horde_Mail_Rfc822_List objects too (
Bug #15020).M lib/Compose.php
https://github.com/horde/imp/commit/db9d2e1ec4d23701c7c02fa03dd951de5e81203c
Assigned to Michael Rubinsky
Priority ⇒ 2. Medium
Type ⇒ Bug
Summary ⇒ Bcc field not saved in sent messages
Queue ⇒ IMP
Milestone ⇒
Patch ⇒ Yes
New Attachment: imp-compose-bcc.patch
State ⇒ Unconfirmed
message is not saved to messages in the "Sent" folder anymore. This is
due to the following commit:
https://github.com/horde/imp/commit/52ebc72bdc21523488102558b949ff31df80cd51
This commit introduces an "is_array" check on the "Bcc" header to be
saved. As it turns out, this check always returns false, because the
$header['bcc'] variable is in fact not an array, but an instance of
Horde_Mail_Rfc822_List. So with the "is_array" check turned on, no
"Bcc" headers ever get saved to the message in the "Sent" folder..
Attached is a patch that uses the "isset()" function instead. I don't
know if that is a good choice, but it sure is better than "is_array()"
which yields undesired results.