Merge pull request rust-lang#1543 from rust-lang/senekor/wqmvzwnnxtuk

Manishearth · web-flow · commit c8bcfd9d45f6 · 2025-03-29T09:17:12.000-07:00
Rename template variables
diff --git a/src/blogs.rs b/src/blogs.rs
@@ -37,12 +37,12 @@ pub struct Blog {
     maintained_by: String,
     index_html: String,
     #[serde(serialize_with = "add_postfix_slash")]
-    prefix: PathBuf,
-    posts: Vec<Post>,
+    path: PathBuf,
+    pages: Vec<Post>,
 }
 
 impl Blog {
-    fn load(prefix: PathBuf, dir: &Path) -> eyre::Result<Self> {
+    fn load(path: PathBuf, dir: &Path) -> eyre::Result<Self> {
         let manifest_content = std::fs::read_to_string(dir.join(MANIFEST_FILE))?
             .strip_prefix("+++\n")
             .unwrap()
@@ -94,8 +94,8 @@ impl Blog {
             maintained_by: manifest.maintained_by,
             index_html: manifest.index_html,
             link_text: manifest.link_text,
-            prefix,
-            posts,
+            path,
+            pages: posts,
         })
     }
 
@@ -111,12 +111,12 @@ impl Blog {
         &self.index_title
     }
 
-    pub(crate) fn prefix(&self) -> &Path {
-        &self.prefix
+    pub(crate) fn path(&self) -> &Path {
+        &self.path
     }
 
     pub(crate) fn posts(&self) -> &[Post] {
-        &self.posts
+        &self.pages
     }
 }
 
@@ -139,10 +139,10 @@ fn load_recursive(base: &Path, current: &Path, blogs: &mut Vec<Blog>) -> eyre::R
             let file_name = path.file_name().and_then(|n| n.to_str());
             if let (Some(file_name), Some(parent)) = (file_name, path.parent()) {
                 if file_name == MANIFEST_FILE {
-                    let prefix = parent
+                    let path = parent
                         .strip_prefix(base)
                         .map_or_else(|_| PathBuf::new(), Path::to_path_buf);
-                    blogs.push(Blog::load(prefix, parent)?);
+                    blogs.push(Blog::load(path, parent)?);
                 }
             }
         }
