Rails 4 вложенные формы ссылка для редактирования не работает в цикле

У меня есть вложенные маршруты / модели / формы в rails. На моей странице индекса я перечисляю todo_lists с todo_items внизу. Я хочу иметь возможность нажать на заголовок списка задач, а затем перейти на страницу редактирования. Я исследую полиморфные маршруты и вложенные маршруты.

Обновить

Это было мое решение, чтобы остановить его от создания фиктивного списка задач. 
<%  @current_todo_lists.each do |list| %>
    <% if list.id %>
        <div class="panel">
            <p><strong><%= link_to list.title ,edit_todo_list_path(list)%></strong></p>
            <% list.todo_items.each do |todo_item| %>
                <p><%= todo_item.description %></p>
            <% end %>
        </div>
    <% end %>
<% end %>

GitHub link

Link_to rails вложенная форма edit

Polymorphic_path не генерирует правильный путь Я провел много исследований в поисках cocoon, направляющих рельсов на полиморфных маршрутах и нескольких других звеньев stackoverflow.

Я не преуспел ни в одной из этих работ.

Вот страница индекса, где перечислены все мои todo_lists с todo_items. Он проходит через цикл, чтобы перечислить каждый список задач с соответствующими элементами, созданными с помощью это

Обновление:

Я уже пробовал <%= link_to list.title, edit_todo_list_path(list) %> и <%= link_to list.title, edit_todo_list_path(@list) %>. Сообщение об ошибке, которое я получаю:

ActionController::UrlGenerationError at /todo_lists No route matches {:action=>"edit", :controller=>"todo_lists", :id=>nil} missing required keys: [:id] Эта же конфигурация с @todo_list дает ту же ошибку. В принципе, он не может найти список задач с идентификатором.

В консоли действительно дает мне результат. Значит, я что-то упускаю. 

>> t = TodoList.find(1)
=> #<TodoList id: 1, title: "First todo List with a modal", created_at: "2014-09-09 23:02:27", updated_at: "2014-09-09 23:02:27", user_id: 1>
>>

Обновление 2: именно здесь происходит ошибка в контроллере списка задач. Он не может найти без удостоверения личности. 

def set_todo_list
@todo_list = TodoList.find(params[:id])
end


<% @todo_lists.each do |list| %> 
    <p><strong><%= link_to list.title, edit_polymorphic_path(@todo_list) %></strong></p>
    <% list.todo_items.each do |todo_item| %>
        <p><%= todo_item.description %></p>
    <% end %>
<% end %>

Пока что <p><strong><%= link_to list.title, edit_polymorphic_path(@todo_list) % параметры были @todo_list (s), @ todo_lists(s), todo_items и так далее.

Модели: 

class TodoList < ActiveRecord::Base
    has_many :todo_items, dependent: :destroy
    accepts_nested_attributes_for :todo_items, allow_destroy: true
    validates_presence_of :title
end

class TodoItem < ActiveRecord::Base
    belongs_to :todo_list
end

Контроллеры: 

Class TodoListsController < ApplicationController
  before_filter :authenticate_user!
  before_filter except: [:index]
  before_action :set_todo_list, only: [:show, :edit, :update, :destroy]

  # GET /todo_lists
  # GET /todo_lists.json
  def index
    #@todo_lists = TodoList.all
    #find current user todo lists/items
    @todo_lists = current_user.todo_lists
    @todo_items = current_user.todo_items
    #create a new user todo list
    @todo_list = current_user.todo_lists.new
    # builder for todo list _form
    3.times{ @todo_list.todo_items.build }
  end

  # GET /todo_lists/1
  # GET /todo_lists/1.json
  def show
  end

  # # GET /todo_lists/new
  def new
    @todo_list = current_user.todo_lists.new
    3.times{ @todo_list.todo_items.build }
  end

  # GET /todo_lists/1/edit
  def edit
    #@todo_list = TodoList.find(todo_list_params)
    @todo_list = TodoList.find(params[:id])
  end

  # POST /todo_lists
  # POST /todo_lists.json
  def create
    #@todo_list = TodoList.new(todo_list_params)
    @todo_list = current_user.todo_lists.new(todo_list_params)
    respond_to do |format|
      if @todo_list.save
        format.html { redirect_to @todo_list, notice: 'Todo list was successfully created.' }
        format.json { render :show, status: :created, location: @todo_list }
      else
        format.html { render :new }
        format.json { render json: @todo_list.errors, status: :unprocessable_entity }
      end
    end
  end

  # PATCH/PUT /todo_lists/1
  # PATCH/PUT /todo_lists/1.json
  def update
    @todo_list = TodoList.find(params[:id])
    respond_to do |format|
      if @todo_list.update(todo_list_params)
        format.html { redirect_to @todo_list, notice: 'Todo list was successfully updated.' }
        format.json { render :show, status: :ok, location: @todo_list }
      else
        format.html { render :edit }
        format.json { render json: @todo_list.errors, status: :unprocessable_entity }
      end
    end
  end

  # DELETE /todo_lists/1
  # DELETE /todo_lists/1.json
  def destroy
    #@todo_list.TodoList.find(params[:id])
    @todo_list.destroy
    respond_to do |format|
      format.html { redirect_to todo_lists_url, notice: 'Todo list was successfully destroyed.' }
      format.json { head :no_content }
    end
  end

  private
    def owns_todolist
      if current_user != TodoList.find(params[:id]).user
        redirect_to todo_lists_path, error: "You can't do that!"
      end
    end
    # Use callbacks to share common setup or constraints between actions.
    def set_todo_list
      @todo_list = TodoList.find(params[:id])
    end

    # Never trust parameters from the scary internet, only allow the white list through.
    def todo_list_params
      params.require(:todo_list).permit(:title, todo_items_attributes: [:description, :_destroy])
    end
