FIX: do not attempt to migrate pre-existing uploads

This makes this job re-runnable just in case cause it will skip creation
of new uploads if an upload already exists
This commit is contained in:
Sam Saffron 2019-06-14 09:39:09 +10:00
parent 4182d8638c
commit 667b98017a
1 changed files with 6 additions and 0 deletions

View File

@ -17,6 +17,12 @@ module Jobs
def execute_onceoff(args) def execute_onceoff(args)
SETTINGS.each do |old_setting, new_setting| SETTINGS.each do |old_setting, new_setting|
if upload_id = SiteSetting.get(new_setting)
logger.warn("Skipping migration of the Site Setting #{new_setting} to url cause upload #{upload_id} already exists for it")
next
end
old_url = DB.query_single( old_url = DB.query_single(
"SELECT value FROM site_settings WHERE name = '#{old_setting}'" "SELECT value FROM site_settings WHERE name = '#{old_setting}'"
).first ).first