"ADVERTENCIA: No se puede asignar en masa los atributos protegidos"

Question

He utilizado técnicas RESTful para generar un modelo (de hecho, estoy usando Devise gem, que hace eso por mí), y he agregado nuevos campos llamados first_name y last_name para el modelo. La migración fue bien. Agregué attr_accessor: first_name,: last_name al modelo y esperaba que simplemente funcionara. Pero cuando trato de asignar en masa nuevas instancias con Doctor.create ({: first_name => "MyName"}) etc., recibo errores que dicen que no puedo asignar en masa los atributos protegidos.

Pensé que el objetivo de usar attr_accessor era evitar la protección de los campos de un modelo. ¿Puedes ayudarme a entender este mensaje?

Edición: oh, y por cierto los registros tampoco se crean. Pensé que deberían ser, ya que esto es solo una advertencia, pero no están en la base de datos.

Edit2: aquí está mi modelo

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 

y el esquema, que no tiene el nombre apellido y apellidos, ya que se crean en la tabla de usuarios, que es el antepasado de los médicos. Usé herencia de mesa única.

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

    t.timestamps 
end 

y esto es la migración de cambiar la tabla de usuarios

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

EDIT: Aquí está el archivo semilla. No incluyo el método truncate_db_table, pero funciona.

%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

No confunda attr_accessor con attr_accessible. Accessor está integrado en Ruby y define un método getter - model_instance.foo # returns something - y un método setter - model_instance.foo = 'bar'.

Accesible está definido por Rails y hace que el atributo se pueda asignar a la masa (hace lo contrario de attr_protected).

Si first_name es un campo en la tabla de la base de datos de su modelo, entonces Rails ya ha definido getters y setters para ese atributo. Todo lo que necesita hacer es agregar attr_accessible :first_name.

Answer 2

No utilice attr_accessor aquí. ActiveRecord los crea automáticamente en el modelo. Además, ActiveRecord no creará un registro si se produce un error de validación o asignación masiva.

EDITAR: No necesita una tabla de médicos, necesita una tabla de usuarios con una columna de tipo para manejar Rails Single Table Inheritance. Las invitaciones estarán en la tabla de usuarios. Ah, veo en tu ejemplo de código agregado que tienes que escribir en los usuarios. Deshágase de la mesa de médicos, envíe invitaciones a los usuarios y creo que debería estar bien. También deshazte del attr_accessor. Innecesario.

Tenga en cuenta que los raíles STI usan la misma tabla para todas las clases y subclases de un modelo en particular. Todos sus registros médicos serán filas en la tabla de usuarios con un tipo de 'doctor'

EDITAR: Además, ¿está seguro de que solo desea validar la presencia de invitaciones en la creación y no las actualizaciones?

Answer 3

piratear su aplicación en conjunto de una manera insegura totalmente inadecuada para el modo de producción:

Ir a/config/aplicación.rb Desplácese hacia abajo hacia el final donde encontrará

{config.active_record.whitelist_attributes = true} 

Establézcalo en falso.

EDITAR/BTW (después de 4 meses de trabajo rubí intenso, incluyendo un taller de 11 semanas): DHH cree que, para noobies (sus palabras), "en marcha y funcionando" es más importante que "muy seguro" .

SE ACONSEJA: Aunque esta respuesta (la primera en stackoverflow, creo) es ahora en +6, ha sido tan bajo como -4 en su historia, lo que significa que muchos desarrolladores de rails experimentados se sintieron muy apasionado por no que desee que haga esto.

ACTUALIZACIÓN: 3 años más tarde, otra manera de hacer esto - de nuevo, no es seguro, pero es mejor que la solución anterior, probablemente porque hay que hacerlo para cada modelo

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

Answer 4

Añadir a su attr_accessible : variable1, variable2 archivo de ruta de tabla.

Answer 5

De acuerdo con la respuesta de @Robert Speicher Pero recomiendo encarecidamente que debe utilizar Strong parameter en lugar de attr_accessible para protegerse de la asignación masiva.

¡Salud!