fix : compatibilité d'Imagick avec Windows #5221

Closed
RealET wants to merge 1 commits from RealET:imagick_windows into master
RealET commented 2 months ago

Imagick a a besoin du chemin absolu et non relatif pour lire et écrire les fichiers sous Windows

ref #5209

Imagick a a besoin du chemin absolu et non relatif pour lire et écrire les fichiers sous Windows ref #5209
RealET added 1 commit 2 months ago
10b1a37d41 fix : compatibilité d'Imagick avec Windows
marcimat reviewed 2 months ago
elseif ($process == 'imagick') {
$vignette = "$destination." . $format_sortie;
// chemin compatible Windows
$vignette = realpath(dirname($destination)) . '/' . basename($destination) . '.' . $format_sortie;
Owner

Est-ce que c’est pas le même problème que précédemment ?
Est-ce qu’on est certain que le répertoire de destination existe ?
(auquel cas realpath retournera false)

Est-ce que c’est pas le même problème que précédemment ? Est-ce qu’on est certain que le répertoire de destination existe ? (auquel cas realpath retournera false)
Owner

ça vaudrait le coup de faire un test pour pas concaténer un false avec une chaine, mais si le répertoire de destination existe pas, l'écriture du fichier echouera dans tous les cas, windows ou pas.

ça vaudrait le coup de faire un test pour pas concaténer un false avec une chaine, mais si le répertoire de destination existe pas, l'écriture du fichier echouera dans tous les cas, windows ou pas.
Poster

Pour tous les cas que j'ai testés, les dossiers existaient.

Mais oui, un test préalable serait encore mieux.

Je le rajoute ?

Pour tous les cas que j'ai testés, les dossiers existaient. Mais oui, un test préalable serait encore mieux. Je le rajoute ?
Owner

intégré (avec un test sur le répertoire).

intégré (avec un test sur le répertoire).
marcimat closed this pull request 2 months ago
b_b deleted branch imagick_windows 2 months ago
Poster

Une raison à ne pas avoir reporté en 4.1 ?

Une raison à ne pas avoir reporté en 4.1 ?
b_b commented 3 weeks ago
Owner

@RealET ça avait bien été reporté, mais certains commits de la 4.1 on disparu...

@RealET ça avait bien été reporté, mais certains commits de la 4.1 on disparu...
This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Loading…
There is no content yet.