Skip to content

Commit

Permalink
fix(PnX-SI/gn_mobile_occtax#232): fetch all taxa area only if 'code_a…
Browse files Browse the repository at this point in the history 
…rea_type' attribute is defined
  • Loading branch information
@sgrimault
sgrimault committed Dec 6, 2023
1 parent 5496c1b commit bd9685c
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion datasync/build.gradle
View file
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ plugins {
id 'org.jetbrains.kotlin.android'
}

version = "0.5.0"
version = "0.5.1"

android {
namespace 'fr.geonature.datasync'
Expand Down
2 changes: 1 addition & 1 deletion datasync/src/main/java/fr/geonature/datasync/sync/usecase/DataSyncUseCase.kt
View file
Original file line number Diff line number Diff line change
Expand Up @@ -756,7 +756,7 @@ class DataSyncUseCase @Inject constructor(
}.getOrDefault(emptyList()))
}

if (withAdditionalData) {
if (withAdditionalData && codeAreaType?.isNotBlank() == true) {
Logger.info { "synchronize taxa additional data..." }

offset = 0
Expand Down
4 changes: 2 additions & 2 deletions datasync/version.properties
View file
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
#Tue Dec 05 19:05:17 CET 2023
VERSION_CODE=1160
#Wed Dec 06 21:08:59 CET 2023
VERSION_CODE=1165

4 comments on commit bd9685c

@camillemonchicourt
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cela veut dire que si on utilise les couleurs de taxons, cela va désormais récupérer les couleurs de taxons pour tout Taxref ?
Cela risque d'être très très long si c'est le cas.
Mais avec le nouveau mécanisme de listes de taxons, où on peut n'avoir aucune liste de taxons définie, je ne vois pas trop comment la récupération des couleurs de taxons va fonctionner/évoluer...

@sgrimault
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

L'appel à la route GET -> /api/synthese/color_taxon fonctionne uniquement avec le paramètre code_area_type (en plus des paramètres de pagination). Donc on récupère toutes les informations liées avec le paramètre code_area_type ensuite mappées avec les taxons récupérés préalablement.

@TheoLechemia
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

La route /api/synthese/color_taxon ne récupère les couleur des taxons présent dans la synthese. Ceux qui ne sont pas retourné par cette route n'auront donc pas de couleur ? (si c'est ça, c'est ok, et ça ne va pas alourdir le mécanisme)

@camillemonchicourt
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK si cela ne prend que les taxons présents au moins une fois dans la synthèse alors c'est parfait, et on ne sera pas plus long qu'actuellement.
Merci pour ces précisions.

Please sign in to comment.