Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[synthèse] URL mal construire pour les médias PDF #3260

Closed
PaulLabruyere opened this issue Nov 19, 2024 · 2 comments
Closed

[synthèse] URL mal construire pour les médias PDF #3260

PaulLabruyere opened this issue Nov 19, 2024 · 2 comments

Comments

@PaulLabruyere
Copy link

PaulLabruyere commented Nov 19, 2024

Version
détectée sur 2.12.3
confirmée sur le site démo (2.14.2)

Description du bug
Bonjour !

Lorsqu'une observation est liée à un média de type "PDF" stockée avec l'instance geonature (dans le dossier backend/media/attachments), le média n'est pas retrouvé lorsqu'on cherche à l'afficher dans la fiche de l'observation (module synthèse / validation).

Le problème semble venir de la construction de l'URL :

  • pour les autres types de fichiers stockés sur le serveur (photos, sons, vidéos), l' API appelle une url correcte ".../geonature/api//media/attachments/{media_path}" (avec un / en trop avant media qui ne semble pas gênant)
  • mais pour les médias typés PDF, l'url construite est ".../geonature/api/undefined/attachments/{media_path}"

Noter que :

  • Si le pdf est stocké sur un autre serveur (avec l'url complète dasn media_url), la fiche synthèse l'affiche correctmeent dans le petit lecteur pdf qui va bien.
  • Si on clique sur le titre du média, il ouvre un nouvel onglet avec l'url correcte, et le pdf est bien retrouvé.
    -> Le problème semble donc se limiter à la construction de l'URL pour le lecteur intégré.

Comportement attendu
Construire l'URL de la même façon quel que soit le type de média stocké.

Comment reproduire
Une observation portant un média "pdf" a été ajoutée sur Occtax. elle est visible ici.

Logs
Avec la console du navigateur activée, ouvrir l'onglet média pour voir la requête en erreur et l'URL erronnée.

@camillemonchicourt
Copy link
Member

Corrigé par 55f7e76
A voir si on peut faire autrement qu'un iframe.

@camillemonchicourt
Copy link
Member

Corrigé dans la 2.15

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants