Bug dans le plugin-dist "medias" suite passage en 4.4.0

Dev
1

Bonjour,
Pour tester j’ai passé un site en 4.4.0 et j’ai une erreur sur une combinaison « Lecteur Multimedia » + « medias » (le premier utilisant le second):

Erreur d’exécution plugins/auto/player/v4.1.0/players/mejs/player.html | File […]/plugins-dist/medias/medias_fonctions.php Line 291 : medias_mime_type_document(): Argument #1 ($id_document) must be of type int, null given, called in […]/ecrire/public/composer.php(96) : eval()'d code on line 58

Dans le fichier « /plugins-dist/medias/medias_fonctions.php » ligne 291 j’ai:

function medias_mime_type_document(int $id_document, string $extension = ''): string {

J’ai corrigé en modifiant comme suit:

function medias_mime_type_document(?int $id_document, ?string $extension = ''): string {

Et là c’est bon, PHP8.2 et PHP8.3 … pas sûr de la raison, le « ? » rends les paramètres « nullable » si je ne m’abuse …? Ça fait sens ou pas ?

2

Je pense qu’il vaudrait mieux corriger en amont mejs pour ne pas passer une valeur nulle. Ca n’a pas tellement de sens normalement ca.

3

J’ai tenté un truc dans « player.html » à la ligne 10 du plugin « Lecteur Multimedia » car je supposais que cela venait de type="#MIME_TYPE" qui serait vide, genre [type="(#MIME_TYPE)"] , mais je ne connais pas assez les entrailles de Spip pour être sûr de l’enchainement des appels donc je laisserai les spécialistes faire …
Mais oui à moi aussi mon « patch » ressemblait plus à une rustine d’où ma question sur le sens de ce que j’avais fait …

4

je t’invite à ouvrir un ticker sur le plugin en question

5

Mais à ton avis sur le plugin « medias » ou « Lecteur Multimedia » ?

6

sur lecteur multimédia

7

Ya la même première erreur en rapport avec les fonctions de mime_type sur contrib

8

J’ai complété le ticket mentionné.