FIX: Use Discourse.deprecate instead of Rails.logger.warn

This will significantly reduce the volume of logs when the condition is hit
This commit is contained in:
David Taylor 2018-11-22 15:59:47 +00:00
parent 2843944315
commit 052bf37578

View File

@ -497,9 +497,9 @@ class Plugin::Instance
provider.authenticator.enabled?
rescue NotImplementedError
provider.authenticator.define_singleton_method(:enabled?) do
Rails.logger.warn("#{provider.authenticator.class.name} should define an `enabled?` function. Patching for now.")
Discourse.deprecate("#{provider.authenticator.class.name} should define an `enabled?` function. Patching for now.")
return SiteSetting.send(provider.enabled_setting) if provider.enabled_setting
Rails.logger.warn("#{provider.authenticator.class.name} has not defined an enabled_setting. Defaulting to true.")
Discourse.deprecate("#{provider.authenticator.class.name} has not defined an enabled_setting. Defaulting to true.")
true
end
end