Tickets :: [#9803] new mail notification is not shown when IMP summary in portal refreshes automatically

6.0.0-beta1

7/5/25

Summary	new mail notification is not shown when IMP summary in portal refreshes automatically
Queue	IMP
Queue Version	Git master
Type	Enhancement
State	Resolved
Priority	1. Low
Owners	slusarz (at) horde (dot) org
Requester	vilius (at) lnk (dot) lt
Created	04/04/2011 (5206 days ago)
Due
Updated	04/14/2011 (5196 days ago)
Assigned
Resolved	04/12/2011 (5198 days ago)
Milestone
Patch	No

04/14/2011 05:23:20 PM	Michael Slusarz	Comment #9	Reply to this comment
Still I'm confused why notifications are not used in Portal screen even though they are used in every other window (including New Message Compose screen). How would those notifications be displayed? Updateable Blocks simply update the HTML contents contained inside of them. They can't display notifications on the page because there is no system to dynamically add notifications to the blocks page (not entirely true - if authenticated to IMP in dynamic mode, it would technically be possible to push notifications via Growler notifications. But this is not available in all circumstances so you can't code for this).

04/14/2011 08:10:47 AM	vilius (at) lnk (dot) lt	Comment #8	Reply to this comment
2011-04-13T10:45:03+03:00 NOTICE: HORDE [horde] PHP ERROR: Undefined offset: 1 [pid 20089 on line 207 of "/usr/share/pear/Horde/Core/Block/Layout/Manager.php"] Can't reproduce. Have you tried deleting and re-adding block? At a minimum, this is not related to the current ticket though. Yes, I reported this on separate ~~bug 9910~~. 2011-04-13T10:45:11+03:00 NOTICE: HORDE [imp] PHP ERROR: Undefined property: IMP_Imap::$imap [pid 19298 on line 79 of "/var/www/html/horde/imp/lib/Block/Summary.php"] Your IMP is not up-to-date with all HEAD commits. True, just updated with today's HEAD and now I can see all needed folders. Still I'm confused why notifications are not used in Portal screen even though they are used in every other window (including New Message Compose screen).

04/13/2011 08:29:09 PM	Michael Slusarz	Comment #7	Reply to this comment
2011-04-13T10:45:03+03:00 NOTICE: HORDE [horde] PHP ERROR: Undefined offset: 1 [pid 20089 on line 207 of "/usr/share/pear/Horde/Core/Block/Layout/Manager.php"] Can't reproduce. Have you tried deleting and re-adding block? At a minimum, this is not related to the current ticket though. 2011-04-13T10:45:11+03:00 NOTICE: HORDE [imp] PHP ERROR: Undefined property: IMP_Imap::$imap [pid 19298 on line 79 of "/var/www/html/horde/imp/lib/Block/Summary.php"] Your IMP is not up-to-date with all HEAD commits.

04/13/2011 07:53:15 AM	vilius (at) lnk (dot) lt	Comment #6	Reply to this comment
After these changes I see only Inbox folder in the summary. Also there is something wrong with saving this block. It fails every second time with notice: "Horde is not activated." PHP error log shows: 2011-04-13T10:45:03+03:00 NOTICE: HORDE [horde] PHP ERROR: Undefined offset: 1 [pid 20089 on line 207 of "/usr/share/pear/Horde/Core/Block/Layout/Manager.php"] and 2011-04-13T10:45:11+03:00 NOTICE: HORDE [imp] PHP ERROR: Undefined property: IMP_Imap::$imap [pid 19298 on line 79 of "/var/www/html/horde/imp/lib/Block/Summary.php"]

04/12/2011 11:16:26 PM	Michael Slusarz	Assigned to Michael Slusarz

04/12/2011 11:16:14 PM	Michael Slusarz	Comment #5 Type ⇒ Enhancement State ⇒ Resolved Priority ⇒ 1. Low	Reply to this comment
Even with new Alarm system in place, I still don't get any notification when portal refreshes automatically. Notifications are sent via the notification system, not the alarm system. Also, for what it's worth, old notification for new email was much better. It was more noticeable and a user have had a link to go straight to the mailbox with new email. Are there any plans to add this functionality back? No. The portal view should be a succinct way to look at your messages. The old way was tremendously inefficient and hackish anyway. But I will compromise some and add recent message information to the summary screen, especially since it doesn't take up much space (or additional processing) and also fits in with the UI.

04/12/2011 11:15:54 PM	Git Commit	Comment #4	Reply to this comment
Changes have been made in Git for this ticket: ~~Request #9803~~: Add RECENT information to IMP summary portal block 1 files changed, 7 insertions(+), 6 deletions(-) http://git.horde.org/horde-git/-/commit/e80b37d483d8e371cb1622c847b1066ef0731cea

04/09/2011 07:48:39 PM	vilius (at) lnk (dot) lt	Comment #3	Reply to this comment
Even with new Alarm system in place, I still don't get any notification when portal refreshes automatically. Also, for what it's worth, old notification for new email was much better. It was more noticeable and a user have had a link to go straight to the mailbox with new email. Are there any plans to add this functionality back?

04/05/2011 04:19:01 AM	Michael Slusarz	Comment #2 State ⇒ Not A Bug	Reply to this comment
See ~~Ticket #9751~~. There are no more popup new mail notifications.

04/04/2011 05:35:38 PM	vilius (at) lnk (dot) lt	Comment #1 State ⇒ Unconfirmed Patch ⇒ No Milestone ⇒ Queue ⇒ IMP Summary ⇒ new mail notification is not shown when IMP summary in portal refreshes automatically Type ⇒ Bug Priority ⇒ 1. Low	Reply to this comment
New mail notification is shown only when I click on Portal icon manually. If I leave Portal window to refresh automatically through AJAX there is no nofitication.