Discussion :

[harlock59]

voici mon code source (en ayant enlevé le superflu):

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
<?
$id = $_GET['id'];
echo $id;
 
include("connect.php");
 
if (isset($_POST['valide'])) 
{
 
  echo $req1="SELECT location.* FROM location WHERE id=$id";
 
  echo "<br>test<br>";
  $sql = mysql_query($req1) or die( mysql_error() );

jusqu'ici, la variable $id est bonne, puis apres, lorsque le formulaire est soumis, $id est annulé.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
  $req2="UPDATE `location` SET `titre` ='".$_POST["titre"]."', `ligne1`='". $_POST["ligne1"]."', `ligne2`='".$_POST["ligne2"]."', `ligne3`='".$_POST["ligne3"]."', `ligne4`= '" . $_POST[ "ligne4" ] . "', `ligne5` = '" . $_POST[ "ligne5" ] . "', `url_photo` = '" . $_FILES[ "userfile" ][ "name" ] . "' WHERE `id` = $id";
 echo $req2; 
  $sql = mysql_query($req) or die( mysql_error() ); 
  echo $sql;
 
  $sql = mysql_query( "SELECT * FROM `location`" ) or die( mysql_error() ); 
 
  $voir = mysql_fetch_assoc( $sql ) . "<br />\n";  
}
?>
 
<form method="post" action="modif_loc2.php" enctype="multipart/form-data">

et c'est ici que $id n'est pas transmis correctement:
voici ce qui s'affiche dans la page cible:
SELECT location.* FROM location WHERE id=
au lieu de id=5 (par exemple)

Code :

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

1
2
3
4
5
6
7
8
9
<? $id=$_GET['id']; ?>
Remplacer par:
<br>
Titre: <input type=text name="titre" size="55" value="<? echo $voir['titre'] ?>">
<br>
 
<input type="hidden" name="MAX_FILE_SIZE" value="4000000">
<input type="file" name="userfile">
<input type="submit" value="Mettre à jour" name="valide">

[CritikKiller]

Bonjour,

Bon en lisant ton code je ne vois pas ou peut se trouver l'erreur.

Le meilleur conseil que je peux te donner c'est de ne pas attribué une variable à une variable déja existante :

Code :

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

1
2
 
$toto = $_GET['toto']

Ceci est inutile, car tu crée une seconde variable avec le meme contenu que ta varible $_GET['toto'] que tu peux utiliser ou tu veux aussi.
Tu simplifies ton code, et tu alleges le serveur (l'assignation d'une variable utilise de la mémoire)

Remplace donc ton code par :

Code :

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

$req2="UPDATE `location` SET `titre` ='".$_POST["titre"]."', `ligne1`='". $_POST["ligne1"]."', `ligne2`='".$_POST["ligne2"]."', `ligne3`='".$_POST["ligne3"]."', `ligne4`= '" . $_POST[ "ligne4" ] . "', `ligne5` = '" . $_POST[ "ligne5" ] . "', `url_photo` = '" . $_FILES[ "userfile" ][ "name" ] . "' WHERE `id` = $_GET[id]";

Il ne faut pas mettre les apostrophes quand la variable est entre des guillemets.

Dis moi si ca marche!

[harlock59]

Il ne faut pas mettre les apostrophes quand la variable est entre des guillemets.

Dis moi si ca marche!

ce sont des simples quotes inversées

[CritikKiller]

Je parlais du :

Code :

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

$_GET[id]

$req2="UPDATE `location` SET `titre` ='".$_POST["titre"]."', `ligne1`='". $_POST["ligne1"]."', `ligne2`='".$_POST["ligne2"]."', `ligne3`='".$_POST["ligne3"]."', `ligne4`= '" . $_POST[ "ligne4" ] . "', `ligne5` = '" . $_POST[ "ligne5" ] . "', `url_photo` = '" . $_FILES[ "userfile" ][ "name" ] . "' WHERE `id` = $_GET[id]";

Pas des quotes inversées ^^

[harlock59]

oui mais comment conserver le $_GET['id'] lors de la soumission du formulaire ?

[CritikKiller]

$req2="UPDATE `location` SET `titre` ='".$_POST["titre"]."', `ligne1`='". $_POST["ligne1"]."', `ligne2`='".$_POST["ligne2"]."', `ligne3`='".$_POST["ligne3"]."', `ligne4`= '" . $_POST[ "ligne4" ] . "', `ligne5` = '" . $_POST[ "ligne5" ] . "', `url_photo` = '" . $_FILES[ "userfile" ][ "name" ] . "' WHERE `id` = $_GET[id]";

C'est pas ca que tu veux?

J'ai peut-etre mal compris ta question.

[Yobs]

pour conserver ton id, tu peux passer par un champ caché:

Code :

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

echo '<input type="hidden" name="id" value="'.$_GET['id'].'" />';

[harlock59]

ok, merci.

[harlock59]

j'ai mis ça, mais ca n'a pas l'air de marcher:

Code :

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

1
2
3
4
<form method="post" action="modif_loc2.php" enctype="multipart/form-data">
<? echo "<input type='hidden' name='id' value='".$_GET['id']."'>"; ?>
Remplacer par:
<br>

[harlock59]

voici les messages d'erreur:


Une erreur est survenue lors du chargement du fichier.SELECT location.* FROM location WHERE id=
test
You have an error in your SQL syntax. Check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1

[Yobs]

tu récupère ta valeur avec $_POST['id']