Merge pull request #2949 from drgrice1/bugfix/show-top-warn-pgcore-warnings

dlglin · web-flow · commit e8c293fd9298 · 2026-04-21T14:29:21.000-06:00
Fix a minor issue resulting from #2935.
diff --git a/lib/WeBWorK/ContentGenerator/GatewayQuiz.pm b/lib/WeBWorK/ContentGenerator/GatewayQuiz.pm
@@ -1512,7 +1512,7 @@ async sub getProblemHTML ($c, $effectiveUser, $set, $formFields, $mergedProblem)
 		if $c->{can}{checkAnswers}
 		&& (!$c->{submitAnswers} || defined $c->param('problem_data_' . $mergedProblem->problem_id));
 
-	$c->stash->{haveProblemWarnings} = 1 if $pg->{warnings} || @{ $pg->{pgwarning} // [] };
+	$c->stash->{haveProblemWarnings} = 1 if $pg->{warnings} || @{ $pg->{warning_messages} // [] };
 
 	return $pg;
 }
diff --git a/templates/ContentGenerator/Problem.html.ep b/templates/ContentGenerator/Problem.html.ep
@@ -65,7 +65,7 @@
 % stash->{footerWidthClass} = 'col-lg-10';
 %
 <%== $c->post_header_text =%>
-% if ($c->{pg}{warnings} || @{ $c->{pgwarning} // [] }) {
+% if ($c->{pg}{warnings} || @{ $c->{pg}{warning_messages} // [] }) {
 	<div class="row g-0 mb-2">
 		<div class="col-12 alert alert-danger m-0">
 			<%== maketext('<strong>Warning</strong>: There may be something wrong with this question. '

Original file line number	Diff line number	Diff line change
`@@ -1512,7 +1512,7 @@ async sub getProblemHTML ($c, $effectiveUser, $set, $formFields, $mergedProblem)`
`1512`	`1512`	`if $c->{can}{checkAnswers}`
`1513`	`1513`	`&& (!$c->{submitAnswers} \|\| defined $c->param('problem_data_' . $mergedProblem->problem_id));`
`1514`	`1514`
`1515`		`- $c->stash->{haveProblemWarnings} = 1 if $pg->{warnings} \|\| @{ $pg->{pgwarning} // [] };`
	`1515`	`+ $c->stash->{haveProblemWarnings} = 1 if $pg->{warnings} \|\| @{ $pg->{warning_messages} // [] };`
`1516`	`1516`
`1517`	`1517`	`return $pg;`
`1518`	`1518`	`}`
Original file line number	Diff line number	Diff line change
`@@ -65,7 +65,7 @@`
`65`	`65`	`% stash->{footerWidthClass} = 'col-lg-10';`
`66`	`66`	`%`
`67`	`67`	`<%== $c->post_header_text =%>`
`68`		`-% if ($c->{pg}{warnings} \|\| @{ $c->{pgwarning} // [] }) {`
	`68`	`+% if ($c->{pg}{warnings} \|\| @{ $c->{pg}{warning_messages} // [] }) {`
`69`	`69`	`<div class="row g-0 mb-2">`
`70`	`70`	`<div class="col-12 alert alert-danger m-0">`
`71`	`71`	`<%== maketext('<strong>Warning</strong>: There may be something wrong with this question. '`