ims-lti 1.1.9 → 1.1.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 60aadf51a888ea3b20d4e01db315fa516b2391d5
-  data.tar.gz: a2de5ab6dc06de39360cc798d1eb084390cde810
+  metadata.gz: c7f1feeb8de427e084dd4e91ec1cd22eb5848dc0
+  data.tar.gz: 54f156903788d5f10a763b6409bc7b443c8cec8c
 SHA512:
-  metadata.gz: 0b81bc2d944853f4aea17886035dd9c157fa5f98de1b33f6e24f4bc0b515ab3504df6542bdafe7345153816b008ece1ac92cf40686bb672e4afcd1acc46fa90c
-  data.tar.gz: c57d082100c2493e07d7d5a426303a6f7ea541f4b57b303febba34513ada770d7d37e639b78f241b31ef4703f1a331c4d04274ae111ac811819432a9fc83d18c
+  metadata.gz: fb94e636da4a9b07e6be2c7ed528b8bb367ff33a2726580f0b085ba0af9547d534d77ca2bbc2375bd782db116408ddda1ac3ac936825f018aa141007812a0b8a
+  data.tar.gz: 3e03fe6b42605945b1b08bf6dc5e95bab898414505ac58ba841bd1c6370fb226971d094774541f76f8189e4636c28377d6006e42fe64dc8fca3f455732ce1cfa

data/lib/ims/lti/tool_config.rb

@@ -115,13 +115,7 @@ module IMS::LTI
           platform = vendor_ext_node.attributes['platform']
           properties = {}
           set_properties(properties, vendor_ext_node)
-          REXML::XPath.each(vendor_ext_node, 'lticm:options', LTI_NAMESPACES) do |options_node|
-            opt_name = options_node.attributes['name']
-            options = {}
-            set_properties(options, options_node)
-            properties[opt_name] = options
-          end
-
+          set_options(properties, vendor_ext_node)
           self.set_ext_params(platform, properties)
         end
 
@@ -175,17 +169,7 @@ module IMS::LTI
             ext_params = @extensions[ext_platform]
             blti_node.blti(:extensions, :platform => ext_platform) do |ext_node|
               ext_params.keys.sort.each do |key|
-                val = ext_params[key]
-                if val.is_a?(Hash)
-                  ext_node.lticm(:options, :name => key) do |type_node|
-                    val.keys.sort.each do |p_key|
-                      p_val = val[p_key]
-                      type_node.lticm :property, p_val, 'name' => p_key
-                    end
-                  end
-                else
-                  ext_node.lticm :property, val, 'name' => key
-                end
+                nest_xml(ext_node, key, ext_params[key])
               end
             end
           end
@@ -199,6 +183,18 @@ module IMS::LTI
 
     private
 
+    def nest_xml(ext_node, key, value)
+      if value.is_a?(Hash)
+        ext_node.lticm(:options, :name => key) do |type_node|
+          value.keys.sort.each do |sub_key|
+            nest_xml(type_node, sub_key, value[sub_key])
+          end
+        end
+      else
+        ext_node.lticm :property, value, 'name' => key
+      end
+    end
+
     def get_node_text(node, path)
       if val = REXML::XPath.first(node, path, LTI_NAMESPACES)
         val.text
@@ -221,5 +217,15 @@ module IMS::LTI
       end
     end
 
+    def set_options(hash, node)
+      REXML::XPath.each(node, 'lticm:options', LTI_NAMESPACES) do |options_node|
+        opt_name = options_node.attributes['name']
+        options = {}
+        set_properties(options, options_node)
+        set_options(options, options_node)
+        hash[opt_name] = options
+      end
+    end
+
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ims-lti
 version: !ruby/object:Gem::Version
-  version: 1.1.9
+  version: 1.1.10
 platform: ruby
 authors:
 - Instructure