Add gallery image JSON

Gemfile

@@ -24,6 +24,7 @@ gem 'feedjira', '~> 2.0'
 gem 'fog', '~> 1.33'
 gem 'globalize', '~> 5.0'
 gem 'globalize-versioning', github: 'globalize/globalize-versioning'
+gem 'jbuilder', '~> 2.6.0'
 gem 'lograge'
 gem 'logstash-event'
 gem 'logstash-logger'

Gemfile.lock

@@ -346,6 +346,9 @@ GEM
     inflecto (0.0.2)
     ipaddress (0.8.3)
     iso-639 (0.2.8)
+    jbuilder (2.6.1)
+      activesupport (>= 3.0.0, < 5.1)
+      multi_json (~> 1.2)
     jquery-rails (4.2.1)
       rails-dom-testing (>= 1, < 3)
       railties (>= 4.2.0)
@@ -629,6 +632,7 @@ DEPENDENCIES
   globalize-versioning!
   htmlcompressor (= 0.3)
   i18n_data
+  jbuilder (~> 2.6.0)
   localeapp (~> 1.0)
   lograge
   logstash-event

app/controllers/galleries_controller.rb

@@ -7,7 +7,7 @@ def index
   end
 
   def show
-    @gallery = Gallery.find_by_slug(params[:id])
+    @gallery = Gallery.find_by_slug(params[:slug])
     @documents = search_api_for_image_metadata(@gallery.images)
   end
 

app/controllers/gallery_images_controller.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+class GalleryImagesController < ApplicationController
+  def show
+    @gallery = Gallery.find_by_slug(params[:gallery_slug])
+    @image = @gallery.images.where(position: params[:position]).first
+    @response, @document = fetch(@image.europeana_record_id, api_query_params)
+
+    respond_to do |format|
+      format.json
+    end
+  end
+
+  protected
+
+  def api_query_params
+    params.slice(:api_url)
+  end
+end

app/views/gallery_images/show.json.jbuilder

@@ -0,0 +1,9 @@
+# frozen_string_literal: true
+presenter = Document::RecordPresenter.new(@document, controller)
+
+json.description presenter.field_value('proxies.dcDescription')
+json.creation_date presenter.field_value('proxies.dcDate')
+json.data_provider presenter.field_value('aggregations.edmDataProvider')
+json.provider presenter.field_value('aggregations.edmProvider')
+json.rights presenter.simple_rights_label_data
+json.url_collection search_url(q: %(edm_datasetName:"#{presenter.field_value('edmDatasetName')}"))

config/routes.rb

@@ -51,8 +51,10 @@
     get 'explore/topics', to: 'explore#topics'
     get 'explore/periods', to: 'explore#periods'
 
-    scope '/explore' do
-      resources :galleries, only: [:show, :index]
+    scope 'explore' do
+      resources :galleries, only: [:show, :index], param: :slug do
+        resources :gallery_images, only: :show, path: 'images', as: 'images', param: :position
+      end
     end
 
     get 'entities/suggest'

spec/controllers/galleries_controller_spec.rb

@@ -35,17 +35,17 @@
       let(:gallery) { Gallery.published.first }
 
       it 'returns http success' do
-        get :show, locale: 'en', id: gallery.slug
+        get :show, locale: 'en', slug: gallery.slug
         expect(response).to have_http_status(:success)
       end
 
       it 'assigns gallery to @gallery' do
-        get :show, locale: 'en', id: gallery.slug
+        get :show, locale: 'en', slug: gallery.slug
         expect(assigns[:gallery]).to eq(gallery)
       end
 
       it 'searches the API for the gallery image metadata' do
-        get :show, locale: 'en', id: gallery.slug
+        get :show, locale: 'en', slug: gallery.slug
         ids = gallery.images.map(&:europeana_record_id)
         api_query = %[europeana_id:("#{ids.join('" OR "')}")]
         expect(an_api_search_request.

spec/controllers/gallery_images_controller_spec.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+RSpec.describe GalleryImagesController do
+  describe '#show' do
+    context 'when gallery is published' do
+      let(:gallery) { Gallery.published.first }
+      let(:image) { gallery.images.first }
+      let(:params) { { locale: 'en', gallery_slug: gallery.slug, position: image.position, format: 'json' } }
+
+      it 'returns http success' do
+        get :show, params
+        expect(response).to have_http_status(:success)
+      end
+
+      it 'assigns gallery to @gallery' do
+        get :show, params
+        expect(assigns[:gallery]).to eq(gallery)
+      end
+
+      it 'assigns image to @image' do
+        get :show, params
+        expect(assigns[:image]).to eq(image)
+      end
+
+      it 'requests image metadata from the record API' do
+        get :show, params
+        expect(an_api_record_request_for(image.europeana_record_id)).
+          to have_been_made.at_least_once
+      end
+    end
+  end
+end

spec/routing/galleries_routing_spec.rb

@@ -5,6 +5,6 @@
   end
 
   it 'routes GET /en/explore/galleries/high-heels to galleries#show' do
-    expect(get('/en/explore/galleries/high-heels')).to route_to('galleries#show', locale: 'en', id: 'high-heels')
+    expect(get('/en/explore/galleries/high-heels')).to route_to('galleries#show', locale: 'en', slug: 'high-heels')
   end
 end

spec/routing/gallery_images_routing_spec.rb

@@ -0,0 +1,6 @@
+# frozen_string_literal: true
+RSpec.describe 'routes for the gallery images controller' do
+  it 'routes GET /en/explore/galleries/high-heels/images/2 to gallery_images#show' do
+    expect(get('/en/explore/galleries/high-heels/images/2')).to route_to('gallery_images#show', locale: 'en', gallery_slug: 'high-heels', position: '2')
+  end
+end