diff --git a/app/controllers/static_controller.rb b/app/controllers/static_controller.rb
index f86b02beb0f..d20cd7de49e 100644
--- a/app/controllers/static_controller.rb
+++ b/app/controllers/static_controller.rb
@@ -27,7 +27,7 @@ class StaticController < ApplicationController
if map.has_key?(@page)
site_setting_key = map[@page][:redirect]
- url = SiteSetting.send(site_setting_key)
+ url = SiteSetting.get(site_setting_key)
return redirect_to(url) unless url.blank?
end
diff --git a/app/jobs/onceoff/migrate_url_site_settings.rb b/app/jobs/onceoff/migrate_url_site_settings.rb
index 49b0911edb6..104793f9bdb 100644
--- a/app/jobs/onceoff/migrate_url_site_settings.rb
+++ b/app/jobs/onceoff/migrate_url_site_settings.rb
@@ -73,7 +73,7 @@ module Jobs
for_site_setting: true
).create_for(Discourse.system_user.id)
- SiteSetting.public_send("#{new_setting}=", upload)
+ SiteSetting.set(new_setting, upload)
end
end
diff --git a/app/jobs/regular/enable_bootstrap_mode.rb b/app/jobs/regular/enable_bootstrap_mode.rb
index 32431ea2afb..1bbc2cf0495 100644
--- a/app/jobs/regular/enable_bootstrap_mode.rb
+++ b/app/jobs/regular/enable_bootstrap_mode.rb
@@ -10,8 +10,8 @@ module Jobs
return unless user.is_singular_admin?
# let's enable bootstrap mode settings
- SiteSetting.set_and_log('default_trust_level', TrustLevel[1]) if SiteSetting.send('default_trust_level') == TrustLevel[0]
- SiteSetting.set_and_log('default_email_digest_frequency', 1440) if SiteSetting.send('default_email_digest_frequency') == 10080
+ SiteSetting.set_and_log('default_trust_level', TrustLevel[1]) if SiteSetting.get('default_trust_level') == TrustLevel[0]
+ SiteSetting.set_and_log('default_email_digest_frequency', 1440) if SiteSetting.get('default_email_digest_frequency') == 10080
SiteSetting.set_and_log('bootstrap_mode_enabled', true)
end
end
diff --git a/app/jobs/scheduled/clean_up_deprecated_url_site_settings.rb b/app/jobs/scheduled/clean_up_deprecated_url_site_settings.rb
index fd12fcbf226..47b62d62850 100644
--- a/app/jobs/scheduled/clean_up_deprecated_url_site_settings.rb
+++ b/app/jobs/scheduled/clean_up_deprecated_url_site_settings.rb
@@ -5,7 +5,7 @@ module Jobs
def execute(args)
Jobs::MigrateUrlSiteSettings::SETTINGS.each do |old_setting, new_setting|
if SiteSetting.where("name = ? AND value IS NOT NULL", new_setting).exists?
- SiteSetting.public_send("#{old_setting}=", nil, warn: false)
+ SiteSetting.set(old_setting, nil, warn: false)
SiteSetting.find_by(name: old_setting).destroy!
end
end
diff --git a/app/jobs/scheduled/disable_bootstrap_mode.rb b/app/jobs/scheduled/disable_bootstrap_mode.rb
index 108a2930620..cf84708c44e 100644
--- a/app/jobs/scheduled/disable_bootstrap_mode.rb
+++ b/app/jobs/scheduled/disable_bootstrap_mode.rb
@@ -7,8 +7,8 @@ module Jobs
total_users = User.human_users.count
if SiteSetting.bootstrap_mode_min_users == 0 || total_users > SiteSetting.bootstrap_mode_min_users
- SiteSetting.set_and_log('default_trust_level', TrustLevel[0]) if SiteSetting.send('default_trust_level') == TrustLevel[1]
- SiteSetting.set_and_log('default_email_digest_frequency', 10080) if SiteSetting.send('default_email_digest_frequency') == 1440
+ SiteSetting.set_and_log('default_trust_level', TrustLevel[0]) if SiteSetting.get('default_trust_level') == TrustLevel[1]
+ SiteSetting.set_and_log('default_email_digest_frequency', 10080) if SiteSetting.get('default_email_digest_frequency') == 1440
SiteSetting.set_and_log('bootstrap_mode_enabled', false)
end
end
diff --git a/app/models/category.rb b/app/models/category.rb
index 18a6c185c06..096ed96aba3 100644
--- a/app/models/category.rb
+++ b/app/models/category.rb
@@ -285,7 +285,7 @@ class Category < ActiveRecord::Base
def remove_site_settings
SiteSetting.all_settings.each do |s|
if s[:type] == 'category' && s[:value].to_i == self.id
- SiteSetting.send("#{s[:setting]}=", '')
+ SiteSetting.set(s[:setting], '')
end
end
diff --git a/app/models/embedding.rb b/app/models/embedding.rb
index a0be5ceea80..3e9b1a38ddf 100644
--- a/app/models/embedding.rb
+++ b/app/models/embedding.rb
@@ -23,7 +23,7 @@ class Embedding < OpenStruct
def save
Embedding.settings.each do |s|
- SiteSetting.send("#{s}=", send(s))
+ SiteSetting.set(s, public_send(s))
end
true
rescue Discourse::InvalidParameters => p
diff --git a/app/models/post_action.rb b/app/models/post_action.rb
index 5fd9d506de5..2d31b641210 100644
--- a/app/models/post_action.rb
+++ b/app/models/post_action.rb
@@ -164,10 +164,10 @@ class PostAction < ActiveRecord::Base
%w(like flag bookmark).each do |type|
if send("is_#{type}?")
- limit = SiteSetting.send("max_#{type}s_per_day")
+ limit = SiteSetting.get("max_#{type}s_per_day")
if is_like? && user && user.trust_level >= 2
- multiplier = SiteSetting.send("tl#{user.trust_level}_additional_likes_per_day_multiplier").to_f
+ multiplier = SiteSetting.get("tl#{user.trust_level}_additional_likes_per_day_multiplier").to_f
multiplier = 1.0 if multiplier < 1.0
limit = (limit * multiplier).to_i
diff --git a/app/models/topic.rb b/app/models/topic.rb
index b8a0b187976..02fe6ddb9c1 100644
--- a/app/models/topic.rb
+++ b/app/models/topic.rb
@@ -1433,7 +1433,7 @@ class Topic < ActiveRecord::Base
end
def apply_per_day_rate_limit_for(key, method_name)
- RateLimiter.new(user, "#{key}-per-day", SiteSetting.send(method_name), 1.day.to_i)
+ RateLimiter.new(user, "#{key}-per-day", SiteSetting.get(method_name), 1.day.to_i)
end
def create_invite_notification!(target_user, notification_type, username)
diff --git a/app/models/user.rb b/app/models/user.rb
index 79d55de023e..cde8851a4a9 100644
--- a/app/models/user.rb
+++ b/app/models/user.rb
@@ -1351,7 +1351,7 @@ class User < ActiveRecord::Base
values = []
%w{watching watching_first_post tracking muted}.each do |s|
- category_ids = SiteSetting.send("default_categories_#{s}").split("|")
+ category_ids = SiteSetting.get("default_categories_#{s}").split("|")
category_ids.each do |category_id|
values << "(#{self.id}, #{category_id}, #{CategoryUser.notification_levels[s.to_sym]})"
end
diff --git a/app/serializers/auth_provider_serializer.rb b/app/serializers/auth_provider_serializer.rb
index f2bc5cdbd13..82310b7bd9f 100644
--- a/app/serializers/auth_provider_serializer.rb
+++ b/app/serializers/auth_provider_serializer.rb
@@ -5,17 +5,17 @@ class AuthProviderSerializer < ApplicationSerializer
:icon
def title_override
- return SiteSetting.send(object.title_setting) if object.title_setting
+ return SiteSetting.get(object.title_setting) if object.title_setting
object.title
end
def pretty_name_override
- return SiteSetting.send(object.pretty_name_setting) if object.pretty_name_setting
+ return SiteSetting.get(object.pretty_name_setting) if object.pretty_name_setting
object.pretty_name
end
def full_screen_login
- return SiteSetting.send(object.full_screen_login_setting) if object.full_screen_login_setting
+ return SiteSetting.get(object.full_screen_login_setting) if object.full_screen_login_setting
return object.full_screen_login if object.full_screen_login
false
end
diff --git a/lib/auth/open_id_authenticator.rb b/lib/auth/open_id_authenticator.rb
index ac80b53e038..7cd54a77316 100644
--- a/lib/auth/open_id_authenticator.rb
+++ b/lib/auth/open_id_authenticator.rb
@@ -10,7 +10,7 @@ class Auth::OpenIdAuthenticator < Auth::Authenticator
end
def enabled?
- SiteSetting.send(@enabled_site_setting)
+ SiteSetting.get(@enabled_site_setting)
end
def description_for_user(user)
diff --git a/lib/plugin/instance.rb b/lib/plugin/instance.rb
index 2611f55688e..28dc167ebbc 100644
--- a/lib/plugin/instance.rb
+++ b/lib/plugin/instance.rb
@@ -81,7 +81,7 @@ class Plugin::Instance
end
def enabled?
- @enabled_site_setting ? SiteSetting.send(@enabled_site_setting) : true
+ @enabled_site_setting ? SiteSetting.get(@enabled_site_setting) : true
end
delegate :name, to: :metadata
@@ -532,7 +532,7 @@ class Plugin::Instance
rescue NotImplementedError
provider.authenticator.define_singleton_method(:enabled?) do
Discourse.deprecate("#{provider.authenticator.class.name} should define an `enabled?` function. Patching for now.")
- return SiteSetting.send(provider.enabled_setting) if provider.enabled_setting
+ return SiteSetting.get(provider.enabled_setting) if provider.enabled_setting
Discourse.deprecate("#{provider.authenticator.class.name} has not defined an enabled_setting. Defaulting to true.")
true
end
diff --git a/lib/rate_limiter/on_create_record.rb b/lib/rate_limiter/on_create_record.rb
index 14374de6a12..3a8483cd8e1 100644
--- a/lib/rate_limiter/on_create_record.rb
+++ b/lib/rate_limiter/on_create_record.rb
@@ -14,9 +14,9 @@ class RateLimiter
limit_key = "create_#{self.class.name.underscore}"
max_setting = if user && user.new_user? && SiteSetting.has_setting?("rate_limit_new_user_#{limit_key}")
- SiteSetting.send("rate_limit_new_user_#{limit_key}")
+ SiteSetting.get("rate_limit_new_user_#{limit_key}")
else
- SiteSetting.send("rate_limit_#{limit_key}")
+ SiteSetting.get("rate_limit_#{limit_key}")
end
@rate_limiter = RateLimiter.new(user, limit_key, 1, max_setting)
end
diff --git a/lib/seed_data/categories.rb b/lib/seed_data/categories.rb
index adf077b894d..abc4a615635 100644
--- a/lib/seed_data/categories.rb
+++ b/lib/seed_data/categories.rb
@@ -95,7 +95,7 @@ module SeedData
def create_category(site_setting_name:, name:, description:, position:, color:, text_color:,
permissions:, force_permissions:, force_existence: false)
- category_id = SiteSetting.send(site_setting_name)
+ category_id = SiteSetting.get(site_setting_name)
if should_create_category?(category_id, force_existence)
category = Category.new(
@@ -111,7 +111,7 @@ module SeedData
category.set_permissions(permissions)
category.save!
- SiteSetting.send("#{site_setting_name}=", category.id)
+ SiteSetting.set(site_setting_name, category.id)
elsif category = Category.find_by(id: category_id)
if description.present? && (category.topic_id.blank? || !Topic.exists?(category.topic_id))
category.description = description
@@ -159,7 +159,7 @@ module SeedData
end
def find_category(site_setting_name)
- category_id = SiteSetting.send(site_setting_name)
+ category_id = SiteSetting.get(site_setting_name)
Category.find_by(id: category_id) if category_id > 0
end
diff --git a/lib/seed_data/topics.rb b/lib/seed_data/topics.rb
index bbf48dac1e8..769b20cad62 100644
--- a/lib/seed_data/topics.rb
+++ b/lib/seed_data/topics.rb
@@ -122,7 +122,7 @@ module SeedData
end
def create_topic(site_setting_name:, title:, raw:, category: nil, static_first_reply: false, after_create: nil)
- topic_id = SiteSetting.send(site_setting_name)
+ topic_id = SiteSetting.get(site_setting_name)
return if topic_id > 0 || Topic.find_by(id: topic_id)
post = PostCreator.create!(
@@ -144,7 +144,7 @@ module SeedData
after_create&.call(post)
- SiteSetting.send("#{site_setting_name}=", post.topic_id)
+ SiteSetting.set(site_setting_name, post.topic_id)
end
def update_topic(site_setting_name:, title:, raw:, static_first_reply: false, skip_changed:)
@@ -163,7 +163,7 @@ module SeedData
end
def find_post(site_setting_name)
- topic_id = SiteSetting.send(site_setting_name)
+ topic_id = SiteSetting.get(site_setting_name)
Post.find_by(topic_id: topic_id, post_number: 1) if topic_id > 0
end
@@ -172,7 +172,7 @@ module SeedData
end
def setting_value(site_setting_key)
- SiteSetting.send(site_setting_key).presence || "#{site_setting_key}"
+ SiteSetting.get(site_setting_key).presence || "#{site_setting_key}"
end
def first_reply(post)
diff --git a/lib/site_icon_manager.rb b/lib/site_icon_manager.rb
index 51d6a9d6b7b..ae3a21e177c 100644
--- a/lib/site_icon_manager.rb
+++ b/lib/site_icon_manager.rb
@@ -65,7 +65,7 @@ module SiteIconManager
def self.resolve_original(info)
info[:settings].each do |setting_name|
- value = SiteSetting.send(setting_name)
+ value = SiteSetting.get(setting_name)
return value if value
end
return Upload.find(SKETCH_LOGO_ID) if info[:fallback_to_sketch]
diff --git a/lib/site_setting_extension.rb b/lib/site_setting_extension.rb
index 83172f10325..80332cc891d 100644
--- a/lib/site_setting_extension.rb
+++ b/lib/site_setting_extension.rb
@@ -390,10 +390,14 @@ module SiteSettingExtension
end
end
- def set(name, value)
+ def set(name, value, options = nil)
if has_setting?(name)
value = filter_value(name, value)
- self.public_send("#{name}=", value)
+ if options
+ self.public_send("#{name}=", value, options)
+ else
+ self.public_send("#{name}=", value)
+ end
Discourse.request_refresh! if requires_refresh?(name)
else
raise Discourse::InvalidParameters.new("Either no setting named '#{name}' exists or value provided is invalid")
@@ -407,7 +411,15 @@ module SiteSettingExtension
value = prev_value = "[FILTERED]" if secret_settings.include?(name.to_sym)
StaffActionLogger.new(user).log_site_setting_change(name, prev_value, value)
else
- raise Discourse::InvalidParameters
+ raise Discourse::InvalidParameters.new("No setting named '#{name}' exists")
+ end
+ end
+
+ def get(name)
+ if has_setting?(name)
+ self.public_send(name)
+ else
+ raise Discourse::InvalidParameters.new("No setting named '#{name}' exists")
end
end
diff --git a/lib/tasks/typepad.thor b/lib/tasks/typepad.thor
index 8197f7f0f57..94bda6ca67c 100644
--- a/lib/tasks/typepad.thor
+++ b/lib/tasks/typepad.thor
@@ -12,7 +12,7 @@ class Typepad < Thor
backup_settings = {}
%w(email_domains_blacklist).each do |s|
- backup_settings[s] = SiteSetting.send(s)
+ backup_settings[s] = SiteSetting.get(s)
end
user = User.where(username_lower: options[:post_as].downcase).first
@@ -26,7 +26,6 @@ class Typepad < Thor
exit 1
end
- inside_block = true
input = ""
entries = []
@@ -103,7 +102,7 @@ class Typepad < Thor
ensure
RateLimiter.enable
backup_settings.each do |s, v|
- SiteSetting.send("#{s}=", v)
+ SiteSetting.set(s, v)
end
end
diff --git a/lib/validators/upload_validator.rb b/lib/validators/upload_validator.rb
index d69bd6cb01c..e6c5fad25e2 100644
--- a/lib/validators/upload_validator.rb
+++ b/lib/validators/upload_validator.rb
@@ -123,7 +123,7 @@ class Validators::UploadValidator < ActiveModel::Validator
max_size_kb = if upload.for_export
SiteSetting.max_export_file_size_kb
else
- SiteSetting.send("max_#{type}_size_kb")
+ SiteSetting.get("max_#{type}_size_kb")
end
max_size_bytes = max_size_kb.kilobytes
diff --git a/lib/wizard/builder.rb b/lib/wizard/builder.rb
index 6fae1d47f23..994230f2da3 100644
--- a/lib/wizard/builder.rb
+++ b/lib/wizard/builder.rb
@@ -278,7 +278,7 @@ class Wizard
protected
def replace_setting_value(updater, raw, field_name)
- old_value = SiteSetting.send(field_name)
+ old_value = SiteSetting.get(field_name)
old_value = field_name if old_value.blank?
new_value = updater.fields[field_name.to_sym]
diff --git a/lib/wizard/step_updater.rb b/lib/wizard/step_updater.rb
index 81d9ba0934d..c7eef71b89f 100644
--- a/lib/wizard/step_updater.rb
+++ b/lib/wizard/step_updater.rb
@@ -37,7 +37,7 @@ class Wizard
value = Upload.get_from_url(value) || ''
end
- SiteSetting.set_and_log(id, value, @current_user) if SiteSetting.send(id) != value
+ SiteSetting.set_and_log(id, value, @current_user) if SiteSetting.get(id) != value
end
def apply_setting(id)
diff --git a/script/import_scripts/base.rb b/script/import_scripts/base.rb
index 8b8f3a841dc..df82530e869 100644
--- a/script/import_scripts/base.rb
+++ b/script/import_scripts/base.rb
@@ -95,7 +95,7 @@ class ImportScripts::Base
@site_settings_during_import = get_site_settings_for_import
@site_settings_during_import.each do |key, value|
- @old_site_settings[key] = SiteSetting.send(key)
+ @old_site_settings[key] = SiteSetting.get(key)
SiteSetting.set(key, value)
end
@@ -108,7 +108,7 @@ class ImportScripts::Base
def reset_site_settings
@old_site_settings.each do |key, value|
- current_value = SiteSetting.send(key)
+ current_value = SiteSetting.get(key)
SiteSetting.set(key, value) unless current_value != @site_settings_during_import[key]
end
diff --git a/spec/components/plugin/instance_spec.rb b/spec/components/plugin/instance_spec.rb
index a13a81b01ba..93692401038 100644
--- a/spec/components/plugin/instance_spec.rb
+++ b/spec/components/plugin/instance_spec.rb
@@ -130,10 +130,11 @@ describe Plugin::Instance do
end
it 'patches the enabled? function for auth_providers if not defined' do
- SiteSetting.stubs(:ubuntu_login_enabled).returns(false)
-
plugin = Plugin::Instance.new
+ # lets piggy back on another boolean setting, so we don't dirty our SiteSetting object
+ SiteSetting.enable_badges = false
+
# No enabled_site_setting
authenticator = Auth::Authenticator.new
plugin.auth_provider(authenticator: authenticator)
@@ -143,26 +144,28 @@ describe Plugin::Instance do
# With enabled site setting
plugin = Plugin::Instance.new
authenticator = Auth::Authenticator.new
- plugin.auth_provider(enabled_setting: 'ubuntu_login_enabled', authenticator: authenticator)
+ plugin.auth_provider(enabled_setting: 'enable_badges', authenticator: authenticator)
plugin.notify_before_auth
expect(authenticator.enabled?).to eq(false)
# Defines own method
plugin = Plugin::Instance.new
- SiteSetting.stubs(:ubuntu_login_enabled).returns(true)
+
+ SiteSetting.enable_badges = true
authenticator = Class.new(Auth::Authenticator) do
def enabled?
false
end
end.new
- plugin.auth_provider(enabled_setting: 'ubuntu_login_enabled', authenticator: authenticator)
+ plugin.auth_provider(enabled_setting: 'enable_badges', authenticator: authenticator)
plugin.notify_before_auth
expect(authenticator.enabled?).to eq(false)
end
context "activate!" do
before do
- SiteSetting.stubs(:ubuntu_login_enabled).returns(false)
+ # lets piggy back on another boolean setting, so we don't dirty our SiteSetting object
+ SiteSetting.enable_badges = false
end
it "can activate plugins correctly" do
diff --git a/spec/components/site_setting_extension_spec.rb b/spec/components/site_setting_extension_spec.rb
index dc351614a9e..f89d3c4eb9d 100644
--- a/spec/components/site_setting_extension_spec.rb
+++ b/spec/components/site_setting_extension_spec.rb
@@ -532,6 +532,23 @@ describe SiteSettingExtension do
end
end
+ describe ".get" do
+ before do
+ settings.setting(:title, "Discourse v1")
+ settings.refresh!
+ end
+
+ it "works correctly" do
+ expect {
+ settings.get("frogs_in_africa")
+ }.to raise_error(Discourse::InvalidParameters)
+
+ expect(settings.get(:title)).to eq("Discourse v1")
+ expect(settings.get("title")).to eq("Discourse v1")
+ end
+
+ end
+
describe ".set_and_log" do
before do
settings.setting(:s3_secret_access_key, "old_secret_key", secret: true)
diff --git a/spec/fixtures/plugins/my_plugin/plugin.rb b/spec/fixtures/plugins/my_plugin/plugin.rb
index 9c2b5881b97..501786da7d6 100644
--- a/spec/fixtures/plugins/my_plugin/plugin.rb
+++ b/spec/fixtures/plugins/my_plugin/plugin.rb
@@ -6,7 +6,7 @@
# authors: Frank Zappa
auth_provider title: 'with Ubuntu',
- authenticator: Auth::OpenIdAuthenticator.new('ubuntu', 'https://login.ubuntu.com', 'ubuntu_login_enabled', trusted: true),
+ authenticator: Auth::OpenIdAuthenticator.new('ubuntu', 'https://login.ubuntu.com', 'enable_badges', trusted: true),
message: 'Authenticating with Ubuntu (make sure pop up blockers are not enbaled)',
frame_width: 1000, # the frame size used for the pop up window, overrides default
frame_height: 800
diff --git a/spec/jobs/clean_up_deprecated_url_site_settings_spec.rb b/spec/jobs/clean_up_deprecated_url_site_settings_spec.rb
index 8d5dcd4d74c..fd52644eaed 100644
--- a/spec/jobs/clean_up_deprecated_url_site_settings_spec.rb
+++ b/spec/jobs/clean_up_deprecated_url_site_settings_spec.rb
@@ -16,8 +16,8 @@ RSpec.describe Jobs::CleanUpDeprecatedUrlSiteSettings do
it 'should clean up the old deprecated site settings correctly' do
logo_upload = Fabricate(:upload)
SiteSetting.logo = logo_upload
- SiteSetting.public_send("logo_url=", '/test/some/url', warn: false)
- SiteSetting.public_send("logo_small_url=", '/test/another/url', warn: false)
+ SiteSetting.set("logo_url", '/test/some/url', warn: false)
+ SiteSetting.set("logo_small_url", '/test/another/url', warn: false)
expect do
described_class.new.execute({})
diff --git a/spec/jobs/migrate_url_site_settings_spec.rb b/spec/jobs/migrate_url_site_settings_spec.rb
index 8e61632e289..c3331f0f654 100644
--- a/spec/jobs/migrate_url_site_settings_spec.rb
+++ b/spec/jobs/migrate_url_site_settings_spec.rb
@@ -67,8 +67,8 @@ RSpec.describe Jobs::MigrateUrlSiteSettings do
twitter_summary_large_image
push_notifications_icon
}.each do |setting|
- expect(SiteSetting.public_send(setting)).to eq(upload)
- expect(SiteSetting.public_send(setting).is_a?(Upload)).to eq(true)
+ expect(SiteSetting.get(setting)).to eq(upload)
+ expect(SiteSetting.get(setting).is_a?(Upload)).to eq(true)
end
end
end
diff --git a/spec/lib/site_settings/validations_spec.rb b/spec/lib/site_settings/validations_spec.rb
index bed04da1570..3087dc850fa 100644
--- a/spec/lib/site_settings/validations_spec.rb
+++ b/spec/lib/site_settings/validations_spec.rb
@@ -11,7 +11,7 @@ describe SiteSettings::Validations do
shared_examples "s3 bucket validation" do
def change_bucket_value(value)
- SiteSetting.public_send("#{other_setting_name}=", value)
+ SiteSetting.set(other_setting_name, value)
end
it "shouldn't raise an error when both buckets are blank" do
diff --git a/spec/models/admin_dashboard_problem_spec.rb b/spec/models/admin_dashboard_problem_spec.rb
index 545bda9c80d..c61b346224c 100644
--- a/spec/models/admin_dashboard_problem_spec.rb
+++ b/spec/models/admin_dashboard_problem_spec.rb
@@ -133,37 +133,37 @@ describe AdminDashboardData do
shared_examples 'problem detection for login providers' do
context 'when disabled' do
it 'returns nil' do
- SiteSetting.public_send("#{enable_setting}=", false)
+ SiteSetting.set(enable_setting, false)
expect(subject).to be_nil
end
end
context 'when enabled' do
before do
- SiteSetting.public_send("#{enable_setting}=", true)
+ SiteSetting.set(enable_setting, true)
end
it 'returns nil when key and secret are set' do
- SiteSetting.public_send("#{key}=", '12313213')
- SiteSetting.public_send("#{secret}=", '12312313123')
+ SiteSetting.set(key, '12313213')
+ SiteSetting.set(secret, '12312313123')
expect(subject).to be_nil
end
it 'returns a string when key is not set' do
- SiteSetting.public_send("#{key}=", '')
- SiteSetting.public_send("#{secret}=", '12312313123')
+ SiteSetting.set(key, '')
+ SiteSetting.set(secret, '12312313123')
expect(subject).to_not be_nil
end
it 'returns a string when secret is not set' do
- SiteSetting.public_send("#{key}=", '123123')
- SiteSetting.public_send("#{secret}=", '')
+ SiteSetting.set(key, '123123')
+ SiteSetting.set(secret, '')
expect(subject).to_not be_nil
end
it 'returns a string when key and secret are not set' do
- SiteSetting.public_send("#{key}=", '')
- SiteSetting.public_send("#{secret}=", '')
+ SiteSetting.set(key, '')
+ SiteSetting.set(secret, '')
expect(subject).to_not be_nil
end
end
@@ -205,7 +205,7 @@ describe AdminDashboardData do
['a', ''].repeated_permutation(keys.size) do |*values|
hash = Hash[keys.zip(values)]
hash.each do |key, value|
- SiteSetting.public_send("#{key}=", value)
+ SiteSetting.set(key, value)
end
yield hash
end
@@ -213,9 +213,9 @@ describe AdminDashboardData do
context 'when setting is enabled' do
before do
- all_setting_keys.each { |key| SiteSetting.public_send("#{key}=", 'foo') }
- SiteSetting.public_send("#{setting[:key]}=", setting[:enabled_value])
- SiteSetting.public_send("#{bucket_key}=", bucket_value)
+ all_setting_keys.each { |key| SiteSetting.set(key, 'foo') }
+ SiteSetting.set(setting[:key], setting[:enabled_value])
+ SiteSetting.set(bucket_key, bucket_value)
end
context 'when bucket is blank' do
@@ -262,7 +262,7 @@ describe AdminDashboardData do
context 'when setting is not enabled' do
before do
- SiteSetting.public_send("#{setting[:key]}=", setting[:disabled_value])
+ SiteSetting.set(setting[:key], setting[:disabled_value])
end
it "always returns nil" do
diff --git a/spec/models/user_spec.rb b/spec/models/user_spec.rb
index a5795c952b8..5a658b20909 100644
--- a/spec/models/user_spec.rb
+++ b/spec/models/user_spec.rb
@@ -1222,7 +1222,6 @@ describe User do
end
it "returns custom color if restrict_letter_avatar_colors site setting is set" do
- colors = SiteSetting.restrict_letter_avatar_colors.split("|")
expect(User.letter_avatar_color("username_one")).to eq("2F70AC")
expect(User.letter_avatar_color("username_two")).to eq("ED207B")
expect(User.letter_avatar_color("username_three")).to eq("AAAAAA")
diff --git a/spec/requests/session_controller_spec.rb b/spec/requests/session_controller_spec.rb
index 7bee8e84333..66f0ef3a8c8 100644
--- a/spec/requests/session_controller_spec.rb
+++ b/spec/requests/session_controller_spec.rb
@@ -267,11 +267,7 @@ RSpec.describe SessionController do
SiteSetting.enable_sso = true
SiteSetting.sso_secret = @sso_secret
- # We have 2 options, either fabricate an admin or don't
- # send welcome messages
Fabricate(:admin)
- # skip for now
- # SiteSetting.send_welcome_message = false
end
let(:headers) { { host: Discourse.current_hostname } }
diff --git a/spec/requests/static_controller_spec.rb b/spec/requests/static_controller_spec.rb
index 04b7b08ae69..9883cf0937d 100644
--- a/spec/requests/static_controller_spec.rb
+++ b/spec/requests/static_controller_spec.rb
@@ -151,7 +151,7 @@ describe StaticController do
context "when #{setting_name} site setting is set" do
before do
- SiteSetting.public_send("#{setting_name}=", 'http://example.com/page')
+ SiteSetting.set(setting_name, 'http://example.com/page')
end
it "redirects to the #{setting_name}" do