From 77a4b5ff8097c911174eb5d41ef6f7945504aa86 Mon Sep 17 00:00:00 2001
From: welpo <welpo@users.noreply.github.com>
Date: Thu, 11 Jul 2024 23:16:05 +0200
Subject: [PATCH] validate settings + use 'date' by default + mention default
 in comments

---
 config.toml                      |  4 ++--
 templates/macros/list_posts.html | 10 ++++++++--
 theme.toml                       |  2 +-
 3 files changed, 11 insertions(+), 5 deletions(-)

diff --git a/config.toml b/config.toml
index 87a093219..29d607b1f 100644
--- a/config.toml
+++ b/config.toml
@@ -147,10 +147,10 @@ show_reading_time = true
 show_date = true
 
 # Determines how dates are displayed in the post listing (e.g. front page or /blog). Options:
-# "date" - Show only the original date of the post.
+# "date" - Show only the original date of the post (default if unset).
 # "updated" - Show only the last updated date of the post.
 # "both" - Show both the original date and the last updated date.
-post_listing_date = "both"
+post_listing_date = "date"
 
 # DEPRECATED!
 # Use Zola's built-in `bottom_footnotes = true` in the [markdown] section instead. (Available since v0.19.0)
diff --git a/templates/macros/list_posts.html b/templates/macros/list_posts.html
index dd23114af..d3760de4a 100644
--- a/templates/macros/list_posts.html
+++ b/templates/macros/list_posts.html
@@ -14,8 +14,14 @@
             {% endif %}
 
             <ul class="bloglist-meta">
-                {%- set show_date = post.date and config.extra.post_listing_date == "date" or post.date and config.extra.post_listing_date == "both" -%}
-                {%- set show_updated = post.updated and config.extra.post_listing_date == "updated" or post.updated and config.extra.post_listing_date == "both" -%}
+                {%- set allowed_post_listing_dates = ["date", "updated", "both"] -%}
+                {%- set post_listing_date = config.extra.post_listing_date | default(value="date") -%}
+                {%- if post_listing_date not in allowed_post_listing_dates -%}
+                    {{ throw(message="ERROR: Invalid value for config.extra.post_listing_date. Allowed values are 'date', 'updated', or 'both'.") }}
+                {%- endif -%}
+
+                {%- set show_date = post.date and post_listing_date == "date" or post.date and post_listing_date == "both" -%}
+                {%- set show_updated = post.updated and post_listing_date == "updated" or post.updated and post_listing_date == "both" -%}
 
                 {%- if show_date or show_updated -%}
                     {%- if show_date -%}
diff --git a/theme.toml b/theme.toml
index c18a988d6..957bd9a3d 100644
--- a/theme.toml
+++ b/theme.toml
@@ -104,7 +104,7 @@ show_reading_time = true
 show_date = true
 
 # Determines how dates are displayed in the post listing (e.g. front page or /blog). Options:
-# "date" - Show only the original date of the post.
+# "date" - Show only the original date of the post (default if unset).
 # "updated" - Show only the last updated date of the post.
 # "both" - Show both the original date and the last updated date.
 post_listing_date = "date"