@@ -1,56 +1,50 @@
class Admin::PostsController < Admin::AdminController
def index
- @posts = Post.all
+ @posts = Post.by_admin_posts
end
def new
end
def create
- # create the article
- @post = Post.new(params[:post])
+ # create the post
+ @post = Post.new_from_params(params)
respond_to do |format|
- if params[:preview]
- format.html {
- session["post_preview"] = @post
- redirect_to(preview_admin_post_url(@post.generate_unique_permalink_from_title))
- }
- else
- @post.status = 'published' if params[:publish]
if @post.save
- format.html { redirect_to(edit_admin_post_url(@post.permalink)) }
+ format.html {
+ params[:preview] ? redirect_to(preview_admin_post_url(@post.permalink)) : redirect_to(edit_admin_post_url(@post.permalink))
+ }
else
format.html { render :action => "new" }
end
- end
end
end
def edit
- @post = Post.get(params[:id])
+ @post = Post.find_by_permalink(params[:id])
end
def update
- @post = Post.get(params[:id])
+ @post = Post.find_by_permalink(params[:id])
+
+ #should we see if they diff
+ @post.preview = params[:post][:body] if params[:preview]
+
+ @post.status = 'published' if params[:publish]
respond_to do |format|
- if params[:preview]
- format.html {
- @new_post = Post.new(params[:post])
- session["post_preview"] = @new_post
- redirect_to(preview_admin_post_url(@new_post.generate_unique_permalink_from_title))
+
+ @post.status = 'published' if params[:publish]
+ if @post.update_attributes(params[:post])
+ format.html {
+ params[:preview] ? redirect_to(preview_admin_post_url(@post.permalink)) : redirect_to(edit_admin_post_url(@post.permalink))
}
else
- @post.status = 'published' if params[:publish]
- if @post.update_attributes(params[:post])
- format.html { redirect_to(edit_admin_post_url(@post.permalink)) }
- else
- format.html { render :action => "edit" }
- end
- end
+ format.html { render :action => "edit" }
+ end
end
end
@@ -58,24 +52,11 @@ class Admin::PostsController < Admin::AdminController
#mark the post as a preview
@post_preview = true
- if session["post_preview"]
- @post = session["post_preview"]
- session["post_preview"] = nil
- end
-
#use id if on query string
- @post = Post.get(params[:id]) if params[:id]
- respond_to do |format|
- format.html { render :template => 'posts/show', :layout => 'application'}
- end
- end
-
- def post_preview
- #mark the post as a preview
- @post_preview = true
+ @post = Post.find_by_permalink(params[:id])
+ @post.body = @post.preview unless @post.preview.nil?
- @post = Post.new(:title => params[:post_title], :body => params[:post_body])
respond_to do |format|
format.html { render :template => 'posts/show', :layout => 'application'}
app/controllers/admin/posts_controller.rb
@@ -11,7 +11,7 @@ class PostsController < ApplicationController
end
def show
- @post = Post.get(params[:id])
+ @post = Post.find_by_permalink(params[:permalink])
respond_to do |format|
if @post.published? && @post.verify_date?(:month => params[:month], :year => params[:year])
app/controllers/posts_controller.rb
@@ -5,11 +5,9 @@ class Post < CouchRest::ExtendedDocument
# -------------------
# Properties
# -------------------
-
- unique_id :permalink
-
property :title
property :body
+ property :preview, :default => nil
property :status, :default => 'draft'
property :permalink
property(:updated_at, :read_only => true, :cast_as => 'Time', :auto_validation => false)
@@ -20,17 +18,11 @@ class Post < CouchRest::ExtendedDocument
validates_present :title, :body
-
+ #validates_is_unique property.name
def set_permalink_from_title
-
self['permalink'] = generate_unique_permalink_from_title if new_record? || title_changed?
-
- # if the title has changed and its an edit
- # set a new id and clean up the old doc
- # TODO look at copy method in couch rest
- cleanup_old_doc if title_changed? and not new_record?
end
# generates a unique permalink from the title
@@ -38,17 +30,53 @@ class Post < CouchRest::ExtendedDocument
permalink = title_to_permalink(title)
permalink = "#{permalink}-#{postfix}" if postfix > 1
-
- # if post exists call again adding postfix
- # if currnt post exists then it will find itself anf +1
- if Post.exists?(permalink)
- generate_unique_permalink_from_title(postfix+1)
+ if Post.permalink_unique?(permalink)
+ # permalink is ok and just return
+ return permalink
else
- return permalink
+ # call the same method again with an incremented number
+ generate_unique_permalink_from_title(postfix+1)
end
end
+ # alternative constructor that return a new post object with some values set
+ def self.new_from_params(params)
+ post = Post.new(params[:post])
+ #set the status
+ if params[:preview]
+ post.status = 'preview'
+ end
+ post.status = 'published' if params[:publish]
+
+ post
+ end
+
+ def set_updated_at(time)
+ self['updated_at'] = Time.parse(time)
+ end
+
+
+ # -------------------
+ # Find by ..
+ # -------------------
+
+ # will return the first post it finds by the pemalink
+ def self.find_by_permalink(permalink)
+ Post.by_permalink(:key => permalink).first
+ end
+
+ def self.find_by_year(year)
+ year = year.to_i
+ self.by_published(:startkey => [year,12,31], :endkey => [year,1,1])
+ end
+
+ def self.find_by_year_and_month(year, month)
+ month = month.to_i
+ year = year.to_i
+ self.by_published(:startkey => [year,month,Time.days_in_month(month)], :endkey => [year,month,1])
+ end
+
# -------------------
# Views
# -------------------
@@ -56,6 +84,17 @@ class Post < CouchRest::ExtendedDocument
view_by :created_at, :descending => true
view_by :status, :created_at, :descending => true
+ view_by :permalink
+
+ # exclude preview posts and order by updated_at
+ view_by :admin_posts, :descending => true,
+ :map =>
+ "function(doc) {
+ if ((doc['couchrest-type'] == 'Post') && (doc['status'] != 'preview') && doc['updated_at']) {
+ emit(doc['updated_at'], 1);
+ }
+ }"
+
# it is implied that all published articles are
# ordered by date, descending
view_by :published, :descending => true,
@@ -74,16 +113,7 @@ class Post < CouchRest::ExtendedDocument
return sum(values);
}"
- def self.find_by_year(year)
- year = year.to_i
- self.by_published(:startkey => [year,12,31], :endkey => [year,1,1])
- end
-
- def self.find_by_year_and_month(year, month)
- month = month.to_i
- year = year.to_i
- self.by_published(:startkey => [year,month,Time.days_in_month(month)], :endkey => [year,month,1])
- end
+
# sum_by_month
#
@@ -92,16 +122,27 @@ class Post < CouchRest::ExtendedDocument
self.by_published :reduce => true, :group_level => 2
end
- def self.exists?(id)
- !get(id).nil?
- end
+ # -------------------
+ # Before save
+ # -------------------
+
+ #TODO remove this and use an explicit set for testing
def set_timestamps
self['updated_at'] = Time.now
self['created_at'] = self['updated_at'] if self.new_document? && self['created_at'].nil?
+ end
+
+ # check if the permalink is unique
+ def self.permalink_unique?(permalink)
+ !Post.find_by_permalink(permalink)
end
+
+ # -------------------
+ # useful methods
+ # -------------------
def draft?
self['status'] == 'draft'
end
@@ -110,6 +151,10 @@ class Post < CouchRest::ExtendedDocument
self['status'] == 'published'
end
+ def preview?
+ self['status'] == 'preview'
+ end
+
def year_and_month
[self['created_at'].year.to_s, self['created_at'].strftime("%m")]
end
@@ -129,20 +174,8 @@ class Post < CouchRest::ExtendedDocument
private
- def cleanup_old_doc
- # store the old id
- id = self['_id']
-
- #keep the same doc but set a new id
- self['_id'] = self['permalink']
-
- # remove old doc
- post = Post.get(id)
- post.destroy
- end
-
def title_changed?
- (self['_id'] != title_to_permalink(self['title']))
+ (self['permalink'] != title_to_permalink(self['title']))
end
app/models/post.rb
@@ -3,6 +3,7 @@
<thead>
<tr>
<th id="title" scope="col">Title</th>
+ <th id="updated" scope="col" class="medium">Last Updated</th>
<th id="created" scope="col" class="medium">Created</th>
<th> </th>
<th> </th>
@@ -13,10 +14,11 @@
<% @posts.each do |post| %>
<tr class="">
<td>
- <strong class="title"><a href="<%= edit_admin_post_path(post.id) %>"><%= post.title %></a></strong>
+ <strong class="title"><a href="<%= edit_admin_post_path(post.permalink) %>"><%= post.title %></a></strong>
</td>
+ <td><%= l post.updated_at, :format => :verbose %></td>
<td><%= l post.created_at.to_date, :format => :long %></td>
- <td><%= link_to "edit", edit_admin_post_path(post.id) %></td>
+ <td><%= link_to "edit", edit_admin_post_path(post.permalink) %></td>
<td><%= button_to "delete", admin_post_path(post.id), :method => :delete
%></td>
</tr>
app/views/admin/posts/index.html.erb
@@ -22,6 +22,7 @@ en:
default: "%a, %d %b %Y %H:%M:%S %z"
short: "%d %b %H:%M"
long: "%B %d, %Y %H:%M"
+ verbose: "%d %B %Y at %H:%M"
am: "am"
pm: "pm"
config/locales/en.yml
@@ -29,7 +29,7 @@ ActionController::Routing::Routes.draw do |map|
map.resource :user_session
map.namespace :admin do |admin|
- admin.resources :posts, :member => { :preview => :get }, :collection => {:post_preview => :post, :preview => :get}
+ admin.resources :posts, :member => { :preview => :get }
end
map.admin_root '/admin', :controller => 'admin/home'
@@ -39,7 +39,7 @@ ActionController::Routing::Routes.draw do |map|
map.archives '/archives', :controller => 'posts', :action => 'archives'
map.by_year ':year', :controller => 'posts', :action => 'by_year', :year => /\d{4}/
map.by_month ':year/:month', :controller => 'posts', :action => 'by_month', :year => /\d{4}/, :month => /\d{2}/
- map.post ':year/:month/:id', :controller => 'posts', :action => 'show'
+ map.post ':year/:month/:permalink', :controller => 'posts', :action => 'show'
# syndication urls
map.feed '/feed.:format', :controller => 'syndication', :action => 'feed'
config/routes.rb
@@ -3,18 +3,20 @@ Feature: Manage posts
As an article writer
I want to edit, delete and view existing blog posts
- Scenario: show a list of created posts
+ Scenario: show a list of created posts with recent updated to the top
Given the following posts:
- | title | body | created_at |
- | a title | the amazing body | 2009/06/24 14:10:27 +0000 |
- | the best title | the amazing body | 2009/06/26 21:07:32 +0000 |
- | a great post | the amazing body | 2009/06/26 21:07:32 +0000 |
+ | title | body | created_at | updated_at |
+ | a title | the amazing body | 2009/06/24 14:10:27 +01:00 | 2009/06/24 14:10:27 +01:00 |
+ | the best title | the amazing body | 2009/06/26 21:07:32 +01:00 | 2009/08/26 21:07:32 +01:00 |
+ | a great post | the amazing body | 2009/06/26 21:07:32 +01:00 | 2009/08/26 22:22:32 +01:00 |
+
And I am logged in as "admin"
And I am on the post admin page
Then I should see the following posts:
- | title | created at |
- | a great post | 26 June 2009 |
- | a title | 24 June 2009 |
+ | title | last updated | created at |
+ | a great post | 26 August 2009 at 22:22 | 26 June 2009 |
+ | the best title | 26 August 2009 at 21:07 | 26 June 2009 |
+ | a title | 24 June 2009 at 14:10 | 24 June 2009 |
Scenario: edit an existing post
Given I am logged in as "admin"
@@ -74,15 +76,15 @@ Feature: Manage posts
Scenario: Delete a post
Given I am logged in as "admin"
And the following posts:
- | title | body |
- | a title | the amazing body |
- | a rubbish post | the crap body |
- | a great post | the amazing body |
+ | title | body |updated_at |
+ | a title | the amazing body |2009/09/24 14:10:27 +01:00 |
+ | a rubbish post | the crap body |2009/08/26 21:07:32 +01:00 |
+ | a great post | the amazing body |2009/07/26 21:07:32 +01:00 |
When I delete the 2nd post
Then I should see the following posts:
- | title |
+ | title |
+ | a title |
| a great post |
- | a title |
features/manage_posts.feature
@@ -1,6 +1,12 @@
Given /^the following posts?:$/ do |table|
table.hashes.each do |hash|
- Post.create!(hash)
+ post = Post.create!(hash)
+ # we need to override the updated_at with the values supplied
+ if hash.include?('updated_at')
+ post.set_updated_at(hash['updated_at'])
+ # save without running the set_timestamps
+ post.save_without_callbacks
+ end
end
end
features/step_definitions/post_steps.rb
@@ -1,5 +1,5 @@
$(document).ready(function(){
- $("#post_preview").bind("click", function(){
+/* $("#post_preview").bind("click", function(){
// get the values from the page
var title = $('input#post_title').val();
var body = $('textarea#post_body').val();
@@ -11,6 +11,7 @@ $(document).ready(function(){
return false;
});
+*/
});
function openPopUp(html, windowName){
public/javascripts/application.js
@@ -43,19 +43,19 @@ describe Admin::PostsController do
describe "with valid params" do
it "exposes a newly created post as @post" do
- Post.should_receive(:new).with({'these' => 'params'}).and_return(mock_post(:save => true))
+ Post.should_receive(:new).with({'these' => 'params'}).and_return(mock_post(:save => true, :preview? => false))
post :create, :post => {:these => 'params'}
assigns(:post).should equal(mock_post)
end
it "redirect to edit after creation" do
- Post.should_receive(:new).with({'these' => 'params'}).and_return(mock_post(:save => true, :status= => "published"))
+ Post.should_receive(:new).with({'these' => 'params'}).and_return(mock_post(:save => true, :status= => "published", :preview? => false))
post :create, :post => {:these => 'params'}, :publish => 'Publish'
response.should redirect_to(edit_admin_post_url(mock_post.permalink))
end
it "should set status to published when published" do
- Post.should_receive(:new).with({'these' => 'params'}).and_return(@foo = mock_post(:save => true))
+ Post.should_receive(:new).with({'these' => 'params'}).and_return(@foo = mock_post(:save => true, :preview? => false))
@foo.should_receive(:status=).with('published').and_return(true)
post :create, :post => {:these => 'params'}, :publish => 'Publish'
end
@@ -80,14 +80,14 @@ describe Admin::PostsController do
describe "rendering preview" do
- it "should write post to session as 'post_preview'" do
- Post.should_receive(:new).with({'these' => 'params'}).and_return(mock_post(:generate_unique_permalink_from_title => 'foo'))
+ it "should set the status to preview" do
+ Post.should_receive(:new).with({'these' => 'params'}).and_return(@foo = mock_post(:save => true, :preview? => true))
+ @foo.should_receive(:status=).with('preview').and_return(true)
post :create, :post => {:these => 'params'}, :preview => 'Preview'
- session["post_preview"].should == mock_post
end
it "should redirect to preview page" do
- Post.should_receive(:new).with({'these' => 'params'}).and_return(mock_post(:generate_unique_permalink_from_title => 'foo'))
+ Post.should_receive(:new).with({'these' => 'params'}).and_return(@foo = mock_post(:save => true, :status= => true, :preview? => true))
post :create, :post => {:these => 'params'}, :preview => 'Preview'
response.should redirect_to(preview_admin_post_url(mock_post.permalink))
end
@@ -100,19 +100,19 @@ describe Admin::PostsController do
describe "with valid params" do
it "updates the requested post" do
- Post.should_receive(:get).with("37").and_return(mock_post)
+ Post.should_receive(:find_by_permalink).with("37").and_return(mock_post)
mock_post.should_receive(:update_attributes).with({'these' => 'params'})
put :update, :id => "37", :post => {:these => 'params'}
end
it "exposes the requested post as @post" do
- Post.stub!(:get).and_return(mock_post(:update_attributes => true))
+ Post.stub!(:find_by_permalink).and_return(mock_post(:update_attributes => true))
put :update, :id => "1"
assigns(:post).should equal(mock_post)
end
it "redirects to the post" do
- Post.stub!(:get).and_return(mock_post(:update_attributes => true))
+ Post.stub!(:find_by_permalink).and_return(mock_post(:update_attributes => true))
put :update, :id => "1"
response.should redirect_to(edit_admin_post_url(mock_post.permalink))
end
@@ -122,19 +122,19 @@ describe Admin::PostsController do
describe "with invalid params" do
it "updates the requested post" do
- Post.should_receive(:get).with("37").and_return(mock_post)
+ Post.should_receive(:find_by_permalink).with("37").and_return(mock_post)
mock_post.should_receive(:update_attributes).with({'these' => 'params'})
put :update, :id => "37", :post => {:these => 'params'}
end
it "exposes the post as @post" do
- Post.stub!(:get).and_return(mock_post(:update_attributes => false))
+ Post.stub!(:find_by_permalink).and_return(mock_post(:update_attributes => false))
put :update, :id => "1"
assigns(:post).should equal(mock_post)
end
it "re-renders the 'edit' template" do
- Post.stub!(:get).and_return(mock_post(:update_attributes => false))
+ Post.stub!(:find_by_permalink).and_return(mock_post(:update_attributes => false))
put :update, :id => "1"
response.should render_template('edit')
end
@@ -143,18 +143,7 @@ describe Admin::PostsController do
describe "rendering preview" do
before do
- Post.stub!(:get).and_return(mock_post(:generate_unique_permalink_from_title => 'foo'))
- Post.should_receive(:new).with({'these' => 'params'}).and_return(mock_post(:generate_unique_permalink_from_title => 'foo'))
- end
-
- it "should create fresh post and expose as @new_post" do
- post :update, :id => "foo", :post => {:these => 'params'}, :preview => 'Preview'
- assigns(:new_post).should equal(mock_post)
- end
-
- it "should write post to session as 'post_preview'" do
- post :update, :id => "foo", :post => {:these => 'params'}, :preview => 'Preview'
- session["post_preview"].should == mock_post
+ Post.stub!(:find_by_permalink).and_return(mock_post(:preview= => true, :update_attributes => true))
end
it "should redirect to preview page" do
@@ -163,55 +152,33 @@ describe Admin::PostsController do
end
end
- end
-
- describe "GET /preview" do
- it "exposes a post initialised from the cache as @post" do
- session[:post_preview] = mock_post
- get :preview
- assigns(:post).should equal(mock_post)
- end
-
- it "should clear the session" do
- session[:post_preview] = mock_post
- get :preview
- session["post_preview"].should == nil
- end
-
- it "should render the 'posts/show' template" do
- session[:post_preview] = mock_post
- get :preview
- response.should render_template('posts/show')
- end
end
- describe "GET /id/preview" do
+ describe "GET /:id/preview" do
it "use the id to init the post" do
- Post.stub!(:get).and_return(mock_post)
+ Post.stub!(:find_by_permalink).and_return(mock_post(:preview => nil))
get :preview, :id => 'foo'
assigns(:post).should equal(mock_post)
end
it "should render the 'posts/show' template" do
- Post.stub!(:get).and_return(mock_post)
+ Post.stub!(:find_by_permalink).and_return(mock_post(:preview => nil))
get :preview, :id => 'foo'
response.should render_template('posts/show')
- end
- end
-
- describe "POST post_preview" do
-
- it "exposes a newly created post as @post" do
- Post.should_receive(:new).with({:title => 'foo', :body => 'bar'}).and_return(mock_post)
- post :post_preview, :post_title => 'foo', :post_body => 'bar'
- assigns(:post).should equal(mock_post)
end
- it "should render the 'posts/show' template" do
- Post.stub!(:new).and_return(mock_post)
- post :post_preview, :post => {}
- response.should render_template('posts/show')
+ it "should mark the post as a preview" do
+ Post.stub!(:find_by_permalink).and_return(mock_post(:preview => nil))
+ get :preview, :id => 'foo'
+ assigns(:post_preview).should equal(true)
end
+
+ it "should set the body as the preview if its there" do
+ Post.stub!(:find_by_permalink).and_return(@foo = mock_post(:preview => 'foo preview'))
+ @foo.should_receive(:body=).with("foo preview")
+ get :preview, :id => 'foo'
+
+ end
end
describe "DELETE destroy" do
spec/controllers/admin/posts_controller_spec.rb
@@ -34,27 +34,27 @@ describe PostsController do
describe "GET 'show'" do
it "should be successful" do
- Post.stub!(:get).and_return(mock_post(:published? => true, :verify_date? => true))
- get :show, :year => "2009", :month => "06", :id => "a-title"
+ Post.stub!(:find_by_permalink).and_return(mock_post(:published? => true, :verify_date? => true))
+ get :show, :year => "2009", :month => "06", :permalink => "a-title"
response.should be_success
end
it "should assign post as @post" do
- Post.stub!(:get).and_return(mock_post(:published? => true, :verify_date? => true))
- get :show, :year => "2009", :month => "06", :id => "a-title"
+ Post.stub!(:find_by_permalink).and_return(mock_post(:published? => true, :verify_date? => true))
+ get :show, :year => "2009", :month => "06", :permalink => "a-title"
assigns[:post].should == mock_post
end
it "should redirect unpublished posts to the 404 page" do
- Post.stub!(:get).and_return(mock_post(:published? => false, :verify_date? => true))
- get :show, :year => "2009", :month => "06", :id => "a-title"
+ Post.stub!(:find_by_permalink).and_return(mock_post(:published? => false, :verify_date? => true))
+ get :show, :year => "2009", :month => "06", :permalink => "a-title"
response.should redirect_to('/404')
end
# we really want to index and cache the correct url
it "should redirect posts with incorrect dates to 404" do
- Post.stub!(:get).and_return(mock_post(:published? => true, :verify_date? => false))
- get :show, :year => "2009", :month => "06", :id => "a-title"
+ Post.stub!(:find_by_permalink).and_return(mock_post(:published? => true, :verify_date? => false))
+ get :show, :year => "2009", :month => "06", :permalink => "a-title"
response.should redirect_to('/404')
end
spec/controllers/posts_controller_spec.rb
@@ -52,20 +52,12 @@ describe Post do
@new.permalink.should == "only-unique-permalinks-times-3"
end
- it "should update the id when title is changed" do
- @post = Post.create!(:title => "title one", :body => "The best blog post in the world")
- @post.title = "title two"
- @post.save!
-
- Post.get("title-two").should_not == nil
- end
-
it "should update the permalink when title is changed" do
@post = Post.create!(:title => "title one", :body => "The best blog post in the world")
@post.title = "title two"
@post.save!
- @foo = Post.get("title-two")
+ @foo = Post.get(@post.id)
@foo.permalink.should == "title-two"
end
@@ -76,14 +68,14 @@ describe Post do
Post.get("title-one").should == nil
end
- describe "exists? method" do
- it "should return true if document exists" do
+ describe "permalink_unique? method" do
+ it "should return false if permalink is being used" do
@post.save!
- Post.exists?(@post.id).should == true
+ Post.permalink_unique?(@post.permalink).should == false
end
- it "should return false when document no in db" do
- Post.exists?("random-id-that-does-not-exist").should == false
+ it "should return true if permalink is not in use" do
+ Post.permalink_unique?("random-id-that-does-not-exist").should == true
end
end
@@ -157,5 +149,31 @@ describe Post do
@posts = Post.find_by_year_and_month('2009', '06')
@posts.size.should == 1
end
+
+ it "should find a post by its permalink" do
+ @post = Post.find_by_permalink('title-one')
+ @post.should_not == nil
+ end
+ end
+
+
+ describe ".new_from_params" do
+
+ it "should set the post status from the params" do
+ @post = Post.new_from_params({:post=>{:body=>"foo bar", :title=>"testing"}, :preview=>"Preview"})
+ @post.status.should == "preview"
+ end
+
+ it "should store the preview" do
+ @post = Post.new_from_params({:post=>{:body=>"foo bar", :title=>"testing"}, :preview=>"Preview"})
+ @post.preview.should == nil
+ end
+
+ end
+
+ it "should be able to override the updated_at value" do
+ @post = Post.create!(:title => "title one", :body => "The best blog post in the world")
+ @post.set_updated_at("2009/06/24 14:10:27")
+ @post.updated_at.should == Time.parse("2009/06/24 14:10:27")
end
end
\ No newline at end of file
spec/models/post_spec.rb
@@ -8,7 +8,7 @@ describe PostsController do
end
it "should map show" do
- route_for(:controller => "posts", :action => "show", :id => "a-title", :month => "01", :year => "2009").should == "/2009/01/a-title"
+ route_for(:controller => "posts", :action => "show", :permalink => "a-title", :month => "01", :year => "2009").should == "/2009/01/a-title"
end
it "should map by_year" do
@@ -26,7 +26,7 @@ describe PostsController do
describe "route recognition" do
it "generates params for #show" do
- params_from(:get, "/2009/01/a-title").should == {:controller => "posts", :action => "show", :id => "a-title", :month => "01", :year => "2009"}
+ params_from(:get, "/2009/01/a-title").should == {:controller => "posts", :action => "show", :permalink => "a-title", :month => "01", :year => "2009"}
end
it "generates params for #by_year" do
spec/routing/posts_routing_spec.rb
app/views/shared/_nav.erb.html
e90d3c82eb04f9c16467c2887e69661546c00061
Ben Aldred
benaldred@gmail.com
http://github.com/benaldred/soapbox/commit/f81a1b09e04be0d34b0d9172f0ee3404465eca6b
f81a1b09e04be0d34b0d9172f0ee3404465eca6b
2009-09-30T06:26:52-07:00
2009-09-30T06:26:52-07:00
refactored a lot
618c395dab123ea678cf911e1b77f7fbbf065002
Ben Aldred
benaldred@gmail.com