Empecher envoi du formulaire plusieurs fois

Version imprimable

Bonjour,

Je sollicite votre aide afin de ne pas envoyer plusieurs fois le $_POST.
En effet, après l'actualisation de ma page, je suis invité à renvoyer l'enregistrement.
Pouvez-vous m'aider svp ?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
<?php
$FR_Email = NULL;
$MSG_OK = NULL;
$MSG_NO = NULL;
 
if(isset($_POST['Envoyer'])){
 
// Nettoyage des entrées
foreach($_POST as $var=>$val){if(!is_array($val)){$$var = strip_tags($val);}else{while(list($arvar,$arval) = each($val)){$$var[$arvar] = strip_tags($arval);}}}
 
if($MSG_NO==""){
	$res = $pdo->query("SELECT Email FROM tb_newsletter_adresse WHERE Email='$FR_Email';"); $data = $res->fetch(PDO::FETCH_ASSOC);
	if($data!=0) {$MSG_NO.= $NEWSLETTER_MSG_NO;}
 
	if($data==0)
	{
	$FR_Source = PAGE;
	$FR_Pays = $PAYS_ISO_2;
	$req_fr = $pdo->prepare ("INSERT INTO tb_newsletter_adresse (Email) VALUES (:FR_Email)");
	$req_fr ->execute(array("FR_Email"=>$FR_Email));
	$MSG_OK.= $NEWSLETTER_MSG_OK;
	}
}
}
?>
 
<form action="<?php echo $_SERVER['PHP_SELF']; ?>" method="post" enctype="multipart/form-data" name="Form1">
<input type="email" id="FR_Email" name="FR_Email" value="<?php echo $FR_Email; ?>" required >
<button type="submit" name="Envoyer">S'inscrire</button>
</form>
 
<?php if($MSG_OK){echo $MSG_OK;} ?>
<?php if($MSG_NO){echo $MSG_NO;} ?>

Tu peux mettre un flag qui n'affiche plus le formulaire si l'adresse est insérée. Une autre solution consiste à ne pas insérer l'adresse email si elle est déjà présente.

$data!=0 ne me semble pas correct.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
 
<?php
$FR_Email = NULL;
$MSG_OK = NULL;
$MSG_NO = NULL;
 
//---- ICI
$affiche = true;
 
if(isset($_POST['Envoyer'])){
 
// Nettoyage des entrées
foreach($_POST as $var=>$val){if(!is_array($val)){$$var = strip_tags($val);}else{while(list($arvar,$arval) = each($val)){$$var[$arvar] = strip_tags($arval);}}}
 
if($MSG_NO==""){
 $res = $pdo->query("SELECT Email FROM tb_newsletter_adresse WHERE Email='$FR_Email';"); $data = $res->fetch(PDO::FETCH_ASSOC);
 if($data!=0) {$MSG_NO.= $NEWSLETTER_MSG_NO;}
 
 if($data==0)
 {
 $FR_Source = PAGE;
 $FR_Pays = $PAYS_ISO_2;
 $req_fr = $pdo->prepare ("INSERT INTO tb_newsletter_adresse (Email) VALUES (:FR_Email)");
 $req_fr ->execute(array("FR_Email"=>$FR_Email));
 $MSG_OK.= $NEWSLETTER_MSG_OK;
 //---- ICI
 $affiche = false;
 }
}
}
?>
<?php
//---- ICI
if ($affiche == true)
{?>
<form action="<?php echo $_SERVER['PHP_SELF']; ?>" method="post" enctype="multipart/form-data" name="Form1">
<input type="email" id="FR_Email" name="FR_Email" value="<?php echo $FR_Email; ?>" required >
<button type="submit" name="Envoyer">S'inscrire</button>
</form>
<?php
//---- ICI
}?>
 
<?php if($MSG_OK){echo $MSG_OK;} ?>
<?php if($MSG_NO){echo $MSG_NO;} ?>

14/07/2018, 23h06
sinistrus

Merci Badaze et merci de t'intéresser à mon problème.

En effet, <form> ne s'affiche plus lorsqu'une adresse mail est injectée, mais l'actualisation renvoie toujours ce message :

Citation:

Pour afficher cette page, les informations précédemment transmises par Firefox doivent être renvoyées. Ceci répétera toute action (telle qu’une recherche ou un ordre d’achat) entreprise précédemment.

Tu penses pouvoir m'aider d'avantage ?

Si tu veux rester sur la même page tu peux faire comme ça.

Cela ne sert à rien d'avoir un MESSAGE_NO et un MESSAGE_OK puisque les deux ne sont jamais renseignés en même temps.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
 
<?php
$FR_Email = NULL;
$MESSAGE = NULL;
 
