qti 2.25.0 → 2.26.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 658160efbc5ba67a6ba35d425c885e76f9d8ece8f6fbe9673287a04bd11b2a3b
-  data.tar.gz: 21bec86a59792ebf56fdbaf1f4833fa41c04fce92c65919824333851b550178a
+  metadata.gz: 9f3871afe8e84ed37180922c577051db9fc09ccd3936a74353e19dfb8a688dc8
+  data.tar.gz: d8be8fe548bc1e89d1e8a595275272291553bb89df682942c22bf53070aa32f6
 SHA512:
-  metadata.gz: e5d3223e5aab2d4d7f1cb2d4f8af2792e3a4c8cc37c9b2ff58d385e388cd2a6d658eae6c6eba842585445d0072e521a01b0bfbd856956373e80ee0e44b56dcd5
-  data.tar.gz: 2f25762910192f38a8c5da3183fc9d8fc7fd3f8109990ec00ee1646f3a111de959226f00e5c74d4b52f64e7da488df95e4d27b111fcd4d1c0265814317d09be0
+  metadata.gz: 570712ae37a5c79f34ac12d5f1ff02f7a5591035fb0ab16d0c01066b61b4a6546317e8049c4427e8ed3c63e013c25a638fc93ec59490fd088d6d17606748d105
+  data.tar.gz: 48b6d948fdbd2f867e98737f7456459ab9206b1446e8c4d77c65b6928f272dce2696ba1fb1b9842b2e5d770f8d5d35c5276019da19e9bdf2d52a6d18987a3880

data/lib/qti/sanitizer.rb

@@ -20,9 +20,58 @@ module Qti
       Sanitize::Config::RELAXED[:attributes][element] + overrides
     end
 
