Redirect to 404 for articles that do not exist

Pomash/Pomash.py

@@ -51,6 +51,8 @@ def get(self, page_id):
 class ArticleHandler(BaseHandler):
     def get(self, article_id):
         article = get_article(article_id)
+        if(not article):
+            self.redirect("/404")
         tags = [tag.strip() for tag in article.tag.split(",")]
         self.render("article.html",
             article = article,
@@ -61,6 +63,7 @@ def get(self, article_id):
             valine_app_key = valine_app_key,
             twitter_card = twitter_card,
             twitter_username = twitter_username,
+            all_articles = get_all_articles(),
             )
 
 class ArticlesHandler(BaseHandler):
@@ -81,9 +84,19 @@ def get(self):
 
 class TagHandler(BaseHandler):
     def get(self, tag_name):
+        all_tag_articles = get_tag_articles(tag_name)
+        year_list = {}
+        for article in all_tag_articles:
+            year = article["datetime"].split("-")[0]
+            if year in year_list:
+                year_list[year].append(article)
+            else:
+                year_list[year] = []
+                year_list[year].append(article)
         self.render("tag.html",
+            all_articles = get_all_articles(),
             tag_name = tag_name,
-            articlesList = get_tag_articles(tag_name),
+            articlesList = year_list,
             )
 
 class TagsHandler(BaseHandler):

Pomash/theme/clean/templates/dropbox.html

@@ -4,7 +4,7 @@
 
 {% block content %}
 <div class="post">
-    <h1 id="title">Dropbox</h1>
+    <h1 id="title">Dropbox Backup</h1>
     <article>
     {% if not authorized %}
         <div class="item">

Pomash/theme/clean/templates/tag.html

@@ -5,10 +5,15 @@
 {% block content %}
 <h2>Tag: {{ tag_name }}</h2>
 <ol id="posts">
-    {% for article in articlesList %}
-    <li>
-        <span class="meta">{{ article.datetime[:10] }}</span> <a href="/article/{{ article.article_id }}">{{ article.title }}</a>
-    </li>
+    {% for year in sorted(articlesList.keys(), reverse = True) %}
+    <h2>{{ year }}</h2>
+    <ol id="posts">
+        {% for i in range(len(articlesList[year])) %}
+        <li>
+            <span class="meta">{{ articlesList[year][i].datetime[:10] }}</span> <a href="/article/{{ articlesList[year][i].article_id }}">{{ articlesList[year][i].title }}</a>
+        </li>
+        {% end %}
+    </ol>
     {% end %}
 </ol>
 {% end %}