Mein Code überspringt die Ansicht und geht direkt in den Controller. Ruby on Rails

Question

Meine Ansicht

<h1>Return a Book</h1> 
<%= form_tag("/orders/returnB", method: "get") do %> 
    <div class="form-group"> 
    <%= label_tag :ID, "ID" %> 
    <%= text_field_tag :ID, nil, class: "form-control" %> 
    </div> 
    <%= submit_tag "Submit", class: "btn btn-default" %> 
<% end %> 

Mein Controller

def returnB 
    getid = params[:ID] 
    @order = Order.find(getid) 
    @order.destroy 
    respond_to do |format| 
    format.html { redirect_to orders_url, notice: 'Order was destroyed.' } 
    format.json { head :no_content } 
    end 
end 

sein Gehen, nur direkt an den Controller und als mein Controller nicht die ID bekommen konnte, so dass es einen Fehler geben. Also ich dies in meiner Route tue:

get 'orders/return' => 'orders#returnB'

Answer 1

In einem einfachen Schiene fließt die Anforderung zuerst den Controller trifft und dann die Ansicht Vorlage mit demselben Namen wie der Controller, wiedergegeben wird.

Ich glaube, Sie haben Ihre Form definiert in

view/orders/returnB.erb 

Deshalb ist die Anforderung, den Controller trifft zuerst, bevor Sie das Formular zu machen. Ich schlage vor, dass Sie das Formular in eine andere Ansichtsvorlage verschieben.

Answer 2

Ihre Routen ändern:

delete 'orders/return' => 'orders#return_order' 

löschen sollte zerstören verwendet werden.

ändern Form:

<h1>Return a Book</h1> 
<%= form_tag("/orders/return", method: "delete") do %> 
<div class="form-group"> 
    <%= label_tag :ID, "ID" %> 
    <%= text_field_tag :ID, nil, class: "form-control" %> 
</div> 
<%= submit_tag "Submit", class: "btn btn-default" %> 
<% end %> 

ändern Aktionsnamen returnB-return_order, da es nicht die richtige Konvention ist es, eine Methode Namen zu schreiben.

Change-Controller-Code:

def return_order 
    @order = Order.find_by_id(params[:ID]) 
    @order.destroy 
    respond_to do |format| 
    format.html { redirect_to orders_url, notice: 'Order was destroyed.' } 
    format.json { head :no_content } 
    end 
end