Corriger le renseignement des JPG où parfois ça mettait jpeg au lieu de jpg fix #5723 #5728

Merged
marcimat merged 2 commits from dev/issue_5723_renseigner_jpg into master 2 weeks ago
Owner

On applique corriger_extension() ici aussi, comme c'est fait dans la fonction pour trouver l'extension depuis les headers. Cela assure d'uniformiser en "jpg" à la fin qui est la seule extension reconnue dans les tables de documents de SPIP.

On applique corriger_extension() ici aussi, comme c'est fait dans la fonction pour trouver l'extension depuis les headers. Cela assure d'uniformiser en "jpg" à la fin qui est la seule extension reconnue dans les tables de documents de SPIP.
rastapopoulos force-pushed dev/issue_5723_renseigner_jpg from e800032160 to 02020edf29 2 weeks ago
Poster
Owner

À reporter au moins en 4.2 ensuite.

À reporter au moins en 4.2 ensuite.
Owner

Pas grand chose à dire là dessus (a priori ok), mais… pourrais tu faire un commit pour le changelog ?

Pas grand chose à dire là dessus (a priori ok), mais… pourrais tu faire un commit pour le changelog ?
rastapopoulos added 1 commit 2 weeks ago
Poster
Owner

Ajouté @marcimat

Ajouté @marcimat
b_b approved these changes 2 weeks ago
CHANGELOG.md Outdated
@ -20,6 +20,7 @@
### Fixed
- #5723 Corrige le renseignement des images JPG pour bien toujours unifier avec l'extension "jpg" et non "jpeg"
b_b commented 2 weeks ago
Owner

Peut-être préciser que ça concerne les "images distantes" ?

Peut-être préciser que ça concerne les "images distantes" ?
Poster
Owner

ajouté

ajouté
b_b marked this conversation as resolved
rastapopoulos force-pushed dev/issue_5723_renseigner_jpg from 6dcb1f414a to c2669963e5 2 weeks ago
rastapopoulos force-pushed dev/issue_5723_renseigner_jpg from c2669963e5 to f9979c7088 2 weeks ago
marcimat approved these changes 2 weeks ago
marcimat merged commit f9979c7088 into master 2 weeks ago
marcimat deleted branch dev/issue_5723_renseigner_jpg 2 weeks ago

Reviewers

b_b approved these changes 2 weeks ago
marcimat approved these changes 2 weeks ago
The pull request has been merged as f9979c7088.
Sign in to join this conversation.
Loading…
There is no content yet.