Bug 67017 - knotify event for new mail not triggerd
Summary: knotify event for new mail not triggerd
Status: RESOLVED FIXED
Alias: None
Product: kontact
Classification: Applications
Component: mail (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords:
: 70228 (view as bug list)
Depends on:
Blocks:
 
Reported: 2003-11-01 21:18 UTC by Thomas Vollmer
Modified: 2007-09-14 12:17 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Thomas Vollmer 2003-11-01 21:18:17 UTC
Version:           CVS HEAD from 29.10.2003 (using KDE Devel)
Installed from:    Compiled sources
Compiler:          gcc 3.2 
OS:          Linux

Hi,

when useing kmail from kontact I dont get notification sounds and popups when new mail arrives.

Thomas
Comment 1 Daniel Molkentin 2003-11-01 21:47:23 UTC
Any idea why this doesn't work?
Comment 2 Daniel Molkentin 2003-11-01 21:50:57 UTC
Any idea why this doesn't work?
Comment 3 Thomas Vollmer 2003-11-07 08:17:43 UTC
Maybe I have a hint for You.

When KMail is active, there is a menu entry under settings to define the notifications like it is in standalone kmail.

When You open this there are no possible notifications and the dialog states "notifications sesstings kontact".

Thomas
Comment 4 Sean Clarke 2003-12-09 01:42:25 UTC
Can confirm it appears to work fine in KDE 3.2 beta 2
Comment 5 Thomas Vollmer 2003-12-09 09:23:24 UTC
Subject: Re:  knotify event for new mail not triggerd

On Tuesday 09 December 2003 01:42, Sean Clarke wrote:
> ------- You are receiving this mail because: -------
> You reported the bug, or are watching the reporter.
>
> http://bugs.kde.org/show_bug.cgi?id=67017
>
>
>
>
> ------- Additional Comments From sean.e.clarke@ntlworld.com 
> 2003-12-09 01:42 ------- Can confirm it appears to work fine in KDE
> 3.2 beta 2

Not for me. We are talking about new mail notifications from within 
Kontact, right?

Thomas
Comment 6 Sean Clarke 2003-12-10 00:41:45 UTC
yes,

I get the "beep" notification and my tray icon displays the number of new messages.
Comment 7 Thomas Vollmer 2003-12-10 09:53:39 UTC
When You use the KDE Control Center under Sound & Multimedia/System Messages to configure a dialog and a differnt sound for KMail, do You still get this notifications?

I dont't think that Your beep and for sure the message count in the systray is related to a KNotify event in any way.

Thomas
Comment 8 Thomas Vollmer 2003-12-17 10:59:12 UTC
Hi,

this is a very simple Problem to solve. The reason is that kontact doesn't have a eventrc file.

I copied over the one from KMail, made some smaller changes (e.g. renaming to kontact and change icon) and voila I can configure the notifications and get them now.

Can somebody do this in CVS? I dont have access to it.

Thomas
Comment 9 Volker Krause 2003-12-17 13:24:48 UTC
Subject: kdepim/kmail

CVS commit by vkrause: 

Mostly fix #67017 (new mail notification in kontact) by helping kmail to find its .eventsrc file when running in kontact.

Remaining issues: Passive popups show up in the left upper corner and taskbar flashing doesn't work.

Reviewed by Ingo Kl
Comment 10 Ingo Klöcker 2003-12-18 00:08:01 UTC
Volker's patch fixes this bug.
Comment 11 Volker Krause 2004-01-02 19:05:17 UTC
*** Bug 70228 has been marked as a duplicate of this bug. ***
Comment 12 Volker Krause 2004-02-24 18:49:33 UTC
CVS commit by vkrause: 

Fix the remaining issues with kontact's new mail notification, i.e. passive popups are now placed correctly and taskbar flashing works.

Reviewed by Ingo Kloecker

CCMAIL: 67017@bugs.kde.org


  M +4 -2      kmmainwidget.cpp   1.144


--- kdepim/kmail/kmmainwidget.cpp  #1.143:1.144
@@ -763,8 +763,10 @@ void KMMainWidget::slotMailChecked(bool 
   if(kmkernel->xmlGuiInstance()) {
     KNotifyClient::Instance instance(kmkernel->xmlGuiInstance());
-    KNotifyClient::event(0, "new-mail-arrived", i18n("New mail arrived"));
+    KNotifyClient::event(topLevelWidget()->winId(), "new-mail-arrived", 
+      i18n("New mail arrived"));
   }
   else
-    KNotifyClient::event(0, "new-mail-arrived", i18n("New mail arrived"));
+    KNotifyClient::event(topLevelWidget()->winId(), "new-mail-arrived", 
+      i18n("New mail arrived"));
   if (mBeepOnNew) {
     KNotifyClient::beep();