Skip to content

Commit

Permalink
Merge pull request #531 from remicollet/issue-php84
Browse files Browse the repository at this point in the history
fgetcsv default values are deprecated with 8.4
  • Loading branch information
viest authored Oct 7, 2024
2 parents 33070cd + 45ddb5e commit d1571c5
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
6 changes: 3 additions & 3 deletions tests/xlsx_to_csv_callback.phpt
Original file line number Diff line number Diff line change
Expand Up @@ -30,8 +30,8 @@ var_dump($csvResult);

$fp = fopen('./tests/file.csv', 'r');

var_dump(fgetcsv($fp));
var_dump(fgetcsv($fp));
var_dump(fgetcsv($fp, 1000, ',', '"', '\\'));
var_dump(fgetcsv($fp, 1000, ',', '"', '\\'));
?>
--CLEAN--
<?php
Expand All @@ -55,4 +55,4 @@ array(4) {
string(2) "10"
[3]=>
string(10) "10.9999995"
}
}
4 changes: 2 additions & 2 deletions tests/xlsx_to_csv_callback_custom_delimiter.phpt
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ if (($csvHandler = fopen('./tests/file.csv', 'r')) === FALSE) {
die('csv file open failure');
}

while (($data = fgetcsv($csvHandler, 1000, ';')) !== FALSE) {
while (($data = fgetcsv($csvHandler, 1000, ';', '"', '\\')) !== FALSE) {
var_dump($data);
}
?>
Expand All @@ -56,4 +56,4 @@ array(4) {
string(2) "10"
[3]=>
string(10) "10.9999995"
}
}
2 changes: 1 addition & 1 deletion tests/xlsx_to_csv_custom_delimiter.phpt
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ if (($csvHandler = fopen('./tests/file.csv', 'r')) === FALSE) {
die('csv file open failure');
}

while (($data = fgetcsv($csvHandler, 1000, ';')) !== FALSE) {
while (($data = fgetcsv($csvHandler, 1000, ';', '"', '\\')) !== FALSE) {
var_dump($data);
}
?>
Expand Down

0 comments on commit d1571c5

Please sign in to comment.