6.0.0-git
2020-08-09

[#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 Assigned
Priority 2. Medium
Owners jan (at) horde (dot) org
Requester wahnes (at) uni-koeln (dot) de
Created 2020-07-06 (34 days ago)
Due
Updated 2020-07-18 (22 days ago)
Assigned 2020-07-14 (26 days ago)
Resolved
Milestone
Patch Yes

History
2020-07-18 07:35:10 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.
2020-07-17 09:53:06 Jan Schneider Comment #3
Assigned to Jan Schneider
Taken from Michael Rubinsky
Reply to this comment
Can you please try this patch instead?
2020-07-17 09:52:15 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
2020-07-14 12:25:45 Michael Rubinsky Assigned to Michael Rubinsky
State ⇒ Assigned
 
2020-07-06 16:03:56 wahnes (at) uni-koeln (dot) de Comment #1
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 Download
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