CSVUnitStorage: Use named arguments

lucaswerkmeister · jenkins-bot · commit f4995317b954 · 2025-10-23T16:00:55.000Z
We’re only interested in the escape parameter. Bug: T386113 Change-Id: I299de0fcdec930d72d31e0091e97e5a4ecce1f06 Follows-Up: I7c2309f348335f36870ec4dd8ed8c9e727711a59
diff --git a/lib/includes/Units/CSVUnitStorage.php b/lib/includes/Units/CSVUnitStorage.php
@@ -39,7 +39,7 @@ protected function loadStorageData() {
 			return null;
 		}
 		$data = [];
-		for ( $row = fgetcsv( $f, 0, ',', '"', "" ); $row !== false; $row = fgetcsv( $f, 0, ',', '"', "" ) ) {
+		for ( $row = fgetcsv( $f, escape: "" ); $row !== false; $row = fgetcsv( $f, escape: "" ) ) {
 			// @phan-suppress-next-line PhanTypeMismatchDimAssignment False positive
 			$data[$row[0]] = array_slice( $row, 1 );
 		}

Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ protected function loadStorageData() {`
`39`	`39`	`return null;`
`40`	`40`	`}`
`41`	`41`	`$data = [];`
`42`		`- for ( $row = fgetcsv( $f, 0, ',', '"', "" ); $row !== false; $row = fgetcsv( $f, 0, ',', '"', "" ) ) {`
	`42`	`+ for ( $row = fgetcsv( $f, escape: "" ); $row !== false; $row = fgetcsv( $f, escape: "" ) ) {`
`43`	`43`	`// @phan-suppress-next-line PhanTypeMismatchDimAssignment False positive`
`44`	`44`	`$data[$row[0]] = array_slice( $row, 1 );`
`45`	`45`	`}`