[HOME] Statistiques générales sur la HOME et permissions

[edelclaux]

Version
develop

Description du bug
La route general_stats de la synthèse a une erreur lors de la vérification du scope des permissions

1. La route s’appuie sur le décorateur permissions_required

@permissions_required("R", module_code="SYNTHESE")

2. Ce décorateur renvoit une liste de permissions, dans la variable permissions

def general_stats(permissions):

3. Cette variable est passée en paramètre à la méthode filter_query_with_cruved

synthese_query_obj.filter_query_with_cruved(g.current_user, permissions)

4. La méthode filter_query_with_cruved attend un scope en paramètre. La variable permissions est interprétée comme un scope

def filter_query_with_cruved(self, user, scope):

5. Le scope est appliqué s'il vaut 1 ou 2. Comme il vaut "list de permission", la fonction ne rentre jamais dans le bloc.

if scope in (1, 2):

6. Hop, on a les stats calculées sur toutes les données, hors scope.
   
   

Suggestion de solution
On peut s'appuyer sur le décorateur

@permissions.check_cruved_scope("R", get_scope=True, module_code="SYNTHESE")

[edelclaux]

Ou alors utiliser:

  def filter_query_with_permissions(self, user, permissions):
      """
      Filter the query with the permissions of a user
      """
      where_clause = self.build_permissions_filter(user=user, permissions=permissions)
      self.query = self.query.where(where_clause)

[jacquesfize]

La version GeoNature 2.15.2 intègre l'appel de la bonne fonction ainsi qu'une sécurité pour que l'appel de la fonction filter_query_with_cruved vérifie le type de la valeur donnée en paramètre.