Change all-time to 30 days and fix canonical url issue

kalenjordan · Jan 21, 2015 · 8622a1a · 8622a1a
1 parent 47d7185
commit 8622a1a
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 1 deletion.
diff --git a/code/Controller/PostList.php b/code/Controller/PostList.php
@@ -25,6 +25,8 @@ protected function _getPosts()
 
         if (! isset($_GET['period']) || $_GET['period'] != 'all-time') {
             $select->where("posts.created_at > DATE_SUB(NOW(), INTERVAL $recentTimePeriod)");
+        } else {
+            $select->where("posts.created_at > DATE_SUB(NOW(), INTERVAL 1 MONTH)");
         }
 
         $postRows = $this->_getContainer()->LocalConfig()->database()->fetchAll($select);

diff --git a/template/base.html.twig b/template/base.html.twig
@@ -15,7 +15,9 @@
         {% block additional_head %} {% endblock %}
 		 <meta name="description" content="Latest news from Magento developers across the globe."</meta>
 		 <meta name="robots" content="index,follow">
+        <!-- commenting out for now - this needs to be updated for post url's
 		 <link rel="canonical" href="http://magehero.com" />
+		 -->
         <link rel="icon" href="/favicon.ico" type="image/x-icon"/>
     {% endblock %}
 </head>

diff --git a/template/post_list.html.twig b/template/post_list.html.twig
@@ -1,7 +1,7 @@
 {% extends 'base.html.twig' %}
 
 {% block content %}
-    <h2 title="All MageHero posts from the last 7 day rolling period">This Week <span class="filter"><a href="/?period=all-time">all time</a></span></h2>
+    <h2 title="All MageHero posts from the last 7 day rolling period">This Week <span class="filter"><a href="/?period=all-time">last 30 days</a></span></h2>
     <ul class="listing">
         {% for post in posts %}
             <li class="listing-post">