Bug 242978

Summary:	Regression: reminders do not work anymore
Product:	[Applications] korganizer	Reporter:	Sabine Faure <sabine>
Component:	general	Assignee:	kdepim bugs <kdepim-bugs>
Status:	VERIFIED FIXED
Severity:	normal	CC:	smartins
Priority:	NOR
Version:	SVN trunk (KDE 4)
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:		Version Fixed In:

Description Sabine Faure 2010-06-27 23:28:06 UTC

Version:           SVN trunk (KDE 4) (using Devel) 
OS:                Linux

After setting a reminder for an event, the deamon never displays a reminder as it should.

Reproducible: Didn't try

Steps to Reproduce:
- Launch KOrg
- Go to Agenda view
- Create a new event for today starting in 5 minutes
- Click on the 'Advanced...' button
- Set the reminder for two minutes before the event starts
- Select the 'Display' radio button and enter a text in the 'Display text:' line edit
- Click on 'Ok' twice
- Click once on the event from the agenda view
- Details about the event are displayed in the details view and a reminder icon (red bell) has correctly been added
- Wait for 4 minutes

Actual Results:  
The reminder dialogue does not appear to remind the user that the event will soon start

Expected Results:  
whereas it should.

This is a regression since it was working when I tested it last.

Trunk, Svn Rev 1142760

Comment 1 Sergio Martins 2010-07-23 00:50:05 UTC

Working fine here.

Is it possible that korgac (alarm deamon) crashed on startup?

Comment 2 Sergio Martins 2010-07-23 19:56:38 UTC

SVN commit 1153663 by smartins:

Ignore changes to collections here, was triggering too many view updates. It was a brute force attempt to detect when access rights are changed.

Now we're listening to Monitor's collectionChanged signal, which contains info about which attributes were changed.


CCBUG: 242978


 M  +1 -19     calendar.cpp  
 M  +0 -1      calendar_p.h  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1153663

Comment 3 Sabine Faure 2010-07-29 01:20:45 UTC

It is corrected now.

The reminder is displayed now on the correct set time.

Trunk, Svn Rev 1156007