fix(plugins): handle queryables with empty details in form

eodag/plugins/search/build_search_result.py

@@ -947,7 +947,7 @@ def queryables_by_form(
 
             prop = {"title": element.get("label", name)}
 
-            details = element.get("details", {})
+            details = element.get("details", {}) or {}
 
             # add values from form if keyword was not in constraints
             values = (
@@ -957,7 +957,8 @@ def queryables_by_form(
             )
 
             # updates the properties with the values given based on the information from the element
-            _update_properties_from_element(prop, element, values)
+            if values is not None:
+                _update_properties_from_element(prop, element, values)
 
             # default value is set with the following priority:
             # input default values > form details default value > no default value

tests/units/test_search_plugins.py

@@ -3996,6 +3996,46 @@ def test_plugins_search_ecmwfsearch_discover_queryables_default_by_form(
                     default_by_form[0], get_args(queryables[ecmwf_name])[1].default
                 )
 
+    @mock.patch(
+        "eodag.plugins.search.build_search_result.ECMWFSearch._fetch_data",
+        autospec=True,
+    )
+    def test_plugins_search_ecmwfsearch_discover_queryables_form_details_none(
+        self, mock__fetch_data
+    ):
+        """Regression: discover_queryables must handle form elements with details set to None."""
+        constraints_path = os.path.join(TEST_RESOURCES_PATH, "constraints.json")
+        with open(constraints_path) as f:
+            constraints = json.load(f)
+        form_path = os.path.join(TEST_RESOURCES_PATH, "form.json")
+        with open(form_path) as f:
+            form = json.load(f)
+
+        for element in form:
+            if element.get("name") == "download_format":
+                element["details"] = None
+                break
+
+        mock__fetch_data.side_effect = [constraints, form]
+
+        default_values = deepcopy(
+            getattr(self.search_plugin.config, "products", {}).get(
+                "CAMS_EU_AIR_QUALITY_RE", {}
+            )
+        )
+        default_values.pop("metadata_mapping", None)
+        params = deepcopy(default_values)
+        params["collection"] = "CAMS_EU_AIR_QUALITY_RE"
+
+        queryables = self.search_plugin.discover_queryables(**params)
+
+        self.assertIsNotNone(queryables)
+        self.assertIn("ecmwf_download_format", queryables)
+        self.assertTrue(
+            get_args(queryables["ecmwf_download_format"])[1].is_required(),
+            "Keyword ecmwf_download_format must be required when details are missing",
+        )
+
     @mock.patch(
         "eodag.plugins.search.build_search_result.ECMWFSearch._fetch_data",
         autospec=True,