From fa8c7c35b5869497df02253cd7b6fc6da566f423 Mon Sep 17 00:00:00 2001 From: Klaus Schmidinger Date: Sat, 17 Apr 2021 09:44:01 +0200 Subject: [PATCH] Fixed the timer indicator in the Schedule menu in case an event is already over, but the timer is still recording --- HISTORY | 4 +++- menu.c | 4 ++-- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/HISTORY b/HISTORY index 9ce48e20..6680d1c4 100644 --- a/HISTORY +++ b/HISTORY @@ -9641,7 +9641,7 @@ Video Disk Recorder Revision History - No longer switching devices for pattern timers (thanks to Helmut Binder). - cTimer::TriggerRespawn() now only acts on local timers. -2021-04-16: +2021-04-17: - When spawning pattern timers, the new function cTimers::GetTimerForEvent() is now used to check whether a matching event already has a local spawned timer. Reason: creating a timer @@ -9656,3 +9656,5 @@ Video Disk Recorder Revision History matching events that would start while the first one is still recording. - Now making sure that spawned timers with reduced start/stop margins actually record with the full margins. +- Fixed the timer indicator in the Schedule menu in case an event is already over, but the + timer is still recording. diff --git a/menu.c b/menu.c index 8685fb1c..2beae50c 100644 --- a/menu.c +++ b/menu.c @@ -4,7 +4,7 @@ * See the main source file 'vdr.c' for copyright information and * how to reach the author. * - * $Id: menu.c 5.3 2021/01/14 10:29:05 kls Exp $ + * $Id: menu.c 5.4 2021/04/17 09:44:01 kls Exp $ */ #include "menu.h" @@ -1602,7 +1602,7 @@ bool cMenuScheduleItem::Update(const cTimers *Timers, bool Force) eTimerMatch OldTimerMatch = timerMatch; bool OldTimerActive = timerActive; const cTimer *Timer = Timers->GetMatch(event, &timerMatch); - if (event->EndTime() < time(NULL) && !event->IsRunning()) + if (event->EndTime() < time(NULL) && !event->IsRunning() && (!Timer || !Timer->Recording())) timerMatch = tmNone; timerActive = Timer && Timer->HasFlags(tfActive); if (Force || timerMatch != OldTimerMatch || timerActive != OldTimerActive) {