if(isset($_POST['Envoyer'])){
 
// Nettoyage des entrées
foreach($_POST as $var=>$val){if(!is_array($val)){$$var = strip_tags($val);}else{while(list($arvar,$arval) = each($val)){$$var[$arvar] = strip_tags($arval);}}}if(isset($_POST)){
 $res = $pdo->query("SELECT Email FROM tb_newsletter_adresse WHERE Email='$FR_Email';"); $data = $res->fetch(PDO::FETCH_ASSOC);
 if($data!=0) {$MESSAGE.= $NEWSLETTER_MSG_NO;}
 
 if($data==0)
 {
 $FR_Source = PAGE;
 $FR_Pays = $PAYS_ISO_2;
 $req_fr = $pdo->prepare ("INSERT INTO tb_newsletter_adresse (Email) VALUES (:FR_Email)");
 $req_fr ->execute(array("FR_Email"=>$FR_Email));
 $MESSAGE.= $NEWSLETTER_MSG_OK;
 //---- ICI
 header("location:".$_SERVER['PHP_SELF']."?message=".$MESSAGE);
 // }
}
}
?>
<?php
//---- ICI
if (isset($_GET['message']) == false)
{?>
<form action="<?php echo $_SERVER['PHP_SELF']; ?>" method="post" enctype="multipart/form-data" name="Form1">
<input type="email" id="FR_Email" name="FR_Email" value="<?php echo $FR_Email; ?>" required >
<button type="submit" name="Envoyer">S'inscrire</button>
</form>
<?php
//---- ICI
}?>
<?php echo $_GET['message'];?>

15/07/2018, 10h01
Invité
Bonjour,
- PRINCIPE de TRAITEMENT et FORMULAIRE sur la même page
- + empêcher renvoi par actualisation.

Merci Jreaux62 et remercie pour ton aide.
Penses-tu que ce que j'ai fais est dans les correct ?
Le message lors de l'actualisation me vient toujours, mais l'enregistrement ne se renvoie plus (je penses :aie:)...

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
<?php
$FR_Email 	= NULL;
$MSG_OK 	= NULL;
$MSG_NO 	= NULL;
$affiche 	= true;
?>
 
<?php if(session_id()=='') { session_start(); } if (isset($_SESSION['random_OK'], $_POST['randomformOK']) && $_POST['randomformOK']==$_SESSION['random_OK']) { ?>
 
<?php $_SESSION['Newsletter']['FR_Email'] = $FR_Email; ?>
 
<?php
if(isset($_POST['Envoyer'])){
 
foreach($_POST as $var=>$val){if(!is_array($val)){$$var=strip_tags($val);}else{while(list($arvar,$arval)=each($val)){$$var[$arvar]=strip_tags($arval);}}}
 
if($MSG_NO=="") {
$res = $pdo->query("SELECT Email FROM tb_newsletter_adresse WHERE Email='$FR_Email';"); $data = $res->fetch(PDO::FETCH_ASSOC);
if($data!=0) {$MSG_NO.= $NEWSLETTER_MSG_NO;}
 
if($data==0) {
 $FR_Source = PAGE;
 $FR_Pays = $PAYS_ISO_2;
 $req_fr = $pdo->prepare ("INSERT INTO tb_newsletter_adresse (Email, Source, Pays) VALUES (:FR_Email, :FR_Source, :FR_Pays)");
 $req_fr ->execute(array("FR_Email"=>$FR_Email, "FR_Source"=>$FR_Source, "FR_Pays"=>$FR_Pays));
 $MSG_OK.= $NEWSLETTER_MSG_OK;
 $affiche = false; }
}
}
?>
 
<?php } unset($_POST); $_SESSION['random_OK'] = rand(100000,999999); ?>
 
<?php unset($_SESSION['Newsletter']); ?>
 
<?php if($affiche == true) { ?>
 
<form action="<?php echo $_SERVER['PHP_SELF']; ?>" method="post" enctype="multipart/form-data" name="Form">
<input type="hidden" name="randomformOK" value="<?php echo $_SESSION['random_OK']; ?>" />
<input type="email" id="FR_Email" name="FR_Email" value="<?php echo $FR_Email; ?>" required >
<button class="btn btn-primary" type="submit" name="Envoyer">Envoyer</button>
 
</form>
 
 
<?php } ?>
<?php if($MSG_OK){ ?><div class="div_20"></div><div class="alert alert-primary col-md-5" role="alert"><?php echo $MSG_OK; ?></div>
<?php } ?>
<?php if($MSG_NO){ ?><div class="div_20"></div><div class="alert alert-danger  col-md-5" role="alert"><?php echo $MSG_NO; ?></div>
<?php } ?>

16/07/2018, 11h15
Invité
Bonjour,

Citation:

Envoyé par sinistrus

Penses-tu que ce que j'ai fais est dans les correct ?

Réponse évidente : non.
Si tu mets les bouts de code n'importe où, sans logique, c'est normal.

De plus, je t'ai indiqué un PRINCIPE de TRAITEMENT et FORMULAIRE sur la même page (qui fonctionne TRÈS bien).
Pourquoi ne l'utilises-tu pas ??
Au lieu de "bricoler" ton code : repars du mien.

Et :
- Mets des commentaires / explications dans ton code ! Ça t'aidera à comprendre ce qui fait quoi et quand.
- INDENTE ton code ! Il sera plus facile à lire, et à débuguer.
- Pourquoi ouvrir et fermer PHP (<?php ... ?>) là où ce n'est pas nécessaire ? (dans toute la 1ère partie du code, où il n'y a QUE du PHP ?)