Merge pull request #1032 from Alex-Jordan/hotfix/NchooseK

HOTFIX: force list context when random_subset used in dprecated macros
openwebwork · Mar 13, 2024 · 26f77df · 26f77df
2 parents 9cefdf9 + 3fb9c0f
commit 26f77df
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/macros/ui/PGchoicemacros.pl b/macros/ui/PGchoicemacros.pl
@@ -800,7 +800,9 @@ sub invert {
 
 sub NchooseK {
 	my ($n, $k) = @_;
-	return random_subset($k, 0 .. $n - 1);
+	# force list context
+	my @return = random_subset($k, 0 .. $n - 1);
+	return @return;
 }
 
 =item [DEPRECATED] shuffle()
@@ -815,7 +817,9 @@ sub NchooseK {
 
 sub shuffle {
 	my ($i) = @_;
-	return random_subset($i, 0 .. $i - 1);
+	# force list context
+	my @return = random_subset($i, 0 .. $i - 1);
+	return @return;
 }
 
 =item [DEPRECATED] match_questions_list()