Merge pull request #11607 from quarto-dev/bugfix/issue-11580

Allow non-strings as `categories` fields

Author: cscheid

news/changelog-1.7.md

@@ -2,5 +2,6 @@ All changes included in 1.7:
 
 ## Regression fixes
 
-- ([#11532](https://github.com/quarto-dev/quarto-cli/issues/11532)): Fix regression for [#660](https://github.com/quarto-dev/quarto-cli/issues/660), which causes files to have incorrect permissions when Quarto is installed in a location not writable by the current user.
 - ([#11509](https://github.com/quarto-dev/quarto-cli/issues/11509)): Fix link-decoration regression in HTML formats.
+- ([#11532](https://github.com/quarto-dev/quarto-cli/issues/11532)): Fix regression for [#660](https://github.com/quarto-dev/quarto-cli/issues/660), which causes files to have incorrect permissions when Quarto is installed in a location not writable by the current user.
+- ([#11580](https://github.com/quarto-dev/quarto-cli/issues/11580)): Fix regression with documents containing `categories` fields that are not strings.

src/project/types/website/website.ts

@@ -89,6 +89,7 @@ import { projectDraftMode } from "./website-utils.ts";
 import { kFieldCategories } from "./listing/website-listing-shared.ts";
 import { pandocNativeStr } from "../../../core/pandoc/codegen.ts";
 import { asArray } from "../../../core/array.ts";
+import { InternalError } from "../../../core/lib/error.ts";
 
 export const kSiteTemplateDefault = "default";
 export const kSiteTemplateBlog = "blog";
@@ -207,8 +208,12 @@ export const websiteProjectType: ProjectType = {
         extras.metadataOverride[kFieldCategories] = asArray(
           format.metadata[kFieldCategories],
         ).map(
-          (category) =>
-            pandocNativeStr(category as string).mappedString().value,
+          (category) => {
+            const strCategory: string = typeof category === "string"
+              ? category
+              : category.toString();
+            return pandocNativeStr(strCategory).mappedString().value;
+          },
         );
       }
 

tests/docs/smoke-all/2024/12/03/issue-11580/.gitignore

@@ -0,0 +1 @@
+/.quarto/

tests/docs/smoke-all/2024/12/03/issue-11580/_quarto.yml

@@ -0,0 +1,21 @@
+project:
+  type: website
+
+website:
+  title: "issue-11580"
+  navbar:
+    right:
+      - about.qmd
+      - icon: github
+        href: https://github.com/
+      - icon: twitter
+        href: https://twitter.com
+format:
+  html:
+    theme:
+      - cosmo
+      - brand
+    css: styles.css
+
+
+

tests/docs/smoke-all/2024/12/03/issue-11580/about.qmd

@@ -0,0 +1,19 @@
+---
+title: "About"
+image: profile.jpg
+about:
+  template: jolla
+  links:
+    - icon: twitter
+      text: Twitter
+      href: https://twitter.com
+    - icon: linkedin
+      text: LinkedIn
+      href: https://linkedin.com
+    - icon: github
+      text: Github
+      href: https://github.com
+
+---
+
+About this blog

tests/docs/smoke-all/2024/12/03/issue-11580/index.qmd

@@ -0,0 +1,16 @@
+---
+title: "issue-11580"
+listing:
+  contents: posts
+  sort: "date desc"
+  type: default
+  categories: true
+  sort-ui: false
+  filter-ui: false
+page-layout: full
+title-block-banner: true
+_quarto:
+  render-project: true
+---
+
+

tests/docs/smoke-all/2024/12/03/issue-11580/posts/_metadata.yml

@@ -0,0 +1,8 @@
+# options specified here will apply to all posts in this folder
+
+# freeze computational output
+# (see https://quarto.org/docs/projects/code-execution.html#freeze)
+freeze: true
+
+# Enable banner style title blocks
+title-block-banner: true

tests/docs/smoke-all/2024/12/03/issue-11580/posts/post-with-code/image.jpg

@@ -0,0 +1,13 @@
+---
+title: "Post With Code"
+author: "Harlow Malloc"
+date: "2024-12-03"
+categories: 
+  - news
+  - code
+  - 3
+  - analysis
+image: "image.jpg"
+---
+
+This is a post with executable code.

tests/docs/smoke-all/2024/12/03/issue-11580/posts/post-with-code/index.qmd

@@ -0,0 +1,12 @@
+---
+title: "Welcome To My Blog"
+author: "Tristan O'Malley"
+date: "2024-11-30"
+categories: [news]
+---
+
+This is the first post in a Quarto blog. Welcome!
+
+![](thumbnail.jpg)
+
+Since this post doesn't specify an explicit `image`, the first image in the post will be used in the listing page of posts.