Szyny - Pobieranie komunikatu o błędzie z modelu, który nie jest błędem sprawdzania poprawności

Question

Mam więc metodę w modelu rezerwacji o nazwie add_equip. Ta metoda sprawdza, czy dodany element wyposażenia jest ważny (nie koliduje z inną rezerwacją).

Kontrola działa. Jeśli dodany ekwipunek nie powinien być dodany, to nie jest, a jeśli tak, powinien.

Problem polega na tym, że nie mogę określić sposobu wysłania wiadomości z powrotem do kontrolera, aby umieścić go w wiadomości flash? Wiem, że muszę czegoś tutaj przegapić, ale mam google przez kilka godzin i nie mogę znaleźć żadnych jasnych wyjaśnień, jak przekazywać błędy z powrotem do kontrolera, chyba że są to błędy sprawdzania poprawności.

add_equip w reservations_controller

def add_equip 
    @reservation = Reservation.find(params[:id]) 
    @addedEquip = Equip.find(params[:equip_id]) 

    respond_to do |format| 
    if @reservation.add_equip(@addedEquip) 
     flash[:notice] = "Equipment was added" 
     format.html { redirect_to(edit_reservation_path(@reservation)) } 
    else 
     flash[:notice] = @reservation.errors 
     format.html { redirect_to(edit_reservation_path(@reservation)) } 
    end 
    end 
    end 

add_equip w rezerwacji modelu

def add_equip equip 
    if self.reserved.find_by_equip_id(equip.id) 
    self.errors.add_to_base("Equipment Already Added") 
    return false 
    elsif !equip.is_available?(self.start, self.end) 
    self.errors.add_to_base("Equipment Already Reserved") 
    return false 
    else 
    r = Reserved.new 
    r.reservation = self 
    r.equip = equip 
    r.save 
    end 
    end 

Każda pomoc będzie mile widziana. Wiem, że tu brakuje czegoś podstawowego.

Answer 1

Zapisanie komunikatu o błędzie wydaje się być dla mnie w porządku, po prostu trzeba dowiedzieć się, jak go wprowadzić do widoku.

Jak o:

flash[:notice] = @reservation.errors.full_messages.to_sentence 

Zakładając masz zamiar ponownie wyświetla formularz, można również prawdopodobnie użyć:

<%= f.error_messages %> 

ewentualnie:

<%= error_messages_for :reservation %> 

także , możesz użyć flash [: error], możesz go inaczej pokolorować za pomocą klasy CSS w widoku.

Answer 2

Myślę, że rozumiem, dlaczego błędy nie są przekazywane użytkownikowi.

Problem polega na wysyłaniu przekierowania do użytkownika, gdy operacja zakończy się niepowodzeniem, a nie tylko renderowaniem, co oznacza, że ​​utracisz wszystkie zmienne skonfigurowane do użycia w żądaniu. Zamiast dodawać błędy do lampy błyskowej, po prostu wyrenderuj stronę edycji i ustaw flash na normalny komunikat i wszystko powinno być w porządku.

Na przykład:

def add_equip 
    @reservation = Reservation.find(params[:id]) 
    @addedEquip = Equip.find(params[:equip_id]) 

    respond_to do |format| 
    if @reservation.add_equip(@addedEquip) 
     flash[:notice] = "Equipment was added" 
     format.html { redirect_to(edit_reservation_path(@reservation)) } 
    else 
     flash[:error] = 'Error adding equipment' 
     format.html { render :action => :edit } 
    end 
    end 
end 

Teraz można nadal używać zwykłych pomocników formularza do wyświetlania komunikatów o błędach.

Ponadto, tylko mała sugestia dotycząca kodu modelu, spróbuj użyć i18n, gdy to możliwe (w tym dla wiadomości flash w kontrolerze). Chociaż jest to przede wszystkim osobiste preferencje, daje logiczne miejsce do domu dla wszystkich wiadomości i określonego tekstu, a alos pozwala tworzyć ogólne lub domyślne komunikaty, które można zmienić w jednym miejscu zamiast duplikowania zmiany w wielu modelach i kontrolerach.

np.

def add_equip equip 
    if self.reserved.find_by_equip_id(equip.id) 
    self.errors.add_to_base(:already_added) 
    return false 
    elsif !equip.is_available?(self.start, self.end) 
    self.errors.add_to_base(:already_reserved) 
    return false 
    else 
    r = Reserved.new 
    r.reservation = self 
    r.equip = equip 
    r.save 
    end 
end