RA-2092: Form Navigator: add configuration parameter to allow empty form

Author: mogoodrich

omod/src/main/webapp/resources/scripts/navigator/navigator.js

@@ -53,10 +53,11 @@ function KeyboardController(formElement) {
         });
 
         var confirmationSection = new ConfirmationSectionModel(
-          $('#confirmation'),
-          breadcrumb, _.clone(sections),
-          formElement.hasClass('skip-confirmation-section'),
-          navButtons);
+            $('#confirmation'),
+            breadcrumb, _.clone(sections),
+formElement.hasClass('skip-confirmation-section') || formElement.find('htmlform').hasClass('skip-confirmation-section'),
+formElement.hasClass('allow-empty-form') || formElement.find('htmlform').hasClass('allow-empty-form'),
+            navButtons);
         sections.push(confirmationSection);
 
         var questions = _.flatten( _.map(sections, function(s) { return s.questions; }), true);

omod/src/main/webapp/resources/scripts/navigator/navigatorModels.js

@@ -607,10 +607,11 @@ SectionModel.prototype.firstInvalidQuestion = function() {
 }
 
 
-function ConfirmationSectionModel(elem, formMenuElem, regularSections, skipConfirmation, navButtons) {
+function ConfirmationSectionModel(elem, formMenuElem, regularSections, skipConfirmation, allowEmptyForm, navButtons) {
     SelectableModel.apply(this, [elem]);
     this.sections = regularSections;
     this.skipConfirmation = skipConfirmation ? skipConfirmation : false;
+    this.allowEmptyForm = allowEmptyForm ? allowEmptyForm : false;
     this.navButtons = navButtons;
 
     var title = this.element.find("span.title").first();
@@ -643,7 +644,6 @@ ConfirmationSectionModel.prototype.select = function() {
 
     if (!this.skipConfirmation) {
         // scan through the form and confirm that at least one of the fields has a value
-        // TODO: move all this out into a separate validator at some point? is the assumption that all forms must have data true?
         // TODO: makes sure this works for radio buttons and checkboxes (once we add them)
         var hasData = _.some(this.sections, function (section) {
             return _.some(section.questions, function (question) {
@@ -653,7 +653,7 @@ ConfirmationSectionModel.prototype.select = function() {
             })
         })
 
-        if (!hasData) {
+        if (!hasData && !this.allowEmptyForm) {
             this.questions[0].confirm.disable();
             this.element.find(".error").show();
         }

omod/src/test/webapp/resources/scripts/navigatorModels.js

@@ -576,7 +576,7 @@ describe("Test for simple form models", function() {
            var menuElement = jasmine.createSpyObj('menu', ['append']);
            var navButtons = jasmine.createSpyObj('navButtons', ['hide','show']);
            var confirmationQuestionModel = jasmine.createSpyObj('confirmationQuestion', ['confirm', 'cancel']);
-           var confirmationSectionModel = new ConfirmationSectionModel( confirmationQuestionModel, menuElement, null, null, navButtons);
+           var confirmationSectionModel = new ConfirmationSectionModel( confirmationQuestionModel, menuElement, null, null, null, navButtons);
            confirmationSectionModel.element = jasmine.createSpyObj('element', ['addClass', 'removeClass', 'triggerHandler']);
            confirmationSectionModel.element.find = function () {  // stub out the find method to simply remove an empty jq call
                return jq();