Merge branch 'ascholerChemeketa-jquery-replacement-fixes'

Author: bnmnetp

bases/rsptx/interactives/runestone/common/js/runestonebase.js

@@ -103,6 +103,20 @@ export default class RunestoneBase {
         });
     }
 
+    // Helper for parsing boolean data-* attributes
+    // Unset/"false"/"no" means false, anything else, including empty string means true
+    parseBooleanAttribute(element, attributeName) {
+        const attrValue = element.getAttribute(attributeName);
+        if (attrValue === null) {
+            return false;
+        }
+        const lowerValue = attrValue.toLowerCase();
+        if (lowerValue === "false" || lowerValue === "no") {
+            return false;
+        }
+        return true;
+    }
+
     // _`logBookEvent`
     //----------------
     // This function sends the provided ``eventInfo`` to the `hsblog endpoint` of the server. Awaiting this function returns either ``undefined`` (if Runestone services are not available) or the data returned by the server as a JavaScript object (already JSON-decoded).

bases/rsptx/interactives/runestone/mchoice/js/mchoice.js

@@ -25,12 +25,12 @@ export default class MultipleChoice extends RunestoneBase {
         this.useRunestoneServices = opts.useRunestoneServices;
         this.multipleanswers = false;
         this.divid = orig.id;
-        if (this.origElem.getAttribute("data-multipleanswers") === "true" || this.origElem.hasAttribute("data-multipleanswers")) {
+        if (this.parseBooleanAttribute(this.origElem, "data-multipleanswers")) {
             this.multipleanswers = true;
         }
         this.children = this.origElem.childNodes;
         this.random = false;
-        if (this.origElem.hasAttribute("[data-random]") || this.origElem.hasAttribute("data-random")) {
+        if (this.parseBooleanAttribute(this.origElem, "data-random")) {
             this.random = true;
         }
         this.correct = null;