Review Request: import csv-file into existing table

Oleg Kukharchuk oleg.kuh at gmail.com
Thu Dec 13 23:35:25 GMT 2012



> On Nov. 25, 2012, 6:15 p.m., Jarosław Staniek wrote:
> > "Fields count does not match" error is displayed on appending if the destination table is identical but contains extra primary-key column (generated by Kexi). I know this case is just not supported now. We would try to import if no primary key is defined for imported data but it is present in the destination table. This could happen silently.
> 
> Oleg Kukharchuk wrote:
>     I think this logic will be added in future with code for columns managment

Fixed


- Oleg


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107330/#review22525
-----------------------------------------------------------


On Dec. 5, 2012, 3:46 p.m., Oleg Kukharchuk wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107330/
> -----------------------------------------------------------
> 
> (Updated Dec. 5, 2012, 3:46 p.m.)
> 
> 
> Review request for Calligra, Jarosław Staniek, Adam Pigg, and Dimitrios Tanis.
> 
> 
> Description
> -------
> 
> * moved to a KAssistantDialog
> * added ability to import into an existent table
> 
> 
> Diffs
> -----
> 
>   kexi/plugins/importexport/csv/kexicsvimportdialog.h 15be3504dfc54877bc45d81eb7e59a72f70c9c5a 
>   kexi/plugins/importexport/csv/kexicsvimportdialog.cpp 5176299599ab034bf256f07ee2651d8a4babb3a0 
> 
> Diff: http://git.reviewboard.kde.org/r/107330/diff/
> 
> 
> Testing
> -------
> 
> tested on local branch
> and additional testing needed
> 
> (code was not cleaned up yet)
> 
> 
> Thanks,
> 
> Oleg Kukharchuk
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20121213/38f35596/attachment.htm>


More information about the calligra-devel mailing list