Pb recherche par Département et Bassin-versant

[raphaelloiseau]

GN 2.4.3 - ZH 1.3.1

Les recherches de zones humides par département ou bassin-versant rencontrent une erreur :

C'est la fonction "filter_area" dans search.py qui récupère l'information du département.

gn_module_ZH/backend/gn_module_zh/search.py

Lines 156 to 175 in 3ffb8d1

	def filter_area(query, json: dict, type_code: str):
	codes = [area.get("code", None) for area in json]
	if any(code is None for code in codes):
	return query
	# Filter on departments
	subquery = (
	select(LAreas.area_name, LAreas.geom, LAreas.id_type, BibAreasTypes.type_code)
	.join(BibAreasTypes, LAreas.id_type == BibAreasTypes.id_type)
	.where(BibAreasTypes.type_code == type_code, LAreas.area_code.in_(codes))
	.subquery()
	)
	# Filter on geom.
	# Need to use (c) on subquery to get the column
	query = query.where(
	func.ST_Transform(func.ST_SetSRID(TZH.geom, 4326), 2154).ST_Intersects(subquery.c.geom)
	)
	return query

Elle est aussi utilisée pour les communes qui contrairement aux départements est en choix multiple dans la recherche.
Pour les département (choix unique) : codes = [area.get("code") for area in json] est reconnu comme un string (au lieu d'une liste)
(alors que cela marche pour les communes)

J'ai ainsi séparer les deux fonctions :

• filter_dept : avec codes = [json.get("code")]
• filter_com : avec codes = [area.get("code") for area in json]

il faut aussi changer :

• gn_module_ZH/backend/gn_module_zh/search.py

  Line 65 in 3ffb8d1

  query = filter_area(query, departement, type_code="DEP")

  avec filter_dept

• gn_module_ZH/backend/gn_module_zh/search.py

  Line 67 in 3ffb8d1

  query = filter_area(query, communes, type_code="COM")

  avec filter_com

De même pour les bassins versant (choix unique):

gn_module_ZH/backend/gn_module_zh/search.py

Line 197 in 3ffb8d1

codes = [area.get("code", None) for area in json]

Changer : codes = [area.get("code") for area in json] par codes = [json.get("code")]

Je ne suis pas trop sur de mes modifs mais les deux recherches marchent bien !

[JulienCorny]

Bonjour,

Merci pour le retour, cette erreur est corrigée et mergée sur la branche develop (#87) et sera intégrée à la prochaine release du module qui sera publiée cette semaine