Discussion :

[choupinou22]

bonjour,

je me retrouve avec un petit problème,initialement j'ai un formulaire de login (login et password) quand l'utilisateur se connecte ,il se trouve sur un autre formulaire dont je récupère le login.
sur ce formulaire je me retrouve avec un tableau et quand l'utilisateur coche une checkbox de résolution son prénom apparaît dans un champ. jusque la tout fonctionne bien sauf que si un autre utilisateur fait la même procédure sur un autre enregistrement; son login écrase ceux qui sont déja enregistré sur les autre enregistrements.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
$NUM=$Resultat['NUM'];
 $login=($_SESSION['login']);
if(isset($_POST['resolu']) ){
     $sql2=$pdo->prepare("UPDATE messages SET RESOLU=:resolu, LOGIN=:login  where NUM=:NUM  " ) or die (print_r($pdo->errorinfo())) ;  
	foreach($_POST['resolu'] as $NUM=> $resolu){
             $sql2->execute(array( 
                ':resolu'=>$resolu,
                ':login'=>$login,
		':NUM'=>$NUM
	    )) ;
	}
}

merci

[Deuzz]

Bonsoir

Si ta requête change tous les logins de ta table c'est donc que where NUM=:NUM ne sert à rien...
Que contient $_POST['resolu'] pour que cette condition soit inutile ?

[sabotage]

et surtout $Resultat c'est quoi ?

[Bovino]

Probablement le résultat d'une requête du type

Code sql :

Sélectionner tout - Visualiser dans une fenêtre à part

SELECT NUM FROM messages

au vu du problème décrit...

[Invité]

Bonjour,
peux-tu nous exposer une bonne fois pour toutes le projet dans son ensemble ?
(du moins, la partie qui nous intéresse)

C'est déjà la 3ème discussion ouverte sur le même sujet...

[choupinou22]

ci joint le code (en fait je me suis aperçu qu'a part le login qui ecrase tout les login dont les cases sont cochées,j'ai le problème avec l'heure(je pense que cela vient du foreach):

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
$sql=$pdo->query( "SELECT * FROM messages " );
 while($Resultat =$sql->fetch()){
?>
<tr>
	    <td><?php echo $Resultat['NUM']; ?></td>
	    <td><?php echo $Resultat['DATE_RESOLUTION']; ?></td> 	
	    <td><input type="checkbox" name="resolu[<?php echo $Resultat['NUM']; ?>]" id="resolu" value="oui"<?php if ($Resultat['RESOLU']=="oui")  { echo ' checked="checked"';} ?> /></td>	
   	    <td><?php echo $Resultat['LOGIN']; ?></td> 
 </tr>
<?php
     }
 
$NUM=$Resultat['NUM'];
 
$login=($_SESSION['login']);
$heure=date('Y-m-d H:i:s');	
 
        if(isset($_POST['resolu'])    ) { 
 
	$sql2=$pdo->prepare("UPDATE messages SET DATE_RESOLUTION=:heure, RESOLU=:resolu  where NUM=:NUM  " ) or die (print_r($pdo->errorinfo())) ;  
	foreach($_POST['resolu'] as $NUM=> $resolu)
	{
 
        $sql2->execute(array(  
			              ':heure'=>$heure,
                                      ':resolu'=>$resolu,                                  
		                      ':NUM'=>$NUM
	 )) ;
 
}
 
?>

[sabotage]

A part plein de choses qui ne servent à rien :
- la ligne 14
- la date en PHP
- le "or die"
- des espaces placés un peu partout
- reprendre la valeur "resolu" du formulaire

Tu peux simplement ne pas mettre à jour les lignes qui sont déjà résolus :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
 }
        if(isset($_POST['resolu'])) { 
 
	$sql2=$pdo->prepare("UPDATE messages SET DATE_RESOLUTION=NOW(), RESOLU='oui' WHERE NUM = :NUM AND RESOLU <> 'oui'");  
	foreach($_POST['resolu'] as $NUM=>$resolu)
	{
        $sql2->execute(array(':NUM'=>$NUM));
        }
} 
?>

Après il serait quand même plus logique de ne plus avoir la checkbox pour des lignes résolus.