Opened 4 years ago

Closed 4 years ago

#896 closed task (fixed)

INCA import should include laterality mapping column

Reported by: olle Owned by: olle
Priority: minor Milestone: Reggie v4.6
Component: net.sf.basedb.reggie Keywords:
Cc:

Description

The first version of INCA import excluded the column used for mapping file entries to laterality for SCAN-B cases, "A030Sida_Beskrivning" from import. It is now desired that this column should also be imported, so the column header name should be removed from the exclude list.

Change History (6)

comment:1 Changed 4 years ago by olle

Owner: changed from Nicklas Nordborg to olle

Ticket reassigned to "olle".

comment:2 Changed 4 years ago by olle

Status: newassigned

Ticket accepted.

comment:3 Changed 4 years ago by olle

Traceability note:

  • Creation of a csv file to be used when requesting information from the INCA database was introduced in Ticket #487 (Export information intended for INCA).
  • Import of an INCA csv file was introduced in Ticket #525 (Import data from INCA).

comment:4 Changed 4 years ago by olle

(In [3999]) Refs #896. INCA import was updated to allow data in laterality mapping column with header "A030Sida_Beskrivning" to be imported:

  1. Java servlet class/file IncaServlet.java in src/net/sf/basedb/reggie/servlet/ updated by removing laterality mapping column header "A030Sida_Beskrivning" from string list unImportedHeaders.

comment:5 Changed 4 years ago by olle

Test:

  • Setup: A new annotation type named "INCA_A030Sida_Beskrivning" with item type Sample, value type String, and belonging to category "INCA", was created in BASE on a local system. An INCA import was performed using a modified INCA file, where personal numbers had been replaced with the fake personal numbers used in the test system.
  • Result: Inspection in BASE of INCA annotations for some cases on the local test system after the INCA import, showed that the annotations of the new annotation type "INCA_A030Sida_Beskrivning" existed with values corresponding to what was expected from the laterality annotations of the cases. The test therefore passed with successful result.

comment:6 Changed 4 years ago by olle

Resolution: fixed
Status: assignedclosed

Ticket closed since the requested change has been added.

Note: See TracTickets for help on using tickets.