[Conception] PB transmission de variables

Version imprimable

Bonjour,

je vien de mettre sur mon site une petite messagerie interne permettant aux membres de s'envoyer des messages.

j'ai une page appellée "liste de membres" avec un hyperlien permettant d'envoyer des messages. En cliquant dessus on est directement redirigé vers la page creation de messages, et le champs destinataire devrait être rempli....mais ce n'est pas le cas ...la transmission de variable ne semble pas se faire...
Quelqu'un voit il la faille ?
merci de votre collaboration.

script : liste membres

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
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
include_once('session.php');
 
//Page du haut
include("sources/haut.php");
 
//Fichier necessaire
include("sources/config.php");
 
// Verifie si on peut se connecter a la base sql
$connect=mysql_connect($host,$user,$pass) or die ("Impossible de se connecter");
mysql_select_db($base, $connect) or die ("Impossible de selectionner la base de donnees");
 
//Verifie si on s'est connecte a l'espace membre
 
$connom=$_SESSION['login'];    
$conpass=$_SESSION['mdp'];
$result=mysql_query ("SELECT * FROM membres WHERE login='$connom' and mdp='$conpass'") or die ("Requete impossible");
$autoconnect = mysql_num_rows($result);
if ($autoconnect==0) {
echo("<script language=\"javascript\">location=\"erreur.php\";</script>");}
?>
<br>
<table width="400" border="0" align="center" bgcolor="<? echo $colp;?>">
  <tr>
    <td>
      <table width="100%" border="0">
        <tr bgcolor="<? echo $cols;?>">
          <td>
            <div align="center"><font face="<? echo $font; ?>" size="<? echo $size; ?>"><b><font color="#FFFFFF">Liste
              des membres</font></b></font></div>
          </td>
        </tr>
      </table>
<?php
 
//Recupere le nombre de messages au total
$result=mysql_query ("SELECT * FROM membres ORDER by login") or die ("Requete impossible");
 
//Nombre de messages recuperer dans $nbmess
$nbmemb = mysql_num_rows($result);
 
//Premiere Page
if(!$start) {$start=0;}
 
//Recupere le nombre de messages au total
$result=mysql_query ("SELECT * FROM membres ORDER by login LIMIT $start,$messpage") or die ("Requete impossible");
 
//Affichage du nombre de membres
echo("<font face=\"$font\" size=\"$size\">&nbsp;Total des membres :</font><font face=\"$font\" size=\"$size\" color=\"#FF0000\">&nbsp;$nbmemb</font><hr>");
 
//Affichage des membres
while ($row = mysql_fetch_array($result)) {
 
$noma = $row["login"];
 
//Affichage des membres
echo("<table width=\"100%\" border=\"0\" bgcolor=\"#FFFFFF\">\n");
echo("<tr>\n");
echo("<center><td width=\"3%\"><a href=\"details.php?det=$noma\"><img src=\"images/detail.gif\" border=\"0\" alt=\"Details\"></a></td></center>\n");
echo("<center><td width=\"3%\"><a href=\"envoyermess.php?nomdest=$noma\"><img src=\"images/message.gif\" border=\"0\" alt=\"Envoyer un message\"></a></td></center>\n");
echo ("<center><td width=\"94%\"><font face=\"$font\" size=\"$size\" color=\"#000000\">$noma</font></td></center>\n");
echo("</tr>\n");
echo ("</table>\n");
}
//Si il n'y a pas encore de message recu
if ($nbmemb=="") {
echo("<table width=\"100%\" border=\"0\" bgcolor=\"#FFFFFF\"><tr>");
echo("<td><font face=\"<? echo $font; ?>\" size=\"$size\"><i>Aucun membre pour le moment</i></font>");
echo("</td></tr></table>");}
 
//Barre de navigation (page précédente - page suivante)
//Affichage ou non de l'url page precedente
echo("<table width=\"100%\" border=\"0\" align=\"center\"><tr><td width=\"30%\">");
if($start>0){
echo("<a href=\"listemembres.php?start=".($start-$messpage)."\"><font face=\"$font\" size=\"$size\">&lt;&lt; Pr&eacute;c&eacute;dente</font></a>");}
if($nbmemb>0){
echo("</td><td width=\"40%\"><div align=\"center\"><font face=\"$font\" size=\"$size\">Page ");
echo ($start + $messpage)/$messpage;
echo (" sur ");
echo ceil($nbmemb/ $messpage);}
echo("</font></div></td><td width=\"30%\">");
//Affichage ou non de l'url Page suivante
if($nbmemb>$start+$messpage){
echo("<div align=\"right\"><a href=\"listemembres.php?start=".($start+$messpage)."\"><font face=\"$font\" size=\"$size\">Suivante &gt;&gt;</font></a></div>");}
echo("</td>\n</tr>\n</table>\n");
?>
</td>
</tr>
</table>
<br>
<div align="center"><font face="<? echo $font; ?>" size="<? echo $size; ?>"><a href="messagerie.php">Retour à votre messagerie</a></font></div>
 
