Schienen - Datensatz Hinzufügen, um zum Tisch vom controller

Ich versuche zu erstellen, die einen Datensatz innerhalb einer join-Tabelle aus der Aktion eine Schaltfläche. Ich hätte eine events-Modell und würde gerne verfolgen ausgewählte Ereignisse " von jedem Benutzer.

Benutzte ich die HABTM-Beziehung weil ich wirklich nicht brauchen extra-Felder.

Benutzer.rb:

has_to_and_belongs_to_many :events

Veranstaltung.rb:

has_to_and_belongs_to_many :users

Events_Users Migration:

[user_id, event_id, id=>false]

Ich bin immer fest auf die eigentliche Gestaltung der Aufnahme. Jemand half mir früher mit dem hinzufügen des Datensatzes in die Konsole:

u = User.find(1)
u.events << Event.find(1)

Nun möchte ich die Aktion durchführen, die als Ergebnis der Klick auf einen link... Ist das in die richtige Richtung?

def add
  @user = User.find(session[:user_id])
  @event = Event.find(params[:id])
  if @user.events.save(params[:user][:event])
    flash[:notice] = 'Event was saved.'
  end
end

Sollte ich hinzufügen, ein @user.events.new irgendwo und wenn ja wo muss ich die Parameter von dem Benutzer und welche Veranstaltung?

InformationsquelleAutor ChrisWesAllen | 2010-03-05
  1. 13

    Folgende code sollte funktionieren (vorausgesetzt, Sie übergeben einen parameter mit dem Namen id entspricht der id, die einem Ereignis-Objekt):

       def add
     @user = User.find(session[:user_id])
     @event = Event.find(params[:id])
     @user.events << @event
     flash[:notice] = 'Event was saved.'
   end

    Die Probleme sehe ich in deinem code sind:

    1. Übergeben Sie einen hash zu .speichern. Speichern sollte nur ein boolean Wert, ob die Validierungen ausgeführt werden soll, und ist standardmäßig true. Aber .erstellen und .neue annehmen können hash-Werte. (.speichern Sie die verwendet werden würde, nach .neue).

    2. Laden Sie ein Ereignis durch params[:id], aber dann Sie versuchen, erstellen Sie ein Ereignis durch params[:user][:event]. Was möchten Sie tun? Erstellen oder laden? (mein Beispiel wird davon ausgegangen Last)

    3. Aktionen, die einen Effekt haben wie dieser passieren soll, wenn ein Benutzer klickt auf einen button und ein Formular abschickt, anstatt einen Klick auf den link'. Dieser code wird möglicherweise anfällig für cross-site request forgery (Jemanden betrügen könnte jemand zum klicken auf einen link auf einer anderen Website, lief diese Aktion). Schienen bildet, wenn richtig umgesetzt, sind gegen dies, weil Sie verwenden eine request forgery protection token.

    4. Meisten wahrscheinlich, dass Sie möchten, leiten Sie den Benutzer nach dieser Aktion. Rendern von Seiten nach der Ausführung von Aktionen wie diese (eher als die Umleitung) wird als schlechte Praxis.

    • meinst du @user.events << @event.name???
    • Nein. Code würde keinen Sinn machen, wenn ich verstehe deine Beschreibung des Problems richtig.
    • Vielen Dank für die Anregungen, ich habe die "<%= link_to image_tag("grid_heart.gif", :border=>0 ), :controller => 'event', :action => 'add_event' %>" zu "<%= button_to "Hinzufügen", :controller => 'event', :action => "add" %>"aber ich bekomme immer noch eine Fehlermeldung, dass es ein "nicht initialisierte Konstante EventController" Habe ich die Methode in der falschen controller?
    • Code ist nicht korrekt, aber das ist nicht das, was die Ursache des Fehlers ist. Verwenden Sie die parameter :controller => 'Ereignis', das davon erzählt, Schienen sich für ein EventController. Rails ist Ihnen zu sagen, dass keine solche existiert in Ihrem Projekt.
    InformationsquelleAutor Gdeglin
  2. 3

    Was du getan hast in der Konsole, die Sie brauchen, um im controller.

    def add
  @user = User.find(session[:user_id])
  @event = Event.find(params[:id])
  @user.events << @event
  flash[:notice] = 'Event was saved.'
end

    Die Sache hier zu beachten ist, dass die << operator für vorhandene Datensätze werden bewirken, dass der Verein beibehalten werden sofort.

    Werfen Sie einen Blick auf die ActiveRecord-Dokumentation für mehr info.

    InformationsquelleAutor Randy Simon
  3. 0

    Wenn die event_id übergeben params[:id] und Sie sind das hinzufügen nur ein Ereignis in diesem rufen Sie dann Folgendes tun können in Ihrem controller-code:

      User.find(session[:user_id]).events << Event.find(params[:id])
  flash[:notice] = 'Event was saved.'

    Brauchen Sie nicht explizit save zum speichern der has_many-Assoziation von einem vorhandenen Modell-Instanz.

    Szenario 1

     u = User.new(..)
 u.events << Event.first
 # Now you need to call `save` in order to save the user object 
 # and the events association
 u.save

    Szenario 2

     u = User.first
 u.events << Event.first
 # Don't need to call `save` on `u` OR `u.events`
    • Im noch immer eine nicht initialisierte Konstante EventController eine Idee, wo könnte es herkommen?
    • Können Sie Ihre controller-code bei Pastie (pastie.org) und geben Sie einen link.
    • Ich kann, aber das ist die einzige Methode, die ich Hinzugefügt habe, die event_controller. Es gibt keine Fehlermeldung, wenn ich entfernen Sie die add-Methode, und der rest wurde der regler erzeugt aus einem Gerüst, so seine ziemlich grundlegende Dinge. Die aktuelle Methode sieht aus wie... def add @user = User.find(session[:user_id]) @event = Ereignis.find(params[:id]) @user.events << @event flash[:notice] = " Ereignis gespeichert wurde.' end
    • Sorry, ich bin nicht sicher, wie Sie Sie zum hinzufügen von code-snippets, um einen Kommentar auf stackoverflow
    • Ich denke das problem ist, Verwandte zu finden, die parameter des Ereignisses...ich;m nicht sicher, wie Sie Sie zu ziehen, dass die Daten
    InformationsquelleAutor Harish Shetty
Schreibe einen Kommentar