-
Notifications
You must be signed in to change notification settings - Fork 35
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
cannot properly parse tsv #35
Comments
vrtám se v tom dál, vypadá to, že za to může enclosure Line 153 in d80c540
null char enclosure $fh = fopen($filenameFrom, "r");
while ($parsed = fgetcsv($fh, null, "\t", chr(0))) {
var_dump($parsed);
}
no enclosure $fh = fopen($filenameFrom, "r");
while ($parsed = fgetcsv($fh, null, "\t")) {
var_dump($parsed);
}
|
když skript rozdělím na $fh = fopen($filenameFrom, "r");
while ($line = fgets($fh)) {
var_dump($line);
var_dump(str_getcsv($line, "\t", chr(0)));
}
fclose($fh);
jsou tam divný ty newliny a otazníky v posledních prvích polí řádků |
$fh = fopen($filenameFrom, "r");
while ($line = fgets($fh)) {
var_dump($line);
var_dump(str_getcsv($line, "\t", "", ""));
}
fclose($fh);
|
možná by to celý stálo za report do php bugtrackeru? |
jenže jinak tech bůgu je tam hlášených několik, např. https://bugs.php.net/bug.php?id=51496 |
https://csv.thephpleague.com/9.0/interoperability/rfc4180-field/ tady je to taky linkované a nějak to zkouší fixovat. Ale naše testovací CSV to moc nedalo https://keboola.slack.com/archives/C02C3GZUS/p1518859662000040 |
V tom testu ale testuješ csv který nemá enclosure ani escape. To nikdy nemůže spolehlivě fungovat. |
Pokud tomu nastavím enclosure na |
Chtělo by to větší sample kde to způsobovalo problém. |
Zkusím do té konfigurace narvat |
Jo, prošlo! OMG, se s tím seru celej den :-( |
Zaleží ale co pak je v tom CSV dál. Jestli nemá enclousure a najednou se někde v datech |
V současanejch datech to nebylo, takže snad cajk! |
php-csv says this file has only one line
demonstrated in #34
The text was updated successfully, but these errors were encountered: