XykoX

La solution dans l’article précédent ne me plaisait pas, alors j’ai fais ma solution … bon j’avoue j’ai pas cherché s’il existe déjà une solution prête à l’emploi.

Personnellement, je préfère que config. reload_plugins soit à false. Mais je ne veux pas mettre dans mes plugins un bout de code qui dépend plus de l’application et du contexte (development/production) que du plugin lui même.

Je pourrais mettre reload_plugin à true, oui mais je n’ai pas forcément envie de recharger toute la masse de plugin à chaque requête.

Dans un premier temps, on va ajouter une directive de configuration à savoir reloadable_plugins, et on va la traiter au moment du chargement des plugins. Voici le code :

Rails::Configuration.class_eval do
  attr_accessor :reloadable_plugins

  def initialize_with_reloadable_plugins
    initialize_without_reloadable_plugins
    self.reloadable_plugins = default_reloadable_plugins
  end

  def default_reloadable_plugins
    []
  end

  alias_method_chain :initialize, :reloadable_plugins
end

Rails::Plugin::Loader.class_eval do
  def add_plugin_load_paths_with_reloadable_plugins
    load_path = add_plugin_load_paths_without_reloadable_plugins
    plugins.each do |plugin|
      if configuration.reloadable_plugins.include?(plugin.name.to_sym)
        ActiveSupport::Dependencies.load_once_paths -= plugin.load_paths
      end
    end
    load_path
  end
  alias_method_chain :add_plugin_load_paths, :reloadable_plugins
end

qu’on va mettre dans lib/configuration_reloadable_plugins.rb par exemple.

Ensuite ajouter dans le fichier config/environnement.rb, juste après le boot.rb (l’ordre est important)

  ...
require File.join(File.dirname(__FILE__), 'boot')
require File.join(File.dirname(__FILE__), '../vendor/plugins/engines/boot')
require 'lib/configuration_reloadable_plugins'
  ...


Et toujours dans <strong>config/environnement.rb</strong> on indique les plugins à recharger :

...
Rails::Initializer.run do |config|
  config.reloadable_plugins = [ :foo, :bar ]
  config.reload_plugins = false
  ...
end
...

Et pour vérifier que c’est bon :

$ script/console
>> ActiveSupport::Dependencies.load_once_paths.select{ |p| p =~ /foo/ }
=> []

Ah les joies de se mettre à jour et son code à l’occasion … Oui depuis longtemps je trainais avec rails 1.2.6 (oui je sais ça commence à dater mais j’ai mes raisons). Il me tardait donc de pouvoir utiliser les nouvelles versions de rails avec son lots d’améliorations.

Bon tout d’abord grosse surprise un bug uniquement présent en development, et pour cause il concerne le système de déchargement/rechargement du code de l’application.

ça donne soit un

stack level too deep

ou alors un pénible

vendor/rails/activerecord/lib/active_record/attribute_methods.rb:142:in `create_time_zone_conversion_attribute?'
vendor/rails/activerecord/lib/active_record/attribute_methods.rb:75:in `define_attribute_methods'
vendor/rails/activerecord/lib/active_record/attribute_methods.rb:71:in `each'
vendor/rails/activerecord/lib/active_record/attribute_methods.rb:71:in `define_attribute_methods'
vendor/rails/activerecord/lib/active_record/attribute_methods.rb:242:in `method_missing'

Le problème semble être valable depuis rails 2.2 et toujours pas corrigé. Il semble que ce ne soit pas une priorité car le problème se pose uniquement en mode development.

Donc la solution est de patcher rails en attendant. Patch qui se trouve dans le ticket chez lighthouseapp :

if ENV['RAILS_ENV'] != 'production'
  class ActiveRecord::Base
    class_eval do
      def self.reset_subclasses
        nonreloadables = []
        subclasses.each do |klass|
          unless ActiveSupport::Dependencies.autoloaded? klass
            nonreloadables << klass
            next
          end
          klass.instance_variables.each { |var| klass.send(:remove_instance_variable, var) }
          klass.instance_methods(false).each { |m| klass.send(:undef_method, m) unless m =~ /^id(=$|\?$|$)/ }

        end
        @@subclasses = {}
        nonreloadables.each { |klass| (@@subclasses[klass.superclass] ||= []) << klass }
      end
    end
  end
end

Voilà et maintenant ça roule on peut développer en paix.