Bug #11646

CSV import should not import blank rows

Added by David Juhasz over 1 year ago. Updated over 1 year ago.

Status:VerifiedStart date:10/25/2017
Priority:MediumDue date:
Assignee:-% Done:

0%

Category:CSV import
Target version:Release 2.5.0
Google Code Legacy ID: Tested version:2.4
Sponsored:No Requires documentation:

Description

Reported on the forum:
https://groups.google.com/forum/?utm_medium=email&utm_source=footer#!msg/ica-atom-users/oov5qBAIwDg/vwKjNjEDAwAJ

The CSV import should skip any rows consist entirely of a string of commas with no data, e.g:
,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,,

History

#1 Updated by Mike Cantelon over 1 year ago

  • Status changed from New to Code Review
  • Assignee set to Nick Wilkinson

#2 Updated by Nick Wilkinson over 1 year ago

  • Assignee changed from Nick Wilkinson to Steve Breker

Hi Steve, passing to you for CR.

#3 Updated by Steve Breker over 1 year ago

  • Status changed from Code Review to Feedback
  • Assignee changed from Steve Breker to Mike Cantelon

Looks good. Radda put a couple notes on the PR about capitalizing some of the new comments which is a good idea.

#4 Updated by Mike Cantelon over 1 year ago

  • Status changed from Feedback to QA/Review
  • Assignee changed from Mike Cantelon to Dan Gillean

Merged into qa/2.5.x.

#5 Updated by Dan Gillean over 1 year ago

  • Target version set to Release 2.5.0

#6 Updated by Dan Gillean over 1 year ago

  • Status changed from QA/Review to Verified
  • Assignee deleted (Dan Gillean)

Also available in: Atom PDF