[Fiche taxon] Ajout d'un onglet "Médias" et un composant de galerie photo paginée

[jules-jean-louis1]

Closes #69854647
Closes #69854746

Backend

Ajout d'une nouvelle route GET /gn_commons/medias/species/ dans gn_commons/medias/.
Cette route permet de récupérer les médias associés à une espèce en utilisant son identifiant cd_nom. La réponse est paginée.

Frontend

Ajout d'un onglet Médias dans la page de la fiche espèce. Le composant DisplayMedia est utilisé pour gérer l'affichage dynamique des différents types de médias (images, vidéos, audios).

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.89%. Comparing base (46ecee3) to head (94dbc1b).
Report is 13 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #3229      +/-   ##
===========================================
+ Coverage    81.95%   83.89%   +1.93%     
===========================================
  Files           86      122      +36     
  Lines         6965     9660    +2695     
===========================================
+ Hits          5708     8104    +2396     
- Misses        1257     1556     +299     

Flag	Coverage Δ
pytest	83.89% <100.00%> (+1.93%)	⬆️

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.

[edelclaux]

Résumé des échangs autour de la route:

Proposition de fonctionnement après réunion technique

Dans l'idéal, faire une route générique dans medias qui récupère les medias associés à un cd_ref.
Le fait que ces médias soit associés à des observations de la synthèse devrait être décrit en paramètre de la requête.

Les permissions devraient pouvoir être gérés en s'appuyant sur has_instance_permissions, ou filter_by_scope.

Si ça devenait un sac de noeud, re-basculer vers la version simpliste de la requête dans la synthese.

Résulat après analyse de NS

Notes route media

Faire une route au niveau des medias: geonature/backend/geonature/core/gn_commons/medias/routes.py

Cette route doit permettre de requêter les medias d'un ou plusieurs module associés à un cd_ref.

L'idée est de rendre cette route agnostique du module en quetsion (synthèse, occtax, etc.), en s'appuyant sur la mécanique existante, BibTablesLocation.

Problèmes rencontrés

BibTablesLocation est incomplète

Problème très mineur. Il suffirait de rajouter une ligne pour la synthèse.

BibTablesLocation n'est pas suffisante

Exemple de récupération des medias dans occtax. Il faudrait récupérer les medias des occurences.

    # Exemple de démo de construction d'un objet dynamiquement, plaqué sur l'objet et la table décrite bib_tables_location. A ce stade, le résultat de la requête dans bib_tables_location est supposé connu.
    field = "unique_id_sinp_occtax"
    table_cor_counting = "cor_counting_occtax"
    t_cor_counting = sa.Table(
        table_cor_counting, metadata, autoload_with=engine, schema="pr_occtax"
    )

    query = (
        select(t_medias)
        .join(
            t_cor_counting,
            t_cor_counting.c[field] == t_medias.c.uuid_attached_row,
        )
    )

Or, le modèle de données dans Occtax est tel que cette requête ne permet pas d'effectuer une sélection sur le cd_nom. Il faut rajouter une jointure sur t_occurrences_occtax.

    # Exemple de démo de construction d'un objet dynamiquement, plaqué sur l'objet et la table décrite bib_tables_location. A ce stade, le résultat de la requête dans bib_tables_location est supposé connu.
    field = "unique_id_sinp_occtax"
    table_cor_counting = "cor_counting_occtax"
    t_cor_counting = sa.Table(
        table_cor_counting, metadata, autoload_with=engine, schema="pr_occtax"
    )

    table_occurences = "t_occurrences_occtax"
    t_occurences = sa.Table(table_occurences, metadata, autoload_with=engine, schema="pr_occtax")

    query = (
        select(t_medias)
        .join(
            t_cor_counting,
            t_cor_counting.c[field] == t_medias.c.uuid_attached_row,
        )
        .join(
            t_occurences,
            t_occurences.c.id_occurrence_occtax == t_cor_counting.c.id_occurrence_occtax,
        )
        .join(Taxref, Taxref.cd_nom == t_occurences.c.cd_nom)
        .where(Taxref.cd_ref == cd_ref)
    )

L'information pour effectuer cette jointure supplémentaire n'est pas contenue actuellement dans bib_table location.

Solution envisagée

Le procédé de jointure de filtrage par cd_ref est propre à la structure du modèle de donnée dans chaque module.

Non seulement la logique portée par la table BibTablesLocations est difficilement transposable à des cas plus complexes, mais ça ne semble en plus vraiment pas souhaitable. On ne souhaite certainement pas exposer au sein de gn_commons la logique de fonctionnement existante au sein des modules, et créer ainsi une connaissance et un couplage qui n'a plus rien à voir avec l'idée d'une route générique.

Il nous a semblé qu'une idée pourrait être de s'appuyer sur une logique plus orientée module.

De la même manière que l'on procède avec les méthodes un peu partagées comme filter_by_scope, ou l'ensemble des méthodes liées à l'import dans ImportActions, les modules pourraient mettre à disposition une interface genre QueryActions / des méthodes d'interactions. C'est évidemment une idée à la fois impactante, et faiblement définie.

Conclusion

Nous ne souhaitons pas intégrer ce niveau d'intrication dans la création d'une route pour récupérer les medias d'un taxon dans la synthèse.
Nous avions convenu que "Si ça devenait un sac de noeud, re-basculer vers la version simpliste de la requête dans la synthese."
Ca y est, c'est un sac de noeud. Nous souhaitons donc maintenir la version simpliste de la requête dans la synthèse :)

[camillemonchicourt]

Quand on affiche un média, il faudrait ajouter ses infos de base (titre, description, auteur...) mais aussi un minimum d'info sur l'observation à laquelle le média est associé (date, commune ?, autre ?).

Attention les médias d'une observation peuvent aussi être des PDF, des audios, des vidéos...
Ils sont affichés aussi ? Ou exclus ?

[edelclaux]

Quand on affiche un média, il faudrait ajouter ses infos de base (titre, description, auteur...) mais aussi un minimum d'info sur l'observation à laquelle le média est associé (date, commune ?, autre ?).

Attention les médias d'une observation peuvent aussi être des PDF, des audios, des vidéos... Ils sont affichés aussi ? Ou exclus ?

Ils sont affichés. On a réutilisé le composant frontend/src/app/GN2CommonModule/form/media/display-medias.component.html

Ok pour rajouter les infos mentionnées. On peut faire la liste ?
Meta-données du media:

• titre
• auteur
• description
• observation:
  • id ? auteur ? date ? renvoit vers la fiche Observation ?

[camillemonchicourt]

OK super si les infos du média sont affichés en reprenant le composant fourni par défaut.
Pour les infos sur l'observation, je sais pas trop. J'aurai dit au moins la date, mais on a la date du média en fait déjà, donc on peut mettre de côté ce sujet certainement. Un lien vers l'observation serait utile par contre, bien vu.