In computeCount() for MigrateSourceCSV there is an issue in the code for embedded newlines.
The first is that the comment for skipping headers is misleading ("Skip all but the last header": it isn't, it's skipping all of them). The second is that the loop to skip headers is using fgets(), which doesn't take into account embedded newlines itself, so if there are any in the header row it will skip the wrong number. The code should be using $this->getNextLine() instead.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | header-embedded-newlines-2770093-3.patch | 536 bytes | karlshea |
Comments
Comment #2
karlsheaComment #3
karlsheaReroll
Comment #4
pifagor commentedComment #6
pifagor commentedComment #8
joseph.olstadhttps://www.drupal.org/project/migrate/releases/7.x-2.12