Problem/Motivation

When a CSV has more than one empty line at the end, the CSV parser never completes parsing as on each iteration it believes that the end of the file has not been reached yet.

Proposed resolution

As soon as parsing yields no more results, set parser progress to completed.

CommentFileSizeAuthor
#2 feeds-csv-extra-lines-infinite-loop-2996253-2.patch5.32 KBmegachriz
#2 feeds-csv-extra-lines-infinite-loop-2996253-2-tests-only.patch4.79 KBmegachriz

Comments

MegaChriz created an issue. See original summary.

megachriz’s picture

megachriz
Primary language Dutch
commented
Status: Active » Needs review
StatusFileSize
new feeds-csv-extra-lines-infinite-loop-2996253-2-tests-only.patch4.79 KB
new feeds-csv-extra-lines-infinite-loop-2996253-2.patch5.32 KB

The last submitted patch, 2: feeds-csv-extra-lines-infinite-loop-2996253-2-tests-only.patch, failed testing. View results

  • MegaChriz committed 32282bb on 8.x-3.x 
    Issue #2996253 by MegaChriz: Fixed CSV with extra empty lines never...
megachriz’s picture

megachriz
Primary language Dutch
commented
Status: Needs review » Fixed

Committed #2.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.