admin: allow editing tmdbid

Author: csmith

cmd/serve/admin/handlers/films.go

@@ -279,6 +279,7 @@ func EditFilmHandler() func(http.ResponseWriter, *http.Request) {
 			ID:        film.ID,
 			Title:     film.Title,
 			Year:      year,
+			TMDBID:    film.TMDBID,
 			Overview:  film.Overview,
 			Runtime:   runtime,
 			Published: film.Published,
@@ -309,6 +310,7 @@ func UpdateFilmHandler() func(http.ResponseWriter, *http.Request) {
 
 		title := r.FormValue("title")
 		year := r.FormValue("year")
+		tmdbIDStr := r.FormValue("tmdb_id")
 		overview := r.FormValue("overview")
 		runtimeStr := r.FormValue("runtime")
 		published := r.FormValue("published") == "true"
@@ -322,13 +324,23 @@ func UpdateFilmHandler() func(http.ResponseWriter, *http.Request) {
 			}
 		}
 
+		var tmdbID *int
+		if tmdbIDStr != "" {
+			value, err := strconv.Atoi(tmdbIDStr)
+			if err != nil {
+				http.Error(w, "Invalid TMDB ID", http.StatusBadRequest)
+				return
+			}
+			tmdbID = &value
+		}
+
 		yearInt := 0
 		if year != "" {
 			yearInt, _ = strconv.Atoi(year)
 		}
 		path := generateFilmPath(title, yearInt)
 
-		if err := db.UpdateFilm(id, 0, title, year, path, overview, runtime, published); err != nil {
+		if err := db.UpdateFilm(id, tmdbID, title, year, path, overview, runtime, published); err != nil {
 			http.Error(w, "Failed to update film", http.StatusInternalServerError)
 			return
 		}

cmd/serve/admin/templates/edit-film.go

@@ -20,6 +20,7 @@ type EditFilmData struct {
 	ID        int
 	Title     string
 	Year      string
+	TMDBID    *int
 	Overview  string
 	Runtime   string
 	Published bool

cmd/serve/admin/templates/edit-film.html.gotpl

@@ -5,6 +5,7 @@
     <form action="/films/edit/{{.ID}}" method="post">
         <label>Title: <input type="text" name="title" value="{{.Title}}" /></label>
         <label>Year: <input type="text" name="year" value="{{.Year}}" /></label>
+        <label>TMDB ID: <input type="text" name="tmdb_id" value="{{if .TMDBID}}{{.TMDBID}}{{end}}" /></label>
         <label>Path: <input type="text" name="path" value="{{.Path}}" /></label>
         <label>Runtime (minutes): <input type="text" name="runtime" value="{{.Runtime}}" /></label>
         <label>

cmd/serve/db/films.go

@@ -105,12 +105,7 @@ func CreateFilm(tmdbID int, title, year, path string, overview string, runtime i
 	return id, nil
 }
 
-func UpdateFilm(id int, tmdbID int, title, year, path string, overview string, runtime int, published bool) error {
-	var tmdbIDPtr *int
-	if tmdbID > 0 {
-		tmdbIDPtr = &tmdbID
-	}
-
+func UpdateFilm(id int, tmdbID *int, title, year, path string, overview string, runtime int, published bool) error {
 	var yearPtr *int
 	if year != "" {
 		y, err := strconv.Atoi(year)
@@ -128,7 +123,7 @@ func UpdateFilm(id int, tmdbID int, title, year, path string, overview string, r
 		UPDATE films
 		SET tmdb_id = $1, title = $2, year = $3, overview = $4, runtime = $5, published = $6, path = $7
 		WHERE id = $8
-	`, tmdbIDPtr, title, yearPtr, overview, runtimePtr, published, path, id)
+	`, tmdbID, title, yearPtr, overview, runtimePtr, published, path, id)
 	if err != nil {
 		return fmt.Errorf("failed to update film: %w", err)
 	}