From 1dbd24300859cf3750a2fb37431a8a8506a00e7d Mon Sep 17 00:00:00 2001
From: Sami Samhuri <sami@samhuri.net>
Date: Sun, 19 Apr 2015 10:40:08 -0700
Subject: [PATCH] tighten up params to HarpBlog#post_path

---
 server/harp_blog.rb | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/server/harp_blog.rb b/server/harp_blog.rb
index 709b169..b99350e 100644
--- a/server/harp_blog.rb
+++ b/server/harp_blog.rb
@@ -55,7 +55,7 @@ class HarpBlog
   def months
     years.map do |year|
       # hack: month dirs (and only month dirs) are always 2 characters in length
-      Dir[post_path(year, '??')].map { |x| [year, File.basename(x)] }
+      Dir[post_path("#{year}/??")].map { |x| [year, File.basename(x)] }
     end.flatten(1).sort
   end
 
@@ -185,8 +185,10 @@ class HarpBlog
     File.join(@path, *components)
   end
 
-  def post_path(*components)
-    path_for('public/posts', *components)
+  def post_path(dir, id = nil)
+    args = ['public/posts', dir]
+    args << "#{id}.md" if id
+    path_for(*args)
   end
 
   def drafts_path(*components)
@@ -203,14 +205,14 @@ class HarpBlog
       unless extra_fields[:draft]
         fields[:slug] = id
       end
-      post_filename = post_path(post_dir, "#{id}.md")
+      post_filename = post_path(post_dir, id)
       fields[:body] = File.read(post_filename)
       Post.new(fields.merge(extra_fields))
     end
   end
 
   def read_post(post_dir, id, extra_fields = nil)
-    post_filename = post_path(post_dir, "#{id}.md")
+    post_filename = post_path(post_dir, id)
     post_data = read_post_data(post_path(post_dir))
     if File.exist?(post_filename) && fields = post_data[id]
       fields[:body] = File.read(post_filename)