Drôle d'erreur avec un jpeg en particulier

SPIP 4.3 sous Git, à jour, aucun plugin activé.
Géré sous ddev, config PHP standard, de base.
GD2 activé dans Fonctions avancées, mais c'est pareil avec CONVERT ou IMAGICK.

J'uploade ce jpg comme logo d'article (zippé pour qu'il ne soit pas altéré) photo-1691950202275-05b7c7ee9af3.zip

et j'ai une erreur fatale :

Fatal error:  Uncaught Error: Undefined constant "IMAGETYPE_SVG" in /var/www/html/plugins-dist/medias/metadata/image.php:54
Stack trace:
#0 /var/www/html/plugins-dist/medias/metadata/image.php(22): decoder_type_image(19)
#1 /var/www/html/plugins-dist/medias/inc/renseigner_document.php(153): metadata_image_dist('../IMG/logo/pho...')
#2 /var/www/html/plugins-dist/medias/action/ajouter_documents.php(203): renseigner_taille_dimension_image('../IMG/logo/pho...', 'jpg')
#3 /var/www/html/plugins-dist/medias/action/ajouter_documents.php(53): action_ajouter_un_document_dist('new', Array, 'article', '1', 'logoon')
#4 /var/www/html/ecrire/action/editer_logo.php(137): action_ajouter_documents_dist('new', Array, 'article', '1', 'logoon')
#5 /var/www/html/prive/formulaires/editer_logo.php(226): logo_modifier('article', '1', 'on', Array)
#6 /var/www/html/ecrire/public/aiguiller.php(288): formulaires_editer_logo_traiter_dist('article', '1', '', 'a:10:{s:4:"exec...')
#7 /var/www/html/ecrire/index.php(70): traiter_formulaires_dynamiques()
#8 {main}
  thrown in <b>/var/www/html/plugins-dist/medias/metadata/image.php</b> on line <b>54</b><br>

IMAGETYPE_SVG ?

Si je tente de l'ajouter en document, rien ne se passe (pas d'erreur, pas de doc ajouté)

Et je ne vois pas trop en quoi ce jpeg est bizarre.
D'après GIMP il est bien en RVB.

Bref, lapin compris.

changed the description

zippé pour qu'il ne soit pas altéré

Tu upload un logo dans un zip, c'est bien ça ? De mon côté, je ne peux pas sélectionner un zip lors de l'ajout d'un logo depuis l'interface privée.

file photo-1691950202275-05b7c7ee9af3.jpg
photo-1691950202275-05b7c7ee9af3.jpg: ISO Media, AVIF Image

Tu es certain que c’est un JPEG ??

Et je reproduis avec ce fichier

De toutes les constantes dans decoder_type_image, seule IMAGETYPE_SVG ne semble pas exister

IMAGETYPE_GIF;
IMAGETYPE_JPEG;
IMAGETYPE_PNG;
IMAGETYPE_SWF;
IMAGETYPE_PSD;
IMAGETYPE_BMP;
IMAGETYPE_TIFF_II;
IMAGETYPE_TIFF_MM;
IMAGETYPE_WEBP;
IMAGETYPE_SVG;

ah il existe un define('IMAGETYPE_SVG', 19); dans ecrire/inc/svg

donc si on met include_spip('inc/svg'); dans decoder_type_image, l’image est chargée… mais renommée en .svg…

Ça m’ajoute ce logo en le renommant en .svg logo/photo-1691950202275-05b7c7ee9af3.jpg.svg

Et ça doit donc être un bug, où on prend la dernière constante après avoir testé les autres, qqchose comme ça, non ?

PS : il faudra se pencher à un moment sur la gestion des .avif justement, ça commence à être de plus en plus répandu.

Tu upload un logo dans un zip, c'est bien ça ?

Non, je l'ai zippé pour l'ajouter ici

Tu es certain que c’est un JPEG ??

Bien vu, ça serait un avif donc.
Téléchargé depuis unsplash.com qui utilise ce format, donc ça ne m'étonne pas.
Avec quoi est ce que tu as affiché ces infos ?

[Edit: avec file oui, bien sûr, je n'y avais pas pensé]

• getimagesize($img);

array:7 [
  0 => 1974
  1 => 2961
  2 => 19
  3 => "width="1974" height="2961""
  "bits" => 8
  "channels" => 3
  "mime" => "image/avif"
]

donc 19, c’est les fichiers avif (pas svg introduit dans SPIP ^^)

Notons qu’il existe une fonction image_type_to_extension dans PHP, proche de decoder_type_image https://www.php.net/manual/fr/function.image-type-to-extension.php

19, c’est les fichiers avif

Ah oui, c'est ballot.

mentioned in commit ecrire@668bbeee

mentioned in merge request ecrire!24 (merged)

cf ecrire!24 (merged) donc pour éviter cette collision ballotte

mentioned in commit ecrire@aa4158d6

mentioned in commit ecrire@b31d8ae5

mentioned in commit ecrire@0db68f28

mentioned in commit ecrire@c7a11983

mentioned in commit spip@afa66e4b

mentioned in commit spip@94644a19

mentioned in commit spip@73f91dc7

mentioned in commit spip@e7694645

mentioned in merge request spip!6124 (merged)

Bon la collusion a été corrigé dans les versions de dev et s'apprete à l'être en version 4.3

Par contre on est d'accord qu'il faut tout de même mettre include_spip('inc/svg'); dans decoder_type_image() ? ou bien il y aurait une manière plus intelligente ?

closed