end



class TodoItemsController < ApplicationController
  before_action :set_todo_item, only: [:show, :edit, :update, :destroy]
  before_action :set_todo_list

  # GET /todo_items
  # GET /todo_items.json
  def index
    @todo_items = TodoItem.all
  end

  # GET /todo_items/1
  # GET /todo_items/1.json
  def show
    @todo_item = TodoItem.find(params[:id])
  end

  # GET /todo_items/new
  def new
    @todo_item = @todo_list.todo_items.build
  end

  # GET /todo_items/1/edit
  def edit
    @todo_item = TodoItem.find(params[:id])
  end

  # POST /todo_items
  # POST /todo_items.json
  def create
    @todo_item = @todo_list.todo_items.build(todo_item_params)

    respond_to do |format|
      if @todo_item.save
        format.html { redirect_to [@todo_list,@todo_item], notice: 'Todo item was successfully created.' }
        format.json { render :show, status: :created, location: @todo_item }
      else
        format.html { render :new }
        format.json { render json: @todo_item.errors, status: :unprocessable_entity }
      end
    end
  end

  # PATCH/PUT /todo_items/1
  # PATCH/PUT /todo_items/1.json
  def update
    @todo_item = TodoItem.find(params[:id])
    respond_to do |format|
      if @todo_item.update(todo_item_params)
        format.html { redirect_to @todo_item, notice: 'Todo item was successfully updated.' }
        format.json { render :show, status: :ok, location: @todo_item }
      else
        format.html { render :edit }
        format.json { render json: @todo_item.errors, status: :unprocessable_entity }
      end
    end
  end

  # DELETE /todo_items/1
  # DELETE /todo_items/1.json
  def destroy
    @todo_item = TodoItem.find(params[:id])
    @todo_item.destroy
    respond_to do |format|
      format.html { redirect_to todo_list_todo_items_url, notice: 'Todo item was successfully destroyed.' }
      format.json { head :no_content }
    end
  end

  private
    # Use callbacks to share common setup or constraints between actions.
    def set_todo_item
      @todo_item = TodoItem.find(params[:id])
    end

    def set_todo_list
      @todo_list = TodoList.find(params[:todo_list_id])
    end

    # Never trust parameters from the scary internet, only allow the white list through.
    def todo_item_params
      params.require(:todo_item).permit(:description, :text, :todo_list_id)
    end
end

И, наконец, форма. Прямо сейчас он позволяет добавлять todo_list и несколько todo_items просто как практика. Я планирую использовать некоторые Ajax для обеспечения динамического создания позже. И иметь другую форму для редактирования. 

<%= form_for(@todo_list) do |f| %>
  <% if @todo_list.errors.any? %>
    <div id="error_explanation">
      <h2><%= pluralize(@todo_list.errors.count, "error") %> prohibited this todo_list from being saved:</h2>

      <ul>
      <% @todo_list.errors.full_messages.each do |message| %>
        <li><%= message %></li>
      <% end %>
      </ul>
    </div>
  <% end %>
  <div class="field">
    <%= f.label :title %><br>
    <%= f.text_field :title %>
  </div>
  <div>
    <%= f.fields_for :todo_items do |builder| %>
      <%= builder.label :description, "Items" %>
      <%= builder.text_field :description %>
      <%= builder.check_box '_destroy' %>
    <% end %>

  </div>
  <div class="actions">
    <%= f.submit %>
  </div>
<% end %>
2 3
ruby-on-rails-4 link-to crud nested-forms

2 ответа:

Я думаю, что проблема на самом деле немного сложнее, потому что edit_todo_list_path(list), кажется, выдает ту же ошибку.

Происходит то, что переменная @todo_lists (Сначала массив сохраненных списков) изменяется при запуске @todo_list = current_user.todo_lists.new. Эта команда фактически добавляет новый (неперсонифицированный) список в конец массива @todo_lists (Мне кажется, что поведение багги, но это случалось со мной раньше), так что когда ваше представление циклически проходит через них, последний не имеет идентификатора, и путь для него не может быть создан.

В решение (я думаю) состоит в том, чтобы сделать эту переменную после того, как вы использовали переменную @todo_lists.

Возьмите @todo_list из контроллера и там, где вы используете его в представлении, вместо этого сделайте current_user.todo_lists.build. Это должно создать экземпляр нового списка без изменения переменной @todo_lists.

2

Вы пытаетесь создать ссылку на ресурс, который еще не был сохранен: 

#create a new user todo list
@todo_list = current_user.todo_lists.new
Поскольку вы никогда не вызываете @todo_list.save, запись не имеет идентификатора и не может быть перенаправлена.

Я думаю, что вы пытаетесь сделать следующее: 

<% @todo_lists.each do |list| %> 
  <p><strong><%= link_to list.title, edit_polymorphic_path(id: list.to_param) #not @todo_list! %></strong></p>
  <% list.todo_items.each do |todo_item| %>
    <p><%= todo_item.description %></p>
  <% end %>
<% end %>

И я бы серьезно подумал о переименовании @todo_list -> @new_todo_list так как это довольно запутанно, как есть.

Обновление

" я даже могу нажать кнопку, чтобы открыть модальный, который создаст новый список задач все с той же страницы."

Что вы должны сделать, так это попросить пользователя создать новый ресурс, разместив форму с запросом AJAX POST в /todo_list, а затем обновить страницу с новым списком (и ссылкой редактирования).

Сейчас вы создаете новый todo_list, который существует только в памяти сервера до того, как rails завершит обработку запроса.

2