From 60a1284cf8485ecf16815905e83023df190b4d11 Mon Sep 17 00:00:00 2001
From: androiddrew <bednar.andrew@gmail.com>
Date: Fri, 16 Nov 2018 13:34:40 -0500
Subject: [PATCH] Corrected sort order and published status on blog index

---
 services/cms/blog/models.py                   | 13 ++++++-
 .../blog/templates/blog/blog_index_page.html  | 36 ++++++++++---------
 2 files changed, 31 insertions(+), 18 deletions(-)

diff --git a/services/cms/blog/models.py b/services/cms/blog/models.py
index 34f5208..f5d40b3 100644
--- a/services/cms/blog/models.py
+++ b/services/cms/blog/models.py
@@ -5,6 +5,7 @@ from wagtail.core.fields import RichTextField
 from wagtail.admin.edit_handlers import FieldPanel
 from wagtail.search import index
 
+
 class BlogIndexPage(Page):
     intro = RichTextField(blank=True)
 
@@ -12,6 +13,16 @@ class BlogIndexPage(Page):
         FieldPanel('intro', classname="full")
     ]
 
+    def get_context(self, request, *args, **kwargs):
+        """
+        Overrides the default context to include only published pages, ordered by reverse chronological order.
+        These child pages can be accessed in the template using `blogpages` instead of page.get_children.
+        """
+        context = super().get_context(request)
+        blogpages = self.get_children().live().order_by("-first_published_at")
+        context["blogpages"] = blogpages
+        return context
+
 
 class BlogPage(Page):
     date = models.DateField("Post date")
@@ -27,4 +38,4 @@ class BlogPage(Page):
         FieldPanel('date'),
         FieldPanel('intro'),
         FieldPanel('body', classname="full"),
-    ]
\ No newline at end of file
+    ]
diff --git a/services/cms/blog/templates/blog/blog_index_page.html b/services/cms/blog/templates/blog/blog_index_page.html
index dd6af10..af2b890 100644
--- a/services/cms/blog/templates/blog/blog_index_page.html
+++ b/services/cms/blog/templates/blog/blog_index_page.html
@@ -9,23 +9,25 @@
 <section class="mw7 center">
   <h1 class="ph3 ph0-l">{{ page.title }}</h1>
 
-  {% for post in page.get_children %}
-  <article class="pv4 bt bb b--black-10 ph3 ph0-l">
-    <a class="db pv4 ph3 ph0-l no-underline black dim" href="{% pageurl post %}" >
-      <div class="flex flex-column flex-row-ns">
-        <div class="w-100 w-60-ns pr3-ns order-2 order-1-ns">
-          <h1 class="f3 mt0 lh-title">{{ post.title }}</h1>
-          <p class="f5 f4-l lh-copy">{{ post.specific.intro }}</p>
-        </div>
-        <div class="pl3-ns order-1 order-2-ns mb4 mb0-ns w-100 w-40-ns">
-          <img src="http://mrmrs.github.io/photos/cpu.jpg" class="db"
-               alt="Photo of a dimly lit room with a computer interface terminal.">
-        </div>
-      </div>
-      <p class="f6 lh-copy gray mv0">By <span class="ttu">{{ post.owner }}</span></p>
-      <time class="f6 db gray">{{ post.specific.date|date }}</time>
-    </a>
-  </article>
+  {% for post in blogpages %}
+    {% with post=post.specific %}
+      <article class="pv4 bt bb b--black-10 ph3 ph0-l">
+        <a class="db pv4 ph3 ph0-l no-underline black dim" href="{% pageurl post %}" >
+          <div class="flex flex-column flex-row-ns">
+            <div class="w-100 w-60-ns pr3-ns order-2 order-1-ns">
+              <h1 class="f3 mt0 lh-title">{{ post.title }}</h1>
+              <p class="f5 f4-l lh-copy">{{ post.intro }}</p>
+            </div>
+            <div class="pl3-ns order-1 order-2-ns mb4 mb0-ns w-100 w-40-ns">
+              <img src="http://mrmrs.github.io/photos/cpu.jpg" class="db"
+                   alt="Photo of a dimly lit room with a computer interface terminal.">
+            </div>
+          </div>
+          <p class="f6 lh-copy gray mv0">By <span class="ttu">{{ post.owner }}</span></p>
+          <time class="f6 db gray">{{ post.date|date }}</time>
+        </a>
+      </article>
+    {% endwith %}
   {% endfor %}
 
 </section>