diff options
| -rw-r--r-- | app/controllers/admin/blog/posts_controller.rb | 46 | ||||
| -rw-r--r-- | app/views/blog/posts/_post.html.erb | 2 | ||||
| -rw-r--r-- | features/authors.feature | 15 | ||||
| -rw-r--r-- | features/support/step_definitions/authors_steps.rb | 11 |
4 files changed, 73 insertions, 1 deletions
diff --git a/app/controllers/admin/blog/posts_controller.rb b/app/controllers/admin/blog/posts_controller.rb index 1b6efc3..e8ffed2 100644 --- a/app/controllers/admin/blog/posts_controller.rb +++ b/app/controllers/admin/blog/posts_controller.rb @@ -10,6 +10,52 @@ class Admin::Blog::PostsController < Admin::BaseController :per_page => BlogPost.per_page }) end + + def create + # if the position field exists, set this object as last object, given the conditions of this class. + if BlogPost.column_names.include?("position") + params[:blog_post].merge!({ + :position => ((BlogPost.maximum(:position, :conditions => "")||-1) + 1) + }) + end + + if BlogPost.column_names.include?("user_id") + params[:blog_post].merge!({ + :user_id => current_user.id + }) + end + + if (@blog_post = BlogPost.create(params[:blog_post])).valid? + (request.xhr? ? flash.now : flash).notice = t( + 'refinery.crudify.created', + :what => "'#{@blog_post.title}'" + ) + + unless from_dialog? + unless params[:continue_editing] =~ /true|on|1/ + redirect_back_or_default(admin_blog_posts_url) + else + unless request.xhr? + redirect_to :back + else + render :partial => "/shared/message" + end + end + else + render :text => "<script>parent.window.location = '#{admin_blog_posts_url}';</script>" + end + else + unless request.xhr? + render :action => 'new' + else + render :partial => "/shared/admin/error_messages", + :locals => { + :object => @blog_post, + :include_object_name => true + } + end + end + end before_filter :find_all_categories, :only => [:new, :edit, :create, :update] diff --git a/app/views/blog/posts/_post.html.erb b/app/views/blog/posts/_post.html.erb index 6a16631..7600d26 100644 --- a/app/views/blog/posts/_post.html.erb +++ b/app/views/blog/posts/_post.html.erb @@ -9,7 +9,7 @@ <details> <time datetime="<%=l @blog_post.published_at.to_date, :format => :default %>" class='posted_at'> <%= t('blog.shared.posts.created_at', :when => l(@blog_post.published_at.to_date, :format => :short)) %>. - </time> + </time> by <%= @blog_post.author.try :login %> <% if (categories = @blog_post.categories).any? %> <aside class='filed_in'> <%= t('blog.posts.show.filed_in') %> diff --git a/features/authors.feature b/features/authors.feature new file mode 100644 index 0000000..5bb5cea --- /dev/null +++ b/features/authors.feature @@ -0,0 +1,15 @@ +Feature: Blog Post Authors + Blog posts can be assigned authors through the given user model + current_user is assumed through admin screens + + Scenario: Saving a blog post in blog_posts#new associates the current_user as the author + Given there is a user named "hubble" + And I am logged in as "hubble" + + When I am on the new blog post form + And I fill in "Title" with "This is my blog post" + And I fill in "Body" with "And I am hubble" + And I press "Save" + + Then there should be 1 blog post + And the blog post should belong to "hubble"
\ No newline at end of file diff --git a/features/support/step_definitions/authors_steps.rb b/features/support/step_definitions/authors_steps.rb new file mode 100644 index 0000000..6f695ca --- /dev/null +++ b/features/support/step_definitions/authors_steps.rb @@ -0,0 +1,11 @@ +Given /^there is a user named "([^\"]*)"$/ do |login| + @user = Factory.create(:user, :login => login, :password => "#{login}-123", :password_confirmation => "#{login}-123") +end + +Then /^there should be (\d+) blog posts?$/ do |num| + BlogPost.all.size == num +end + +Then /^the blog post should belong to "([^\"]*)"$/ do |login| + BlogPost.last.author.login == login +end
\ No newline at end of file |