From 99942fb0f27d9caa0d8acb59685e3aeaa46f3709 Mon Sep 17 00:00:00 2001 From: alexfurmenkov Date: Mon, 16 Feb 2026 17:15:52 +0100 Subject: [PATCH 1/3] #1267 changed full path to original path in stdm reporting --- cdisc_rules_engine/services/reporting/sdtm_report_data.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/cdisc_rules_engine/services/reporting/sdtm_report_data.py b/cdisc_rules_engine/services/reporting/sdtm_report_data.py index b895f9059..b197c8cf8 100644 --- a/cdisc_rules_engine/services/reporting/sdtm_report_data.py +++ b/cdisc_rules_engine/services/reporting/sdtm_report_data.py @@ -194,7 +194,9 @@ def get_dataset_details_data(self) -> list[dict]: { "filename": dataset.name, "label": dataset.label, - "path": str(Path(dataset.full_path or "").parent), + "path": str( + Path(dataset.original_path or dataset.full_path or "").parent + ), "modification_date": dataset.modification_date, "size_kb": (dataset.file_size or 0) / 1000, "length": dataset.record_count, From 7f7b5c0cb5fb9e1b6d1a7bfc488b2d98a47ba357 Mon Sep 17 00:00:00 2001 From: alexfurmenkov Date: Mon, 23 Feb 2026 09:54:34 +0100 Subject: [PATCH 2/3] #1267 left only original_path and fixed test_get_export dataset_metadata to include original_path --- cdisc_rules_engine/services/reporting/sdtm_report_data.py | 4 +--- tests/unit/test_services/test_reporting/test_excel_export.py | 1 + 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/cdisc_rules_engine/services/reporting/sdtm_report_data.py b/cdisc_rules_engine/services/reporting/sdtm_report_data.py index b197c8cf8..2a9369259 100644 --- a/cdisc_rules_engine/services/reporting/sdtm_report_data.py +++ b/cdisc_rules_engine/services/reporting/sdtm_report_data.py @@ -194,9 +194,7 @@ def get_dataset_details_data(self) -> list[dict]: { "filename": dataset.name, "label": dataset.label, - "path": str( - Path(dataset.original_path or dataset.full_path or "").parent - ), + "path": str(Path(dataset.original_path or "").parent), "modification_date": dataset.modification_date, "size_kb": (dataset.file_size or 0) / 1000, "length": dataset.record_count, diff --git a/tests/unit/test_services/test_reporting/test_excel_export.py b/tests/unit/test_services/test_reporting/test_excel_export.py index aa305b49a..5a47b7a4e 100644 --- a/tests/unit/test_services/test_reporting/test_excel_export.py +++ b/tests/unit/test_services/test_reporting/test_excel_export.py @@ -39,6 +39,7 @@ def test_get_export(mock_validation_results): "filename": "test.xpt", "label": "Test Data", "full_path": str(Path("tests/unit/text.xpt")), + "original_path": str(Path("tests/unit/text.xpt")), "modification_date": "2022-04-19T16:17:45", "file_size": 20000, "record_count": 700, From f7f683c8a1ae5344209e0161252762bc5f40ffd8 Mon Sep 17 00:00:00 2001 From: alexfurmenkov Date: Wed, 25 Feb 2026 20:14:44 +0100 Subject: [PATCH 3/3] #1267 returned full_path for cases when original path is not filled --- cdisc_rules_engine/services/reporting/sdtm_report_data.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/cdisc_rules_engine/services/reporting/sdtm_report_data.py b/cdisc_rules_engine/services/reporting/sdtm_report_data.py index 2a9369259..b197c8cf8 100644 --- a/cdisc_rules_engine/services/reporting/sdtm_report_data.py +++ b/cdisc_rules_engine/services/reporting/sdtm_report_data.py @@ -194,7 +194,9 @@ def get_dataset_details_data(self) -> list[dict]: { "filename": dataset.name, "label": dataset.label, - "path": str(Path(dataset.original_path or "").parent), + "path": str( + Path(dataset.original_path or dataset.full_path or "").parent + ), "modification_date": dataset.modification_date, "size_kb": (dataset.file_size or 0) / 1000, "length": dataset.record_count,