From eac9b401fefe21c56f05006aadfaaaf2549ff51f Mon Sep 17 00:00:00 2001
From: Alex Koufos <exoticdft@gmail.com>
Date: Thu, 26 Sep 2024 15:39:47 -0700
Subject: [PATCH] Issue #1587: Fixes a bug in the JSON API for jobs

- Adds a `,` for every element in our jobs array
- Updates formatting to see the liquid logic more clearly
- Moves nowunix capture outside of the for loop for jobs
---
 pages/api/jobs.json | 22 +++++++++++++++++-----
 1 file changed, 17 insertions(+), 5 deletions(-)

diff --git a/pages/api/jobs.json b/pages/api/jobs.json
index d8292ed96..0b9748892 100644
--- a/pages/api/jobs.json
+++ b/pages/api/jobs.json
@@ -3,10 +3,22 @@ layout: none
 permalink: /api/jobs.json
 excluded_in_search: true
 ---
-{% assign sorted_jobs = site.data.jobs | sort: "posted" | reverse %}[{% for job in sorted_jobs %}{% capture nowunix %}{{'now' | date: '%s'}}{% endcapture %}{% capture expires %}{{ job.expires | date: '%s'}}{% endcapture %}{% capture posted %}{{ job.posted | date: '%b %d, %Y'}}{% endcapture %}{% if expires > nowunix %}{
+{%- assign sorted_jobs = site.data.jobs | sort: "posted" | reverse -%}
+{%- capture nowunix -%}{{'now' | date: '%s'}}{%- endcapture -%}
+[
+{%- for job in sorted_jobs -%}
+   {%- capture expires -%}{{ job.expires | date: '%s'}}{%- endcapture -%}
+   {%- capture posted -%}{{ job.posted | date: '%b %d, %Y'}}{%- endcapture -%}
+   {% if expires > nowunix %}
+   {
         "title": {{ job.name | jsonify }},
-        {% if posted != '' %}"published": "{{ posted }}",{% endif %}
-        "location": {{ job.location| jsonify }},
+        {%- if posted != '' %}
+        "published": "{{ posted }}",
+        {%- endif %}
+        "location": {{ job.location | jsonify }},
         "url": "{{ job.url }}"
-   }{% if forloop.last %}{% else %},{% endif %}{% endif %}
-{% endfor %}{}]
+   },
+   {%- endif %}
+{%- endfor %}
+   {}
+]