I have an event model and a Q+A model inside the event.
I'm new to rails so not sure if there is an alternate way to do this
Right now, inside an event show.html.erb, I have both a form to post a question, and each question will have a form to post an answer
The routes sit like this right now
resources :events do
resources :event_questions, only: [:create, :destroy]
end
When you create a question inside my show.html.erb, it's being routed through EventQuestionsController and I access the event_id via params. Do I have to do it the same way with my event_answer?
And by that I mean, do I have to nest event_answers inside event_questions. I will need to know the event_id and the event_question_id.
If that's the only way to access the params. Would it look like this?
resources :events do
resources :event_questions, only: [:create, :destroy] do
resources :event_answers, only: [:create, :destroy]
end
end
event_questions_controller.rb
def create
#event_question = EventQuestion.new(event_question_params)
if #event_question.save
event = Event.find(params[:event_id])
#event_question.event = event
redirect_to event
else
redirect_to :back
end
end
show.html.erb
<%= form_for(#event_question, :url => event_event_questions_path(#event)) do |f| %>
# form stuff
<% end %>
I started changing my answer form to this
<%= #event.event_questions.each do |q| %>
<%= q.question %>
<%= q.fields_for :answers do |a| %>
<%= a.label :answer %>
<%= a.text_field :answer %>
<%= a.submit "Answer" %>
<% end %>
<% end %>
You shouldn't need to directly access the params to create your nested records. For example, if in your event#new action, you may have something like
#event = Event.new
3.times do
question = #event.questions.build
5.times do
question.answers.build
end
end
This will create a new Event and take you to the New page. Within your New page, you should have nested fields to create and update the nested models.
In the form, you can use fields_for to access the associated model's records to that event record. In the new action case, we are building 3 questions and each question has 5 answers.
<%= simple_form_for #event do |f| %>
<%= f.input :attribute1 %>
<%= f.input :attribute2 %>
<%= f.fields_for :questions do |builder| %>
<%= render 'questions', f: builder %>
<% end%>
<% end %>
I generally like to use a partial when doing nested associations, but if you wanted to keep it all in one file (see below), it would looks something like this.
<%= simple_form_for #event do |f| %>
<%= f.input :attribute1 %>
<%= f.input :attribute2 %>
<%= f.fields_for :questions do |q| %>
<%= q.input :question_text %>
<%= q.fields_for :answers do |a| %>
<%= a.input :answer_text %>
<% end %>
<% end%>
<% end %>
In your show action, you may have something like this
#event = Event.find(params[:id])
#questions = #event.questions
#answers = #question.answers if #questions
From here, you have three instance variables assigned and able to access the associated records to your events and questions.
With some help I derived this answer from a bit of kobaltz answer and some other help.
The alternate method would be putting the information in hidden fields and passing it to the controller
<%= #event.event_questions.each do |q| %>
<%= q.question %><br>
<% q.event_answers.each do |a| %>
<%= a.answer %>
<% end %>
<%= form_for(EventAnswer.new) do |f| %>
<%= hidden_field_tag :event_question_id, q.id %>
<%= hidden_field_tag :event_id, #event.id %>
<%= f.label :answer %>
<%= f.text_field :answer %>
<%= f.submit "Answer" %>
<% end %>
<% end %>
Related
invites_controller.rb
def index
#invites = Invites.all
end
invite.rb
has_one :answer
answers_controller.rb
def new
#invite = Invite.find(params[:invite_id])
#answer = #invite.build_answer
end
def create
#invite = Invite.find(params[:invite_id])
#answer = #invite.create_answer
redirect_to :back
end
answer.rb
belongs_to :invite
routes.rb
resources :invites, only: [:index] do
resources :answers, only: [:new, :create]
end
Is it possible to create invites/index.html where for each line-item will be exist working form for answer without any javascript?
With some logic like this:
<% #invites.each do |i| %>
<%= form_for(i, i.build_answer) do |f| %>
<%= f.hidden_field :accept, value: true %>
<%= f.submit "accept" %>
<% end %>
<% end %>
What are the routing and the rendering of forms will be?
ps: I can solve my problem by creating accept column at Invite model and simply updating each line item like this
<% #invites.each do |i| %>
<%= form_for(i) do |f| %>
<%= f.hidden_field :accept, value: true %>
<%= f.submit "accept" %>
<% end %>
<% end %>
but interesting about other way))
Yes, you can achieve it as
<% #invites.each do |i| %>
<%= form_for([i,i.answer.build]) do |f| %>
<%= f.text_field :name_of_you_answer_field %>
<%= f.submit %>
<% end %>
<% end %>
I want to create a form for "Comments" route which is a member of Article Resources:
resources :articles do
member do
post 'comments'
end
end
I want the comment form to be in Articles#Show page. The problem i got an error:
First argument in form cannot contain nil or be empty
If the for is like this:
<div>
<%= form_for #comm do |c| %>
<%= c.label :Your_comment %>
<%= c.text_area :commBody %>
<%= c.submit 'submit' %>
<% end %>
</div>
So how to do it ?
If this is your controller,
def show
#article = Article.find(params[:id])
end
and you want to create a form for a new Comment related to #article that points to POST /articles/3/comments:
<%= form_for([#article, Comment.new], as: :article, url: comments_article_path(#article)) do |f| %>
<%= f.label :body %>
<%= f.text_area :body %>
<%= f.submit 'Submit' %>
<% end %>
Don't forget to add accepts_nested_attributes_for :comments in the Article model. And also don't forget to setup the whitelisted params in the ArticleController.
Another thing: don't use abbreviations for your variable names. Use #article and #comment, not #art and #comm.
#config/routes.rb
resources :articles do
post :comment, on: :member #-> url.com/articles/:id/comment
end
#app/controllers/articles_controller.rb
class ArticlesController < ApplicationController
def show
#article = Article.find params[:id]
#comment = #article.comments.new
end
end
#app/views/articles/show.html.erb
Comment:
<%= form_for [#article, #comment], url: article_comment_path(#article) do |c| %>
<%= c.label "Your Comment" %>
<%= c.text_area :commBody %>
<%= c.submit %>
<% end %>
Hi I have a working "blog module" on my page.
From the start. My routes looks like this:
root 'static_pages#home'
resources :announcements, only: [:new,:create,:update,:edit,:destroy,:show] do
resources :comments, only: [:create, :destroy]
end
My controller actions looks like this:
static_pages_controllers.rb
def home
#announcements = Announcement.page(params[:page]).order('id DESC')
end
announcements_controller.rb
def new
#announcement = Announcement.new
end
def create
#announcement = Announcement.new(announcement_params)
if #announcement.save
flash[:success] = "Post został pomyślnie dodany"
redirect_to root_path
else
render 'new'
end
end
comments_controller.rb
def create
#announcement = Announcement.find(params[:announcement_id])
#comment = #announcement.comments.new(comments_params)
if #comment.save
flash[:success] = "Komentarz dodano"
redirect_to root_path }
else
render 'staticpages#home'
end
end
Part of the home.html.erb which lists all anouncements and under each of them it adds a form to add a comment:
<% #announcements.each do |announcement| %>
.... some code ....
<% if current_user && user_signed_in? %>
<div class="create_comment" >
<%= render partial: 'shared/create_comments_form', locals: {announcement: announcement } %>
</div>
<% end %>
... some code ...
<% end %>
And _create_comment.html.erb :
<%= form_for announcement.comments.create, url: announcement_comments_path(announcement.id) do |form| %>
<%= form.label :author, "#{current_user.username}:" %>
<%= form.hidden_field :author, value: current_user.username %>
<%= form.hidden_field :user_id, value: current_user.id %>
<%= form.text_field :content, class: "form-control content_length" %>
<div class="char_counter">255</div>
<%= form.submit "Dodaj!", class: "btn btn-primary" %>
<% end %>
My question is there a better way of writing this form, because It just doesn`t look rails for me when I have to specify everywhere all those parameters. Eg. when I send parameters to _create_comment I have to pass announcement object, and in form it self I have to specify the url as it looks for create_comment_path by default.
I think you wanted just to handle nested resources, and need a form to create such a nested resource.
Then this should work:
form_for( [announcement, announcement.comments.new] ) do |form|
Do not use create in the form template, as it actually saves it to the database, you end with lots of empty comments just because somebody viewed the form.
You can use cokoon gem for this. This gem will create Dynamic nested forms using jQuery.
Also I have one more suggestion. Rather then using only you can use except. Because you have used all 6 routes without index.
root 'static_pages#home'
resources :announcements, except: [:index] do
resources :comments, only: [:create, :destroy]
end
Update:
You can use like below:
Form:
<%= form_for :comments do |form| %>
<%= form.label :author, "#{current_user.username}:" %>
<%= form.hidden_field :author, value: current_user.username %>
<%= form.hidden_field :user_id, value: current_user.id %>
<%= form.text_field :content, class: "form-control content_length" %>
<div class="char_counter">255</div>
<%= form.submit "Dodaj!", class: "btn btn-primary" %>
<% end %>
For more info: http://www.sitepoint.com/complex-rails-forms-with-nested-attributes/
I'm trying to make two forms for two different models in the same view.
I have a model named category and a model named post.
Im trying to make a form for categories in the same view i have a form for posts.
The form for posts works fine, but when i'm trying to add the form for categories i get this error:
undefined method `model_name' for Category::ActiveRecord_Relation:Class
category.rb - model
has_many :posts
post.rb - model
has_many :categories
posts_controller
def index
#posts = new.Post
#categories = new.Category
end
def create
#posts = Post.create(post_params)
#posts.save
redirect_to :back
end
def create_cate
#categories = Categroy.create(categories_params)
#categroies.save
redirect_to :back
end
posts view - index.html.erb
<%= form_for(#posts) do |f| %>
<%= f.text_field :title %>
<%= f.text_area :content %>
<%= f.submit %>
<% end %>
<%= form_for(#categories) do |f| %>
<%= f.text_field :name %>
<%= f.submit %>
<% end %>
routes.rb
resources :posts
resources :categories
root 'posts#index'
I have tried to search after if, but i can only find solutions for two models, one form.
Thanks in advance. :-)
Since you say its in the index action:
def index
#post = Post.new
#category = Category.new
end
In your view:
<%= form_for(#post) do |f| %>
<%= f.text_field :title %>
<%= f.text_area :content %>
<%= f.submit %>
<% end %>
<%= form_for(#category) do |f| %>
<%= f.text_field :name %>
<%= f.submit %>
<% end %>
i have some problems with my app, i have posts, posts has_many responces
when i create new responce to the post, not added in the db 'responce' post_id
my routes.rb
resources :categories do
resources :posts
end
resources :posts do
resources :responces
end
controller
class ResponcesController < ApplicationController
def new
#post = Post.find(params[:post_id])
#responce = #post.responces.new(post_id:params[:post_id])
end
def create
#responce = current_user.responces.build(responce_params)
#post = Post.find(params[:post_id])
if #responce.save
flash[:success] = "Вы откликнулись на задание"
redirect_to post_path #post
else
render 'new'
end
end
def show
end
private
def responce_params
params.require(:responce).permit(:price, :comment, :post_id)
end
end
view
<%= form_for([#post, #responce]) do |f| %>
<%= f.text_area :price %>
<%= f.submit "GO", class: "btn btn-large btn-primary" %>
<% end %>
but if add to the view this
<%= f.collection_select :post_id, Post.all, :id, :name %>
rails create post_id to the db
help
You are doing several things the wrong way.
First: I don't think you need two separate resources for the same model. I'd recomend nesting all three resources upon each other like this.
resource :categories do
resource :posts do
resource :responces
end
end
This way you'll be able to find the needed category_id and post_id in the params hash.
I'd also recomend adding :shalow => true to the :categories resource to make your routes a bit prettier.
Second: you need to assign the params[:post_id] in your create action, like this.
#responce = current_user.responces.build(responce_params)
#responce.post_id = params[:post_id]
#post = #responce.post
Alternatevely you can just add a hidden field to your form like I show below, but it I don't like that approach, 'cause it can lead to security risks.
<%= form_for([#post, #responce]) do |f| %>
<%= f.text_area :price %>
<%= f.hidden_field :post_id, :value => #post.id %>
<%= f.submit "GO", class: "btn btn-large btn-primary" %>
<% end %>
In your form you aren't passing in the post_id. You probably want something like this:
<%= form_for([#post, #responce]) do |f| %>
<%= f.text_area :price %>
<%= f.hidden_field :post_id, :value => #post.id %>
<%= f.submit "GO", class: "btn btn-large btn-primary" %>
<% end %>
The hidden field will pass the id of the current post into your form as the post_id parameter.