From 426696b03360fc6b4b3c29206f6b0264cb93b2fc Mon Sep 17 00:00:00 2001 From: Francis Lachapelle Date: Wed, 22 Aug 2012 13:49:45 +0000 Subject: [PATCH] See ChangeLog. Monotone-Parent: 6722e59df0723e9698a938cf20033b44e842756f Monotone-Revision: 52bc1ab2a2dcc4de7809fd592d0e9c41c556cf17 Monotone-Author: flachapelle@inverse.ca Monotone-Date: 2012-08-22T13:49:45 --- SOPE/NGCards/ChangeLog | 6 ++++++ SOPE/NGCards/iCalWeeklyRecurrenceCalculator.m | 6 ++---- 2 files changed, 8 insertions(+), 4 deletions(-) diff --git a/SOPE/NGCards/ChangeLog b/SOPE/NGCards/ChangeLog index 1fe2c27c8..e5922829f 100644 --- a/SOPE/NGCards/ChangeLog +++ b/SOPE/NGCards/ChangeLog @@ -1,3 +1,9 @@ +2012-08-22 Francis Lachapelle + + * iCalWeeklyRecurrenceCalculator.m + (-recurrenceRangesWithinCalendarDateRange:): fixed validation of + the interval when a day mask is used on multiple days. + 2012-08-21 Francis Lachapelle * iCalRepeatableEntityObject.m (-exceptionDatesWithTimeZone:): we diff --git a/SOPE/NGCards/iCalWeeklyRecurrenceCalculator.m b/SOPE/NGCards/iCalWeeklyRecurrenceCalculator.m index f49536646..140d05531 100644 --- a/SOPE/NGCards/iCalWeeklyRecurrenceCalculator.m +++ b/SOPE/NGCards/iCalWeeklyRecurrenceCalculator.m @@ -118,8 +118,8 @@ currentStartDate = [firStart copy]; [currentStartDate autorelease]; ranges = [NSMutableArray array]; - i = 0; count = 0; + i = [currentStartDate dayOfWeek]; // Set the first day of the week as Sunday and ignore WKST if (dayMask == nil) { @@ -158,7 +158,7 @@ [startDate compare: currentStartDate] == NSOrderedSame) { // If the rule count is defined, stop once the count is reached. - if (i == 0) + if ([currentStartDate compare: firStart] == NSOrderedSame) { // Always add the start date of the recurring event if within // the lookup range. @@ -166,8 +166,6 @@ } else { - // The following always set the first day of the week as the day - // of the master event start date, ie WKST is ignored. week = i / 7; if ((week % interval) == 0 &&