From 31b6a716f213d5a2c47950fcf53e437c3eb55a96 Mon Sep 17 00:00:00 2001 From: Raphael Malikian Date: Fri, 19 Jun 2026 05:28:33 -0700 Subject: [PATCH] fix: wrap comma-containing field in quotes in v5.4 Field_Level CSV The userGuidance field for cdm_source.source_documentation_reference contains a comma ('...source data, e.g. a data dictionary.') that was not wrapped in quotes, causing CSV parsers to split it into 14 fields instead of 13. Adds double quotes around the field value to properly escape the comma. Fixes #775 Fixes #744 Signed-off-by: Raphael Malikian --- inst/csv/OMOP_CDMv5.4_Field_Level.csv | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/inst/csv/OMOP_CDMv5.4_Field_Level.csv b/inst/csv/OMOP_CDMv5.4_Field_Level.csv index 9e463df0..22fb656e 100644 --- a/inst/csv/OMOP_CDMv5.4_Field_Level.csv +++ b/inst/csv/OMOP_CDMv5.4_Field_Level.csv @@ -396,7 +396,7 @@ cdm_source,cdm_source_name,Yes,varchar(255),The name of the CDM instance.,NA,No, cdm_source,cdm_source_abbreviation,Yes,varchar(25),The abbreviation of the CDM instance.,NA,No,No,NA,NA,NA,NA,NA cdm_source,cdm_holder,Yes,varchar(255),The holder of the CDM instance.,NA,No,No,NA,NA,NA,NA,NA cdm_source,source_description,No,varchar(MAX),The description of the CDM instance.,NA,No,No,NA,NA,NA,NA,NA -cdm_source,source_documentation_reference,No,varchar(255),Refers to a publication or web resource describing the source data, e.g. a data dictionary.,NA,No,No,NA,NA,NA,NA,NA +cdm_source,source_documentation_reference,No,varchar(255),"Refers to a publication or web resource describing the source data, e.g. a data dictionary.",NA,No,No,NA,NA,NA,NA,NA cdm_source,cdm_etl_reference,No,varchar(255),NA,Version of the ETL script used. e.g. link to the Git release,No,No,NA,NA,NA,NA,NA cdm_source,source_release_date,Yes,date,The date the data was extracted from the source system. In some systems that is the same as the date the ETL was run. Typically the latest even date in the source is on the source_release_date.,NA,No,No,NA,NA,NA,NA,NA cdm_source,cdm_release_date,Yes,date,The date the ETL script was completed. Typically this is after the source_release_date.,NA,No,No,NA,NA,NA,NA,NA