Restructuring the mutation of the Manifest Object to use "rendering"

Removing the mutator method ManifestBuilder::FasterIIIFManifest#rendering=
pulibrary · Oct 23, 2017 · 103724f · 103724f
1 parent e1ef1e4
commit 103724f
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 11 deletions.
diff --git a/app/services/manifest_builder/faster_iiif_manifest.rb b/app/services/manifest_builder/faster_iiif_manifest.rb
@@ -48,10 +48,6 @@ def license=(license)
  inner_hash["license"] = license
  end
 
- def rendering=(rendering)
- inner_hash["rendering"] = rendering
- end
-
  def initial_attributes
  {
  "@context" => "http://iiif.io/api/presentation/2/context.json",

diff --git a/app/services/manifest_builder/rendering_builder.rb b/app/services/manifest_builder/rendering_builder.rb
@@ -15,13 +15,7 @@ def initialize(resource)
  # @return [IIIF::Presentation::Manifest]
  def apply(manifest)
  # This is currently here to work around https:/iiif-prezi/osullivan/issues/56
- if identifier?
- if manifest.is_a? FasterIIIFManifest
- manifest.rendering = rendering_hash
- else
- manifest.insert(-1, 'rendering', rendering_hash)
- end
- end
+ manifest['rendering'] = rendering_hash if identifier?
  manifest
  end