Patrick Rother wrote:
> Hello.
>
> There seems to be a bug in timer handling.
>
> Please see this SVDRP snapshot:
>
> 220 vdr SVDRP VideoDiskRecorder 1.3.30; Tue Sep 6 10:50:38 2005
> newt 1:S19.2E-133-33-46:----F--:2213:2250:90:30:Sechserpack:
> 550 Timer already defined: 12 1:5:-D-----:2213:2250:40:10:Mensch_Markus:
> lstt 12
> 250 12 1:5:-D-----:2213:2250:40:10:Mensch_Markus:
> newt 1:S19.2E-133-33-46:----F--:2213:2250:90:30:Sechserpack:
> 550 Timer already defined: 12 1:5:-D-----:2213:2250:40:10:Mensch_Markus:
>
> vdr 1.3.30 refuses to install a weekly timer with same start/stop
> time for a different day of week.
>
> I get the new timer in by modifying start to by one minute, so this
> has no actual service impact, but this shall be fixed anyway.
Please try the attached patch.
Klaus
-------------- next part --------------
--- timers.c 2005/09/04 10:53:21 1.35
+++ timers.c 2005/09/09 15:22:33
@@ -519,7 +519,10 @@
cTimer *cTimers::GetTimer(cTimer *Timer)
{
for (cTimer *ti = First(); ti; ti = Next(ti)) {
- if (ti->Channel() == Timer->Channel() && ti->Day() == Timer->Day() && ti->Start() == Timer->Start() && ti->Stop() == Timer->Stop())
+ if (ti->Channel() == Timer->Channel() &&
+ (ti->WeekDays() && ti->WeekDays() == Timer->WeekDays() || !ti->WeekDays() && ti->Day() == Timer->Day()) &&
+ ti->Start() == Timer->Start() &&
+ ti->Stop() == Timer->Stop())
return ti;
}
return NULL;