<?php
//Ferme la base de donnees
mysql_free_result($result);
mysql_close($connect);
 
//Page du bas
include("sources/bas.php");
 
?>

et mon script de la page d'envoi de messages

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
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
include_once('session.php');
 
//Page du haut
include("sources/haut.php");
 
//Fichier necessaire
include("sources/config.php");
 
// Verifie si on peut se connecter a la base sql
$connect=mysql_connect($host,$user,$pass) or die ("Impossible de se connecter");
mysql_select_db($base, $connect) or die ("Impossible de selectionner la base de donnees");
 
//Verifie si on s'est connecte a l'espace membre
 
$connom=$_SESSION['login'];    
$conpass=$_SESSION['mdp'];
$result=mysql_query ("SELECT * FROM membres WHERE login='$connom' and mdp='$conpass'") or die ("Requete impossible");
$autoconnect = mysql_num_rows($result);
if ($autoconnect==0) {
echo("<script language=\"javascript\">location=\"erreur.php\";</script>");}
?>
 
<script type="text/javascript" language="javascript1.1">
function ajsmiley(smiley){
var messtext = document.envoyermess.messtext.value;
avecsmiley = messtext+smiley;
document.envoyermess.messtext.value=avecsmiley;
}
</script>
 
<form method="post" action="validmess.php" name="envoyermess">
  <table width="500" border="0" align="center" bgcolor="<? echo $colp;?>" height="137">
    <tr>
      <td height="133">
        <table width="100%" border="0">
          <tr bgcolor="<? echo $cols;?>">
            <td>
              <div align="center"><font face="<? echo $font; ?>" size="<? echo $size; ?>"><b><font color="#FFFFFF">Messagerie
                 - Envoyer un message</font> </b></font></div>
            </td>
          </tr>
        </table>
        <table width="100%" border="0">
          <tr>
            <td width="20%"><font face="<? echo $font; ?>" size="<? echo $size; ?>"><b>Destinataire
              :</b></font></td>
            <td width="80%">
              <?php echo ("<input type=\"text\" name=\"messdest\" value=\"$nomdest\">"); ?>
              <font face="<? echo $font; ?>" size="<? echo $size; ?>"><a href="listemembres.php">Liste des membres</a></font></td>
          </tr>
          <tr>
            <td width="20%"><font face="<? echo $font; ?>" size="<? echo $size; ?>"><b>Sujet
              :</b></font></td>
            <td width="80%">
            <input type="text" name="messtitre">
          </tr>
          <tr>
            <td width="20%"><font face="<? echo $font; ?>" size="<? echo $size; ?>"><b>Message
              :</b></font></td>
            <td width="80%">
              <div align="left">
                <textarea name="messtext" cols="25" rows="5"></textarea>
                <font face="<? echo $font; ?>" size="<? echo $size; ?>"></font></div>
            </td>
          </tr>
        </table>
 
      <table width="100%" border="0">
          <tr>
            <td>
              <center><input type="submit" name="Submit" value="Valider"></center>
            </td>
          </tr>
        </table>
      </td>
    </tr>
  </table>
</form>
 
<div align="center">
  <font face="<? echo $font; ?>" size="<? echo $size; ?>"><a href="messagerie.php">Retour à votre messagerie</a></font></div>
 
<?php
mysql_free_result($result);
mysql_close($connect);
 
 
 
//Page du bas
include("sources/bas.php");
?>

11/09/2006, 16h57
druidev

Salut !

Je me trompe peut-être, mais je ne vois pas de:

Code:

$nomdest = $_GET['nomdest'];

tu pense qu'il faudrait que je remplace :

Code:

<?php echo ("<input type=\"text\" name=\"messdest\" value=\"$nomdest\">"); ?>

par

Code:

<?php echo ("<input type=\"text\" name=\"messdest\" value=\"$_GET['nomdest']\">"); ?>

en fait c'est un script que j'ai récupéré et adapté, car il utilisait un espace membre basé sur l'utilisation de cookies ....alors que mon espace membre utilise simplement des variables de session.

