Bug #54692 for Schedule-Cron: [patch] check_entry returns incorrect indexes in the case of delete

Wed Feb 17 08:48:23 2010 bakhtin [...] amt.ru - Ticket created

Subject:	[patch] check_entry returns incorrect indexes in the case of delete_entry is called before it
Date:	Wed, 17 Feb 2010 16:19:51 +0300
To:	"bug-Schedule-Cron [...] rt.cpan.org" <bug-Schedule-Cron [...] rt.cpan.org>
From:	"Bakhtin, Alexander" <bakhtin [...] amt.ru>

Hello, If we call check_entry after delete_entry it returns incorrect index because delete_entry doesn't update $self->{map} hash with new indexes. The patch was made for clearenes, not for speed, because I'm assuming that delete_entry should not be invoked frequently. Executed on current version (0.99) script output is: Show quoted text

> perl test_cron_bug.pl

Test1 is OK. Test2 is FAIL. Test3 is OK. After pathc: Show quoted text

> perl -I . test_cron_bug.pl

Test1 is OK. Test2 not found Test3 is OK. --- Alex Bakhtin, CCIE #8439. Head of Technical Assistance Center AMT Group

Message body is not shown because sender requested not to inline it.

Mon Mar 01 13:03:52 2010 roland [...] cpan.org - Correspondence added

Hi Alex, thanx for your patch. I will include it in the final 1.00 which I will do in some weeks.

Mon Mar 01 13:03:53 2010 The RT System itself - Status changed from 'new' to 'open'

Fri May 14 12:19:23 2010 roland [...] cpan.org - Correspondence added

1.00 is out with your patch added. Thanx for fix. 1.01 will include your test case as well. thx ... ...roland

Fri May 14 12:19:31 2010 roland [...] cpan.org - Taken

Fri May 14 12:19:49 2010 roland [...] cpan.org - Correspondence added

Fixed in 1.00.

Fri May 14 12:19:51 2010 roland [...] cpan.org - Status changed from 'open' to 'resolved'

Bug #54692 for Schedule-Cron: [patch] check_entry returns incorrect indexes in the case of delete_entry is called before it