Merge pull request #6668 from avalonmediasystem/section_thumbnails

Pass section thumbnail urls to iiif_manifest

Author: cjcolvar

app/models/iiif_canvas_presenter.rb

@@ -228,7 +228,8 @@ def manifest_attributes(quality, media_type, mimetype: 'application/x-mpegURL')
         height: (master_file.height || MasterFile::AUDIO_HEIGHT).to_i,
         duration: stream_info[:duration],
         type: media_type,
-        format: mimetype
+        format: mimetype,
+        thumbnail: [{ id: thumbnail_url, type: 'Image' }]
       }.compact
 
       if master_file.media_object.visibility == 'public'
@@ -308,4 +309,12 @@ def auth_service(quality)
         ]
       }
     end
+
+    def thumbnail_url
+      if master_file.is_video?
+        Rails.application.routes.url_helpers.thumbnail_master_file_url(master_file.id)
+      else
+        ActionController::Base.helpers.asset_url('audio_icon.png')
+      end
+    end
 end

app/models/iiif_playlist_canvas_presenter.rb

@@ -201,7 +201,8 @@ def manifest_attributes(quality, media_type, mimetype: 'application/x-mpegURL')
         height: (master_file.height || MasterFile::AUDIO_HEIGHT).to_i,
         duration: stream_info[:duration],
         type: media_type,
-        format: mimetype
+        format: mimetype,
+        thumbnail: [{ id: thumbnail_url, type: 'Image' }]
       }.compact
 
       if master_file.media_object.visibility == 'public'
@@ -263,4 +264,12 @@ def auth_service(quality)
         ]
       }
     end
+
+    def thumbnail_url
+      if master_file.is_video?
+        Rails.application.routes.url_helpers.thumbnail_master_file_url(master_file.id)
+      else
+        ActionController::Base.helpers.asset_url('audio_icon.png')
+      end
+    end
 end

spec/models/iiif_canvas_presenter_spec.rb

@@ -89,6 +89,10 @@
         expect(subject.height).to eq 40
       end
 
+      it 'has thumbnail' do
+        expect(subject.thumbnail.first[:id]).to start_with "http://test.host/assets/audio_icon"
+      end
+
       context 'with mp3 file' do
         let(:mp3_url) { 'https://streaming.example.com/dir/file.mp3' }
         let(:derivative) { FactoryBot.build(:derivative, hls_url: mp3_url, mime_type: 'audio/mpeg' ) }
@@ -113,6 +117,10 @@
         expect(subject.height).to eq 768
       end
 
+      it 'has thumbnail' do
+        expect(subject.thumbnail.first[:id]).to eq "http://test.host/master_files/#{master_file.id}/thumbnail"
+      end
+
       context 'with mp4 file' do
         let(:mp4_url) { 'https://streaming.example.com/dir/file.mp4' }
         let(:derivative) { FactoryBot.build(:derivative, hls_url: mp4_url, mime_type: 'video/mp4' ) }

spec/models/iiif_playlist_canvas_presenter_spec.rb

@@ -91,6 +91,10 @@
         expect(subject.height).to eq 40
       end
 
+      it 'has thumbnail' do
+        expect(subject.thumbnail.first[:id]).to start_with "http://test.host/assets/audio_icon"
+      end
+
       context 'with mp3 file' do
         let(:mp3_url) { 'https://streaming.example.com/dir/file.mp3' }
         let(:derivative) { FactoryBot.build(:derivative, hls_url: mp3_url, mime_type: 'audio/mpeg' ) }
@@ -119,6 +123,10 @@
         expect(subject.height).to eq 768
       end
 
+      it 'has thumbnail' do
+        expect(subject.thumbnail.first[:id]).to eq "http://test.host/master_files/#{master_file.id}/thumbnail"
+      end
+
       context 'with mp4 file' do
         let(:mp4_url) { 'https://streaming.example.com/dir/file.mp4' }
         let(:derivative) { FactoryBot.build(:derivative, hls_url: mp4_url, mime_type: 'video/mp4' ) }