"CẢNH BÁO: Không thể gán số lượng lớn các thuộc tính được bảo vệ"

Question

Tôi đã sử dụng các kỹ thuật RESTful để tạo mô hình (trên thực tế, tôi đang sử dụng đá quý Devise, điều đó cho tôi) và tôi đã thêm các trường mới gọi là first_name và last_name cho mô hình. Di chuyển đã ổn. Tôi đã thêm attr_accessor: first_name,: last_name vào mô hình và dự kiến ​​nó sẽ hoạt động. Nhưng khi tôi cố gắng đại diện cho các trường hợp mới với Doctor.create ({: first_name => "MyName"}) vv, tôi nhận được lỗi nói rằng tôi không thể gán các thuộc tính được bảo vệ theo khối lượng.

Tôi nghĩ rằng toàn bộ điểm sử dụng attr_accessor là để có được xung quanh sự bảo vệ của các lĩnh vực của một mô hình. Bạn có thể giúp tôi hiểu được thông điệp này không?

Chỉnh sửa: oh, và nhân tiện các bản ghi cũng không được tạo. Tôi nghĩ rằng họ nên được vì đây chỉ là một cảnh báo, nhưng họ không phải trên cơ sở dữ liệu.

Edit2: đây là mô hình của tôi

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 

và giản đồ, mà không có first_name và last_name vì họ được tạo ra trong bảng người dùng, đó là tổ tiên của các bác sĩ. Tôi đã sử dụng thừa kế bảng đơn.

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

    t.timestamps 
end 

và đây là sự di cư để thay đổi bảng người dùng

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

EDIT: đây là file giống. Tôi không bao gồm phương thức truncate_db_table, nhưng nó hoạt động.

%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

Đừng nhầm lẫn attr_accessor với attr_accessible. Accessor được xây dựng trong Ruby và định nghĩa một phương thức getter - model_instance.foo # returns something - và một phương thức setter - model_instance.foo = 'bar'.

Quyền truy cập được xác định bởi Rails và làm cho thuộc tính có thể chuyển nhượng hàng loạt (ngược lại với attr_protected).

Nếu first_name là một trường trong bảng cơ sở dữ liệu của mô hình của bạn, thì Rails đã xác định getters và setters cho thuộc tính đó. Tất cả những gì bạn cần làm là thêm attr_accessible :first_name.

Answer 2

Không sử dụng attr_accessor tại đây. ActiveRecord tự động tạo các mô hình đó trên mô hình. Ngoài ra, ActiveRecord sẽ không tạo bản ghi nếu lỗi xác thực hoặc gán khối lượng được ném.

EDIT: Bạn không cần bảng bác sĩ, bạn cần bảng người dùng có cột loại để xử lý Rails Single Table Inheritance. Lời mời sẽ nằm trên bảng người dùng. Ah, tôi thấy trong mẫu mã đã thêm của bạn, bạn có loại trên người dùng. Loại bỏ bảng bác sĩ, chuyển lời mời đến người dùng và tôi nghĩ bạn nên ổn. Cũng loại bỏ attr_accessor. Không cần thiết.

Hãy nhớ rằng đường ray STI sử dụng cùng một bảng cho tất cả các lớp và lớp con của một mô hình cụ thể. Tất cả hồ sơ Bác sĩ của bạn sẽ là các hàng trong bảng người dùng có loại 'doctor'

EDIT: Ngoài ra, bạn có chắc chắn chỉ muốn xác thực sự hiện diện của lời mời khi tạo và không cập nhật không?

Answer 3

Để hack ứng dụng của bạn với nhau theo cách không an toàn hoàn toàn không phù hợp với chế độ sản xuất:

Đi tới/config/application.rb Cuộn xuống phía cuối nơi bạn sẽ tìm thấy

{config.active_record.whitelist_attributes = true} 

Đặt thành false.

EDIT/btw (sau 4 tháng làm việc ruby ​​chuyên sâu bao gồm một hội thảo 11 tuần): DHH tin rằng, cho noobies (lời nói của ông), "và chạy" quan trọng hơn "rất an toàn" .

ĐƯỢC TƯ VẤN: Mặc dù câu trả lời này (đầu tiên tôi dựa trên lưu lượng truy cập mà tôi nghĩ) hiện ở mức +6, thấp đến -4 trong lịch sử của nó, ý nghĩa của điều đó là rất nhiều nhà phát triển có kinh nghiệm cảm thấy rất đam mê về không phải muốn bạn làm điều này.

UPDATE: 3 năm sau, một cách khác để làm điều này - một lần nữa, không an toàn, nhưng tốt hơn so với các giải pháp trên có lẽ là bởi vì bạn phải làm điều đó cho mỗi mô hình

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

Answer 4

Thêm attr_accessible : variable1, variable2 để bạn tệp tuyến đường bảng.

Answer 5

Đồng ý với câu trả lời @Robert Speicher Nhưng tôi thực sự khuyên bạn nên sử dụng Strong parameter thay vì attr_accessible để bảo vệ khỏi việc sắp xếp hàng loạt.

Chúc mừng!