diff --git a/lib/HardcopyRenderedProblem.pm b/lib/HardcopyRenderedProblem.pm index 76380647d0..fc2f9b0f32 100644 --- a/lib/HardcopyRenderedProblem.pm +++ b/lib/HardcopyRenderedProblem.pm @@ -265,8 +265,7 @@ sub write_problem_tex { $correctTeX .= "\\item\n\$\\displaystyle " . ($rh_result->{answers}{$_}{correct_ans_latex_string} - || "\\text{$rh_result->{answers}{$_}{correct_ans}}") - . "\$\n"; + || "\\text{$rh_result->{answers}{$_}{correct_ans}}") . "\$\n"; } $correctTeX .= "\\end{itemize}}\\par\n"; diff --git a/lib/WeBWorK/Authen/LTIAdvantage.pm b/lib/WeBWorK/Authen/LTIAdvantage.pm index 0d2b22f792..48f8c848c9 100644 --- a/lib/WeBWorK/Authen/LTIAdvantage.pm +++ b/lib/WeBWorK/Authen/LTIAdvantage.pm @@ -323,8 +323,7 @@ sub authenticate ($self) { "Account creation blocked by block_lti_create_user setting. Did not create user $self->{user_id}."; if ($ce->{debug_lti_parameters}) { warn $c->maketext('Account creation is currently disabled in this course. ' - . 'Please speak to your instructor or system administrator.') - . "\n"; + . 'Please speak to your instructor or system administrator.') . "\n"; } return 0; } else { diff --git a/lib/WeBWorK/ConfigValues.pm b/lib/WeBWorK/ConfigValues.pm index a30a0785ff..58955fc34e 100644 --- a/lib/WeBWorK/ConfigValues.pm +++ b/lib/WeBWorK/ConfigValues.pm @@ -1202,7 +1202,7 @@ sub getConfigValues ($ce) { @$configValues, [ x('LTI'), - map { $LTIConfigValues->{$_} } + map { $LTIConfigValues->{$_} } grep { defined $LTIConfigValues->{$_} } @{ $ce->{LTIConfigVariables} } ] ); diff --git a/lib/WeBWorK/ContentGenerator/GatewayQuiz.pm b/lib/WeBWorK/ContentGenerator/GatewayQuiz.pm index 438e46f5ad..1bc6960f6a 100644 --- a/lib/WeBWorK/ContentGenerator/GatewayQuiz.pm +++ b/lib/WeBWorK/ContentGenerator/GatewayQuiz.pm @@ -498,7 +498,7 @@ async sub pre_header_initialize ($c) { if ($setVersionNumber && !$c->{invalidSet} && $setID ne 'Undefined_Set') { my @setVersionIDs = $db->listSetVersions($effectiveUserID, $setID); - my @setVersions = $db->getSetVersions(map { [ $effectiveUserID, $setID,, $_ ] } @setVersionIDs); + my @setVersions = $db->getSetVersions(map { [ $effectiveUserID, $setID, $_ ] } @setVersionIDs); for (@setVersions) { $totalNumVersions++; $currentNumVersions++ diff --git a/lib/WeBWorK/ContentGenerator/Hardcopy.pm b/lib/WeBWorK/ContentGenerator/Hardcopy.pm index ecfc9bc34f..c618e4a185 100644 --- a/lib/WeBWorK/ContentGenerator/Hardcopy.pm +++ b/lib/WeBWorK/ContentGenerator/Hardcopy.pm @@ -1007,8 +1007,7 @@ async sub write_set_tex ($c, $FH, $TargetUser, $themeTree, $setID) { { print $FH '\\def\\webworkReducedScoringDate{' . ($c->formatDateTime($MergedSet->{reduced_scoring_date}, $ce->{studentDateDisplayFormat}) =~ - s/\x{202f}/ /gr) - . "}%\n"; + s/\x{202f}/ /gr) . "}%\n"; } # write set header (theme presetheader, then PG header, then theme postsetheader) @@ -1181,10 +1180,10 @@ async sub write_problem_tex ($c, $FH, $TargetUser, $MergedSet, $themeTree, $prob problemID => $MergedProblem->problem_id, ), $MergedProblem->problem_id == 0 - # link for a fake problem (like a header file) + # link for a fake problem (like a header file) ? (params => { sourceFilePath => $MergedProblem->source_file, problemSeed => $MergedProblem->problem_seed }) - # link for a real problem + # link for a real problem : (), ); diff --git a/lib/WeBWorK/ContentGenerator/Instructor/UserList.pm b/lib/WeBWorK/ContentGenerator/Instructor/UserList.pm index 62df5978c3..bd31a8d6c9 100644 --- a/lib/WeBWorK/ContentGenerator/Instructor/UserList.pm +++ b/lib/WeBWorK/ContentGenerator/Instructor/UserList.pm @@ -237,15 +237,15 @@ sub pre_header_initialize ($c) { # Always have a definite sort order in case the first three sorts don't determine things. $c->{sortedUserIDs} = [ map { $_->user_id } - sort { - $primarySortSub->() + sort { + $primarySortSub->() || $secondarySortSub->() || $ternarySortSub->() || byLastName() || byFirstName() || byUserID() } - grep { $c->{visibleUserIDs}{ $_->user_id } } (values %allUsers) + grep { $c->{visibleUserIDs}{ $_->user_id } } (values %allUsers) ]; return;