Commit 412ad055

mo k <mo@mokhan.ca>
2012-06-10 00:11:04
extract command to create command.
1 parent eee4627
app/controllers/comments_controller.rb
@@ -5,15 +5,17 @@ class CommentsController < ApplicationController
     @comment = Comment.new
   end
 
+  #def create
+    #creation = Creation.find(params[:creation_id])
+    #comment = Comment.build_from(creation, current_user.id, params[:comment][:body])
+    #comment.save
+    #flash[:notice] = params[:comment][:body]
+    #redirect_to :controller => 'creations', :action => 'show', :id => params[:creation_id]
+  #end
   def create
-    creation = Creation.find(params[:creation_id])
-    comment = Comment.build_from(creation, current_user.id, params[:comment][:body])
-    if comment.save
-      flash[:notice] = "Your comment was added!"
-      redirect_to creation_path(creation)
-    else
-      flash[:error] = "Ooops... we couldn't save your comment at this time."
-      #render 'creations/new'
-    end
+    CommentOnCreationCommand.new(current_user).run(params)
+
+    flash[:notice] = params[:comment][:body]
+    redirect_to :controller => 'creations', :action => 'show', :id => params[:creation_id]
   end
 end
app/services/comment_on_creation_command.rb
@@ -0,0 +1,10 @@
+class CommentOnCreationCommand
+  def initialize(user)
+    @current_user = user
+  end
+  def run(params)
+    creation = Creation.find(params[:creation_id])
+    comment = Comment.build_from(creation, @current_user.id, params[:comment][:body])
+    comment.save
+  end
+end
config/application.rb
@@ -16,7 +16,8 @@ module Cake
     # -- all .rb files in that directory are automatically loaded.
 
     # Custom directories with classes and modules you want to be autoloadable.
-    # config.autoload_paths += %W(#{config.root}/extras)
+    #config.autoload_paths += %W(#{config.root}/app/services)
+    config.autoload_paths += %W(#{config.root}/app/services)
 
     # Only load the plugins named here, in the order given (default is alphabetical).
     # :all can be used as a placeholder for all plugins not explicitly named.
config/sphinx.yml
@@ -1,10 +0,0 @@
-development:
-
-test:
-
-staging:
-
-production:
-  enable_star: 1
-  min_infix_len: 1
-
spec/controllers/comments_controller_spec.rb
@@ -16,6 +16,9 @@ describe CommentsController do
     end
     it "should not display an errors" do
       @flash_hash[:error].should be_nil
+    end
+    it "should redirect to the creation#show page" do
+     
     end
     before(:each) do
       creation = fake
spec/spec_helper.rb
@@ -1,43 +1,16 @@
 require 'rubygems'
 require 'spork'
-#uncomment the following line to use spork with the debugger
-#require 'spork/ext/ruby-debug'
 
 Spork.prefork do
-  # Loading more in this block will cause your tests to run faster. However,
-  # if you change any configuration or code from libraries loaded here, you'll
-  # need to restart spork for it take effect.
-  # This file is copied to spec/ when you run 'rails generate rspec:install'
   ENV["RAILS_ENV"] ||= 'test'
   require File.expand_path("../../config/environment", __FILE__)
   require 'rspec/rails'
   require 'capybara/rspec'
 
-  # Requires supporting ruby files with custom matchers and macros, etc,
-  # in spec/support/ and its subdirectories.
   Dir[Rails.root.join("spec/support/**/*.rb")].each {|f| require f}
-
   RSpec.configure do |config|
-    # == Mock Framework
-    #
-    # If you prefer to use mocha, flexmock or RR, uncomment the appropriate line:
-    #
-    # config.mock_with :mocha
-    # config.mock_with :flexmock
-    # config.mock_with :rr
     config.mock_with :rspec
-
-    # Remove this line if you're not using ActiveRecord or ActiveRecord fixtures
-    # config.fixture_path = "#{::Rails.root}/spec/fixtures"
-
-    # If you're not using ActiveRecord, or you'd prefer not to run each of your
-    # examples within a transaction, remove the following line or assign false
-    # instead of true.
     config.use_transactional_fixtures = true
-
-    # If true, the base class of anonymous controllers will be inferred
-    # automatically. This will be the default behavior in future versions of
-    # rspec-rails.
     config.infer_base_class_for_anonymous_controllers = false
   end
 end
@@ -47,32 +20,3 @@ Spork.each_run do
   Devise.stretches = 1
   Rails.logger.level = 4
 end
-
-# --- Instructions ---
-# Sort the contents of this file into a Spork.prefork and a Spork.each_run
-# block.
-#
-# The Spork.prefork block is run only once when the spork server is started.
-# You typically want to place most of your (slow) initializer code in here, in
-# particular, require'ing any 3rd-party gems that you don't normally modify
-# during development.
-#
-# The Spork.each_run block is run each time you run your specs.  In case you
-# need to load files that tend to change during development, require them here.
-# With Rails, your application modules are loaded automatically, so sometimes
-# this block can remain empty.
-#
-# Note: You can modify files loaded *from* the Spork.each_run block without
-# restarting the spork server.  However, this file itself will not be reloaded,
-# so if you change any of the code inside the each_run block, you still need to
-# restart the server.  In general, if you have non-trivial code in this file,
-# it's advisable to move it into a separate file so you can easily edit it
-# without restarting spork.  (For example, with RSpec, you could move
-# non-trivial code into a file spec/support/my_helper.rb, making sure that the
-# spec/support/* files are require'd from inside the each_run block.)
-#
-# Any code that is left outside the two blocks will be run during preforking
-# *and* during each_run -- that's probably not what you want.
-#
-# These instructions should self-destruct in 10 seconds.  If they don't, feel
-# free to delete them.