fix : _T ne peut pas recevoir NULL en paramètre en PHP 8.1 #4891

Closed
RealET wants to merge 3 commits from RealET:bug_taille_contrainte_81 into master
RealET commented 2 months ago

Cas d'usage, n'avoir qu'une des 2 valeurs xxx_WIDTH ou xxx_HEIGHT dans les contraintes (vu sur l'ajout d'un site référencé automatisé avec un logo pas assez large).

Dans mpn mes_options.php, il y a :

if (!defined('_LOGO_MIN_WIDTH'))
	define('_LOGO_MIN_WIDTH', 1920);
if (!defined('_LOGO_MAX_WIDTH'))
	define('_LOGO_MAX_WIDTH', 3840);
Cas d'usage, n'avoir qu'une des 2 valeurs `xxx_WIDTH` ou `xxx_HEIGHT` dans les contraintes (vu sur l'ajout d'un site référencé automatisé avec un logo pas assez large). Dans mpn mes_options.php, il y a : ``` if (!defined('_LOGO_MIN_WIDTH')) define('_LOGO_MIN_WIDTH', 1920); if (!defined('_LOGO_MAX_WIDTH')) define('_LOGO_MAX_WIDTH', 3840); ```
RealET added 1 commit 2 months ago
RealET added 1 commit 2 months ago
b_b added the
bug
label 2 months ago
b_b added this to the spip-4.1 milestone 2 months ago
b_b approved these changes 2 months ago
Owner

Je ne comprends pas. Et @b_b qui valide ?


'largeur_vignette' => $max_width && '∞',

Ça fait toujours true. Tu es sûr que tu ne voulais pas ?? à la place de && ?

Je ne comprends pas. Et @b_b qui valide ? ```php 'largeur_vignette' => $max_width && '∞', ``` Ça fait toujours `true`. Tu es sûr que tu ne voulais pas `??` à la place de `&&` ?
marcimat requested changes 2 months ago
marcimat left a comment
Owner

C’est certainement pas $truc && '∞'

C’est certainement pas `$truc && '∞'`
Owner

Est-ce qu'il y a un ticket associé avec une description plus complète ainsi que l'erreur ou la notice php produite ?

Est-ce qu'il y a un ticket associé avec une description plus complète ainsi que l'erreur ou la notice php produite ?
RealET added 1 commit 2 months ago
Poster

Mais oui, tu as raison @marcimat !
C'est ça de tester en 4.1 et de reporter manuellement en Master après.

Je me suis aussi rendu compte que infini, ça n'a pas de sens pour mini : c'est 0 dans ce cas là.

Mais oui, tu as raison @marcimat ! C'est ça de tester en 4.1 et de reporter manuellement en Master après. Je me suis aussi rendu compte que infini, ça n'a pas de sens pour mini : c'est 0 dans ce cas là.
RealET requested review from marcimat 2 months ago
marcimat approved these changes 2 months ago
Owner

Intégré.

Intégré.
marcimat closed this pull request 2 months ago
RealET deleted branch bug_taille_contrainte_81 2 months ago

Reviewers

b_b approved these changes 2 months ago
marcimat approved these changes 2 months ago
This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
4 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.