Discussion :

[nanard25]

Bonjour à Toutes et à Tous,

Avec PHP 7.2.9 et Kcfinder un problème survient lors de l"upload d'une image

Kcfinder => lib => class_image.php

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
} elseif (is_array($image)) {
            list($key, $width) = each($image);
            list($key, $height) = each($image);
            $img = $this->getBlankImage($width, $height);

Modifier les lignes 119 et 120

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
} elseif (is_array($image)) {
 
        list($key, $width) = $image;
 
 	list($key, $height) = $image;

La seule méthode trouvée, cela fonctionne

[rawsrc]

Salut,

tu peux également faire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
[$key, $width]  = $image;
[$key, $height] = $image;

[CosmoKnacki]

Modifier les lignes 119 et 120

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
} elseif (is_array($image)) {
 
        list($key, $width) = $image;
 
 	list($key, $height) = $image;

La seule méthode trouvée, cela fonctionne

J'en doute. Ce n'est pas parce que tu n'as plus d'erreur qui s'affiche que "cela fonctionne".

La fonction each() renvoie la clef et la valeur à la position du pointeur interne d'un tableau, puis avance ce pointeur à l'élément suivant. Donc si ton tableau est par exemple [1024, 768] et que le pointeur du tableau est au début, alors:

Code php<7.2 :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
list($key, $width) = each($image); // $key vaut 0 et $width 1024, le pointeur avance à l'élément suivant
list($key, $height) = each($image); // $key vaut 1 et $height 768, le pointeur est à la fin du tableau.

En écrivant list($key, $width) = $image, $key vaudra 1024 et $width 768. À la ligne suivante $height vaudra également 768. Donc ce que fait ton code c'est d'envoyer deux fois la hauteur en guise de largeur et de hauteur à la méthode getBlankImage (bienvenue au pays des carrées).

Si on voulait strictement reproduire le comportement des deux lignes de code originales, il faudrait écrire en PHP 7.2:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
list($width, $key) = [current($image), key($image)];
list($height, $key) = [next($image), key($image)];
next($image);

Mais dans le cas présent on se fiche aussi bien des clefs du tableau que de la position du pointeur interne. Ces deux lignes sont en fait une écriture vieillotte pour récupérer les deux premières et probablement les seules valeurs d'un tableau pour les affecter à $width et $height sans se soucier des clefs du tableau, donc on peut remplacer ces deux lignes par:

• à partir de PHP 7.1:

  Code :

  Sélectionner tout - Visualiser dans une fenêtre à part

  [$width, $height] = $image;

• ou avant PHP 7.1:

  Code :

  Sélectionner tout - Visualiser dans une fenêtre à part

  list($width, $height) = $image;