When handling a report, megaqc checks to see if that SampleDataType already exists. However it only checks on the basis of data_id, but ignores data_section. Therefore if multiple report types (data sections) reuse the same data_id, currently this will reuse that SampleDataType even if data_section is wrong for the incoming report.
This PR adjusts the logic so that it checks for previous SampleDataType entries based on data_key, which includes both data_id and data_section.
PR Checklist
docsis updateddocs/changelog.mdis updatedDescription of changes
SampleDataTypealready exists. However it only checks on the basis ofdata_id, but ignoresdata_section. Therefore if multiple report types (data sections) reuse the samedata_id, currently this will reuse thatSampleDataTypeeven ifdata_sectionis wrong for the incoming report.SampleDataTypeentries based ondata_key, which includes bothdata_idanddata_section.