Merge pull request #2271 from Alex-Jordan/randomize-seeds

button to randomize individual seeds on set detail page for one user
openwebwork · Jan 15, 2024 · 2ed01cf · 2ed01cf
2 parents a45ca4b + b0dfa9b
commit 2ed01cf
Show file tree

Hide file tree

Showing 3 changed files with 59 additions and 1 deletion.
diff --git a/htdocs/js/ProblemSetDetail/problemsetdetail.js b/htdocs/js/ProblemSetDetail/problemsetdetail.js
@@ -414,4 +414,27 @@
 		comboBoxSelect.addEventListener('change',
 			() => comboBoxText.value = comboBoxSelect.options[comboBoxSelect.selectedIndex].value);
 	});
+
+	// Set up seed randomization buttons.
+	const randomize_seeds_button = document.getElementById('randomize_seeds');
+	const randomize_seed_buttons = document.querySelectorAll('.randomize-seed-btn');
+	if (randomize_seeds_button) {
+		randomize_seeds_button.addEventListener('click',
+			() => (randomize_seed_buttons.forEach((btn) => {
+				const exclude_correct = document.getElementById('excludeCorrect').checked;
+				const input = document.getElementById(btn.dataset.seedInput);
+				const stat  = document.getElementById(btn.dataset.statusInput).value || 0;
+				if (input) {
+					if (!exclude_correct || (exclude_correct && stat < 1)) {
+						input.value = Math.floor(Math.random() * 10000);
+					}
+				}
+			}))
+		)
+	}
+	for (const btn of randomize_seed_buttons) {
+		const input = document.getElementById(btn.dataset.seedInput);
+		if (input) btn.addEventListener('click', () => (input.value = Math.floor(Math.random() * 10000)));
+	}
+
 })();
diff --git a/lib/WeBWorK/ContentGenerator/Instructor/ProblemSetDetail.pm b/lib/WeBWorK/ContentGenerator/Instructor/ProblemSetDetail.pm
@@ -937,14 +937,38 @@ sub fieldHTML ($c, $userID, $setID, $problemID, $globalRecord, $userRecord, $fie
 			my $value = $forUsers ? $userValue : $globalValue;
 			$value = format_set_name_display($value =~ s/\s*,\s*/,/gr) if $field eq 'restricted_release';
 
-			$inputType = $c->text_field(
+			my @field_args = (
 				"$recordType.$recordID.$field", $value,
 				id    => "$recordType.$recordID.${field}_id",
 				data  => { override => "$recordType.$recordID.$field.override_id" },
 				class => 'form-control form-control-sm',
 				$forUsers && $check ? (aria_labelledby => "$recordType.$recordID.$field.label") : (),
 				$field eq 'restricted_release' || $field eq 'source_file' ? (dir => 'ltr')      : ()
 			);
+			if ($field eq 'problem_seed') {
+				# Insert a randomization button
+				$inputType = $c->tag(
+					'div',
+					class => 'input-group input-group-sm',
+					style => 'min-width: 7rem',
+					$c->c(
+						$c->number_field(@field_args, min => 0),
+						$c->tag(
+							'button',
+							type  => 'button',
+							class => 'randomize-seed-btn btn btn-sm btn-secondary',
+							title => 'randomize',
+							data  => {
+								seed_input   => "$recordType.$recordID.problem_seed_id",
+								status_input => "$recordType.$recordID.status_id"
+							},
+							$c->tag('i', class => 'fa-solid fa-shuffle')
+						)
+					)->join('')
+				);
+			} else {
+				$inputType = $c->text_field(@field_args);
+			}
 		}
 	} elsif ($choose) {
 		# If $field matches /:/, then multiple fields are used.

diff --git a/templates/ContentGenerator/Instructor/ProblemSetDetail.html.ep b/templates/ContentGenerator/Instructor/ProblemSetDetail.html.ep
@@ -348,6 +348,17 @@
 						<%= maketext('Hide All') =%>
 					</button>
 				</div>
+			% } else {
+				<div class="input-group d-inline-flex flex-nowrap w-auto py-1 me-3">
+					<button id="randomize_seeds" type="button" class="btn btn-secondary">
+						<%= maketext('Randomize Seeds') =%>
+					</button>
+					<label class="form-check-label input-group-text ps-0">
+						<%= check_box excludeCorrect => 0,
+							id => 'excludeCorrect', class => 'form-check-input mx-2' =%>
+						<%= maketext('if status less than 1') =%>
+					</label>
+				</div>
 			% }
 			% if (!@editForUser) {
 				<div class="btn-group w-auto me-3 py-1">