Pour faire fonctionner Imagick sous Windows, il faut ajouter realpath
#5209
Closed
RealET
wants to merge 4 commits from <deleted>:4.1
into 4.1
pull from: <deleted>:4.1
merge into: spip:4.1
spip:1.8
spip:1.9.1
spip:1.9.2
spip:2.0
spip:2.1
spip:3.0
spip:3.1
spip:3.2
spip:4.0
spip:4.1
spip:boutons-danger
spip:debug_ecrire_fichier
spip:dev-sortable
spip:dev/autoloader
spip:dev/instituer_ergo
spip:dev_infos_image
spip:fix/valider_url_distante
spip:fix_modifier_login
spip:issue_4101
spip:issue_4678
spip:issue_4705
spip:issue_4717
spip:issue_4946
spip:issue_5016
spip:issue_5056_composer_road
spip:issue_5273
spip:master
spip:refactor_texte_safety
Reviewers
Request review
No reviewers
Labels
Amélioration, nouvelle fonctionnalité APIs authentification base de données bug
Ca ne fonctionne pas code généré compilo css divers documentation doublon
Ce ticket est un doublon ergonomie espace privé filtres et balises formulaires Inscription installation invalide
Ticket invalide javascript langues LDAP plugin PostgreSQL refusé
Ignoré, c'est comme Ca... sécurité traduction
Apply labels
Clear labels
accessibilité
amélioration
Amélioration, nouvelle fonctionnalité APIs authentification base de données bug
Ca ne fonctionne pas code généré compilo css divers documentation doublon
Ce ticket est un doublon ergonomie espace privé filtres et balises formulaires Inscription installation invalide
Ticket invalide javascript langues LDAP plugin PostgreSQL refusé
Ignoré, c'est comme Ca... sécurité traduction
No Label
accessibilité
amélioration
APIs
authentification
base de données
bug
code généré
compilo
css
divers
documentation
doublon
ergonomie
espace privé
filtres et balises
formulaires
Inscription
installation
invalide
javascript
langues
LDAP
plugin
PostgreSQL
refusé
sécurité
traduction
Milestone
Set milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Assign users
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.
No due date set.
Dependencies
This pull request currently doesn't have any dependencies.
Reference in new issue
There is no content yet.
Delete Branch '<deleted>:4.1'
Deleting a branch is permanent. It CANNOT be undone. Continue?
No
Yes
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
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 ???
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