Skip to content

feat: calcul dynamique de `#MIME_TYPE` pour prendre en compte les multiples mime possibles pour une même extension

cerdic a demandé de fusionner fix_mime_type_mp4 vers master

cf #5008 Selon son contenu un fichier .mp4 peut-être une video, de l'audio ou ni l'un ni l'autre.

  • On detecte a l'upload le type via la fonction metada_mp4_dist()
  • on calcule le mime-type via une balise calculée qui prend en compte id_document + extension et cherche l'existence de fonction type calculer_mime_type_xxx associée au mime-type par défaut pour traiter les cas particuliers
  • on implémente une fonction calculer_mime_type_application_mp4 pour traiter le cas de mp4
  • la fonction inc_vignette_dist() prend aussi le media en second argument pour permettre de décliner la vignette selon le type de fichier
  • dans la galerie de la mediatheque on utilise un filtre |titre_mime_type pour afficher un titre mieux adapté et décliner la variante quand c'est nécessaire (pour les mp4 donc)

Il reste le cas du filtre |vignette qui ne prend pas en compte le media, mais il semble quasi inutilisé sur la zone hormis deux cas d'appel avec l'extension zip : https://git.spip.net/spip-contrib-squelettes/ressourcotheque/blob/master/inclure/documents.html#L33 et https://git.spip.net/spip-galaxie/contrib.spip.net/blob/master/inclure/plugins/telechargements.html#L32 A voir si on le laisse comme ça pour le moment, ou si on le refactore aussi donc...

Rapports de requête de fusion

Chargement en cours