[#5234] Add user hook to add formatting to a message list row
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 Michael Slusarz <slusarz (at) horde (dot) org>
Requester robb (at) wtg (dot) cw (dot) com
Created 04/10/2007 (499 days ago)
Due
Updated 01/03/2008 (231 days ago)
Assigned
Resolved 01/03/2008 (231 days ago)
Attachments
Milestone 4.2
Patch No

History
01/03/2008 Michael Slusarz Comment #10
State ⇒ Resolved
Summary ⇒ Add user hook to add formatting to a message list row
Reply to this comment
Merged to 4.2.
01/03/2008 Matt Selsky Comment #9 Reply to this comment
Spoke too soon.  Thanks, Michael.
01/03/2008 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 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 Michael Slusarz Comment #6
Priority ⇒ 2. Medium
Assigned to Michael Slusarz
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 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/01/2007 Chuck Hagenbuch State ⇒ Accepted
 
05/01/2007 Jan Schneider Comment #4 Reply to this comment
The hook idea sounds good.
04/30/2007 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 Chuck Hagenbuch State ⇒ Feedback
 
04/10/2007 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 robb (at) wtg (dot) cw (dot) com Comment #1
Summary ⇒ show a spam icon for email with header X-Spam-Status: Yes
Queue ⇒ IMP
Type ⇒ Enhancement
State ⇒ New
Priority ⇒ 1. Low
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.