Taille minimum des images à l'upload #9

Bonjour,

J'ai adapté https://core.spip.net/issues/2378 à SPIP 3.3 (bien possible que ça soit pareil en 3.2 d'ailleurs).

À noter : le plugin vérifier a lui aussi une fonction verifier_fichier_dimension_max mais je n'ai pas trouvé d'appel de sa fonction mère nul part (verifier_fichiers_dist)

Bonjour, J'ai adapté https://core.spip.net/issues/2378 à SPIP 3.3 (bien possible que ça soit pareil en 3.2 d'ailleurs). À noter : le plugin vérifier a lui aussi une fonction verifier_fichier_dimension_max mais je n'ai pas trouvé d'appel de sa fonction mère nul part (verifier_fichiers_dist)

J'ai testé le patch et ça marche bien.

C'est encore plus souvent utile probablement avec les squelettes html5up.

J'ai testé le patch et ça marche bien. C'est encore plus souvent utile probablement avec les squelettes html5up.

tu peux renommer ce titre bizarre et mettre ce que ça fait directement dans la description ?

Copie d'écran de ce que ça donne quand le logo colle pas avec les specifications des 2 constantes

Capture d’écran de 2020-12-13 21-39-29.png

27 KiB

#9/files#issuecomment-2663

https://git.spip.net/spip/medias/pulls/9/files#issuecomment-2663

#9/files#issuecomment-2663
J'ai beau chercher, je comprends pas ce commentaire.

> https://git.spip.net/spip/medias/pulls/9/files#issuecomment-2663 J'ai beau chercher, je comprends pas ce commentaire.

Si les constantes de tailles min sont incohérentes avec les constantes de tailles max elles doivent être ignorées et ça doit générer un log, pas bloquer l'upload de document en générant systématiquement une erreur comme actuellement - qui plus est une erreur qui ne parle pas aux utilisateurs mais uniquement au webmestre...

Et lors du copie colle de chaine de langue, l'argument @maxi@ a été conservé tel quel alors que c'est maintenant une taille @mini@

Si les constantes de tailles min sont incohérentes avec les constantes de tailles max elles doivent être ignorées et ça doit générer un log, pas bloquer l'upload de document en générant systématiquement une erreur comme actuellement - qui plus est une erreur qui ne parle pas aux utilisateurs mais uniquement au webmestre... Et lors du copie colle de chaine de langue, l'argument `@maxi@` a été conservé tel quel alors que c'est maintenant une taille `@mini@`

Si les constantes de tailles min sont incohérentes avec les constantes de tailles max elles doivent être ignorées et ça doit générer un log, pas bloquer l'upload de document en générant systématiquement une erreur comme actuellement - qui plus est une erreur qui ne parle pas aux utilisateurs mais uniquement au webmestre...

Sauf que si on n'affiche pas une erreur à ce moment là, on va avoir une erreur encore pire ensuite : quelle que soit la taille de l'image, elle sera toujours trop petite ou trop grande, sans jamais avoir une taille qui passe.

Et lors du copie colle de chaine de langue, l'argument @maxi@ a été conservé tel quel alors que c'est maintenant une taille @mini@

Bien vu !

> Si les constantes de tailles min sont incohérentes avec les constantes de tailles max elles doivent être ignorées et ça doit générer un log, pas bloquer l'upload de document en générant systématiquement une erreur comme actuellement - qui plus est une erreur qui ne parle pas aux utilisateurs mais uniquement au webmestre... Sauf que si on n'affiche pas une erreur à ce moment là, on va avoir une erreur encore pire ensuite : quelle que soit la taille de l'image, elle sera toujours trop petite ou trop grande, sans jamais avoir une taille qui passe. > > Et lors du copie colle de chaine de langue, l'argument `@maxi@` a été conservé tel quel alors que c'est maintenant une taille `@mini@` Bien vu !

#9/files#issuecomment-2663
J'ai beau chercher, je comprends pas ce commentaire.

C'est normal, ton dernier commit a supprimé mon commentaire inline...

