Merge lp:~jri-openerp/enapps-csv-import-tool/python_csv_reader into lp:~enapps-london-uk/enapps-csv-import-tool/old_trunk
Proposed by
Juan Rial (OpenERP)
Status: | Needs review | ||||
---|---|---|---|---|---|
Proposed branch: | lp:~jri-openerp/enapps-csv-import-tool/python_csv_reader | ||||
Merge into: | lp:~enapps-london-uk/enapps-csv-import-tool/old_trunk | ||||
Diff against target: |
77 lines (+11/-19) 2 files modified
ea_import/ea_import_chain.py (+4/-16) ea_import/ea_import_template.py (+7/-3) |
||||
To merge this branch: | bzr merge lp:~jri-openerp/enapps-csv-import-tool/python_csv_reader | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Enapps Ltd OpenERP Partner London UK | Pending | ||
Review via email: mp+106373@code.launchpad.net |
Description of the change
Fixes 2 issues:
1) LP Bug #999180: multiline text fields weren't properly handled since the file was read and parsed line by line
2) When the separator occurs within a text field (enclosed by string delimiters), the field it split in two, resulting in an incorrect line being parsed
To post a comment you must log in.
Unmerged revisions
- 4. By Juan Rial (OpenERP)
-
[FIX] LP Bug #999180: inline trivial file_to_list() functionality
- 3. By Juan Rial (OpenERP)
-
[FIX] LP Bug #999180: first commit, switching to python's own csv reader.
- 2. By Enapps Ltd OpenERP Partner London UK
-
fix invalid model reference