From 9cd4943950fe6ddace97ac09671b7883cdb44949 Mon Sep 17 00:00:00 2001
From: Christian Soffke <christian.soffke@gmail.com>
Date: Tue, 17 Oct 2023 19:28:47 +0200
Subject: Properties/Tagcache Track Info Retrieval: Skip files with errors (2)

Addendum to e3b2293

Don't abort even when the database has returned a filename,
since metadata retrieval may still fail.

Change-Id: I9e397c44a4c80f24e937f085efbd540f274822a0
---
 apps/plugins/properties.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/apps/plugins/properties.c b/apps/plugins/properties.c
index ef9dc4c7f7..ce3c03694c 100644
--- a/apps/plugins/properties.c
+++ b/apps/plugins/properties.c
@@ -375,10 +375,8 @@ static bool determine_file_or_dir(void)
 #ifdef HAVE_TAGCACHE
 bool mul_id3_add(const char *file_name)
 {
-    if (!file_name)
+    if (!file_name || rb->mp3info(&id3, file_name))
         skipped_count++;
-    else if (rb->mp3info(&id3, file_name))
-        return false;
     else
     {
         collect_id3(&id3, mul_id3_count == 0);
-- 
cgit v1.2.3