Bug 279410 - Csv Importer plugin doesn't detect badly formatted date
Summary: Csv Importer plugin doesn't detect badly formatted date
Status: RESOLVED FIXED
Alias: None
Product: kmymoney
Classification: Applications
Component: general (show other bugs)
Version: SVN
Platform: Ubuntu Linux
: NOR normal
Target Milestone: ---
Assignee: KMyMoney Devel Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-04 23:05 UTC by allan
Modified: 2011-08-05 12:48 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description allan 2011-08-04 23:05:52 UTC
Version:           SVN trunk (using KDE 4.6.0) 
OS:                Linux

Part of the date validation detects an embedded space, treats it as a date field separator and replaces it with a '/' character.  If the actual date is prefixed or suffixed with a space, that too is converted to a separator and an invalid date is detected and flagged, causing the import to fail.

Reproducible: Always

Steps to Reproduce:
Import a csv file containing the date "26/06/11 " .  The plugin replaces the trailing ' ' character with a '/', producing a date "26/06/11/".  This generates four sub-fields, to which the plugin objects.

Actual Results:  
As above.

Expected Results:  
The output date should be ""26/06/2011" - no trailing '/'.

Is there time for this to be committed prior to next release?  It's a single (or half-) line patch, to trim the text date during checking/conversion.
Comment 1 Thomas Baumgart 2011-08-05 06:44:10 UTC
The release is planned for 8. August. If you can apply the fix sooner than that, it should be OK.
Comment 2 allan 2011-08-05 12:48:21 UTC
SVN commit 1245148 by allananderson:

BUG:279410
Part of the date validation detects an embedded space, treats it as a date
field separator and replaces it with a '/' character.  If the actual date is
prefixed or suffixed with a space, that too is converted to a separator and an
invalid date is detected and flagged, causing the import to fail.

 M  +1 -1      convdate.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1245148