A bit of code cleanup.

pstaabp · pstaabp · commit 44cce4cfbaa4 · 2025-07-08T15:00:25.000-04:00
diff --git a/lib/WeBWorK/Utils/Rendering.pm b/lib/WeBWorK/Utils/Rendering.pm
@@ -257,7 +257,7 @@ sub renderPG ($c, $effectiveUser, $set, $problem, $psvn, $formFields, $translati
 		my @deprecated_macros;
 		for (keys %{ $pg->{pgcore}{PG_loadMacros}{macroFileList} }) {
 			my @dirs = split('/', $_);
-			push(@deprecated_macros, $dirs[$#dirs]) if $dirs[ $#dirs - 1 ] eq 'deprecated';
+			push(@deprecated_macros, $dirs[-1]) if $dirs[-2] eq 'deprecated';
 		}
 
 		if (ref($pg->{pgcore}) eq 'PGcore') {

Original file line number	Diff line number	Diff line change
`@@ -257,7 +257,7 @@ sub renderPG ($c, $effectiveUser, $set, $problem, $psvn, $formFields, $translati`
`257`	`257`	`my @deprecated_macros;`
`258`	`258`	`for (keys %{ $pg->{pgcore}{PG_loadMacros}{macroFileList} }) {`
`259`	`259`	`my @dirs = split('/', $_);`
`260`		`- push(@deprecated_macros, $dirs[$#dirs]) if $dirs[ $#dirs - 1 ] eq 'deprecated';`
	`260`	`+ push(@deprecated_macros, $dirs[-1]) if $dirs[-2] eq 'deprecated';`
`261`	`261`	`}`
`262`	`262`
`263`	`263`	`if (ref($pg->{pgcore}) eq 'PGcore') {`