¿Cómo comprobar si existe un registro antes de crear uno nuevo en rails3?

Question

Esto es lo que estoy tratando de lograr:

• que tienen un sistema de etiquetado en su lugar.
• etiquetas se crean, cuando se crean mensajes (mensajes has_many: etiquetas,: a través de =>: tag_joins
• Una etiqueta de unión se crea automáticamente cuando se crea un post con las etiquetas.).

Quiero comprobar si la etiqueta ya existe. Si lo hace, quiero usar la etiqueta existente para el registro tag_join, en lugar de crear un nuevo registro de etiqueta.

Aquí está mi código actual, que no está funcionando.

class Tag < ActiveRecord :: Base 
    belongs_to :user 
    belongs_to :tag_join 
    belongs_to :post 

    before_create :check_exists 

    def check_exists 
    tag = Tag.where(:name => self.name, :user_id => current_user.id) 
    if tag.nil? 
     tag = Tag.create(:name => self.name, :user_id => current_user.id) 
    end 
    end 

end 

Esto no funciona sin embargo, estoy recibiendo un error en la creación de tareas ... (el servidor es en realidad el tiempo de espera - no recibo un error específico).

¿Alguna idea?

Tokland dijo que estaba creando un bucle infinito contando para crear la etiqueta de nuevo - por lo que intentó esto:

def check_exists 
     tag = Tag.find_by_name_and_user_id(:name => self.name, :user_id => current_user.id) 
     if tag != nil 
     self.id = tag.id 
     end 
    end 

y aún así obtener el tiempo de espera del servidor

Editar: No estoy seguro de si esto importa, pero la forma en que se agregan las etiquetas es similar a "http://railscasts.com/episodes/73-complex-forms-part-1

están anidados en el formulario de publicación y usan algo de esta manera:

def tag_attributes=(tag_attributes) 
    tag_attributes.each do |attributes| 
    tags.build(attributes) 
    end 
end 

Me pregunto si esto está impidiendo que todo funcione. Además, el uso current_user.id en el modelo sin duda parece ser un problema ...

EDIT:

Algo que he descubierto: esto tenía que cambiar, el formato que estábamos usando antes era incorrecto sintaxis: generalmente se usa para un método .where.

def check_exists 
    @tag = Tag.find_by_name_and_user_id(self.name, self.user_id) 
    if @tag != nil 
     #return false 
     #self=@tag 
    end 
    end 

El problema ahora es este, puedo saber si la etiqueta ya existe. ¿Pero entonces, qué? Si voy con la opción de devolución falsa, hay un error al crear el mensaje, y el registro de combinación no se crea ... La otra opción "self = @ tag" obviamente no funciona.

Answer 1

Le resultará difícil hacerlo desde el modelo Tag. Parece que lo que desea es actualizar el mensaje utilizando atributos anidados, así:

post = Post.create 
post.update_attributes(:tags_attributes=>{"0"=>{:name=>"fish",:user_id=>"37"}}) 

Esto es en realidad muy simple de hacer mediante el uso de un método atributo incubadora virtual:

class Post < AR::Base 
    has_many :tags 

    def tags_attributes=(hash) 
    hash.each do |sequence,tag_values| 
     tags << Tag.find_or_create_by_name_and_user_id(tag_values[:name],\ 
     tag_values[:user_id]) 
    end 
    end 

> post = Post.create 
> post.update_attributes(:tags_attributes=>{"0"=>{:name=>"fish",:user_id=>"37"}}) 
> Tag.count # => 1 
# updating again does not add dups 
> post.update_attributes(:tags_attributes=>{"0"=>{:name=>"fish",:user_id=>"37"}}) 
> Tag.count # => 1 

Answer 2

¿Sabía que hay una función find_by_or_create_by_ integrada en Rails, ¿verdad?

# No 'Summer' tag exists 
Tag.find_or_create_by_name("Summer") # equal to Tag.create(:name => "Summer") 

# Now the 'Summer' tag does exist 
Tag.find_or_create_by_name("Summer") # equal to Tag.find_by_name("Summer") 

http://api.rubyonrails.org/classes/ActiveRecord/Base.html (bajo buscadores dinámicos basados ​​en atributos)

Answer 3

Usted desea utilizar el método mágico find_or_create_by

def check_exists 
    tag = Tag.find_or_create_by_name_and_user_id(:name => self.name, :user_id => current_user.id) 
end 

Mira la ActiveRecord::Base documentos para obtener más información

Answer 4

probar esto

def check_exists 
    tag = Tag.where(:name => self.name, :user_id => current_user.id).first 
    tag = Tag.new({:name => self.name, :user_id => current_user.id}) unless tag 
    end 

uso Tag.new en lugar de Tag.create

Answer 5

La pregunta que originalmente pregunté puso bastante distorsionada por el final. Entonces lo estoy separando.

Las personas que están tratando de hacer lo que le pedía inicialmente puede intentar esto:

before_create :check_tag exists 

private 

def check_tag_exists 
    @tag = Tag.find_by_name_and_user_id(self.name, self.user_id) 
    if @tag != nil 
     # 
    end 
    end 

Esto le permitirá comprobar si ya se ha creado su registro. Cualquier lógica adicional que puedas incluir en eso si es una declaración.

Answer 6

donde devuelve una vacíe ActiveRecord al no encontrar ninguna coincidencia.

Answer 7

Creo que las otras respuestas son un poco anticuadas. Así es como probablemente debería lograr esto para Rails 4

tag = Tag.first_or_initialize(:name => self.name, :user_id => current_user.id) 
if !tag.new_record? 
    tag.id = self.id 
    tag.save 
end