Différences
Ci-dessous, les différences entre deux révisions de la page.
Les deux révisions précédentes Révision précédente Prochaine révision | Révision précédente | ||
txs:framadate-p18:resolution_de_l_issue_237 [2018/06/20 15:22] – [Merge request] stph.crzt | txs:contrib:framadate_p18:resolution_de_l_issue_237 [2020/09/16 11:24] (Version actuelle) – modification externe 127.0.0.1 | ||
---|---|---|---|
Ligne 1: | Ligne 1: | ||
+ | ====== Résolution d'une issue 1 : #237 ====== | ||
+ | ===== Description de l' | ||
+ | ==== Lien ==== | ||
+ | https:// | ||
+ | ==== Texte ==== | ||
+ | Lors de la création d'un framadate, si l' | ||
+ | |||
+ | |||
+ | ==== Justification du choix ==== | ||
+ | C'est une issue qui est très simple | ||
+ | ===== Solution apportée ===== | ||
+ | ====La fonction trim ==== | ||
+ | La fonction //trim// prend en valeur d' | ||
+ | |||
+ | |||
+ | |||
+ | |||
+ | |||
+ | |||
+ | |||
+ | === Démarche des recherches | ||
+ | |||
+ | === Itération 1 : travail sur create_poll.php=== | ||
+ | |||
+ | J'ai tout d' | ||
+ | < | ||
+ | $mail=' | ||
+ | $mail= trim($mail); | ||
+ | echo $mail; | ||
+ | var_dump($mail); | ||
+ | var_dump($title); | ||
+ | var_dump($name); | ||
+ | </ | ||
+ | Avec ce code, et en ayant mis en commentaire | ||
+ | |||
+ | === Itération 2 : mail services=== | ||
+ | |||
+ | De l' | ||
+ | isValidemail | ||
+ | < | ||
+ | public | ||
+ | $email=$trim($mail); | ||
+ | return filter_var($mail, | ||
+ | </ | ||
+ | |||
+ | Ce code ne marchait pas, tout simplement parce que la classe ne modifiait pas la chaîne, mais faisait simplement une vérification si elle était bien conforme. | ||
+ | === Itération 3 : input services=== | ||
+ | |||
+ | La function filterMail prend en entrée une chaîne de caractères , qui retourne vraie si elle est valide selon le[[http:// | ||
+ | |||
+ | < | ||
+ | public function filterMail($mail){ | ||
+ | $mail=trim($mail); | ||
+ | return filter_var($mail, | ||
+ | } | ||
+ | </ | ||
+ | Ainsi le message d' | ||
+ | |||
+ | |||
+ | |||
+ | |||
+ | |||
+ | |||
+ | |||
+ | |||
+ | |||
+ | ===== Merge request ===== | ||
+ | https:// | ||
+ | |||
+ | **La contribution a été intégrée.** |