History | Log In     View a printable version of the current page.  
Issue Details (XML | Word | Printable)

Key: QUARTZ-680
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: James House
Reporter: Wessel van Norel
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Quartz Scheduler

AnnualCalendar isDayExcluded (and with that getNextIncludedTime) doesn't take the basecalendar into account

Created: 15/Jul/08 06:47 AM   Updated: 02/Aug/08 09:29 PM
Component/s: None
Affects Version/s: 1.6
Fix Version/s: None


 Description  « Hide
The isDayExcluded method doesn't check the base calendar. and because of that the getNextIncludedTime function can return an invalid date, according to the base calendar.

If you just add

        // Check baseCalendar first
        if (! super.isTimeIncluded(day.getTime().getTime())) {
         return true;
        }

to the top of the isDayExcluded method (just belof the check if (day == null), it should be fine, if I'm not mistaken.

 All   Comments   Change History      Sort Order:
James House - [02/Aug/08 09:29 PM ]

fix applied. thanks!