Merge pull request #37461 from code-dot-org/fix-destroy-error

Pass IDs rather than objects to `.destroy`
code-dot-org · Oct 26, 2020 · c9a5d81 · c9a5d81
2 parents d068f14 + 25c5942
commit c9a5d81
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/dashboard/lib/services/script_seed.rb b/dashboard/lib/services/script_seed.rb
@@ -235,7 +235,7 @@ def self.import_levels_script_levels(levels_script_levels_data, seed_context)
   def self.destroy_outdated_objects(model_class, all_objects, imported_objects, seed_context)
     objects_to_keep_by_seeding_key = imported_objects.index_by {|o| o.seeding_key(seed_context)}
     should_keep = all_objects.group_by {|o| objects_to_keep_by_seeding_key.include?(o.seeding_key(seed_context))}
-    model_class.destroy(should_keep[false]) if should_keep.include?(false)
+    model_class.destroy(should_keep[false].pluck(:id)) if should_keep.include?(false)
     should_keep[true]
   end