From 9db468ab46370482334bd538cabb29fc813675a3 Mon Sep 17 00:00:00 2001
From: Jack Reed <phillipjreed@gmail.com>
Date: Wed, 4 Apr 2018 18:26:32 -0600
Subject: [PATCH] canvas_index to canvas_id

---
 CHANGELOG.md                             | 6 ++++++
 app/helpers/application_helper.rb        | 6 +++---
 config/initializers/blacklight_oembed.rb | 2 +-
 spec/helpers/application_helper_spec.rb  | 4 ++--
 4 files changed, 12 insertions(+), 6 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index ade19d45e..7f704a6d7 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -13,6 +13,12 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 ### Fixed
 ### Security
 
+## [1.17.3] - 2018-04-04
+
+### Fixed
+ - Use the correct parameter that sul-embed expects for canvas id #1145
+
+
 ## [1.17.2] - 2018-04-04
 
 ### Security
diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb
index e8064b078..b119e4ace 100644
--- a/app/helpers/application_helper.rb
+++ b/app/helpers/application_helper.rb
@@ -43,11 +43,11 @@ def render_viewer_in_context(document, block)
   ##
   #
   # @param [SolrDocument] document
-  # @param [Integer] canvas_index
-  def custom_render_oembed_tag_async(document, canvas_index)
+  # @param [Integer] canvas_id
+  def custom_render_oembed_tag_async(document, canvas_id)
     url = context_specific_oembed_url(document)
 
-    content_tag :div, '', data: { embed_url: blacklight_oembed_engine.embed_url(url: url, canvas_index: canvas_index) }
+    content_tag :div, '', data: { embed_url: blacklight_oembed_engine.embed_url(url: url, canvas_id: canvas_id) }
   end
 
   ##
diff --git a/config/initializers/blacklight_oembed.rb b/config/initializers/blacklight_oembed.rb
index 538bc19b3..1cc2456b4 100644
--- a/config/initializers/blacklight_oembed.rb
+++ b/config/initializers/blacklight_oembed.rb
@@ -1 +1 @@
-Blacklight::Oembed::Engine.config.additional_params = [:canvas_index]
+Blacklight::Oembed::Engine.config.additional_params = [:canvas_id]
diff --git a/spec/helpers/application_helper_spec.rb b/spec/helpers/application_helper_spec.rb
index 235d5a369..5a8b71a9f 100644
--- a/spec/helpers/application_helper_spec.rb
+++ b/spec/helpers/application_helper_spec.rb
@@ -30,7 +30,7 @@
         )
         rendered = helper.custom_render_oembed_tag_async(document, 3)
         expect(rendered).to have_css '[data-embed-url="http://test.host/oembed/e'\
-          'mbed?canvas_index=3&url=http%3A%2F%2Fexample.com%2Fstuff"]'
+          'mbed?canvas_id=3&url=http%3A%2F%2Fexample.com%2Fstuff"]'
       end
     end
 
@@ -41,7 +41,7 @@
         )
         rendered = helper.custom_render_oembed_tag_async(document, 3)
         expect(rendered).to have_css '[data-embed-url="http://test.host/oembed/e'\
-          'mbed?canvas_index=3&url=https%3A%2F%2Fsul-purl-uat.stanford.edu%2Fabc123"]'
+          'mbed?canvas_id=3&url=https%3A%2F%2Fsul-purl-uat.stanford.edu%2Fabc123"]'
       end
     end
   end