Summary | Broken notification in compose window |
Queue | IMP |
Queue Version | Git master |
Type | Bug |
State | Resolved |
Priority | 1. Low |
Owners | slusarz (at) horde (dot) org |
Requester | jan (at) horde (dot) org |
Created | 04/21/2011 (5138 days ago) |
Due | |
Updated | 05/18/2011 (5111 days ago) |
Assigned | 05/03/2011 (5126 days ago) |
Resolved | 05/18/2011 (5111 days ago) |
Milestone | |
Patch | No |
State ⇒ Resolved
discussed below, this is a limitation of the current Notification
system. Will add a new Enhancement ticket to track this.
between the head and body tags.
Bug #9950: Fix undefined error1 files changed, 0 insertions(+), 1 deletions(-)
http://git.horde.org/horde-git/-/commit/71215a4e0f57a15614556de24efc5f7f20d7c442
Bug #9950: Fix notifications in dimp popup screens4 files changed, 6 insertions(+), 1 deletions(-)
http://git.horde.org/horde-git/-/commit/1e9e1316ee5f730cd724f13ed215098e5686475b
NOTICE: HORDE [imp] PHP ERROR: Undefined variable: status [pid 26255
on line 279 of "/var/www/html/hordetest/imp/compose-dimp.php
between the head and body tags.
between the head and body tags.
Bug #9950: Fix notifications in dimp popup screens4 files changed, 6 insertions(+), 1 deletions(-)
http://git.horde.org/horde-git/-/commit/1e9e1316ee5f730cd724f13ed215098e5686475b
piling up. Though this might be a problem of audio notifications not
being used in the main interface. It makes them pile up, and they
are all outputed at once (making a terrible noise) if you open a
page that supports audio notifications. I'm not sure if we should
implement a duplicate check in the audio notification listener. But
for now we should either support audio notifications throughout DIMP
or check for duplicates where we output the notifications in DIMP.
limitation) in the current Notification system. At present, there
really is no way to push a notification with the intent of "I want the
notification to be displayed on this page access and, if not, you can
go ahead and discard the notification". Especially for things like
newmail notifications, this is time-critical information. It becomes
useless once the page access is complete.
I've worked around the limitation in IMP (somewhat) by using
Decorators. Decorators were used simply because they are triggered
whenever notify() is called. Thus, I only create notifications if I
know the notifications are about to be displayed.
However, an alternative method would be to indicate, at the time we
are pushing the notification, whether the notification should persist
in the session. (Although an advantage of doing the Decorator
approach is to prevent potentially costly activities, such as mailbox
polling, when there is absolutely no chance it would be used on a page).
State ⇒ Assigned
the head and body tags.
And they are raised several times if you had several notifications
piling up. Though this might be a problem of audio notifications not
being used in the main interface. It makes them pile up, and they are
all outputed at once (making a terrible noise) if you open a page that
supports audio notifications. I'm not sure if we should implement a
duplicate check in the audio notification listener. But for now we
should either support audio notifications throughout DIMP or check for
duplicates where we output the notifications in DIMP.
State ⇒ Resolved
State ⇒ Feedback
Bug #9950: Fix notifications in compose window (dynamic view)1 files changed, 4 insertions(+), 0 deletions(-)
http://git.horde.org/horde-git/-/commit/d24291a4c369b4a81a04c01541026acce6e823a0
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ Broken notification in compose window
Queue ⇒ IMP
Assigned to Michael Slusarz
Milestone ⇒
Patch ⇒ No
State ⇒ Assigned
notification isn't displayed anywhere. If that notification contains
audio, the audio tag is printed before starting the page output.
Commit fdd357ede9d2452c6db34f8a6282d456cfb9402c was supposed to fix
that, but it broke other things.