From 088287955265330ef2eb2479b507d88574754655 Mon Sep 17 00:00:00 2001
From: Tom Arild Jakobsen <tajakobsen@gmail.com>
Date: Mon, 28 Nov 2016 14:54:33 +0100
Subject: [PATCH 1/3] Allow subContentId in group validation. Relates to:
 #HFP-285

---
 h5p.classes.php | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/h5p.classes.php b/h5p.classes.php
index 4d7223b..03d5136 100644
--- a/h5p.classes.php
+++ b/h5p.classes.php
@@ -3063,6 +3063,11 @@ class H5PContentValidator {
     }
     else {
       foreach ($group as $key => &$value) {
+        // If subContentId is set, keep value
+        if($key == 'subContentId' && $value == TRUE){
+          continue;
+        }
+
         // Find semantics for name=$key
         $found = FALSE;
         foreach ($semantics->fields as $field) {

From 5494443533aa25b3a3a117f299b3f87200a35897 Mon Sep 17 00:00:00 2001
From: Tom Arild Jakobsen <tajakobsen@gmail.com>
Date: Tue, 29 Nov 2016 15:44:32 +0100
Subject: [PATCH 2/3] validateGroup must handle isSubContent

 * Update validateGroup in h4p.classes.php to allow subContentId into the data
 * Fix h5p-content-upgrade-process.js to take into account isSubContent

Relates to: #HFP-285
---
 h5p.classes.php                   | 6 ++++--
 js/h5p-content-upgrade-process.js | 2 +-
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/h5p.classes.php b/h5p.classes.php
index 03d5136..a403856 100644
--- a/h5p.classes.php
+++ b/h5p.classes.php
@@ -3056,7 +3056,9 @@ class H5PContentValidator {
     $function = null;
     $field = null;
 
-    if (count($semantics->fields) == 1 && $flatten) {
+    $isSubContent = isset($semantics->isSubContent) && $semantics->isSubContent === TRUE;
+
+    if (count($semantics->fields) == 1 && $flatten && !$isSubContent) {
       $field = $semantics->fields[0];
       $function = $this->typeMap[$field->type];
       $this->$function($group, $field);
@@ -3064,7 +3066,7 @@ class H5PContentValidator {
     else {
       foreach ($group as $key => &$value) {
         // If subContentId is set, keep value
-        if($key == 'subContentId' && $value == TRUE){
+        if($isSubContent && ($key == 'subContentId')){
           continue;
         }
 
diff --git a/js/h5p-content-upgrade-process.js b/js/h5p-content-upgrade-process.js
index 1d886cc..e54dbb7 100644
--- a/js/h5p-content-upgrade-process.js
+++ b/js/h5p-content-upgrade-process.js
@@ -182,7 +182,7 @@ H5P.ContentUpgradeProcess = (function (Version) {
         break;
 
       case 'group':
-        if (field.fields.length === 1) {
+        if (field.fields.length === 1 && field.isSubContent !== true) {
           // Single field to process, wrapper will be skipped
           self.processField(field.fields[0], params, function (err, upgradedParams) {
             if (upgradedParams) {

From 32d06356b19a584c20dced4fc7eedbc583ed9974 Mon Sep 17 00:00:00 2001
From: Tom Arild Jakobsen <tajakobsen@gmail.com>
Date: Tue, 29 Nov 2016 16:44:20 +0100
Subject: [PATCH 3/3] Bump to version 1.11

Relates to: #HFP-285
---
 h5p.classes.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/h5p.classes.php b/h5p.classes.php
index a403856..f564114 100644
--- a/h5p.classes.php
+++ b/h5p.classes.php
@@ -1676,7 +1676,7 @@ class H5PCore {
 
   public static $coreApi = array(
     'majorVersion' => 1,
-    'minorVersion' => 10
+    'minorVersion' => 11
   );
   public static $styles = array(
     'styles/h5p.css',