Opened 15 years ago

Closed 15 years ago

#566 closed enhancement (fixed)

Plugins using the flat file parser should warn about complex column mappings

Reported by: Nicklas Nordborg Owned by: Nicklas Nordborg
Priority: major Milestone: BASE 2.4
Component: coreplugins Version:
Keywords: Cc:

Description

Most mappings are from a single file column to a single data base column, ie. \Row\ The flat file parser allows combined mappings that can use multiple columns and/or constant strings in any combination, ie. \Row\,\Coloumn\

This kind of mapping is unusual and in three recent cases it was an error (an extra space or slash at the end of the column mapping).

By default, plugins should only allow the simple case and return an error message in the complex case. Then, a user may correct an incorrect mapping or use an extra parameter to specify if the complext mapping should be allowed or not.

See also ticket #563.

Change History (8)

comment:1 Changed 15 years ago by Jari Häkkinen

Milestone: BASE 2.4BASE 2.3

Milestone BASE 2.4 deleted

comment:2 Changed 15 years ago by Nicklas Nordborg

Owner: changed from Johan Enell to Nicklas Nordborg
Status: newassigned

comment:3 Changed 15 years ago by Nicklas Nordborg

Resolution: fixed
Status: assignedclosed

(In [3493]) Fixes #566

comment:4 Changed 15 years ago by Nicklas Nordborg

Resolution: fixed
Status: closedreopened

The 'Complex column mapping' parameter is sneaking itself into the 'Column mappings' part of the 'Test with file' function. It is not a mapping expression and should not be visible there.

comment:5 Changed 15 years ago by Nicklas Nordborg

Status: reopenednew

comment:6 Changed 15 years ago by Nicklas Nordborg

Status: newassigned

comment:7 Changed 15 years ago by Nicklas Nordborg

The AnnotationFlatFileImporter? doesn't check the column mappings as it should.

comment:8 Changed 15 years ago by Nicklas Nordborg

Resolution: fixed
Status: assignedclosed

(In [3598]) Fixes #566: Plugins using the flat file parser should warn about complex column mappings

Note: See TracTickets for help on using tickets.