j'ai donc adapté la première partie
Code:

1 2 3 4 5 6 7 8 9 //Verifie si on s'est connecte a l'espace membre $connom=$_SESSION['login']; $conpass=$_SESSION['mdp']; $result=mysql_query ("SELECT * FROM membres WHERE login='$connom' and mdp='$conpass'") or die ("Requete impossible"); $autoconnect = mysql_num_rows($result); if ($autoconnect==0) { echo("<script language=\"javascript\">location=\"erreur.php\";</script>");} ?>
je me demande si ça ne vient pas de là.....y aurait il un lien lien ?

merci d'avance

11/09/2006, 17h21
druidev

Je pense pas qu'il y ait un rapport...essayes ce que je t'ai dit plus haut ca sera bon....
11/09/2006, 17h44
heteroclite

quand je met ça :

Code:

<?php echo ("<input type=\"text\" name=\"messdest\" value=\"$_GET['nomdest']\">"); ?>

j'ai une erreur de syntaxe....je vois pas où...

Code:

Parse error: syntax error, unexpected T_ENCAPSED_AND_WHITESPACE, expecting T_STRING or T_VARIABLE or T_NUM_STRING in /home/serevo/public_html/mess/envoyermess.php on line 55

Fais comme ca:

Code:

1
2
3
4
<?
$nomdest = $_GET['nomdest'];
echo "<input type='text' name='messdest' value='".$nomdest."'>"; 
?>

Sinon je vois pas...

voilà ...j'ai suivi tes recommandations......mais ça ne marche toujours pas ....le champs destinataire ne prends pas la variable transmise....une idée ?

mon nouveau script:

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
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
include_once('session.php');
 
//Page du haut
include("sources/haut.php");
 
//Fichier necessaire
include("sources/config.php");
 
// Verifie si on peut se connecter a la base sql
$connect=mysql_connect($host,$user,$pass) or die ("Impossible de se connecter");
mysql_select_db($base, $connect) or die ("Impossible de selectionner la base de donnees");
 
//Verifie si on s'est connecte a l'espace membre
 
$connom=$_SESSION['login'];    
$conpass=$_SESSION['mdp'];
$result=mysql_query ("SELECT * FROM membres WHERE login='$connom' and mdp='$conpass'") or die ("Requete impossible");
$autoconnect = mysql_num_rows($result);
if ($autoconnect==0) {
echo("<script language=\"javascript\">location=\"erreur.php\";</script>");}
?>
 
<script type="text/javascript" language="javascript1.1">
function ajsmiley(smiley){
var messtext = document.envoyermess.messtext.value;
avecsmiley = messtext+smiley;
document.envoyermess.messtext.value=avecsmiley;
}
</script>
 
<form method="post" action="validmess.php" name="envoyermess">
  <table width="500" border="0" align="center" bgcolor="<? echo $colp;?>" height="137">
    <tr>
      <td height="133">
        <table width="100%" border="0">
          <tr bgcolor="<? echo $cols;?>">
            <td>
              <div align="center"><font face="<? echo $font; ?>" size="<? echo $size; ?>"><b><font color="#FFFFFF">Messagerie
                 - Envoyer un message</font> </b></font></div>
            </td>
          </tr>
        </table>
        <table width="100%" border="0">
          <tr>
            <td width="20%"><font face="<? echo $font; ?>" size="<? echo $size; ?>"><b>Destinataire
              :</b></font></td>
            <td width="80%">
              <?$nomdest = $_GET['nomdest'];echo "<input type='text' name='messdest' value='".$nomdest."'>"; ?>
              <font face="<? echo $font; ?>" size="<? echo $size; ?>"><a href="listemembres.php">Liste des membres</a></font></td>
          </tr>
          <tr>
            <td width="20%"><font face="<? echo $font; ?>" size="<? echo $size; ?>"><b>Sujet
              :</b></font></td>
            <td width="80%">
            <input type="text" name="messtitre">
          </tr>
          <tr>
            <td width="20%"><font face="<? echo $font; ?>" size="<? echo $size; ?>"><b>Message
              :</b></font></td>
            <td width="80%">
              <div align="left">
                <textarea name="messtext" cols="25" rows="5"></textarea>
                <font face="<? echo $font; ?>" size="<? echo $size; ?>"></font></div>
            </td>
          </tr>
        </table>
 
      <table width="100%" border="0">
          <tr>
            <td>
              <center><input type="submit" name="Submit" value="Valider"></center>
            </td>
          </tr>
        </table>
      </td>
    </tr>
  </table>