Je disais donc, il serait opportun d'aligner les deux items de langue, cf :

	'info_image_max_taille' => 'Les images doivent obligatoirement faire moins de @maxi@ (ce fichier fait @actuel@).',
	'info_image_min_taille' => 'Les images doivent faire au moins @mini@ (ce fichier ne fait que @actuel@).',

donc plutôt 'info_image_min_taille' => 'Les images doivent obligatoirement faire plus de @mini@ (ce fichier fait @actuel@).',

> > https://git.spip.net/spip/medias/pulls/9/files#issuecomment-2663 > J'ai beau chercher, je comprends pas ce commentaire. > > C'est normal, ton dernier commit a supprimé mon commentaire inline... Je disais donc, il serait opportun d'aligner les deux items de langue, cf : ``` 'info_image_max_taille' => 'Les images doivent obligatoirement faire moins de @maxi@ (ce fichier fait @actuel@).', 'info_image_min_taille' => 'Les images doivent faire au moins @mini@ (ce fichier ne fait que @actuel@).', ``` donc plutôt `'info_image_min_taille' => 'Les images doivent obligatoirement faire plus de @mini@ (ce fichier fait @actuel@).',`

Si les constantes de tailles min sont incohérentes avec les constantes de tailles max elles doivent être ignorées et ça doit générer un log, pas bloquer l'upload de document en générant systématiquement une erreur comme actuellement - qui plus est une erreur qui ne parle pas aux utilisateurs mais uniquement au webmestre...

Sauf que si on n'affichae pas une erreur à ce moment là, on va avoir une erreur encore pire ensuite : quelle que soit la taille de l'image, elle sera toujours trop petite ou trop grande, sans jamais avoir une taille qui passe.

À moins que tu veuille dire : ne traiter la vérification de taille minimum que si pas d'incohérence ?

> > Si les constantes de tailles min sont incohérentes avec les constantes de tailles max elles doivent être ignorées et ça doit générer un log, pas bloquer l'upload de document en générant systématiquement une erreur comme actuellement - qui plus est une erreur qui ne parle pas aux utilisateurs mais uniquement au webmestre... > Sauf que si on n'affichae pas une erreur à ce moment là, on va avoir une erreur encore pire ensuite : quelle que soit la taille de l'image, elle sera toujours trop petite ou trop grande, sans jamais avoir une taille qui passe. À moins que tu veuille dire : ne traiter la vérification de taille minimum que si pas d'incohérence ?

donc plutôt 'info_image_min_taille' => 'Les images doivent obligatoirement faire plus de @mini@ (ce fichier fait @actuel@).',

Alors, j'y ai pensé, mais ça ne le ferait pas parce que ça dirait par exemple que ça devrai faire plus de 1920 x 1080 (donc 1921 x 1081) alors que ça doit juste faire au moins 1920 x 1080.

> donc plutôt `'info_image_min_taille' => 'Les images doivent obligatoirement faire plus de @mini@ (ce fichier fait @actuel@).',` Alors, j'y ai pensé, mais ça ne le ferait pas parce que ça dirait par exemple que ça devrai faire plus de 1920 x 1080 (donc 1921 x 1081) alors que ça doit juste faire au moins 1920 x 1080.

À moins que tu veuille dire : ne traiter la vérification de taille minimum que si pas d'incohérence ?

Implémenté par la 4e révision fd51767792

> À moins que tu veuille dire : ne traiter la vérification de taille minimum que si pas d'incohérence ? Implémenté par la 4e révision https://git.spip.net/spip/medias/commit/fd51767792b0495fc6f920240d36de44b881b0d7

Bonsoir,

Il manquerait quoi pour que ce soit intégré ?

Bonsoir, Il manquerait quoi pour que ce soit intégré ?

Rahh, j'ai fermé cette PR par erreur, en supprimant un fork que je pensais concerner un autre ticket.

@RealET pour le coup tu as gagné le droit de tester et proposer une correction si besoin sur #11

@RealET pour le coup tu as gagné le droit de tester et proposer une correction si besoin sur https://git.spip.net/spip/medias/pulls/11

Labels Milestones

Taille minimum des images à l'upload #9