Is anyone using this hook? I've already implemented performance improvement as described in item 1. of my request: http://trac.roundcube.net/ticket/1485690
I use BODY[] instead of BODY.PEEK[] in FETCH command to set the \Seen flag implicitly. This gives one STORE command less. Now the hook... it is executed in show.inc when message hasn't got the \Seen flag set. With my changes the hook will be never executed. I propose to remove this hook.
Could be useful for servers which do not support MDN Flags like hMailServer!
Regards, Roland
----- Original Message ----- From: "A.L.E.C" alec@alec.pl To: "RoundCube Dev" dev@lists.roundcube.net Sent: Friday, July 10, 2009 7:25 PM Subject: [RCD] message_read hook
Is anyone using this hook? I've already implemented performance improvement as described in item 1. of my request: http://trac.roundcube.net/ticket/1485690
I use BODY[] instead of BODY.PEEK[] in FETCH command to set the \Seen flag implicitly. This gives one STORE command less. Now the hook... it is executed in show.inc when message hasn't got the \Seen flag set. With my changes the hook will be never executed. I propose to remove this hook.
-- Aleksander 'A.L.E.C' Machniak http://alec.pl gg:2275252 LAN Management System Developer http://lms.org.pl Roundcube Webmail Developer http://roundcube.net _______________________________________________ List info: http://lists.roundcube.net/dev/
List info: http://lists.roundcube.net/dev/
Roland Liebl wrote:
Could be useful for servers which do not support MDN Flags like hMailServer!
Ok, I've got another idea. To make new messages displaying more responsive we could set the \Seen flag after page is sent to the browser. See attached patch. This makes that 'message_read' hook is executed later and will be not possible to send some data to the browser from this hook actions. Do we need this?