refactor: simplify error handling in updateParticipants and toModels methods

Signed-off-by: Deluan <deluan@navidrome.org>
2026-04-28 03:19:38 +00:00 · 2026-03-03 08:14:54 -05:00 · 2026-03-03 08:14:54 -05:00 · 24ba655dc3
commit 24ba655dc3
parent ed4c0ef432
1 changed files with 3 additions and 4 deletions
--- a/persistence/album_repository.go
+++ b/persistence/album_repository.go
@ -203,12 +203,11 @@ func (r *albumRepository) Put(al *model.Album) error {
 	}
 	al.ID = id
 	if len(al.Participants) > 0 {
-		err = r.updateParticipants(al.ID, al.Participants)
-		if err != nil {
+		if err = r.updateParticipants(al.ID, al.Participants); err != nil {
 			return err
 		}
 	}
-	return err
+	return nil
 }

 // TODO Move external metadata to a separated table
@ -242,7 +241,7 @@ func (r *albumRepository) GetAll(options ...model.QueryOptions) (model.Albums, e
 	if err != nil {
 		return nil, err
 	}
-	return res.toModels(), err
+	return res.toModels(), nil
 }

 func (r *albumRepository) CopyAttributes(fromID, toID string, columns ...string) error {