Merge pull request #6030 from avalonmediasystem/default_manifest

Provide default required metadata in IIIF Manifest

app/models/iiif_manifest_presenter.rb

@@ -45,7 +45,7 @@ def description
   end
 
   def to_s
-    media_object.title
+    media_object.title || media_object.id
   end
 
   def manifest_metadata
@@ -180,7 +180,7 @@ def display_lending_period(media_object)
 
   def iiif_metadata_fields
     fields = [
-      metadata_field('Title', media_object.title),
+      metadata_field('Title', media_object.title, media_object.id),
       metadata_field('Date', combined_display_date(media_object), 'Not provided'),
       metadata_field('Main contributor', media_object.creator),
       metadata_field('Summary', display_summary(media_object)),

spec/models/iiif_manifest_presenter_spec.rb

@@ -19,6 +19,19 @@
   let(:master_file) { FactoryBot.build(:master_file, media_object: media_object) }
   let(:presenter) { described_class.new(media_object: media_object, master_files: [master_file]) }
 
+  context 'to_s' do
+    it "returns the media object's title" do
+      expect(presenter.to_s).to eq media_object.title
+    end
+
+    context 'when media object is missing title' do
+      it "returns the media object's id" do
+        allow(media_object).to receive(:title).and_return(nil)
+        expect(presenter.to_s).to eq media_object.id
+      end
+    end
+  end
+
   context 'homepage' do
     subject { presenter.homepage.first }