maverick/docbrown
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Only migrate config vars that were explicitly set (#13593)

Browse source
This commit is contained in:
Jamie Gaskins 2021-04-30 10:15:33 -04:00 • committed by GitHub
parent 69f4c039da
commit 8c9a0dc55a
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 6 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
lib/data_update_scripts/20210405034117_move_campaign_settings.rb
View file

@ -16,10 +16,9 @@ module DataUpdateScripts
# All these fields got renamed so we migrate them explicitly
CAMPAIGN_SETTINGS.each do |setting|
Settings::Campaign.public_send(
"#{setting}=",
SiteConfig.public_send("campaign_#{setting}"),
)
if (value = SiteConfig.public_send("campaign_#{setting}"))
Settings::Campaign.public_send("#{setting}=", value)
end
end
end
end

4
lib/data_update_scripts/20210420050256_move_mascot_settings.rb
View file

@ -10,7 +10,9 @@ module DataUpdateScripts
def run
SETTINGS.each do |setting|
Settings::Mascot.public_send("#{setting}=", SiteConfig.public_send("mascot_#{setting}"))
if (value = SiteConfig.public_send("mascot_#{setting}"))
Settings::Mascot.public_send("#{setting}=", value)
end
end
Settings::Mascot.mascot_user_id = SiteConfig.mascot_user_id
end