RoR: Comment modifier le contrôleur pour accepter la mise à jour de plusieurs paramètres seulement?

Question

Voici le code généré par des rails:

def update 
    @user = User.find(params[:id]) 

    respond_to do |format| 
    if @user.update_attributes(params[:user]) 
     flash[:notice] = 'User was successfully updated.' 
     format.html { redirect_to(@user) } 
     format.xml { head :ok } 
    else 
     format.html { render :action => "edit" } 
     format.xml { render :xml => @user.errors, :status => :unprocessable_entity } 
    end 
    end 
end  

Mais je ne veux pas l'utilisateur de mettre à jour l'utilisateur tout, supposons que mon utilisateur a fname, lname et le sexe, au lieu de supprimer le sexe de la vue, Je veux limiter que la méthode de mise à jour accepte UNIQUEMENT fname et lname seulement, si il/elle veut mettre à jour le genre, je ne lui permettrai pas de le faire. Comment puis-je empêcher l'utilisateur de le faire? Je vous remercie.

Answer 1

Utilisez les paramètres Hash des update_attributes

@user = User.find(params[:id]) 
@user.update_attributes(:fname=>params[:user][:fname], :lname=>params[:user][:lname]) 

Answer 2

Vous pouvez delete attributs indésirables du param[:user] Hash:

# ... 
attributes = params[:user] 
gender = attributes.delete :gender 
raise SomeError unless gender.blank? 
if @user.update_attributes(attributes) 
    # ... 
end 
# ... 

Ce code supprime :gender du Hash et vérifie si elle est remplie si. donc, une exception est soulevée. Bien sûr, vous pouvez donner un avertissement gentil ou ignorer en silence le fait que le sexe a été rempli.

Answer 3

ou ajouter une méthode personnalisée @user.update_only(), ce qui le rend aussi plus facile à réutiliser dans des contextes différents ...

class User 
    def update_only(attrs = {}, *limit_to) 
    update_attributes(attrs.delete_if { |k,v| !limit_to.include?(k.to_sym) }) 
    end 
end 

Ensuite, faire quelque chose le long des lignes de

@user.update_only(params[:user], :fname, :lname) 

Answer 4

Il existe deux méthodes ActiveRecord qui viennent très pratique dans des cas comme ceux-ci, attr_protected et attr_accessible.

-vous les utiliser comme ceci:

class MyModel < ActiveRecord::Base 
    attr_accessible :fname, :lname #Allow mass-assignment 
    attr_protected :secret #Do not allow mass-assignment 
end 

model = MyModel.new(:fname => "Firstname", :lname => "Lastname", :secret => "haha") 
puts model.fname # "Firstname" 
puts model.lname # "Lastname" 
puts model.secret = nil # Can not be set through mass-assignment 
model.secret = "mysecret" # May only be assigned like this 
puts model.secret # "mysecret" 

Cependant, si vous avez besoin de cette fonctionnalité à un endroit, puis la solution de Salil fonctionnera tout aussi bien.

Une chose à noter est que vous devez utiliser attr_acessible aux attributs qui sont OK de whitelist à masse assign, et de faire tout autre attribut protégé. Ce faisant, vous empêchez les gens de mettre à jour des données qu'ils ne sont pas censés toucher.

Voir the docs pour plus d'informations.