Rails 3 concevoir, current_user n'est pas accessible dans un Modèle ?

dans mon projet.rb modèle, je suis en train de créer un champ avec une variable dynamique:

scope :instanceprojects, lambda { 
    where("projects.instance_id = ?", current_user.instance_id)
}

J'obtiens l'erreur suivante: 

undefined local variable or method `current_user' for #<Class:0x102fe3af0>

Où dans le contrôleur, je peux accéder à current_user.instance_id... Est-il une raison pour laquelle le modèle ne pouvez pas y accéder et un moyen pour obtenir l'accès? Aussi, est-ce le bon endroit pour créer un champ d'application comme ci-dessus, ou n'appartenant dans le contrôleur?

InformationsquelleAutor AnApprentice | 2010-09-18
  1. 73

    Cela ne fait pas beaucoup de sens, comme vous l'avez déjà fait. Le current_user n'appartient pas au modèle de la logique, elle doit être traitée sur le contrôleur de niveau.

    Mais vous pouvez toujours créer portée comme ça, il suffit de passer le paramètre à partir du contrôleur:

    scope :instanceprojects, lambda { |user|
    where("projects.instance_id = ?", user.instance_id)
}

    Maintenant, vous pouvez appeler dans le contrôleur:

    Model.instanceprojects(current_user)
    • c'est génial. l'essayer maintenant
    • A très bien fonctionné. Merci!!!!
    • J'ai appelé le champ " de " de sorte qu'il serait bien lire. Project.of current_user
    • Je vais avoir un moment difficile de googler pour savoir pourquoi -> ne fonctionne pas (se plaint de la pipe). Je pensais que cela signifiait "lambda" mais je ne crois pas... (je suis un noob par rapport à ruby.)
    • à l'aide de -> les paramètres sont passés dans les choses différemment ->(user){ ... }
    InformationsquelleAutor mdrozdziel
  2. 31

    Déjà accepté de répondre fournit une très bonne façon d'atteindre cet.

    Mais voici la version thread-safe de User.current_user truc.

    class User
  class << self
    def current_user=(user)
      Thread.current[:current_user] = user
    end

    def current_user
      Thread.current[:current_user]
    end
  end
end

class ApplicationController
  before_filter :set_current_user

  def set_current_user
    User.current_user = current_user
  end
end

    Cela fonctionne comme prévu, mais il peut être considéré comme sale, parce que nous avons essentiellement de définir une variable globale ici.

    • Merci - j'ai eu une situation particulière où j'ai vraiment besoin de cela et n'avait aucun moyen de passer en. BTW, ce code a quelques erreurs, vous devez utiliser l'Utilisateur.current_user= méthode de ApplicationController (sinon, pourquoi l'avoir), et aussi qu'il est défini dans :current_user, mais le lecteur lit à partir de :utilisateur (m'a Pris de manière à bien voir ça!)
    • Merci pour cette remarque. Réponse de mise à jour.
    • J'ai essayé et fonctionne parfaitement. Toutefois, dans mon environnement de développement il n'y a pas de multi utilisateur de la situation. Je suis toujours préoccupé par les commentaires dans d'autres réponses sur le thread-safe question. Quelqu'un d'autre a testé ce dans un environnement de production? Cheers!
    InformationsquelleAutor Michał Szajbe
  3. 8

    Ryan Bates, établit un moyen assez sûr pour mettre en œuvre ce type de stratégie dans ce railscast

    Ce payée de l'épisode (à ne pas voter de moi!) mais vous pouvez parcourir le code source gratuitement

    Ici, il crée un current_tenant méthode, mais vous pouvez facilement remplacer current_user à la place.

    Voici les principaux morceaux de code...

    #application_controller.rb
around_filter :scope_current_tenant

private

def current_tenant
  Tenant.find_by_subdomain! request.subdomain
end
helper_method :current_tenant

def scope_current_tenant
  Tenant.current_id = current_tenant.id
  yield
ensure
  Tenant.current_id = nil
end

#models/tenant.rb

def self.current_id=(id)
  Thread.current[:tenant_id] = id
end

def self.current_id
  Thread.current[:tenant_id]
end

    Ensuite dans le modèle, vous pouvez faire quelque chose comme...

    default_scope { where(tenant_id: Tenant.current_id) }
    InformationsquelleAutor Mark Locklear
  4. 0

    Vous n'avez pas besoin d'utiliser des étendues. Si vous avez défini les associations dans les modèles, à la suite de morceau de code placé dans le contrôleur devrait faire l'affaire:

    @projects = current_user.instance.projects
    InformationsquelleAutor sampi