Calendar - Change to recurring (AKA Yet another bug fix)

Print Page | Close Window

Change to recurring (AKA Yet another bug fix)

Printed From: Codejock Forums
Category: Codejock Products
Forum Name: Calendar
Forum Description: Topics Related to Codejock Calendar
URL: http://forum.codejock.com/forum_posts.asp?TID=7241
Printed Date: 11 May 2024 at 5:11pm
Software Version: Web Wiz Forums 12.04 - http://www.webwizforums.com

Topic: Change to recurring (AKA Yet another bug fix)

Posted By: JohnCrenshaw
Subject: Change to recurring (AKA Yet another bug fix)
Date Posted: 25 May 2007 at 1:15pm

This should only an issue when using the MAPI data provider (I didn't test the other, so I can't be 100% sure.)

In xtpcalendarmapidataprovider.cpp at line 3542 we are inside the MAPI callback, dealing with the fact that the event changed, and updating the cache. On this line, there is a silly mistake.

ptrEvent2->SetRecurrencePatternID(dwPatternID);

the pattern ID was just grabbed from another event pointer, and now it is being coppied over to this one. This is fine, except that in this case, the old event HAD no pattern ID, and the new one does. This causes the code inside _AddRPatternWithExceptions (called by ChangeEvent which is called on the next line) to fail when it assumes that we DO have an ID. The fix is fairly basic.

if (dwPatternID != XTP_CALENDAR_UNKNOWN_RECURRENCE_PATTERN_ID)
{
    ptrEvent2->SetRecurrencePatternID(dwPatternID);
}

Replies:

Posted By: JohnCrenshaw
Date Posted: 24 August 2007 at 4:17pm

Just installed 11.1.3 and this still hasn't been fixed, just want to make sure that this gets into the next build.

Posted By: JohnCrenshaw
Date Posted: 19 October 2007 at 6:57pm

11.2.0 is fixed, thanks a bunch.