</form>
 
<div align="center">
  <font face="<? echo $font; ?>" size="<? echo $size; ?>"><a href="messagerie.php">Retour à votre messagerie</a></font></div>
 
<?php
mysql_free_result($result);
mysql_close($connect);
 
 
 
//Page du bas
include("sources/bas.php");
?>

11/09/2006, 20h28
zulot

bon alors j'ai pas lu le code mais je vais te donner des indications.

Le lien que tu cree dans la liste des membres doit renvoyer vers ta pages d'envoie de messages plus le nom a qui envoyer du style envoie.php?dest=XXX

Puis dans la page de messagerie tu dois recupérer le nom du destinataire par get comme ca t'as été montré et remplir la case value de ton champs destinataire par cette valeure.
11/09/2006, 22h21
heteroclite
ok ....ça marche !
Mais j'ai encore un petit probleme ailleurs .....
Je dois pouvoir supprimer mes messages et cela se fait par une requete qui doit recuperer une variable par "get"....mais j'ai encore une erreur de syntaxe....une idée ?
Code:

1 2 3 //Supression du message if ($suppr) { $result1=mysql_query ("DELETE FROM messages WHERE id= $_GET['idmess'];") or die ("Erreur de suppression");}
12/09/2006, 08h51
druidev
Premièrement, tu devrais récupérer tes variables en GET dans des variables, et les utiliser ensuite.

Code:

$var1 = $_GET['var1'];

Personnellement je trouve ca plus clair.

Deuxièmement, n'oublies pas la concaténation:
Code:

1 2 3 //Supression du message if ($suppr) { $result1=mysql_query ("DELETE FROM messages WHERE id=".$_GET['idmess']) or die ("Erreur de suppression");}

ok merci ....je penserai à declarer les variables avant....
Il me reste encore un problème
Je n'arrive pas à supprimer un message.....la variable supp est bien transmise dans l'URL, je tente de la recuperer en GET dans la requete de suppression....mais rien à faire .....ça marche pas:aie:
Voyez vous la faille ?

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
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
include_once('session.php');
//Page du haut
include("sources/haut.php");
 
//Fichier necessaire
include("sources/config.php");
 
// Verifie si on peut se connecter a la base sql
$connect=mysql_connect($host,$user,$pass) or die ("Impossible de se connecter");
mysql_select_db($base, $connect) or die ("Impossible de selectionner la base de donnees");
 
//Verifie si on s'est connecte a l'espace membre
 
$connom=$_SESSION['login'];    
$conpass=$_SESSION['mdp'];
$result=mysql_query ("SELECT * FROM membres WHERE login='$connom' and mdp='$conpass'") or die ("Requete impossible");
$autoconnect = mysql_num_rows($result);
if ($autoconnect==0) {
echo("<script language=\"javascript\">location=\"erreur.php\";</script>");}
?>
<br>
<table width="500" border="0" align="center" bgcolor="<? echo $colp;?>">
  <tr>
    <td>
      <table width="100%" border="0">
        <tr bgcolor="<? echo $cols;?>">
          <td>
            <div align="center"><font face="<? echo $font; ?>" size="<? echo $size; ?>"><b><font color="#FFFFFF">Messagerie
               - Messages re&ccedil;us</font></b> </font></div>
          </td>
        </tr>
      </table>
<?php
 
$idmess = $_GET["id"];
$envmess = $_GET["env"];
$titremess = $_GET["titre"];
$datemess = $_GET["date"];
 
 
 
//Premiere Page
if(!$start) {$start=0;}
 
//Supression du message
if ($suppr) {
$result=mysql_query ("DELETE * FROM messages WHERE id=".$_GET['suppr']) or die ("Erreur de suppression");}
 
//Recupere le nombre de messages au total
$result=mysql_query ("SELECT * FROM messages WHERE dest='$connom' ORDER by date DESC, heure DESC LIMIT $start,$messpagerec") or die ("Requete impossible");
 
//Nombre de messages recuperer dans $nbmessrec
$nbmessrec = mysql_num_rows($result);
 
