From 103724f137d9ddfdde919e55e56b0c30469a7eaa Mon Sep 17 00:00:00 2001
From: "James R. Griffin III" <jrgriffiniii@gmail.com>
Date: Mon, 23 Oct 2017 09:39:11 -0400
Subject: [PATCH] Restructuring the mutation of the Manifest Object to use
 "rendering"

Removing the mutator method ManifestBuilder::FasterIIIFManifest#rendering=
---
 app/services/manifest_builder/faster_iiif_manifest.rb | 4 ----
 app/services/manifest_builder/rendering_builder.rb    | 8 +-------
 2 files changed, 1 insertion(+), 11 deletions(-)

diff --git a/app/services/manifest_builder/faster_iiif_manifest.rb b/app/services/manifest_builder/faster_iiif_manifest.rb
index 5d81e2b7be..94f250d256 100644
--- 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
index 2d671fed89..14e2d6e6e3 100644
--- 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://github.com/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