From 50615eca2ccf480349ffdddb66a53b4c3b9959d6 Mon Sep 17 00:00:00 2001 From: Chris Colvard Date: Wed, 31 Jul 2024 15:39:50 -0400 Subject: [PATCH] Get rid of nil ids when migrating to section lists This fixes an edge case where ordered_master_file_ids contain nils. --- app/models/media_object.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/models/media_object.rb b/app/models/media_object.rb index e2030cd738..c1eb6aa305 100644 --- a/app/models/media_object.rb +++ b/app/models/media_object.rb @@ -177,7 +177,7 @@ def section_ids return @section_ids if @section_ids # Do migration - self.section_ids = self.ordered_master_file_ids if self.section_list.nil? + self.section_ids = self.ordered_master_file_ids.compact if self.section_list.nil? return [] if self.section_list.nil? @section_ids = JSON.parse(self.section_list)