##// END OF EJS Templates
Fixed: Project#enabled_module_names= does not test if a module is already enabled (#4200)....
Jean-Philippe Lang -
r2922:28f0b5ce4ab4
parent child
Show More
@@ -389,7 +389,7 class Project < ActiveRecord::Base
389 # remove disabled modules
389 # remove disabled modules
390 enabled_modules.each {|mod| mod.destroy unless module_names.include?(mod.name)}
390 enabled_modules.each {|mod| mod.destroy unless module_names.include?(mod.name)}
391 # add new modules
391 # add new modules
392 module_names.each {|name| enabled_modules << EnabledModule.new(:name => name)}
392 module_names.reject {|name| module_enabled?(name)}.each {|name| enabled_modules << EnabledModule.new(:name => name)}
393 else
393 else
394 enabled_modules.clear
394 enabled_modules.clear
395 end
395 end
General Comments 0
You need to be logged in to leave comments. Login now