echo("<table width=\"100%\" border=\"0\">\n<tr>\n<td>\n");
echo("<div align=\"left\"><font face=\"<? echo $font; ?>\" size=\"$size\">Messages ");
echo("re&ccedil;us :</font> <font face=\"<? echo $font; ?>\" size=\"$size\" color=\"#FF0000\">$nbmessrec</font><br>\n");
echo("<font face=\"<? echo $font; ?>\" size=\"$size\"></font></div>");
echo("</td>\n</tr>\n</table><hr>");
echo("<table width=\"100%\" border=\"0\">\n<tr>");
echo("<td width=\"20%\"><font face=\"<? echo $font; ?>\" size=\"$size\"><center><b>Date</b></center></font></td>\n");
echo("<td width=\"20%\"><font face=\"<? echo $font; ?>\" size=\"$size\"><center><b>Expediteur</b></center></font></td>\n");
echo("<td width=\"47%\"><font face=\"<? echo $font; ?>\" size=\"$size\"><center><b>Sujet</b></center></font></td>\n");
echo ("<td width=\"13%\"><font face=\"<? echo $font; ?>\" size=\"$size\"><center><b>Effacer</b></center></a></font></td>");
echo("</tr>\n</table>");
 
//Affichage des messages trier par l'affichage par page
while ($row = mysql_fetch_array($result)) {
 
$idmess = $row["id"];
$envmess = $row["env"];
$titremess = $row["titre"];
$datemess = $row["date"];
 
//Date version dd mm yyyy
list($y,$m,$d) = explode("-",$datemess);
$tiret= "-";
$datemess2 = $d.$tiret.$m.$tiret.$y;
 
echo("<table width=\"100%\" border=\"0\" bgcolor=\"#FFFFFF\">\n<tr>");
echo("<td width=\"20%\"><font face=\"<? echo $font; ?>\" size=\"$size\" color=\"#FF0000\"><center>$datemess2</center></font></td>\n");
echo("<td width=\"20%\"><font face=\"<? echo $font; ?>\" size=\"$size\"><center>$envmess</center></font></td>\n");
echo("<td width=\"47%\"><font face=\"<? echo $font; ?>\" size=\"$size\"><a href=\"message.php?aff=$idmess\"><center>$titremess</center></a></font></td>\n");
echo ("<td width=\"13%\"><font face=\"<? echo $font; ?>\" size=\"$size\"><a href=\"messagerie.php?suppr=$idmess\"><center>Effacer</center></a></font></td>");
echo("</tr>\n</table>");
}
//Si il n'y a pas encore de message  recu
if ($nbmessrec=="") {
echo("<table width=\"100%\" border=\"0\" bgcolor=\"#FFFFFF\"><tr>");
echo("<td><font face=\"<? echo $font; ?>\" size=\"$size\"><i>Aucun message pour le moment</i></font>");
echo("</td></tr></table>");}
?>
</td></tr>
  <tr>
 <td>
<?php
//Barre de navigation (page précédente - page suivante)
//Affichage ou non de l'url page precedente
echo("<table width=\"100%\" border=\"0\" align=\"center\"><tr><td width=\"30%\">");
if($start>0){
echo("<a href=\"messagerie.php?start=".($start-$messpagerec)."\"><font face=\"$font\" size=\"$size\">&lt;&lt; Pr&eacute;c&eacute;dent</font></a>");}
if($nbmessrec>0){
echo("</td><td width=\"40%\"><div align=\"center\"><font face=\"$font\" size=\"$size\">Page ");
echo ($start + $messpagerec)/$messpagerec;
echo (" sur ");
echo ceil($nbmessrec/ $messpagerec);}
echo("</font></div></td><td width=\"30%\">");
//Affichage ou non de l'url Page suivante
if($nbmessrec>$start+$messpagerec){
echo("<div align=\"right\"><a href=\"messagerie.php?start=".($start+$messpagerec)."\"><font face=\"$font\" size=\"$size\">Suivant &gt;&gt;</font></a></div>");}
echo("</td>\n</tr>\n</table>\n");
?>
</td></tr>
    <tr>
    <td><hr>
    <table width="100%" border="0">
  <tr>
    <td><a href="envmessagerie.php"><font face="<? echo $font; ?>" size="<? echo $size; ?>">Messages envoy&eacute;s</font></a></td>
    <td><div align="right"><a href="envoyermess.php"><font face="<? echo $font; ?>" size="<? echo $size; ?>">Envoyer un message</font></a></div></td>
  </tr>
</table>
     </td>
  </tr>
</table><br>
<div align="center">
 
 
<?php
//Ferme la base de donnees
mysql_free_result($result);
mysql_close($connect);
 
 
//Page du bas
include("sources/bas.php");
?>

13/09/2006, 02h12
druidev

Si j'ai bien lu (il est tard), tu mets

Code:

if ($suppr)

alors que tu devrais mettre

Code:

if ($_GET['suppr'])
13/09/2006, 02h20
heteroclite

MERCI pour ton analyse ! ça marche:D