Merge pull request #815 from BlackPythonDevs/issue-707-author-list

Refactor author frontmatter to list

Author: TimOsahenru

.frontmatter_check.yaml

@@ -8,4 +8,3 @@ patterns:
       - field_name: lang
       - field_name: layout
       - field_name: title
-      - field_name: author

_layouts/blog-list.html

@@ -12,7 +12,7 @@ <h2><a href="{{ page }}.html">
           {% if page.description %}
           <p>{{page.description}}</p>
           {% endif %}
-          <small>{{page.author | join(", ")}} - <strong>{{page.date | format_datetime}}</strong></small>
+          <small>{% if page.author %}{{page.author | join(", ")}} - {% endif %}<strong>{{page.date | format_datetime}}</strong></small>
         </div>
       </article>
     {% endfor %}

_layouts/event-list.html

@@ -7,7 +7,7 @@ <h1>{{title}}</h1>
         <h2><a href="{{ page }}.html">
           {{ page.title }}
         </a></h2>
-        <small>{{page.author}} {{page.date}}</small>
+        <small>{% if page.author %}{{page.author | join(", ")}} {% endif %}{{page.date}}</small>
       </article>
     {% endfor %}
     </section>

_layouts/post.html

@@ -15,13 +15,14 @@
     <header class="post-header">
       <h1 class="post-title p-name" itemprop="name headline">{{ title }}</h1>
       <p class="post-meta">
+        {% if author %}
         <span itemprop="author" itemscope itemtype="http://schema.org/Person">
           {% for person in author %}
           <span class="p-author h-card" itemprop="name">{{ person }}</span>
           {% if not loop.last %}, {% endif %}
           {% endfor %}
         </span>
-
+        {% endif %}
       </p>
     </header>
     <div class="post-content e-content" itemprop="articleBody">
@@ -30,13 +31,17 @@ <h1 class="post-title p-name" itemprop="name headline">{{ title }}</h1>
   </article>
 </section>
 
-{% set author = authors %}
 <section>
-  {% if author and author.bio %}
-  <hr />
-    <h3>About {{author.name}}</h3>
-    <p>{{author.bio|e}}</p>
-  </section>
+  {% if author %}
+  {% for author_name in author %}
+    {% set author_data = SITE_AUTHORS | selectattr("name", "equalto", author_name) | first %}
+    {% if author_data and author_data.bio %}
+      <hr />
+      <h3>About {{author_data.name}}</h3>
+      <p>{{author_data.bio|e}}</p>
+    {% endif %}
+  {% endfor %}
   {% endif %}
+</section>
 
 {% endblock %}

app.py

@@ -31,6 +31,9 @@
 app.site_vars["navigation"] = navigation
 app.site_vars["DATETIME_FORMAT"] = "%d %b %Y"
 app.site_vars["year"] = str(datetime.date.today().year)
+app.site_vars["SITE_AUTHORS"] = json.loads(
+    pathlib.Path("_data/authors.json").read_text()
+)
 
 
 @app.page

scripts/check_author_list.py

@@ -17,6 +17,11 @@ def check_authors():
                 if not isinstance(author, list):
                     print(f"Error: Author in {file_path} is not a list: {author}")
                     failed = True
+                elif len(author) == 0:
+                    print(
+                        f"Error: Author list in {file_path} is empty. Remove the field if there is no author."
+                    )
+                    failed = True
         except Exception as e:
             print(f"Error processing {file_path}: {e}")
             failed = True