diff --git a/src/lib.rs b/src/lib.rs
@@ -106,7 +106,7 @@ impl Generator {
     }
 
     fn render_blog(&self, blog: &Blog) -> eyre::Result<()> {
-        std::fs::create_dir_all(self.out_directory.join(blog.prefix()))?;
+        std::fs::create_dir_all(self.out_directory.join(blog.path()))?;
 
         let path = self.render_index(blog)?;
 
@@ -135,24 +135,24 @@ impl Generator {
             .map(|other_blog| {
                 json!({
                     "link_text": other_blog.link_text(),
-                    "url": other_blog.prefix().join("index.html"),
+                    "url": other_blog.path().join("index.html"),
                 })
             })
             .collect();
 
         let data = json!({
             "title": blog.index_title(),
-            "blog": blog,
+            "section": blog,
             "other_blogs": other_blogs,
         });
-        let path = blog.prefix().join("index.html");
+        let path = blog.path().join("index.html");
         self.render_template(&path, "index.html", data)?;
         Ok(path)
     }
 
     fn render_post(&self, blog: &Blog, post: &Post) -> eyre::Result<PathBuf> {
         let path = blog
-            .prefix()
+            .path()
             .join(format!("{:04}", &post.year))
             .join(format!("{:02}", &post.month))
             .join(format!("{:02}", &post.day));
@@ -164,8 +164,8 @@ impl Generator {
 
         let data = json!({
             "title": format!("{} | {}", post.title, blog.title()),
-            "blog": blog,
-            "post": post,
+            "section": blog,
+            "page": post,
         });
 
         let path = path.join(filename);
@@ -176,12 +176,12 @@ impl Generator {
     fn render_feed(&self, blog: &Blog) -> eyre::Result<()> {
         let posts: Vec<_> = blog.posts().iter().take(10).collect();
         let data = json!({
-            "blog": blog,
-            "posts": posts,
+            "section": blog,
+            "pages": posts,
             "feed_updated": chrono::Utc::now().with_nanosecond(0).unwrap().to_rfc3339(),
         });
 
-        self.render_template(blog.prefix().join("feed.xml"), "feed.xml", data)?;
+        self.render_template(blog.path().join("feed.xml"), "feed.xml", data)?;
         Ok(())
     }
 
@@ -193,8 +193,8 @@ impl Generator {
             .map(|post| ReleasePost {
                 title: post.title.clone(),
                 url: blog
-                    .prefix()
-                    .join(post.url.clone())
+                    .path()
+                    .join(post.path.clone())
                     .to_string_lossy()
                     .to_string(),
             })
@@ -204,7 +204,7 @@ impl Generator {
             feed_updated: chrono::Utc::now().with_nanosecond(0).unwrap().to_rfc3339(),
         };
         fs::write(
-            self.out_directory.join(blog.prefix()).join("releases.json"),
+            self.out_directory.join(blog.path()).join("releases.json"),
             serde_json::to_string(&data)?,
         )?;
         Ok(())
diff --git a/src/posts.rs b/src/posts.rs
@@ -19,7 +19,7 @@ pub struct Post {
     pub(crate) month: u8,
     pub(crate) day: u8,
     pub(crate) contents: String,
-    pub(crate) url: String,
+    pub(crate) path: String,
     pub(crate) published: String,
     pub(crate) updated: String,
     pub(crate) release: bool,
@@ -122,7 +122,7 @@ impl Post {
             month,
             day,
             contents,
-            url,
+            path: url,
             published,
             updated,
             release,
diff --git a/templates/feed.xml b/templates/feed.xml
@@ -1,29 +1,29 @@
 {% import "macros.html" as macros %}
 <?xml version="1.0" encoding="utf-8"?>
 <feed xmlns="http://www.w3.org/2005/Atom" xml:lang="en">
-    <generator uri="https://blog.rust-lang.org/{{blog.prefix}}" version="0.1.0">{{blog.title}}</generator>
-    <link href="https://blog.rust-lang.org/{{blog.prefix}}feed.xml" rel="self" type="application/atom+xml" />
-    <link href="https://blog.rust-lang.org/{{blog.prefix}}" rel="alternate" type="text/html" />
-    <id>https://blog.rust-lang.org/{{blog.prefix}}</id>
-    <title>{{blog.title}}</title>
-    <subtitle>{{blog.description}}</subtitle>
+    <generator uri="https://blog.rust-lang.org/{{ section.path }}" version="0.1.0">{{ section.title }}</generator>
+    <link href="https://blog.rust-lang.org/{{ section.path }}feed.xml" rel="self" type="application/atom+xml" />
+    <link href="https://blog.rust-lang.org/{{ section.path }}" rel="alternate" type="text/html" />
+    <id>https://blog.rust-lang.org/{{ section.path }}</id>
+    <title>{{ section.title }}</title>
+    <subtitle>{{ section.description }}</subtitle>
     <author>
-        <name>Maintained by {{blog.maintained_by}}.</name>
+        <name>Maintained by {{ section.maintained_by }}.</name>
         <uri>https://github.com/rust-lang/blog.rust-lang.org/</uri>
     </author>
     <updated>{{feed_updated}}</updated>
 
-    {% for post in posts %}
+    {% for page in pages %}
     <entry>
-        <title>{{ macros::escape_hbs(input=post.title) }}</title>
-        <link rel="alternate" href="https://blog.rust-lang.org/{{blog.prefix}}{{ macros::escape_hbs(input=post.url) }}" type="text/html" title="{{ macros::escape_hbs(input=post.title) }}" />
-        <published>{{ macros::escape_hbs(input=post.published) }}</published>
-        <updated>{{ macros::escape_hbs(input=post.updated) }}</updated>
-        <id>https://blog.rust-lang.org/{{blog.prefix}}{{ macros::escape_hbs(input=post.url) }}</id>
-        <content type="html" xml:base="https://blog.rust-lang.org/{{blog.prefix}}{{ macros::escape_hbs(input=post.url) }}">{{ macros::escape_hbs(input=post.contents) }}</content>
+        <title>{{ macros::escape_hbs(input=page.title) }}</title>
+        <link rel="alternate" href="https://blog.rust-lang.org/{{ section.path }}{{ macros::escape_hbs(input=page.path) }}" type="text/html" title="{{ macros::escape_hbs(input=page.title) }}" />
+        <published>{{ macros::escape_hbs(input=page.published) }}</published>
+        <updated>{{ macros::escape_hbs(input=page.updated) }}</updated>
+        <id>https://blog.rust-lang.org/{{ section.path }}{{ macros::escape_hbs(input=page.path) }}</id>
+        <content type="html" xml:base="https://blog.rust-lang.org/{{ section.path }}{{ macros::escape_hbs(input=page.path) }}">{{ macros::escape_hbs(input=page.contents) }}</content>
 
         <author>
-            <name>{{ macros::escape_hbs(input=post.author) }}</name>
+            <name>{{ macros::escape_hbs(input=page.author) }}</name>
         </author>
     </entry>
     {%- endfor %}
diff --git a/templates/headers.html b/templates/headers.html
@@ -1,15 +1,15 @@
-{% macro headers(title, blog) -%}
+{% macro headers(title, section) -%}
  <!-- Twitter card -->
  <meta name="twitter:card" content="summary">
  <meta name="twitter:site" content="@rustlang">
  <meta name="twitter:creator" content="@rustlang">
  <meta name="twitter:title" content="{{ macros::escape_hbs(input=title) }}">
- <meta name="twitter:description" content="{{blog.description}}">
+ <meta name="twitter:description" content="{{ section.description }}">
 <meta name="twitter:image" content="https://www.rust-lang.org/static/images/rust-social.jpg">
 
 <!-- Facebook OpenGraph -->
 <meta property="og:title" content="{{ macros::escape_hbs(input=title) }}" />
-<meta property="og:description" content="{{blog.description}}">
+<meta property="og:description" content="{{ section.description }}">
 <meta property="og:image" content="https://www.rust-lang.org/static/images/rust-social-wide.jpg" />
 <meta property="og:type" content="website" />
 <meta property="og:locale" content="en_US" />
@@ -36,7 +36,7 @@
 <meta name="theme-color" content="#ffffff">
 
  <!-- atom -->
- <link type="application/atom+xml" rel="alternate" href="https://blog.rust-lang.org/{{blog.prefix}}feed.xml" title="{{blog.title}}" />
+ <link type="application/atom+xml" rel="alternate" href="https://blog.rust-lang.org/{{ section.path }}feed.xml" title="{{ section.title }}" />
 
 <!-- theme switcher -->
 <script src="/scripts/theme-switch.js"></script>
diff --git a/templates/index.html b/templates/index.html
@@ -4,7 +4,7 @@
 <header class="mt3 mt0-ns mb4-ns">
   <div class="container flex flex-column flex-row-l justify-between-l">
     <div class="mw6-l">
-      <p>{{blog.index_html}}</p>
+      <p>{{ section.index_html }}</p>
     </div>
   </div>    
   <div class="container flex flex-column flex-row-l justify-between-l">
@@ -23,14 +23,14 @@
    <div class="w-100 mw-none ph3 mw8-m mw9-l center f3">
 
     <table class="post-list collapse w-100 f2-l f2-m f3-s">
-    {%- for post in blog.posts %}
-      {% if post.show_year %}<tr>
+    {%- for page in section.pages %}
+      {% if page.show_year %}<tr>
         <td class="bn"></td>
-        <td class="bn"><h3 class="f0-l f1-m f2-s mt4 mb0">Posts in {{post.year}}</h3></td>
+        <td class="bn"><h3 class="f0-l f1-m f2-s mt4 mb0">Posts in {{ page.year }}</h3></td>
       </tr>{% endif %}
       <tr>
-        <td class="tr o-60 pr4 pr5-l bn">{{ macros::month_name(num=post.month) }}&nbsp;{{post.day}}</td>
-        <td class="bn"><a href="{{post.url}}">{{ macros::escape_hbs(input=post.title) }}</a></td>
+        <td class="tr o-60 pr4 pr5-l bn">{{ macros::month_name(num=page.month) }}&nbsp;{{ page.day }}</td>
+        <td class="bn"><a href="{{ page.path }}">{{ macros::escape_hbs(input=page.title) }}</a></td>
       </tr>
     {%- endfor %}
     </table>
diff --git a/templates/layout.html b/templates/layout.html
@@ -9,10 +9,10 @@
     <title>{{ macros::escape_hbs(input=title) }}</title>
     <meta name="viewport" content="width=device-width,initial-scale=1.0">
 <meta name="description" content="Empowering everyone to build reliable and efficient software.">
-     {{ headers::headers(title=title, blog=blog) | indent(prefix="    ", blank=true) }}
+     {{ headers::headers(title=title, section=section) | indent(prefix="    ", blank=true) }}
   </head>
   <body>
-    {{ nav::nav(blog=blog) | indent(prefix="    ", blank=true) }}
+    {{ nav::nav(section=section) | indent(prefix="    ", blank=true) }}
     {%- block page %}{% endblock page %}
     {{ footer::footer() | indent(prefix="    ", blank=true) }}
   </body>
diff --git a/templates/nav.html b/templates/nav.html
@@ -1,9 +1,9 @@
-{% macro nav(blog) -%}
+{% macro nav(section) -%}
 <nav class="flex flex-row justify-center justify-end-l items-center flex-wrap ph2 pl3-ns pr4-ns">
   <div class="brand flex-auto w-100 w-auto-l self-start tc tl-l">
-    <a href="/{{blog.prefix}}">
+    <a href="/{{ section.path }}">
       <img class="v-mid ml0-l rust-logo" alt="Rust Logo" src="/images/rust-logo-blk.svg">
-      <span class="dib ml1 ml0-l">{{blog.title}}</span>
+      <span class="dib ml1 ml0-l">{{ section.title }}</span>
     </a>
   </div>
 
diff --git a/templates/post.html b/templates/post.html
@@ -1,19 +1,19 @@
 {% import "macros.html" as macros %}
 {% extends "layout.html" %}
 {% block page %}
-<section id="{{ macros::escape_hbs(input=post.title) }}" class="white">
+<section id="{{ macros::escape_hbs(input=page.title) }}" class="white">
   <div class="w-100 mw-none ph3 mw8-m mw8-l center f3">
     <header>
-      <h2>{{ macros::escape_hbs(input=post.title) }}</h2>
+      <h2>{{ macros::escape_hbs(input=page.title) }}</h2>
       <div class="highlight mt2 mb3"></div>
     </header>
 
-    <div class="publish-date-author">{{ macros::month_name(num=post.month) }} {{post.day}}, {{post.year}} &middot; {{ macros::escape_hbs(input=post.author) }}
-    {% if post.has_team %} on behalf of <a href="{{post.team_url}}">{{post.team}}</a> {% endif %}
+    <div class="publish-date-author">{{ macros::month_name(num=page.month) }} {{ page.day }}, {{ page.year }} &middot; {{ macros::escape_hbs(input=page.author) }}
+    {% if page.has_team %} on behalf of <a href="{{ page.team_url }}">{{ page.team }}</a> {% endif %}
     </div>
 
     <div class="post">
-      {{ post.contents }}
+      {{ page.contents }}
     </div>
   </div>
 </section>

Original file line number	Diff line number	Diff line change
`@@ -37,12 +37,12 @@ pub struct Blog {`
`37`	`37`	`maintained_by: String,`
`38`	`38`	`index_html: String,`
`39`	`39`	`#[serde(serialize_with = "add_postfix_slash")]`
`40`		`- prefix: PathBuf,`
`41`		`- posts: Vec<Post>,`
	`40`	`+ path: PathBuf,`
	`41`	`+ pages: Vec<Post>,`
`42`	`42`	`}`
`43`	`43`
`44`	`44`	`impl Blog {`
`45`		`- fn load(prefix: PathBuf, dir: &Path) -> eyre::Result<Self> {`
	`45`	`+ fn load(path: PathBuf, dir: &Path) -> eyre::Result<Self> {`
`46`	`46`	`let manifest_content = std::fs::read_to_string(dir.join(MANIFEST_FILE))?`
`47`	`47`	`.strip_prefix("+++\n")`
`48`	`48`	`.unwrap()`
`@@ -94,8 +94,8 @@ impl Blog {`
`94`	`94`	`maintained_by: manifest.maintained_by,`
`95`	`95`	`index_html: manifest.index_html,`
`96`	`96`	`link_text: manifest.link_text,`
`97`		`- prefix,`
`98`		`- posts,`
	`97`	`+ path,`
	`98`	`+ pages: posts,`
`99`	`99`	`})`
`100`	`100`	`}`
`101`	`101`
`@@ -111,12 +111,12 @@ impl Blog {`
`111`	`111`	`&self.index_title`
`112`	`112`	`}`
`113`	`113`
`114`		`- pub(crate) fn prefix(&self) -> &Path {`
`115`		`- &self.prefix`
	`114`	`+ pub(crate) fn path(&self) -> &Path {`
	`115`	`+ &self.path`
`116`	`116`	`}`
`117`	`117`
`118`	`118`	`pub(crate) fn posts(&self) -> &[Post] {`
`119`		`- &self.posts`
	`119`	`+ &self.pages`
`120`	`120`	`}`
`121`	`121`	`}`
`122`	`122`
`@@ -139,10 +139,10 @@ fn load_recursive(base: &Path, current: &Path, blogs: &mut Vec<Blog>) -> eyre::R`
`139`	`139`	`let file_name = path.file_name().and_then(\|n\| n.to_str());`
`140`	`140`	`if let (Some(file_name), Some(parent)) = (file_name, path.parent()) {`
`141`	`141`	`if file_name == MANIFEST_FILE {`
`142`		`- let prefix = parent`
	`142`	`+ let path = parent`
`143`	`143`	`.strip_prefix(base)`
`144`	`144`	`.map_or_else(\|_\| PathBuf::new(), Path::to_path_buf);`
`145`		`- blogs.push(Blog::load(prefix, parent)?);`
	`145`	`+ blogs.push(Blog::load(path, parent)?);`
`146`	`146`	`}`
`147`	`147`	`}`
`148`	`148`	`}`