Merge pull request #919 from sul-dlss/only-cocina

Don't try to deserialize XML as JSON
sul-dlss · Aug 23, 2024 · 72bc782 · 72bc782
2 parents f2b63be + a5e92a7
commit 72bc782
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/app/models/public_json.rb b/app/models/public_json.rb
@@ -10,6 +10,8 @@ def data
   end
 
   def cocina_hash
+    return {} unless data_type == 'cocina' # Legacy metadata (xml), doesn't set this
+
     JSON.parse(data)
   end
 end
diff --git a/spec/factories/public_jsons.rb b/spec/factories/public_jsons.rb
@@ -1,5 +1,7 @@
 FactoryBot.define do
   factory :public_json do
+    data_type { 'cocina' }
+
     data do
       Cocina::RSpec::Factories.build(:dro_with_metadata, id: purl.druid)
                               .new(access: { view: 'world' },