From be83d68956f02f695d213fdc51745047da86d0c9 Mon Sep 17 00:00:00 2001 From: Deluan Date: Fri, 25 Jul 2025 17:54:51 -0400 Subject: [PATCH] fix(scanner): fix misleading custom tag split config message. See https://github.com/navidrome/navidrome/discussions/3901#discussioncomment-13883185 Signed-off-by: Deluan --- model/tag_mappings.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/model/tag_mappings.go b/model/tag_mappings.go index d54f51f4..0365ed56 100644 --- a/model/tag_mappings.go +++ b/model/tag_mappings.go @@ -138,7 +138,7 @@ func compileSplitRegex(tagName TagName, split []string) *regexp.Regexp { escaped = append(escaped, regexp.QuoteMeta(s)) } // If no valid separators remain, return the original value. - if len(escaped) == 0 { + if len(split) > 0 && len(escaped) == 0 { log.Warn("No valid separators found in split list", "split", split, "tag", tagName) return nil } @@ -147,7 +147,7 @@ func compileSplitRegex(tagName TagName, split []string) *regexp.Regexp { pattern := "(?i)(" + strings.Join(escaped, "|") + ")" re, err := regexp.Compile(pattern) if err != nil { - log.Error("Error compiling regexp", "pattern", pattern, "tag", tagName, "err", err) + log.Warn("Error compiling regexp for split list", "pattern", pattern, "tag", tagName, "split", split, err) return nil } return re