i think that's all the api endpoints!

Signed-off-by: ari melody <ari@arimelody.me>

api/artist.go

@@ -12,25 +12,8 @@ import (
 
 func ServeAllArtists() http.Handler {
     return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		type (
-			creditJSON struct {
-				Role    string `json:"role"`
-				Primary bool   `json:"primary"`
-			}
-		)
-
-        var artists = []model.Artist{}
-        for _, artist := range global.Artists {
-            artists = append(artists, model.Artist{
-                ID: artist.ID,
-                Name: artist.Name,
-                Website: artist.Website,
-                Avatar: artist.Avatar,
-            })
-        }
-
 		w.Header().Add("Content-Type", "application/json")
-        err := json.NewEncoder(w).Encode(artists)
+        err := json.NewEncoder(w).Encode(global.Artists)
 		if err != nil {
 			http.Error(w, http.StatusText(http.StatusInternalServerError), http.StatusInternalServerError)
 			return
@@ -55,24 +38,24 @@ func ServeArtist() http.Handler {
 				Credits map[string]creditJSON `json:"credits"`
 			}
 		)
-		var res = artistJSON{}
+		var artist = artistJSON{}
 
-		res.ID = r.URL.Path[1:]
-		var artist = global.GetArtist(res.ID)
-		if artist == nil {
+		artist.ID = r.URL.Path[1:]
+		var a = global.GetArtist(artist.ID)
+		if a == nil {
 			http.NotFound(w, r)
 			return
 		}
-		res.Name = artist.Name
-		res.Website = artist.Website
-		res.Credits = make(map[string]creditJSON)
+		artist.Name = a.Name
+		artist.Website = a.Website
+		artist.Credits = make(map[string]creditJSON)
 
 		for _, release := range global.Releases {
 			for _, credit := range release.Credits {
-				if credit.Artist.ID != res.ID {
+				if credit.Artist.ID != artist.ID {
 					continue
 				}
-				res.Credits[release.ID] = creditJSON{
+				artist.Credits[release.ID] = creditJSON{
 					Role:    credit.Role,
 					Primary: credit.Primary,
 				}
@@ -80,7 +63,7 @@ func ServeArtist() http.Handler {
 		}
 
 		w.Header().Add("Content-Type", "application/json")
-        err := json.NewEncoder(w).Encode(res)
+        err := json.NewEncoder(w).Encode(artist)
 		if err != nil {
 			http.Error(w, http.StatusText(http.StatusInternalServerError), http.StatusInternalServerError)
 			return
@@ -136,5 +119,104 @@ func CreateArtist() http.Handler {
         w.Header().Add("Content-Type", "application/json")
         w.WriteHeader(http.StatusCreated)
         err = json.NewEncoder(w).Encode(artist)
+        if err != nil {
+            http.Error(w, http.StatusText(http.StatusInternalServerError), http.StatusInternalServerError)
+            return
+        }
+    })
+}
+
+func UpdateArtist() http.Handler {
+    return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+        if r.Method != http.MethodPut {
+            http.NotFound(w, r)
+            return
+        }
+
+        if r.URL.Path == "/" {
+            http.Error(w, http.StatusText(http.StatusBadRequest), http.StatusBadRequest)
+            return
+        }
+
+        var data model.Artist
+        err := json.NewDecoder(r.Body).Decode(&data)
+        if err != nil {
+            fmt.Printf("Failed to update artist: %s\n", err)
+            http.Error(w, http.StatusText(http.StatusBadRequest), http.StatusBadRequest)
+            return
+        }
+
+        var artistID = r.URL.Path[1:]
+        var artist = global.GetArtist(artistID)
+        if artist == nil {
+            http.Error(w, fmt.Sprintf("Artist %s does not exist\n", artistID), http.StatusBadRequest)
+            return
+        }
+
+        if data.ID == "" { data.ID = artist.ID }
+
+        if data.Name == "" {
+            http.Error(w, "Artist name cannot be blank\n", http.StatusBadRequest)
+            return
+        }
+
+        err = controller.UpdateArtistDB(global.DB, &data)
+        if err != nil {
+            fmt.Printf("Failed to update artist %s: %s\n", artist.ID, err)
+            http.Error(w, http.StatusText(http.StatusInternalServerError), http.StatusInternalServerError)
+            return
+        }
+
+        artist.ID = data.ID
+        artist.Name = data.Name
+        artist.Website = data.Website
+        artist.Avatar = data.Avatar
+
+        w.Header().Add("Content-Type", "application/json")
+        err = json.NewEncoder(w).Encode(artist)
+        if err != nil {
+            http.Error(w, http.StatusText(http.StatusInternalServerError), http.StatusInternalServerError)
+            return
+        }
+    })
+}
+
+func DeleteArtist() http.Handler {
+    return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+        if r.Method != http.MethodDelete {
+            http.NotFound(w, r)
+            return
+        }
+
+        if r.URL.Path == "/" {
+            http.Error(w, http.StatusText(http.StatusBadRequest), http.StatusBadRequest)
+            return
+        }
+
+        var artistID = r.URL.Path[1:]
+        var artist = global.GetArtist(artistID)
+        if artist == nil {
+            http.Error(w, fmt.Sprintf("Artist %s does not exist\n", artistID), http.StatusBadRequest)
+            return
+        }
+
+        err := controller.DeleteArtistDB(global.DB, artist)
+        if err != nil {
+            fmt.Printf("Failed to delete artist %s: %s\n", artist.ID, err)
+            http.Error(w, http.StatusText(http.StatusInternalServerError), http.StatusInternalServerError)
+            return
+        }
+
+        global.Artists = func () []*model.Artist {
+            var artists = []*model.Artist{}
+            for _, a := range global.Artists {
+                if a.ID == artist.ID { continue }
+                artists = append(artists, a)
+            }
+            return artists            
+        }()
+
+        w.WriteHeader(http.StatusOK)
+        w.Write([]byte(fmt.Sprintf("Artist %s has been deleted\n", artist.ID)))
     })
 }