Summary | Bcc field not saved in sent messages |
Queue | IMP |
Queue Version | FRAMEWORK_5_2 |
Type | Bug |
State | Assigned |
Priority | 2. Medium |
Owners | jan (at) horde (dot) org |
Requester | wahnes (at) uni-koeln (dot) de |
Created | 2020-07-06 (195 days ago) |
Due | |
Updated | 2020-07-18 (183 days ago) |
Assigned | 2020-07-14 (187 days ago) |
Resolved | |
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
State ⇒ Assigned
Type ⇒ Bug
State ⇒ Unconfirmed
Priority ⇒ 2. Medium
Summary ⇒ Bcc field not saved in sent messages
Queue ⇒ IMP
Milestone ⇒
Patch ⇒ Yes
New Attachment: imp-compose-bcc.patch
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.