Tickets :: [#5234] Add user hook to add formatting to a message list row

6.0.0-beta1

7/10/25

Summary	Add user hook to add formatting to a message list row
Queue	IMP
Queue Version	HEAD
Type	Enhancement
State	Resolved
Priority	2. Medium
Owners	slusarz (at) horde (dot) org
Requester	robb (at) wtg (dot) cw (dot) com
Created	04/10/2007 (6666 days ago)
Due
Updated	01/03/2008 (6398 days ago)
Assigned
Resolved	01/03/2008 (6398 days ago)
Milestone	4.2
Patch	No

01/03/2008 07:08:29 AM	Michael Slusarz	Comment #10 Summary ⇒ Add user hook to add formatting to a message list row State ⇒ Resolved	Reply to this comment
Merged to 4.2.

01/03/2008 07:06:30 AM	Matt Selsky	Comment #9	Reply to this comment
Spoke too soon. Thanks, Michael.

01/03/2008 07:05:46 AM	Matt Selsky	Comment #8	Reply to this comment
Latest commit shows: Fatal error: Call to undefined method IMP_UI_Mailbox::getAttachmentType() in /etc/httpd/htdocs/horde/imp/config/hooks.php on line 253 getAttachmentType() comes from IMP_UI_Message...

12/26/2007 11:07:27 PM	Michael Slusarz	Comment #7 State ⇒ Feedback	Reply to this comment
The testing code has been added to HEAD. I will test for a few days in HEAD before merging to FW_3.

12/26/2007 08:40:49 AM	Michael Slusarz	Comment #6 Assigned to Michael Slusarz Priority ⇒ 2. Medium	Reply to this comment
I am currently testing some code that implements a new hook to do what is discussed in this ticket. This should be marked as blocking for IMP 4.2 since it requires a configuration change (2 config options are removed in exchange for an added hook), and config changes like this makes the most sense for a minor release, not a dot release.

05/02/2007 09:45:25 AM	Michael Slusarz	Comment #5	Reply to this comment
While we are at it, we should probably move all icons that require more than an IMAP flag to a hook (i.e. attachments). Although with these cases, I am unsure how the hook would be able to take advantage of message caching.

05/02/2007 02:00:36 AM	Chuck Hagenbuch	State ⇒ Accepted

05/01/2007 10:20:21 PM	Jan Schneider	Comment #4	Reply to this comment
The hook idea sounds good.

04/30/2007 08:59:18 PM	Chuck Hagenbuch	Comment #3	Reply to this comment
Actually while I understand the request I'm not really in favor of adding more icons and stuff like this. Jan, Michael, what are your thoughts on this feature? It seems like the idea would be to make it possible through a hook, but not to put it in to Horde out of the box.

04/13/2007 09:50:19 PM	Chuck Hagenbuch	State ⇒ Feedback

04/10/2007 03:49:54 PM	Chuck Hagenbuch	Comment #2	Reply to this comment
We need a decent way to allow hooks like this that require MailboxCache data to work effectively... thoughts?

04/10/2007 02:59:30 PM	robb (at) wtg (dot) cw (dot) com	Comment #1 Priority ⇒ 1. Low State ⇒ New Queue ⇒ IMP Summary ⇒ show a spam icon for email with header X-Spam-Status: Yes Type ⇒ Enhancement	Reply to this comment
for email that has "X-Spam-Status: Yes" header show a spam icon next to the message in the folder view.