@pie-element/ebsr 14.2.7 → 14.2.9

package/module/controller.js

@@ -2710,10 +2710,6 @@ const getShuffledChoices = (choices, session, updateSession, choiceKey) =>
  * - true - that means that the order of the choices will be ordinal (as is created in the configure item)
  * - false - that means the getShuffledChoices above will be called and that in turn means that we either
  * return the shuffled values on the session (if any exists) or we shuffle the choices
- *
- * Note: the role (student/instructor) is intentionally not considered here — instructor mode
- * will respect the same `lockChoiceOrder` value as students, instead of forcing the order to be locked.
- *
  * @param model - model to check if we should lock order
  * @param session - session to check if we should lock order
  * @param env - env to check if we should lock order
@@ -2730,6 +2726,21 @@ const lockChoices = (model, session, env) => {
     return true;
   }
 
+  const role = get$1(env, 'role', 'student');
+
+  if (role === 'instructor') {
+    // TODO: .. in the future the instructor can toggle between ordinal and shuffled here, so keeping this code until then
+    /*const alreadyShuffled = hasShuffledValues(session);
+
+    if (alreadyShuffled) {
+      return false;
+    }
+
+    return true;*/
+    return true;
+  }
+
+  // here it's a student, so don't lock and it will shuffle if needs be
   return false;
 };
 

package/module/demo.js

@@ -8,7 +8,7 @@ generate.model = (id, element) => ({
   id,
   element,
   partA: {
-    choiceMode: 'radio',
+    choiceMode: 'checkbox',
     choices: [
       {
         value: 'yellow',
@@ -30,7 +30,7 @@ generate.model = (id, element) => ({
     promptEnabled: true,
   },
   partB: {
-    choiceMode: 'radio',
+    choiceMode: 'checkbox',
     choices: [
       {
         value: 'orange',