From 6c94f194df4605efb24599ccf99c8d487ef5d7f4 Mon Sep 17 00:00:00 2001
From: sentriz <senan@senan.xyz>
Date: Fri, 22 Jul 2022 14:13:01 +0100
Subject: [PATCH] rewrite `""` search queries as ``

---
 server/ctrlsubsonic/handlers_by_folder.go | 7 ++-----
 server/ctrlsubsonic/handlers_by_tags.go   | 5 +----
 2 files changed, 3 insertions(+), 9 deletions(-)

diff --git a/server/ctrlsubsonic/handlers_by_folder.go b/server/ctrlsubsonic/handlers_by_folder.go
index 3bfa3ace..c8e4040d 100644
--- a/server/ctrlsubsonic/handlers_by_folder.go
+++ b/server/ctrlsubsonic/handlers_by_folder.go
@@ -7,9 +7,9 @@ import (
 
 	"github.com/jinzhu/gorm"
 
+	"go.senan.xyz/gonic/db"
 	"go.senan.xyz/gonic/server/ctrlsubsonic/params"
 	"go.senan.xyz/gonic/server/ctrlsubsonic/spec"
-	"go.senan.xyz/gonic/db"
 )
 
 // the subsonic spec mentions "artist" a lot when talking about the
@@ -177,10 +177,7 @@ func (c *Controller) ServeSearchTwo(r *http.Request) *spec.Response {
 	if err != nil {
 		return spec.NewError(10, "please provide a `query` parameter")
 	}
-	if query == `""` {
-		query = ""
-	}
-	query = fmt.Sprintf("%%%s%%", strings.TrimSuffix(query, "*"))
+	query = fmt.Sprintf("%%%s%%", strings.Trim(query, `*"'`))
 
 	results := &spec.SearchResultTwo{}
 
diff --git a/server/ctrlsubsonic/handlers_by_tags.go b/server/ctrlsubsonic/handlers_by_tags.go
index e1025b41..7be3e4b6 100644
--- a/server/ctrlsubsonic/handlers_by_tags.go
+++ b/server/ctrlsubsonic/handlers_by_tags.go
@@ -182,10 +182,7 @@ func (c *Controller) ServeSearchThree(r *http.Request) *spec.Response {
 	if err != nil {
 		return spec.NewError(10, "please provide a `query` parameter")
 	}
-	if query == `""` {
-		query = ""
-	}
-	query = fmt.Sprintf("%%%s%%", strings.TrimSuffix(query, "*"))
+	query = fmt.Sprintf("%%%s%%", strings.Trim(query, `*"'`))
 	results := &spec.SearchResultThree{}
 
 	// search "artists"