+    # Copied from Canvas (Classic Quizzes)
+    # canvas_sanitize/lib/canvas_sanitize/canvas_sanitize.rb:142
+    MATHML_TAGS =
+      %w[annotation
+         annotationml
+         maction
+         maligngroup
+         malignmark
+         mark
+         math
+         menclose
+         merror
+         mfenced
+         mfrac
+         mglyph
+         mi
+         mlabeledtr
+         mlongdiv
+         mmultiscripts
+         mn
+         mo
+         mover
+         mpadded
+         mphantom
+         mprescripts
+         mroot
+         mrow
+         ms
+         mscarries
+         mscarry
+         msgroup
+         msline
+         mspace
+         msqrt
+         msrow
+         mstack
+         mstyle
+         msub
+         msubsup
+         msup
+         mtable
+         mtd
+         mtext
+         mtr
+         munder
+         munderover
+         none
+         semantics].freeze
+
     CONFIG =
       {
-        elements: Sanitize::Config::RELAXED[:elements] + FILTER_TAGS,
+        elements: Sanitize::Config::RELAXED[:elements] + MATHML_TAGS + FILTER_TAGS,
         protocols:
           {
             'iframe' => { 'src' => PROTOCOLS },

data/lib/qti/v1/models/interactions/hot_spot_interaction.rb

@@ -31,22 +31,32 @@ module Qti
           end
 
           def shape_type
-            @shape_type ||= response_label&.attributes&.[]('rarea')&.value
+            @shape_type ||= extract_shape_type(response_label)
           end
 
           def coordinates
-            @coordinates ||= begin
-              raw_coordinates_array = raw_coordinates.split(',')
-              return [] unless (raw_coordinates_array.length % 2).zero?
+            @coordinates ||= parse_coordinates(raw_coordinates)
+          end
 
-              raw_coordinates_array.each_slice(2).map do |point|
-                { x: point[0].to_f, y: point[1].to_f }
-              end
-            end
+          def scoring_data_structs
+            @scoring_data_structs ||= parse_scoring_data
           end
 
           private
 
+          def parse_scoring_data
+            raw_scoring_data = answer_nodes&.map do |answer|
+              {
+                id: extract_identifier(answer),
+                type: extract_shape_type(answer),
+                coordinates: parse_coordinates(answer&.text || '')
+              }
+            end
+            raw_scoring_data.map do |values|
+              Models::ScoringData.new(values, rcardinality)
+            end
+          end
+
           def response_label
             @response_label ||= begin
               node = @node.dup
@@ -55,10 +65,35 @@ module Qti
             end
           end
 
+          def answer_nodes
+            @answer_nodes ||= begin
+              node = @node.dup
+              presentation = node.at_xpath('.//xmlns:presentation')
+              presentation.xpath('.//xmlns:response_label')
+            end
+          end
+
           def raw_coordinates
             @raw_coordinates ||= response_label&.text || ''
           end
 
+          def extract_shape_type(answer)
+            answer&.attributes&.[]('rarea')&.value
+          end
+
+          def extract_identifier(answer)
+            answer&.attributes&.[]('ident')&.value
+          end
+
+          def parse_coordinates(raw_coordinates)
+            raw_coordinates_array = raw_coordinates.split(',')
+            return [] unless (raw_coordinates_array.length % 2).zero?
+
+            raw_coordinates_array.each_slice(2).map do |point|
+              { x: point[0].to_f, y: point[1].to_f }
+            end
+          end
+
           def coordinate_hash(coordinate_node)
             %w[x y].map do |axis|
               [axis.to_sym, coordinate_node&.attributes&.[](axis)&.value&.to_f]

data/lib/qti/version.rb

@@ -1,3 +1,3 @@
 module Qti
-  VERSION = '2.25.0'.freeze
+  VERSION = '2.26.0'.freeze
 end

data/lib/qti.rb

@@ -1,3 +1,4 @@
+require 'logger'
 require 'action_view'
 require 'active_support/core_ext/array/access'
 require 'active_support/core_ext/string'

data/spec/fixtures/items_1.2/hot_spot.xml

@@ -39,7 +39,7 @@
                 <material>
                   <matimage uri="$IMS-CC-FILEBASE$/Uploaded Media/someuuid"/>
                 </material>
-                <response_label ident="response1" rarea="square">0.372,0.1376,0.616,0.4256</response_label>
+                <response_label ident="response1" rarea="rectangle">0.372,0.1376,0.616,0.4256</response_label>
               </render_hotspot>
             </response_xy>
           </flow>
@@ -76,7 +76,7 @@
                 <material>
                   <matimage uri="$IMS-CC-FILEBASE$/Uploaded Media/someuuid"/>
                 </material>
-                <response_label ident="response1" rarea="square">0.372</response_label>
+                <response_label ident="response1" rarea="rectangle">0.372</response_label>
               </render_hotspot>
             </response_xy>
           </flow>
@@ -113,7 +113,46 @@
                 <material>
                   <matimage uri="$IMS-CC-FILEBASE$/Uploaded Media/someuuid"/>
                 </material>
-                <response_label ident="response1" rarea="square"></response_label>
+                <response_label ident="response1" rarea="rectangle"></response_label>
+              </render_hotspot>
+            </response_xy>
+          </flow>
+        </presentation>
+      </item>
+      <item ident="g515255148f5a3cae1832584071aa8529" title="hot-spot with multiple selections">
+        <itemmetadata>
+          <qtimetadata>
+            <qtimetadatafield>
+              <fieldlabel>question_type</fieldlabel>
+              <fieldentry>hot_spot_question</fieldentry>
+            </qtimetadatafield>
+            <qtimetadatafield>
+              <fieldlabel>points_possible</fieldlabel>
+              <fieldentry>1.0</fieldentry>
+            </qtimetadatafield>
+            <qtimetadatafield>
+              <fieldlabel>original_answer_ids</fieldlabel>
+              <fieldentry/>
+            </qtimetadatafield>
+            <qtimetadatafield>
+              <fieldlabel>assessment_question_identifierref</fieldlabel>
+              <fieldentry>gd6ff15355f3dd3f532816fa268230130</fieldentry>
+            </qtimetadatafield>
+          </qtimetadata>
+        </itemmetadata>
+        <presentation>
+          <flow>
+            <response_xy ident="response1" rcardinality="Multiple" rtiming="No">
+              <material>
+                <mattext texttype="text/html">&lt;p&gt;point something on the image&lt;/p&gt;</mattext>
+              </material>
+              <render_hotspot>
+                <material>
+                  <matimage uri="$IMS-CC-FILEBASE$/Uploaded Media/someuuid"/>
+                </material>
+                <response_label ident="response1" rarea="rectangle">0.1,0.2,0.3,0.4</response_label>
+                <response_label ident="response2" rarea="ellipse">0.5,0.6,0.7,0.8</response_label>
+                <response_label ident="response3" rarea="bounded">0.9,0.1,0.2,0.3</response_label>
               </render_hotspot>
             </response_xy>
           </flow>

data/spec/lib/qti/sanitizer_spec.rb

@@ -63,5 +63,11 @@ describe Qti::Sanitizer do
       expect(sanitizer.clean(html)).to include 'height: 294px;'
       expect(sanitizer.clean(html)).to include 'display: inline-block;'
     end
+
+    Qti::Sanitizer::MATHML_TAGS.each do |tag|
+      it "allows MathML tag: #{tag}" do
+        expect(sanitizer.clean("<#{tag}>")).to eq("<#{tag}></#{tag}>")
+      end
+    end
   end
 end

data/spec/lib/qti/v1/models/interactions/hot_spot_interaction_spec.rb

@@ -28,7 +28,7 @@ describe Qti::V1::Models::Interactions::HotSpotInteraction do
   describe '#shape_type' do
     let(:loaded_class) { described_class.new(assessment_item_refs.first, test_object) }
     it 'returns correct shape type' do
-      expect(loaded_class.shape_type).to eq 'square'
+      expect(loaded_class.shape_type).to eq 'rectangle'
     end
   end
 
@@ -54,4 +54,35 @@ describe Qti::V1::Models::Interactions::HotSpotInteraction do
       end
     end
   end
+
+  describe '#scoring_data_structs' do
+    context 'with a single hotspot' do
+      let(:loaded_class) { described_class.new(assessment_item_refs.first, test_object) }
+
+      it 'returns scoring data with a single hotspot' do
+        scoring_data = loaded_class.scoring_data_structs
+        expect(scoring_data.size).to eq(1)
+        expect(scoring_data.first.values[:type]).to eq('rectangle')
+        expect(scoring_data.first.values[:coordinates]).to eq([{ x: 0.372, y: 0.1376 }, { x: 0.616, y: 0.4256 }])
+      end
+    end
+
+    context 'with multiple hotspots' do
+      let(:loaded_class) { described_class.new(assessment_item_refs[3], test_object) }
+
+      it 'returns scoring data with multiple hotspots' do
+        scoring_data = loaded_class.scoring_data_structs
+        expect(scoring_data.size).to eq(3)
+
+        expect(scoring_data[0].values[:type]).to eq('rectangle')
+        expect(scoring_data[0].values[:coordinates]).to eq([{ x: 0.1, y: 0.2 }, { x: 0.3, y: 0.4 }])
+
+        expect(scoring_data[1].values[:type]).to eq('ellipse')
+        expect(scoring_data[1].values[:coordinates]).to eq([{ x: 0.5, y: 0.6 }, { x: 0.7, y: 0.8 }])
+
+        expect(scoring_data[2].values[:type]).to eq('bounded')
+        expect(scoring_data[2].values[:coordinates]).to eq([{ x: 0.9, y: 0.1 }, { x: 0.2, y: 0.3 }])
+      end
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: qti
 version: !ruby/object:Gem::Version
-  version: 2.25.0
+  version: 2.26.0
 platform: ruby
 authors:
 - Adrian Diaz
@@ -12,7 +12,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-10-23 00:00:00.000000000 Z
+date: 2025-03-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionview