From 5a93f45652d5fdc0240117ac52ac33fbbc5fdaa9 Mon Sep 17 00:00:00 2001
From: Kai Sommerfeld <kai.sommerfeld@gmx.com>
Date: Tue, 25 Jan 2022 14:17:42 +0100
Subject: [PATCH 1/4] When parsing recording add/update messages, always set
 the correct error string.

---
 src/Tvheadend.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/Tvheadend.cpp b/src/Tvheadend.cpp
index ad27581e..cc9d9c62 100644
--- a/src/Tvheadend.cpp
+++ b/src/Tvheadend.cpp
@@ -2595,8 +2595,8 @@ void CTvheadend::ParseRecordingAddOrUpdate(htsmsg_t* msg, bool bAdd)
       rec.SetState(PVR_TIMER_STATE_ABORTED);
     else if (strstr(str, "missing") != nullptr)
       rec.SetState(PVR_TIMER_STATE_ERROR);
-    else
-      rec.SetError(str);
+
+    rec.SetError(str);
   }
 
   /* A running recording will have an active subscription assigned to it */

From 290badb2e2243c772e8c0eb4651a6174cad89d13 Mon Sep 17 00:00:00 2001
From: Kai Sommerfeld <kai.sommerfeld@gmx.com>
Date: Tue, 25 Jan 2022 14:19:41 +0100
Subject: [PATCH 2/4] Reduce debug log spam. Do not log recording descriptions.

---
 src/Tvheadend.cpp | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/src/Tvheadend.cpp b/src/Tvheadend.cpp
index cc9d9c62..cefe6684 100644
--- a/src/Tvheadend.cpp
+++ b/src/Tvheadend.cpp
@@ -2641,11 +2641,10 @@ void CTvheadend::ParseRecordingAddOrUpdate(htsmsg_t* msg, bool bAdd)
   /* Update */
   if (rec != comparison)
   {
-    std::string error = rec.GetError().empty() ? "none" : rec.GetError();
+    const std::string error = rec.GetError().empty() ? "n/a" : rec.GetError();
 
-    Logger::Log(LogLevel::LEVEL_DEBUG, "recording id:%d, state:%s, title:%s, desc:%s, error:%s",
-                rec.GetId(), state, rec.GetTitle().c_str(), rec.GetDescription().c_str(),
-                error.c_str());
+    Logger::Log(LogLevel::LEVEL_DEBUG, "recording id:%d, state:%s, title:%s, error:%s",
+                rec.GetId(), state, rec.GetTitle().c_str(), error.c_str());
 
     if (m_asyncState.GetState() > ASYNC_DVR)
     {

From bfe38352af8490832bbccfc3be12274705cadf52 Mon Sep 17 00:00:00 2001
From: Kai Sommerfeld <kai.sommerfeld@gmx.com>
Date: Tue, 25 Jan 2022 14:48:34 +0100
Subject: [PATCH 3/4] Ignore recordings without a file (e.g. removed
 recordings).

---
 src/Tvheadend.cpp | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/src/Tvheadend.cpp b/src/Tvheadend.cpp
index cefe6684..37460047 100644
--- a/src/Tvheadend.cpp
+++ b/src/Tvheadend.cpp
@@ -2299,6 +2299,11 @@ void CTvheadend::ParseRecordingAddOrUpdate(htsmsg_t* msg, bool bAdd)
       !htsmsg_get_u32(msg, "duplicate", &dup) && dup == 1)
     return;
 
+  /* Ignore recordings without a file (e.g. removed recordings) */
+  const char* error = htsmsg_get_str(msg, "error");
+  if (error && (strstr(error, "missing") != nullptr))
+    return;
+
   /* Get/create entry */
   Recording& rec = m_recordings[id];
   Recording comparison = rec;
@@ -2588,15 +2593,12 @@ void CTvheadend::ParseRecordingAddOrUpdate(htsmsg_t* msg, bool bAdd)
   }
 
   /* Error */
-  str = htsmsg_get_str(msg, "error");
-  if (str)
+  if (error)
   {
-    if (!std::strcmp(str, "300"))
+    if (!std::strcmp(error, "300"))
       rec.SetState(PVR_TIMER_STATE_ABORTED);
-    else if (strstr(str, "missing") != nullptr)
-      rec.SetState(PVR_TIMER_STATE_ERROR);
 
-    rec.SetError(str);
+    rec.SetError(error);
   }
 
   /* A running recording will have an active subscription assigned to it */

From f275837ddf0540955781d68793fdcf4d42fb753d Mon Sep 17 00:00:00 2001
From: Kai Sommerfeld <kai.sommerfeld@gmx.com>
Date: Mon, 31 Jan 2022 18:42:58 +0100
Subject: [PATCH 4/4] Add-on version bump and changelog.

---
 pvr.hts/addon.xml.in  | 2 +-
 pvr.hts/changelog.txt | 7 +++++++
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/pvr.hts/addon.xml.in b/pvr.hts/addon.xml.in
index e6f2d618..2d753723 100644
--- a/pvr.hts/addon.xml.in
+++ b/pvr.hts/addon.xml.in
@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <addon
   id="pvr.hts"
-  version="19.0.4"
+  version="19.0.5"
   name="Tvheadend HTSP Client"
   provider-name="Adam Sutton, Sam Stenvall, Lars Op den Kamp, Kai Sommerfeld">
   <requires>@ADDON_DEPENDS@</requires>
diff --git a/pvr.hts/changelog.txt b/pvr.hts/changelog.txt
index 72e4b8d1..c7ded3c3 100644
--- a/pvr.hts/changelog.txt
+++ b/pvr.hts/changelog.txt
@@ -1,3 +1,10 @@
+v19.0.5
+- Translations updates from Weblate
+  - pl_pl
+- Ignore recordings without a file (e.g. removed recordings).
+- When parsing recording add/update messages, always set the correct error string.
+- Reduce debug log spam. Do not log recording descriptions.
+
 v19.0.4
 - Translations updates from Weblate
 	- es_mx