Merge pull request #3347 from ConnectGrid/optimize-worksheet-from-array

Check $strictNullComparison outside of loops
This commit is contained in:
oleibman
2024-10-06 05:01:55 +00:00
committed by GitHub
+14 -7
View File
@@ -2784,23 +2784,30 @@ class Worksheet implements IComparable
[$startColumn, $startRow] = Coordinate::coordinateFromString($startCell);
// Loop through $source
foreach ($source as $rowData) {
$currentColumn = $startColumn;
foreach ($rowData as $cellValue) {
if ($strictNullComparison) {
if ($strictNullComparison) {
foreach ($source as $rowData) {
$currentColumn = $startColumn;
foreach ($rowData as $cellValue) {
if ($cellValue !== $nullValue) {
// Set cell value
$this->getCell($currentColumn . $startRow)->setValue($cellValue);
}
} else {
++$currentColumn;
}
++$startRow;
}
} else {
foreach ($source as $rowData) {
$currentColumn = $startColumn;
foreach ($rowData as $cellValue) {
if ($cellValue != $nullValue) {
// Set cell value
$this->getCell($currentColumn . $startRow)->setValue($cellValue);
}
++$currentColumn;
}
++$currentColumn;
++$startRow;
}
++$startRow;
}
return $this;