Citation :J'aimerai que quelqu'un me donne un éclaircissement sur le callback after_add, j'ai fait

has_and_belongs_to_many :teams, :after_add => et_statut

 

def set_statut

   if (self.teams.count == 6)

      self.update_attribute(tatut, 1)

      self.save

   end

end

mais ca n'a pas l'air de faire ce que je voudrais que ca fasse 12
je ne vois pas dans ma console s'exécuter le count par exemple...

Citation :Crée plutôt un nouveau sujet pour ce genre de question, assez spécifique. 16

Je n'utilise pas souvent ces callbacks (ils sont peu utilisés de manière générale, car leur documentation est assez succinte et que leur implémentation initiale était un peu bancale) et je ne peux pas faire de test dans l'immédiat, mais j'ai peut-être des pistes.
Es-tu sûr que self correspond bien à ton objet (je ne sais pas dans quelle classe tu es puisque tu as tronqué ton modèle) ?
La méthode update_attribute effectue une sauvegarde de l'objet, tu n'as pas besoin d'appeler save derrière.

Enfin, cette manipulation devrait plutôt se faire dans le callback after_create de ton modèle de relation, puisque after_add est appelé même si l'association n'est pas sauvegardé en base de données alors que dans l'esprit, tu ne veux passer le status à 1 que si tu as atteint les 6 joueurs. D'ailleurs, je te conseille d'utiliser une constante avec un nom parlant plutôt qu'un nombre (même si cette constante contient en effet un nombre).

Exemple si j'implémentais ça comme je te le conseille (en extrapolant sur les modèles) :

class GameTeam < ActiveRecord::Base

 

  belongs_to :game

  belongs_to :team

 

 

  after_create et_status

 

 

  private

 

  def set_status

    if game.teams.count == 6

      game.update_attribute(tatus, GAME::STARTED)

    end

  end

 

end

Quand je serais chez moi, je ferais quelques essais pour t'indiquer comment ça aurait pu avoir le comportement souhaité.


Sephi-Chan

class League < ActiveRecord::Base

	has_and_belongs_to_many :teams, :after_add => et_statut

	

	def self.find_or_create_open_league

		League.where(:level => 1, tatut => 0).first || League.create

	end

	

	private



		def set_statut

			if (teams.count == 6)

				update_attribute(tatut, 1)

				save

			end

		end

	

end



class Team < ActiveRecord::Base



	has_and_belongs_to_many :leagues	

	before_create :add_to_league

	

	private

	

		def add_to_league

			league = League.find_or_create_open_league

			self.leagues = [league]

		end

end