Return 404 for out of bounds archive pages. Fixes #5

rgrove · Jul 14, 2009 · 24c99a1 · 24c99a1
1 parent 222333c
commit 24c99a1
Showing 1 changed file with 2 additions and 5 deletions.
diff --git a/lib/thoth/controller/archive.rb b/lib/thoth/controller/archive.rb
@@ -35,14 +35,11 @@ class ArchiveController < Controller
 
     def index(page = 1)
       page = page.to_i
-      page = 1 unless page >= 1
+      error_404 unless page >= 1
 
       @posts = Post.recent(page, 10)
 
-      if page > @posts.page_count && @posts.page_count > 0
-        page = @posts.page_count
-        @posts = Post.recent(page, 10)
-      end
+      error_404 if page > @posts.page_count && @posts.page_count > 0
 
       @title = "#{Config.site['name']} Archives (page #{page} of " <<
           "#{@posts.page_count > 0 ? @posts.page_count : 1})"