"OSTRZEŻENIE: Nie można masowo przypisywać chronionych atrybutów"

Question

Użyłem technik RESTful do wygenerowania modelu (w rzeczywistości używam Devise gem, co robi to dla mnie), i dodałem nowe pola o nazwie first_name i last_name do modelu. Migracja poszła dobrze. Dodałem attr_accessor: first_name,: last_name do modelu i oczekiwałem, że po prostu zadziała. Ale kiedy próbuję masowo przypisać nowe wystąpienia za pomocą Doctor.create ({: first_name => "MyName"}) itp., Dostaję błędy mówiąc, że nie mogę masowo przypisywać chronionych atrybutów.

Uważam, że cały sens korzystania z attr_accessor było obejście ochrony pól modelu. Czy możesz mi pomóc zrozumieć tę wiadomość?

Edycja: oh, a przy okazji także rekordy nie są tworzone. Myślałem, że powinny być, ponieważ to tylko ostrzeżenie, ale nie ma ich w bazie danych.

Edit2: tu jest mój model

class Doctor < User 
    has_many :patients 
    has_many :prescriptions, :through=> :patients 

    validates_presence_of :invitations, :on => :create, :message => "can't be blank" 

    attr_accessor :invitations 
end 

i schemat, który nie ma first_name i last_name ponieważ są one tworzone w tabeli użytkowników, który jest przodkiem lekarzy. Użyłem dziedziczenia pojedynczego stołu.

create_table :doctors do |t| 
    t.integer :invitations 

    t.timestamps 
end 

i to jest migracja do zmiany tabeli użytkowników

add_column :users, :first_name, :string 
add_column :users, :last_name, :string 
add_column :users, :type, :string 

EDIT: tutaj jest plik nasion. Nie włączam metody truncate_db_table, ale działa.

%w{doctors patients}.each do |m| 
    truncate_db_table(m) 
end 

Doctor.create(:invitations=>5, :email=>"email@gmail.com", :first_name=>"Name", :last_name=>"LastName") 
Patient.create(:doctor_id=>1, :gender=>"male", :date_of_birth=>"1991-02-24") 

Answer 1

Nie mylić attr_accessor z attr_accessible. Accessor jest wbudowany w Ruby i definiuje metodę pobierającą - model_instance.foo # returns something - oraz metodę ustawiającą - model_instance.foo = 'bar'.

Dostęp jest definiowany przez Railsy i powoduje, że atrybut jest przypisywany masowo (przeciwnie niż attr_protected).

Jeśli first_name jest polem w tabeli bazy danych modelu, to Railsy już zdefiniowały obiekty pobierające i ustawiające dla tego atrybutu. Wszystko, co musisz zrobić, to dodać attr_accessible :first_name.

Answer 2

Nie używaj tutaj attr_accessor. ActiveRecord tworzy je automatycznie na modelu. Ponadto, ActiveRecord nie utworzy rekordu, jeśli zgłoszony zostanie błąd sprawdzania poprawności lub przypisania masy.

EDYCJA: Nie potrzebujesz tabeli lekarzy, potrzebujesz tabeli użytkowników z kolumną typu do obsługi Railsów Single Table Inheritance. Zaproszenia będą w tabeli użytkowników. Ach, widzę w twoim dodanym przykładowym kodzie, że masz typ na użytkownikach. Pozbądź się tabeli lekarzy, przenieś zaproszenia do użytkowników i myślę, że powinieneś być w porządku. Pozbądź się także attr_accessor. Nie są potrzebne.

Należy pamiętać, że STI korzysta z tej samej tabeli dla wszystkich klas i podklas określonego modelu. Wszystkie rekordy lekarza będą wiersze w tabeli użytkowników z typem "lekarza"

EDYCJA: Czy na pewno chcesz potwierdzić obecność zaproszeń przy tworzeniu, a nie aktualizacje?

Answer 3

włamać swoją aplikację razem w niebezpieczny sposób całkowicie nienadające się do trybie produkcyjnym:

do/config/aplikacji.rb Przewiń w dół do końca, gdzie znajdziesz:

{config.active_record.whitelist_attributes = true} 

Ustaw na false.

EDIT/btw (po 4 miesiącach ruby-intensywnej pracy w tym warsztacie 11 tygodniu): DHH uważa, że ​​dla Noobies (jego słowa), „gotowy do pracy” jest ważniejsze niż „bardzo bezpieczna” .

ZOSTAĆ ZADOWOLONY: Chociaż ta odpowiedź (mój pierwszy na stackoverflow, jak sądzę) jest teraz na +6, to było tak niskie, jak -4 w swojej historii, znaczenie tego było dużo doświadczonych deweloperów szyny czuł namiętny o nie chcąc, abyś to zrobił.

UPDATE: 3 lata później inny sposób to zrobić - ponownie, nie jest bezpieczne, ale lepsze niż powyższego roztworu prawdopodobnie dlatego, że trzeba to zrobić dla każdego modelu

class ModelName < ActiveRecord::Base 
    column_names.each do |col| 
    attr_accessible col.to_sym 
    end 
    ... 
end 

Answer 4

Dodaj do listy attr_accessible : variable1, variable2 plik trasy tabeli.

Answer 5

Zgadzam się z odpowiedzią @Robert Speicher Ale zdecydowanie zalecam użycie Strong parameter zamiast attr_accessible w celu ochrony przed wyrównaniem masy.

Pozdrawiam!