6.0.0-beta1
7/15/25

[#15020] Bcc field not saved in sent messages
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

History
01/19/2021 06:06:56 PM Jan Schneider State ⇒ Resolved
 
07/18/2020 07:35:10 AM samuel (dot) wolf (at) wolf-maschinenbau (dot) de Comment #4 Reply to this comment
Can you please try this patch instead?
Jan, this patch works, thank you.
07/17/2020 09:53:06 AM Jan Schneider Comment #3
Assigned to Jan Schneider
Taken from Michael Rubinsky
Reply to this comment
Can you please try this patch instead?
07/17/2020 09:52:15 AM Git Commit Comment #2 Reply to this comment
Changes have been made in Git (FRAMEWORK_5_2):

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
07/14/2020 12:25:45 PM Michael Rubinsky State ⇒ Assigned
Assigned to Michael Rubinsky
 
07/06/2020 04:03:56 PM wahnes (at) uni-koeln (dot) de Comment #1
Priority ⇒ 2. Medium
Type ⇒ Bug
Summary ⇒ Bcc field not saved in sent messages
Queue ⇒ IMP
Milestone ⇒
Patch ⇒ Yes
New Attachment: imp-compose-bcc.patch Download
State ⇒ Unconfirmed
Reply to this comment
Starting with version 6.2.25 of Imp, the list of "Bcc" recipients of a 
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.

Saved Queries