From a9b797bda79c84d06701b8c222232f4be733c2b8 Mon Sep 17 00:00:00 2001 From: root Date: Tue, 8 Sep 2026 11:48:56 +0000 Subject: [PATCH 1/2] Allow results_metadata to be explicitly None. --- openeo_driver/jobregistry.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/openeo_driver/jobregistry.py b/openeo_driver/jobregistry.py index 5e5632f4..9b91cfcf 100644 --- a/openeo_driver/jobregistry.py +++ b/openeo_driver/jobregistry.py @@ -199,7 +199,10 @@ def map_safe(prop: str, f): return f(value) if value else None def get_results_metadata(result_metadata_prop: str): - return job_info.get("results_metadata", {}).get(result_metadata_prop) + # Note: `results_metadata` can be explicitly `None` (not just absent) when result metadata + # is stored externally instead (see `results_metadata_uri`), so `.get(..., {})` alone is not + # enough to avoid `NoneType has no attribute 'get'`. + return (job_info.get("results_metadata") or {}).get(result_metadata_prop) def map_results_metadata_safe(result_metadata_prop: str, f): value = get_results_metadata(result_metadata_prop) From 98c5a7b3368ae09a609191ca1542e447ef8c65fd Mon Sep 17 00:00:00 2001 From: Emile Sonneveld Date: Tue, 8 Sep 2026 13:57:42 +0200 Subject: [PATCH 2/2] Remove verbose comment. --- openeo_driver/jobregistry.py | 3 --- 1 file changed, 3 deletions(-) diff --git a/openeo_driver/jobregistry.py b/openeo_driver/jobregistry.py index 9b91cfcf..b3054407 100644 --- a/openeo_driver/jobregistry.py +++ b/openeo_driver/jobregistry.py @@ -199,9 +199,6 @@ def map_safe(prop: str, f): return f(value) if value else None def get_results_metadata(result_metadata_prop: str): - # Note: `results_metadata` can be explicitly `None` (not just absent) when result metadata - # is stored externally instead (see `results_metadata_uri`), so `.get(..., {})` alone is not - # enough to avoid `NoneType has no attribute 'get'`. return (job_info.get("results_metadata") or {}).get(result_metadata_prop) def map_results_metadata_safe(result_metadata_prop: str, f):