Est-ce qu’on est sûr que ça n’impacte pas les autres OS ? Tu avais regardé Real3t aussi ?
D’autres avis dessus ?
Et par ailleurs il faudrait proposer en master, pas 4.1 la PR :)
Je n’ai aucun moyen de vérifier cela.
Mais effectivement d’autres personnes l’indiquent, mais c’est il y a 10 ans sur PHP 5.4. Ça n’a pas changé depuis ?
- https://www.php.net/manual/en/imagick.construct.php#108764
- https://stackoverflow.com/questions/9954764/imagick-php-5-4-extension-does-not-work-with-relative-paths-windows
Est-ce qu’on est sûr que ça n’impacte pas les autres OS ? Tu avais regardé Real3t aussi ?
D’autres avis dessus ?
Et par ailleurs il faudrait proposer en master, pas 4.1 la PR :)
Je suis curieux : comment je teste en 4.1 et pourrais faire une PR en master ? (à part aller directement utiliser l'éditeur de code source de Gitea) ?
Ben j'ai testé sous Windows avec PHP 8.1.
Je n'ai pas de SPIP 4.1 installé sous Linux :(
Je suis curieux : comment je teste en 4.1 et pourrais faire une PR en master ? (à part aller directement utiliser l'éditeur de code source de Gitea) ?
Par ailleurs, on a déjà un exemple dans SPIP qui fonctionne:
https://git.spip.net/spip/filtres_images/src/branch/master/filtres/images_typo.php#L524
Et d'ailleurs, il restait ça à faire : https://git.spip.net/spip/filtres_images/pulls/4700
On cherche à écrire une vignette, donc par défaut elle n’existe pas, donc realpath($vignette) retourne false.
Tu avais vraiment pu faire fonctionner ça ???
J’ai testé trop rapidement. J’ai bien des soucis
Ça va pas sur `writeImage`
On cherche à écrire une vignette, donc par défaut elle n’existe pas, donc `realpath($vignette)` retourne false.
Tu avais vraiment pu faire fonctionner ça ???
Le cas est sans doute anecdotique, mais j'en ai eu besoin pour tester Imagick en local
Je n’ai aucun moyen de vérifier cela.
Mais effectivement d’autres personnes l’indiquent, mais c’est il y a 10 ans sur PHP 5.4. Ça n’a pas changé depuis ?
Est-ce qu’on est sûr que ça n’impacte pas les autres OS ? Tu avais regardé Real3t aussi ?
D’autres avis dessus ?
Et par ailleurs il faudrait proposer en master, pas 4.1 la PR :)
Ben j'ai testé sous Windows avec PHP 8.1.
Je n'ai pas de SPIP 4.1 installé sous Linux :(
Je suis curieux : comment je teste en 4.1 et pourrais faire une PR en master ? (à part aller directement utiliser l'éditeur de code source de Gitea) ?
Par ailleurs, on a déjà un exemple dans SPIP qui fonctionne:
https://git.spip.net/spip/filtres_images/src/branch/master/filtres/images_typo.php#L524
Et d'ailleurs, il restait ça à faire : spip/filtres_images#4700
J’ai testé que ça cassait fonctionnait encore sans souci apparents.
Donc hop, c’est fusionné.
J’ai testé trop rapidement. J’ai bien des soucis
Ça va pas sur
writeImageOn cherche à écrire une vignette, donc par défaut elle n’existe pas, donc
realpath($vignette)retourne false.Tu avais vraiment pu faire fonctionner ça ???
Du coup j’ai enlevé les realpath sur les writeImage.
Je ne sais pas ce que ça donne sous Windows par contre.
Je vais tester, je te dis.
N’oublie pas pour tester
Bon, j'ai mieux testé et voici ma proposition :
ad944cd642