Projet

Général

Profil

Actions

Anomalie #4128

fermé

ApiUsage::addRequiredParameter() retourne bool(true) si le paramètre n'a pas de valeur

Ajouté par Jérôme Augé il y a presque 13 ans. Mis à jour il y a presque 13 ans.

Statut:
Intégré
Priorité:
Normal
Assigné à:
Nicolas Thing-Leoh
Version cible:
Début:
28/06/2013
Echéance:
% réalisé:

100%

Temps estimé:
Version source:
Solution proposée:

La valeur retourné doit être une chaîne vide dans ce cas.
=> le verify retournera alors une erreur

Principaux fichiers impactés:
Complexité:
Contrôle:
Thème:
Régression:

Description

Si un paramètre est requis, qu'on spécifie le paramètre, mais pas de valeur ('=xxx'), alors l'usage est validé et la valeur du paramètre est bool(true).

Le paramètre semble se comporter comme un addEmptyParameter().

Script test.php :

$usage = new ApiUsage();
$docid = $usage->addRequiredParameter("docid", "Document id");
$usage->verify();
var_dump($docid);

Résultat obtenu :

$ ./wsh.php --api=test --docid
bool(true)

Résultat attendu :

$ ./wsh.php --api=test --docid
*message d'erreur de validation de l'usage*

Mis à jour par Marc Claverie il y a presque 13 ans

  • Statut changé de Nouveau à À analyser
  • Assigné à mis à Éric Brison
  • Version cible mis à 3.2.10

Mis à jour par Éric Brison il y a presque 13 ans

  • Assigné à changé de Éric Brison à Nicolas Thing-Leoh

Mis à jour par Éric Brison il y a presque 13 ans

  • Solution proposée mis à jour (diff)

Mis à jour par Nicolas Thing-Leoh il y a presque 13 ans

  • Statut changé de À analyser à Assigné

Mis à jour par Nicolas Thing-Leoh il y a presque 13 ans

  • % réalisé changé de 0 à 100

Mis à jour par Éric Brison il y a presque 13 ans

  • Statut changé de Assigné à Intégré

Appliqué par commit commit:78968c15d06764ef5a13188af8a9cd9e3f8a29d2.

Actions

Formats disponibles : Atom PDF