From e263b3cb41c374a93f7f22861e8704a2ae11c67c Mon Sep 17 00:00:00 2001
From: Eryn Wells <eryn@erynwells.me>
Date: Tue, 29 Nov 2022 09:06:18 -0800
Subject: [PATCH] Include all photos as links in the RSS and Atom feeds,
 instead of just the thumbnail

- Move photo_thumbnail.html to photos/thumbnail.html
- Add a photos/list.html partial that returns a list of photos for the current
  page
- Use the template above everywhere we need a list of photos for the page
---
 layouts/partials/photos/list.html                        | 9 +++++++++
 .../photo_thumbnail.html => photos/thumbnail.html}       | 0
 layouts/photos/atom_entry.atom                           | 7 +++----
 layouts/photos/li_thumbnail_in_grid.html                 | 4 ++--
 layouts/photos/rss_item.rss                              | 8 +++++---
 layouts/photos/single.html                               | 4 +---
 layouts/shortcodes/photo.html                            | 2 +-
 7 files changed, 21 insertions(+), 13 deletions(-)
 create mode 100644 layouts/partials/photos/list.html
 rename layouts/partials/{images/photo_thumbnail.html => photos/thumbnail.html} (100%)

diff --git a/layouts/partials/photos/list.html b/layouts/partials/photos/list.html
new file mode 100644
index 0000000..5d9a675
--- /dev/null
+++ b/layouts/partials/photos/list.html
@@ -0,0 +1,9 @@
+{{ $thumbnailResourcesNames := slice "thumbnail" "Thumbnail" "thumbnail.jpg" "Thumbnail.jpg" }}
+{{ with index .Params "thumbnail" }}
+    {{ $thumbnailResourcesNames = $thumbnailResourcesNames | append . }}
+{{ end }}
+
+{{ $imageResources := .Resources.ByType "image" }}
+{{ $photos := where $imageResources "Name" "not in" $thumbnailResourcesNames }}
+
+{{ return $photos }}
diff --git a/layouts/partials/images/photo_thumbnail.html b/layouts/partials/photos/thumbnail.html
similarity index 100%
rename from layouts/partials/images/photo_thumbnail.html
rename to layouts/partials/photos/thumbnail.html
diff --git a/layouts/photos/atom_entry.atom b/layouts/photos/atom_entry.atom
index 7603e5d..9d69922 100644
--- a/layouts/photos/atom_entry.atom
+++ b/layouts/photos/atom_entry.atom
@@ -1,11 +1,10 @@
-{{- $thumbnail := partial "images/photo_thumbnail.html" . -}}
+{{- $thumbnail := partial "photos/thumbnail.html" . -}}
 <entry>
   {{ partial "atom_entry_metadata.xml" . }}
   <link rel="enclosure" href="{{ $thumbnail.Permalink }}" type="{{ $thumbnail.MediaType }}" length="{{ len $thumbnail.Content }}" />
-  {{- $inlineThumbnail := partial "images/photo_thumbnail.html" (dict "Page" . "Width" 1280 "Height" 1280) -}}
   <content type="html">{{ `<![CDATA[` | safeHTML }}
-    {{- with $inlineThumbnail -}}
-    <img src="{{ .Permalink }}">
+    {{- range partial "photos/list.html" . -}}
+    <img src="{{ .Permalink }}"{{ with .Params.alt }} alt="{{ . }}"{{ end }}>
     {{- end -}}
     {{- .Content -}}
   ]]></content>
diff --git a/layouts/photos/li_thumbnail_in_grid.html b/layouts/photos/li_thumbnail_in_grid.html
index c766d5b..e35b156 100644
--- a/layouts/photos/li_thumbnail_in_grid.html
+++ b/layouts/photos/li_thumbnail_in_grid.html
@@ -1,6 +1,6 @@
-{{- $thumbnail := partial "images/photo_thumbnail.html" (dict "Page" . "Width" 600 "Height" 600) -}}
+{{- $thumbnail := partial "photos/thumbnail.html" (dict "Page" . "Width" 600 "Height" 600) -}}
 {{- $thumbnail = $thumbnail.Crop "600x600" -}}
 {{- $altText := $thumbnail.Params.alt -}}
 <a href="{{ .RelPermalink }}" title="{{ .Title }}">
-  <img src="{{ $thumbnail.RelPermalink }}" {{ with $altText }}alt="{{ . }}"{{ end }}>
+  <img src="{{ $thumbnail.RelPermalink }}"{{ with $altText }} alt="{{ . }}"{{ end }}>
 </a>
diff --git a/layouts/photos/rss_item.rss b/layouts/photos/rss_item.rss
index 751c950..9baafb0 100644
--- a/layouts/photos/rss_item.rss
+++ b/layouts/photos/rss_item.rss
@@ -1,8 +1,10 @@
-{{- $thumbnail := partial "images/photo_thumbnail.html" (dict "Page" . "Width" 1280 "Height" 1280) -}}
+{{- $thumbnail := partial "photos/thumbnail.html" (dict "Page" . "Width" 1280 "Height" 1280) -}}
 <item>
   {{ partial "rss_item_metadata.rss" . }}
   <description>{{ `<![CDATA[` | safeHTML }}
-    <img src="{{ $thumbnail.Permalink }}">
-    {{ .Content }}
+    {{- range (partial "photos/list.html" .) -}}
+    <img src="{{ .Permalink }}"{{ with .Params.alt }} alt="{{ . }}"{{ end }}>
+    {{- end -}}
+    {{- .Content -}}
   ]]></description>
 </item>
diff --git a/layouts/photos/single.html b/layouts/photos/single.html
index 8a1aeae..2eab4b2 100644
--- a/layouts/photos/single.html
+++ b/layouts/photos/single.html
@@ -3,13 +3,11 @@
 {{ end }}
 
 {{ define "main" }}
-{{- $photos := where (.Resources.ByType "image")
-               "Name" "not in" (slice "thumbnail" "Thumbnail" "thumbnail.jpg" "Thumbnail.jpg") -}}
-
 {{- if .Title -}}
   {{ partial "content_header.html" . }}
 {{- end -}}
 
+{{- $photos := partial "photos/list.html" . -}}
 {{ if eq (len $photos) 0 }}
   {{ errorf "Missing photo from photos page %q" .Path }}
 {{ end }}
diff --git a/layouts/shortcodes/photo.html b/layouts/shortcodes/photo.html
index cb3bd6f..d5e6fae 100644
--- a/layouts/shortcodes/photo.html
+++ b/layouts/shortcodes/photo.html
@@ -1,5 +1,5 @@
 {{- with $photoPage := $.Page.GetPage (printf "photos/%s" (.Get 0)) -}}
-  {{- $thumbnail := partial "images/photo_thumbnail.html" (dict "Page" . "Width" 1280 "Height" 1280) -}}
+  {{- $thumbnail := partial "photos/thumbnail.html" (dict "Page" . "Width" 1280 "Height" 1280) -}}
   {{- $altText := $thumbnail.Params.alt | default .Title -}}
   <a href="{{ .RelPermalink }}" title="{{ .Title }}">
     <img src="{{ $thumbnail.RelPermalink }}"{{ with $altText }} alt="{{ . }}"{{ end }}>