Valider e094ba0a rédigé par Maïeul's avatar Maïeul
Parcourir les fichiers

BUGfix Lorsqu'un champ est obligatoire ET qu'il y a une vérification, il faut

aussi détecter lorsque le champ n'a pas été remplie, même si ce n'est
pas un entier.

Bon, tout ca n'arriverait pas si cextras utilisait l'API de saisies, qui
normalement fait proprement les choses (poke @rastapopoulos).

Petit cour de logique booléenne.
On définit $a = 'a';

Alors:
1. $a != 'b' renvoie true;
2. !$a == 'b' renvoie false, car la négation d'une chaîne, c'est false;
3. !($a == 'b') renvoie true, car la négation de false, c'est true.

On évitera donc d'utiliser la syntaxe 2. à la place de la syntaxe 1.,
même en croyant utiliser la syntaxe $a (qui n'est en général pas
parlante.)
parent 0d899e8f
Chargement en cours
Chargement en cours
Chargement en cours
Chargement en cours
+1 −1
Numéro de ligne d'origine Numéro de ligne de diff Ligne de diff
@@ -318,7 +318,6 @@ function cextras_verifier_saisie($saisie, $valeur, &$normaliser = null, $depuis_
	if (is_null($verifier)) {
		$verifier = charger_fonction('verifier', 'inc', true);
	}

	// verifier obligatoire
	// si la verification porte sur un entier, on teste la longueur de la chaîne afin de laisser passer la valeur 0
	if (
@@ -329,6 +328,7 @@ function cextras_verifier_saisie($saisie, $valeur, &$normaliser = null, $depuis_
			or ((isset($saisie['verifier']['type']) and $saisie['verifier']['type'] == 'entier') and !strlen($valeur))
		)
	) {

		return _T('info_obligatoire');
	}

+1 −1
Numéro de ligne d'origine Numéro de ligne de diff Ligne de diff
<paquet
	prefix="cextras"
	categorie="outil"
	version="3.13.0"
	version="3.13.1"
	etat="stable"
	compatibilite="[3.0.0;3.2.*]"
	logo="images/cextras-64.png"