diff options
author | Rafael Mendonça França <rafaelmfranca@gmail.com> | 2013-07-07 08:26:11 -0700 |
---|---|---|
committer | Rafael Mendonça França <rafaelmfranca@gmail.com> | 2013-07-07 08:26:11 -0700 |
commit | a67cc28311b0ecb4282ca7d80b5ece959d22628f (patch) | |
tree | d4749f530b9a26531f96baa5f8cbb5bac0d0c72b /guides/code/getting_started/app | |
parent | 3e953d48a2db35969de8fb7ca3324b65b2e4b5fb (diff) | |
parent | 53f12b073cc238757431708e6730e60b10b4d5fe (diff) | |
download | rails-a67cc28311b0ecb4282ca7d80b5ece959d22628f.tar.gz rails-a67cc28311b0ecb4282ca7d80b5ece959d22628f.tar.bz2 rails-a67cc28311b0ecb4282ca7d80b5ece959d22628f.zip |
Merge pull request #11347 from ankit8898/refactoring-sp
Refactored strong parameters usage and updated Gemfile.lock with Rails 4
Diffstat (limited to 'guides/code/getting_started/app')
3 files changed, 16 insertions, 3 deletions
diff --git a/guides/code/getting_started/app/controllers/comments_controller.rb b/guides/code/getting_started/app/controllers/comments_controller.rb index 0e3d2a6dde..b2d9bcdf7f 100644 --- a/guides/code/getting_started/app/controllers/comments_controller.rb +++ b/guides/code/getting_started/app/controllers/comments_controller.rb @@ -4,7 +4,7 @@ class CommentsController < ApplicationController def create @post = Post.find(params[:post_id]) - @comment = @post.comments.create(params[:comment].permit(:commenter, :body)) + @comment = @post.comments.create(comment_params) redirect_to post_path(@post) end @@ -14,4 +14,10 @@ class CommentsController < ApplicationController @comment.destroy redirect_to post_path(@post) end + + private + + def comment_params + params.require(:comment).permit(:commenter, :body) + end end diff --git a/guides/code/getting_started/app/controllers/posts_controller.rb b/guides/code/getting_started/app/controllers/posts_controller.rb index 6aa1409170..02689ad67b 100644 --- a/guides/code/getting_started/app/controllers/posts_controller.rb +++ b/guides/code/getting_started/app/controllers/posts_controller.rb @@ -17,7 +17,7 @@ class PostsController < ApplicationController def update @post = Post.find(params[:id]) - if @post.update(params[:post].permit(:title, :text)) + if @post.update(post_params) redirect_to action: :show, id: @post.id else render 'edit' @@ -29,7 +29,7 @@ class PostsController < ApplicationController end def create - @post = Post.new(params[:post].permit(:title, :text)) + @post = Post.new(post_params) if @post.save redirect_to action: :show, id: @post.id @@ -44,4 +44,10 @@ class PostsController < ApplicationController redirect_to action: :index end + + private + + def post_params + params.require(:post).permit(:title, :text) + end end diff --git a/guides/code/getting_started/app/views/welcome/index.html.erb b/guides/code/getting_started/app/views/welcome/index.html.erb index 738e12d7dc..56be8dd3cc 100644 --- a/guides/code/getting_started/app/views/welcome/index.html.erb +++ b/guides/code/getting_started/app/views/welcome/index.html.erb @@ -1,3 +1,4 @@ <h1>Hello, Rails!</h1> <%= link_to "My Blog", controller: "posts" %> +<%= link_to "New Post", new_post_path %> |