lyrics

commit e247dcb1d107584440cbedb1a8d5fe041d73abb5
parent dedabec83aec63d8171ad04a667d530dd8381fd8
Author: brookjeynes <me@brookjeynes.dev>
Date:   Sun,  7 Jun 2026 17:45:17 +1000

fix: handle optional artist fields correctly

Signed-off-by: brookjeynes <me@brookjeynes.dev>

Diffstat:
M
internal/server/new-artist.go
 | 
12
++++++++++--

1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/internal/server/new-artist.go b/internal/server/new-artist.go
@@ -39,20 +39,28 @@ func parseNewArtistForm(form NewArtistForm) (domain.Artist, error) {
 	}
 
 	// bio
+	var bio *string
 	if form.Bio != "" && !domain.NoTrailingOrLeadingSpaceRegex.MatchString(form.Bio) {
 		return artist, fmt.Errorf("Artist biography must not be empty or have leading/trailing whitespace")
 	}
+	if form.Bio != "" {
+		bio = &form.Bio
+	}
 
 	// disambiguation
+	var disambiguation *string
 	if form.Disambiguation != "" && !domain.NoTrailingOrLeadingSpaceRegex.MatchString(form.Disambiguation) {
 		return artist, fmt.Errorf("Artist disambiguation must not be empty or have leading/trailing whitespace")
 	}
+	if form.Disambiguation != "" {
+		disambiguation = &form.Disambiguation
+	}
 
 	artist = domain.Artist{
 		Name:           form.Name,
 		LatinName:      latinName,
-		Bio:            &form.Bio,
-		Disambiguation: &form.Disambiguation,
+		Bio:            bio,
+		Disambiguation: disambiguation,
 	}
 
 	return artist, nil