[PHPiCalendar-DEV] Another repeat issue - what's this for?

[Jim Hu <ji...@ta...>]

This is in the case 'WEEKLY' part of ical_parser.php.

// Since this renders events from $next_range_time to  
$next_range_time + 1 week, I need to handle intervals
// as well. This checks to see if $next_date_time is after $day_start  
(i.e., "next week"), and thus
// if we need to add $interval weeks to $next_date_time.
if ($next_date_time > strtotime($week_start_day, $next_range_time) &&  
$interval > 1) {
  	$next_date_time = strtotime('+'.($interval - 1).' '.$freq_type,  
$next_date_time);
}

This appears to be the cause of a bug reported on the bbs, where  
events that repeat every 2 weeks gain a week in successive months.   
Commenting it out seems to fix that bug, and my test calendar works  
with repeats of 1, 2, 3, 4, and 5 weeks.  But presumably the code was  
put in the parser for a reason.  Or is it now redundant with  
something else? $next_range_time must be bumped by $interval elsewhere.

This has NOT been added to CVS, but the patch is noted on the bbs.
=====================================
Jim Hu