Merge pull request #339 from oat-sa/fix/TR-3424/unmarshall-list-insid…

…e-record-container fix: Record container handle list when unmarshalling
oat-sa · Nov 2, 2022 · 9378bc2 · 9378bc2
2 parents 0d7c8a3 + ee50a19
commit 9378bc2
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 0 deletions.
diff --git a/qtism/runtime/pci/json/Unmarshaller.php b/qtism/runtime/pci/json/Unmarshaller.php
@@ -254,6 +254,9 @@ protected function unmarshallUnit(array $unit)
                 case 'identifier':
                     return $this->unmarshallIdentifier($unit);
                     break;
+                case 'list':
+                    return $this->unmarshallList($unit['base']);
+                    break;
 
                 default:
                     throw new UnmarshallingException("Unknown QTI baseType '" . $keys[0] . "'");

diff --git a/test/qtismtest/runtime/pci/json/JsonUnmarshallerTest.php b/test/qtismtest/runtime/pci/json/JsonUnmarshallerTest.php
@@ -353,6 +353,22 @@ public function unmarshallRecordProvider()
         $json = '{ "record" : [ { "name": "A" } ] }';
         $returnValue[] = [$record, $json];
 
+        $record = new RecordContainer(
+            [
+                'A' => new MultipleContainer(
+                    BaseType::STRING,
+                    [
+                        new QtiString('p'),
+                        new QtiString('a'),
+                        new QtiString('p'),
+                        new QtiString('e'),
+                        new QtiString('r'),
+                    ]
+                )
+            ]);
+        $json = '{ "record" : [ {"name": "A","list": {"string": ["p","a","p","e","r"]} } ] }';
+        $returnValue[] = [$record, $json];
+
         return $returnValue;
     }