From 49c1da36277eb145beeff20c19330df8c98e7765 Mon Sep 17 00:00:00 2001
From: thomasmars <thomasmars87@gmail.com>
Date: Mon, 26 Sep 2016 11:45:15 +0200
Subject: [PATCH] Added option for disabling backward navigation

HFP-5
---
 js/questionset.js | 15 ++++++++++-----
 library.json      |  2 +-
 semantics.json    | 10 +++++++++-
 3 files changed, 20 insertions(+), 7 deletions(-)

diff --git a/js/questionset.js b/js/questionset.js
index 912b8b3..7e7e645 100644
--- a/js/questionset.js
+++ b/js/questionset.js
@@ -38,7 +38,7 @@ H5P.QuestionSet = function (options, contentId) {
           '  <% } %>' +
           '  <div class="qs-footer">' +
           '    <div class="qs-progress">' +
-          '      <% if (progressType == "dots") { %>' +
+          '      <% if (progressType == "dots" && !disableBackwardsNavigation) { %>' +
           '        <ul class="dots-container" role="navigation">' +
           '          <% for (var i=0; i<questions.length; i++) { %>' +
           '            <li class="progress-item"><a href="#" class="progress-dot unanswered" ' +
@@ -113,7 +113,8 @@ H5P.QuestionSet = function (options, contentId) {
       retryButtonText: 'Retry',
       showAnimations: false,
       skipButtonText: 'Skip video'
-    }
+    },
+    disableBackwardsNavigation: false
   };
 
   var template = new EJS({text: texttemplate});
@@ -125,7 +126,7 @@ H5P.QuestionSet = function (options, contentId) {
   var $myDom;
   var scoreBar;
   var up;
-  renderSolutions = false;
+  var renderSolutions = false;
 
   var $template = $(template.render(params));
   // Set overrides for questions
@@ -325,9 +326,13 @@ H5P.QuestionSet = function (options, contentId) {
       _displayEndGame();
 
     }
-    else {
+    else if (!params.disableBackwardsNavigation || questionInstances[currentQuestion].getAnswerGiven()) {
+      // Allow movement if backward navigation enabled or answer given
       _showQuestion(currentQuestion + direction);
     }
+    else {
+      //TODO: Give an error message ? or disable/grey out previous button when not allowed
+    }
   };
 
   /**
@@ -594,7 +599,7 @@ H5P.QuestionSet = function (options, contentId) {
       }
 
       // Add previous question button
-      if (questionInstances[0] !== question) {
+      if (questionInstances[0] !== question && !params.disableBackwardsNavigation) {
         question.addButton('prev', '', moveQuestion.bind(this, -1), true, {
           href: '#', // Use href since this is a navigation button
           'aria-label': params.texts.prevButton
diff --git a/library.json b/library.json
index c334485..232a3be 100644
--- a/library.json
+++ b/library.json
@@ -4,7 +4,7 @@
   "contentType": "question",
   "majorVersion": 1,
   "minorVersion": 9,
-  "patchVersion": 0,
+  "patchVersion": 1,
   "embedTypes": [
     "iframe"
   ],
diff --git a/semantics.json b/semantics.json
index be5a854..eaaa265 100644
--- a/semantics.json
+++ b/semantics.json
@@ -193,6 +193,14 @@
       }
     ]
   },
+  {
+    "name": "disableBackwardsNavigation",
+    "type": "boolean",
+    "label": "Disable backwards navigation",
+    "description": "This option will only allow you to move forward in Question Set",
+    "optional": true,
+    "default": false
+  },
   {
     "name": "endGame",
     "type": "group",
@@ -385,4 +393,4 @@
       }
     ]
   }
-]
\ No newline at end of file
+]