From 457be89445a28f91478c391df5758ff10c83b272 Mon Sep 17 00:00:00 2001
From: Sam Saffron <sam.saffron@gmail.com>
Date: Fri, 14 Jun 2019 09:52:02 +1000
Subject: [PATCH] DEV: only skip migration if a non seeded upload exists

Followup to 667b9801
---
 app/jobs/onceoff/migrate_url_site_settings.rb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/jobs/onceoff/migrate_url_site_settings.rb b/app/jobs/onceoff/migrate_url_site_settings.rb
index 4ddb2f3270c..c2641b7e1f1 100644
--- a/app/jobs/onceoff/migrate_url_site_settings.rb
+++ b/app/jobs/onceoff/migrate_url_site_settings.rb
@@ -18,8 +18,8 @@ module Jobs
     def execute_onceoff(args)
       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")
+        if (upload = SiteSetting.get(new_setting)) && upload.id >= Upload::SEEDED_ID_THRESHOLD
+          logger.warn("Skipping migration of the Site Setting #{new_setting} to url cause upload #{upload} already exists for it")
           next
         end