fix #3324

[jbrieuclp]

Corrige le premier point du #3324

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.97%. Comparing base (0f031f4) to head (4853d51).
Report is 47 commits behind head on release-2-15-3.

Additional details and impacted files

@@               Coverage Diff               @@
##           release-2-15-3    #3325   +/-   ##
===============================================
  Coverage           83.97%   83.97%           
===============================================
  Files                 122      122           
  Lines                9712     9712           
===============================================
  Hits                 8156     8156           
  Misses               1556     1556           

Flag	Coverage Δ
pytest	83.97% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[bouttier]

La colonne géométrique est utilisée à plusieurs endroits :

• GeoNature/backend/geonature/core/gn_synthese/utils/query_select_sqla.py

  Lines 442 to 446 in 6f6b91a

  	geo_filter = func.ST_DWithin(
  	func.ST_GeogFromWKB(self.geom_column),
  	func.ST_GeogFromWKB(geom_wkb),
  	radius,
  	)

  → attendue 4326

• GeoNature/backend/geonature/core/gn_synthese/utils/query_select_sqla.py

  Line 448 in 6f6b91a

  geo_filter = self.geom_column.ST_Intersects(geom_wkb)

  → attendue 4326

• GeoNature/backend/geonature/core/gn_synthese/utils/query_select_sqla.py

  Lines 480 to 483 in 6f6b91a

  	l_areas_cte = LAreas.query.filter(LAreas.id_area.in_(value)).cte("area_filter")
  	self.query = self.query.where(
  	func.ST_Intersects(self.geom_column, l_areas_cte.c.geom)
  	)

  → attendue srid local

Pour une mise en cohérence, je pense qu’il vaut mieux utiliser du 4326, sachant que le ST_Intersect et facile à corriger en utilisant l_areas_cte.c.geom_4326 au lieu de l_areas_cte.c.geom depuis PnX-SI/RefGeo#17.

[camillemonchicourt]

Il n'y a pas un soucis de précision si on utilise les géométries reprojetées en 4326 pour les intersections ?

[bouttier]

Pas si évident de dire que les géométries 4326 sont celles reprojetées, et non les géométries locales (import en 4326, sélection sur leaflet). Aucune idée de la précision inhérente aux reprojections.

[camillemonchicourt]

OK, en effet.

[jacquesfize]

Travail continué dans #3372

J'ai opté pour refaire les filtres dans SyntheseQuery pour prendre en compte la potentielle différence de SRID