Tickets :: [#9771] Add sentmail entries on new messages too

6.0.0-beta1

11/8/25

Summary	Add sentmail entries on new messages too
Queue	IMP
Queue Version	Git master
Type	Bug
State	Resolved
Priority	1. Low
Owners
Requester	rui.carneiro (at) portugalmail (dot) net
Created	04/01/2011 (5335 days ago)
Due
Updated	04/01/2011 (5335 days ago)
Assigned
Resolved	04/01/2011 (5335 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	Yes

04/01/2011 07:03:32 PM	rui (dot) carneiro (at) portugalmail (dot) net	Comment #5	Reply to this comment
Not really. Although storing with integers would be more efficient storage, not sure we want to go through the trouble of converting existing tables at this time. So fixed IMP_Sentmail to continue to insert text entries for the action type. Fair enough :)

04/01/2011 06:27:00 PM	Michael Slusarz	Comment #4 State ⇒ Resolved	Reply to this comment
Related bug: http://bugs.horde.org/ticket/9763 Patch attached. Altered and fixed. The sentmail table is being updated with numbers (IMP_Compose constants). This is the expected behavior? Not really. Although storing with integers would be more efficient storage, not sure we want to go through the trouble of converting existing tables at this time. So fixed IMP_Sentmail to continue to insert text entries for the action type.

04/01/2011 06:25:23 PM	Git Commit	Comment #3	Reply to this comment
Changes have been made in Git for this ticket: ~~Bug #9771~~: Clean up IMP_Sentmail 7 files changed, 83 insertions(+), 82 deletions(-) http://git.horde.org/horde-git/-/commit/300963658dd52062a4b459b38b2923ea5e874f72

04/01/2011 06:25:20 PM	Git Commit	Comment #2	Reply to this comment
Changes have been made in Git for this ticket: ~~Bug #9771~~: Add constant entry for new message 1 files changed, 2 insertions(+), 1 deletions(-) http://git.horde.org/horde-git/-/commit/ad556edd3da098245ca334ba0edd75a0efb0c319

04/01/2011 11:28:17 AM	rui (dot) carneiro (at) portugalmail (dot) net	Comment #1 Priority ⇒ 1. Low New Attachment: 0001-Add-sentmail-entries-on-new-messages-too.patch Patch ⇒ Yes Milestone ⇒ Queue ⇒ IMP Summary ⇒ Add sentmail entries on new messages too Type ⇒ Bug State ⇒ Unconfirmed	Reply to this comment
Related bug: http://bugs.horde.org/ticket/9763 Patch attached. The sentmail table is being updated with numbers (IMP_Compose constants). This is the expected behavior?