Show containing lists on film pages

Author: csmith

cmd/serve/assets/stylesheet/films.css

@@ -200,6 +200,10 @@ a.film-list {
     background: var(--accent-colour);
   }
 
+  & + & {
+    margin-top: var(--medium-space);
+  }
+
   *, *:hover, *:active, *:focus {
     margin: 0;
     padding: 0;
@@ -339,9 +343,9 @@ ol.recent-films {
   display: grid;
 
   @media(min-width: 1200px) {
-    grid-template-areas: "header sidebar" "review sidebar";
+    grid-template-areas: "header sidebar" "review sidebar" "review -";
     grid-template-columns: calc(var(--content-width) - 300px) 300px;
-    grid-template-rows: auto 1fr;
+    grid-template-rows: auto auto 1fr;
     column-gap: var(--medium-space);
 
     aside {

cmd/serve/content/shortcodes/filmlist/render.go

@@ -27,6 +27,10 @@ func RenderFromText(args []string, _ *context.Context) (string, error) {
 		return "", fmt.Errorf("invalid film list ID: %s", args[0])
 	}
 
+	return Render(id)
+}
+
+func Render(id int) (string, error) {
 	list, count, entries, err := db.GetFilmListWithFilms(id)
 	if err != nil {
 		return "", fmt.Errorf("failed to get film list: %w", err)
@@ -58,7 +62,7 @@ func RenderFromText(args []string, _ *context.Context) (string, error) {
 		return "", fmt.Errorf("failed to render film list description: %w", err)
 	}
 
-	return Render(Data{
+	return renderTemplate(Data{
 		ID:          list.ID,
 		Title:       list.Title,
 		Description: description,
@@ -68,7 +72,7 @@ func RenderFromText(args []string, _ *context.Context) (string, error) {
 	})
 }
 
-func Render(data Data) (string, error) {
+func renderTemplate(data Data) (string, error) {
 	buf := &bytes.Buffer{}
 	err := tmpl.Execute(buf, data)
 	if err != nil {

cmd/serve/db/film_lists.go

@@ -328,3 +328,18 @@ func GetNextPosition(listID int) (int, error) {
 	}
 	return position, nil
 }
+
+func GetFilmListsContainingFilm(filmID int) ([]FilmList, error) {
+	var lists []FilmList
+	err := db.Select(&lists, `
+		SELECT DISTINCT fl.id, fl.title, fl.description, fl.published, fl.path
+		FROM film_lists fl
+		JOIN film_list_entries fle ON fl.id = fle.film_list_id
+		WHERE fl.published = true AND fle.film_id = $1
+		ORDER BY fl.title
+	`, filmID)
+	if err != nil {
+		return nil, err
+	}
+	return lists, nil
+}

cmd/serve/handlers/films.go

@@ -10,6 +10,7 @@ import (
 	"chameth.com/chameth.com/cmd/serve/assets"
 	"chameth.com/chameth.com/cmd/serve/content"
 	"chameth.com/chameth.com/cmd/serve/content/markdown"
+	"chameth.com/chameth.com/cmd/serve/content/shortcodes/filmlist"
 	"chameth.com/chameth.com/cmd/serve/content/shortcodes/rating"
 	"chameth.com/chameth.com/cmd/serve/db"
 	"chameth.com/chameth.com/cmd/serve/templates"
@@ -102,6 +103,21 @@ func Film(w http.ResponseWriter, r *http.Request) {
 		posterPath = *film.PosterPath
 	}
 
+	lists, err := db.GetFilmListsContainingFilm(film.ID)
+	if err != nil {
+		slog.Error("Failed to get film lists containing film", "film_id", film.ID, "error", err)
+	}
+
+	var filmLists []template.HTML
+	for _, list := range lists {
+		listHTML, err := filmlist.Render(list.ID)
+		if err != nil {
+			slog.Error("Failed to render film list", "list_id", list.ID, "error", err)
+		} else {
+			filmLists = append(filmLists, template.HTML(listHTML))
+		}
+	}
+
 	w.Header().Set("Content-Type", "text/html; charset=utf-8")
 	w.WriteHeader(http.StatusOK)
 	err = templates.RenderFilm(w, templates.FilmData{
@@ -113,6 +129,7 @@ func Film(w http.ResponseWriter, r *http.Request) {
 		TimesWatched:  timesWatched,
 		AverageRating: template.HTML(ratingHTML),
 		PosterPath:    posterPath,
+		FilmLists:     filmLists,
 		PageData: templates.PageData{
 			Title:        fmt.Sprintf("%s (%s) · Chameth.com", film.Title, year),
 			Stylesheet:   assets.GetStylesheetPath(),

cmd/serve/templates/film.go

@@ -26,6 +26,7 @@ type FilmData struct {
 	TimesWatched  int
 	AverageRating template.HTML
 	PosterPath    string
+	FilmLists     []template.HTML
 }
 
 type FilmReviewData struct {

cmd/serve/templates/film.html.gotpl

@@ -20,6 +20,14 @@
                 </blockquote>
             </section>
             {{end}}
+            {{if .FilmLists}}
+            <section class="film-lists">
+                <h3>Appears in these lists</h3>
+                {{range .FilmLists}}
+                    {{.}}
+                {{end}}
+            </section>
+            {{end}}
         </div>
 
         <aside class="film-sidebar">