Bug Formulaire PHP-MySQL

**Davboc** · 01/03/2006, 13h55

Salut
Je fais des formulaires PHP pour gérer une base de données MySQL, et j'ai une erreur quand je poste mon formulaire :

Code :

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

Parse error: parse error, expecting `T_STRING' or `T_VARIABLE' or `T_NUM_STRING' in d:\interfaceadminbdd\modifyproject-sendtodb.php on line 6

Le script PHP ou se trouve l'erreur est le suivant :

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
 
<?php
	if(isset($_POST["Project"])){
		mysql_connect("localhost","root","tulaurapamoncode");
		mysql_select_db("database");
		$query="UPDATE project p, prgmgr prg, prjmgr prj, busorg b, rdorg ro, customer c, platform pt
				SET p.Name=$_POST["ProjectName"],
				p.PrgMgrID=$_POST["ProgramManager"],
				p.PrjMgrID=$_POST["ProjectManager"],
				p.BusOrgID=$_POST["BusOrg"],
				p.RDOrgID=$_POST["RDOrg"],
				p.CustomerID=$_POST["Customer"],
				p.PlatformID=$_POST["Platform"],
				p.Type=$_POST["Type"],
				p.Archived=$_POST["Archived"]
				WHERE	p.ProjectID=$_POST["Project"]
				DELETE FROM siteprj WHERE ProjectID=$_POST["Project"];
				INSERT INTO siteprj
				VALUES($_POST["RDSite"], $_POST["Project"]);";
		mysql_query($query);
	}
?>

l'erreur actuelles est à la ligne SET p.Name=$_POST["ProjectName"],
ProjectName correspond à l'attribut "name" d'un input de type "text" et p.Name est un VarChar(200) dans ma Bdd.

Auriez vus une idée de là d'ou peut venir mon erreur ?
Merci !

**zulkifli** · 01/03/2006, 13h59

utilise plutot la variable $_POST avec des guillemet simple comme ceci $_POST['ProjectName']

**Yobs** · 01/03/2006, 14h02

ou tu peux placer ta requete entre des ' ' à la place de " " à toi de voir

**Davboc** · 01/03/2006, 14h03

Merci !
Malheureusement la modif ne change rien à mon problème, d'ailleurs dans d'autres parties ça marche avec des double quotes partout...

[edit] C'est pour la lisibilité ?

**Yobs** · 01/03/2006, 14h07

en fait non il faut que tu remplace toutes tes variables $_POST ou plus simple tu mets ta requete entre ' ' .

**Davboc** · 01/03/2006, 14h09

je l'ai fait j'ai mis toutes mes variables post comme ça :
$_POST['Var']

Mais ça n'a pas fait évoluer le bin's...

**zulkifli** · 01/03/2006, 14h10

essaie comme cela

Code :

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

1
2
3
4
 
      $query="UPDATE project p, prgmgr prg, prjmgr prj, busorg b, rdorg ro, customer c, platform pt
            SET p.Name=".$_POST['ProjectName'].", 
            p.PrgMgrID=".$_POST['ProgramManager'].", .....

**Bluespike62** · 01/03/2006, 14h11

Il y a des trucs louche dans ta requête :

- j'ai pas trop compris le DELETE et INSERT dans ta clause WHERE

- il te faut des quotes pour ton p.Name

Essaye ce code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
$query="UPDATE project p, prgmgr prg, prjmgr prj, busorg b, rdorg ro, customer c, platform pt 
            SET p.Name='".$_POST["ProjectName"]."', 
            p.PrgMgrID=".$_POST["ProgramManager"].", 
            p.PrjMgrID=".$_POST["ProjectManager"].", 
            p.BusOrgID=".$_POST["BusOrg"].", 
            p.RDOrgID=".$_POST["RDOrg"].", 
            p.CustomerID=".$_POST["Customer"].", 
            p.PlatformID=".$_POST["Platform"].", 
            p.Type='".$_POST["Type"]."', 
            p.Archived='".$_POST["Archived"]."'  
            WHERE   p.ProjectID=".$_POST["Project"];

**Davboc** · 01/03/2006, 14h12

ok merci !
Le delete et le insert c'est deux autres requetes je voulais tester si je pouvais mettre les trois ensemble... Mais ça vient pas de là si ?

**Bluespike62** · 01/03/2006, 14h14

Ton problème non, mais le fait de faire comme tu l'as fait deja ce n'est pas correct, et ca ne te donnera aucuns résultats, et c'est logique, ca veut rien dire ^^

Fait tes requetes séparement

**Davboc** · 01/03/2006, 14h31

ok donc j'ai remis comme tu l'as dit et séparé les requete

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
 
1 <?php
2 	if(isset($_POST['Project'])){
3 		mysql_connect("localhost","root","t4r4t4t4");
4 		mysql_select_db("pmodatabase");
5 		$query="UPDATE project p, prgmgr prg, prjmgr prj, busorg b, rdorg ro, customer c, platform pt
6				SET p.Name='".$_POST['ProjectName']."',
7				p.PrgMgrID=".$_POST['ProgramManager'].",
8				p.PrjMgrID=".$_POST['ProjectManager'].",
9				p.BusOrgID=".$_POST['BusOrg'].",
10				p.RDOrgID=".$_POST['RDOrg'].",
11				p.CustomerID=".$_POST['Customer'].",
12				p.PlatformID=".$_POST['Platform'].",
13				p.Type='".$_POST['Type']."',
14				p.Archived='".$_POST['Archived']."'
15				WHERE	p.ProjectID=."$_POST['Project']".";
16		mysql_query($query);
17		$query="DELETE FROM siteprj WHERE ProjectID=."$_POST['Project'];
18		mysql_query($query);
19		$query="INSERT INTO siteprj	VALUES($_POST['RDSite'], $_POST['Project]);";
20		mysql_query($query);
21	}
22 ?>

Et ça a evolué. L'erreur est maintenant la suivante

Code :

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

1
2
 
Parse error: parse error in d:\interfaceadminbdd\modifyproject-sendtodb.php on line 15

Il me semblait manquer une " et un . et meme en les ajoutant ça n'a rien changé. A moins que je ne les aie mal mis..

Au cas ou vous ne l'ayez pas compris je fais mon autoformation alors n'ayez pas peur de ma signaler des trucs qui vous semblent évidents !

**Yobs** · 01/03/2006, 14h45

essaie sa:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
$query='UPDATE project p, prgmgr prg, prjmgr prj, busorg b, rdorg ro, customer c, platform pt 
            SET p.Name="'.$_POST['ProjectName'].'", 
            p.PrgMgrID="'.$_POST['ProgramManager'].'", 
            p.PrjMgrID="'.$_POST['ProjectManager'].'", 
            p.BusOrgID="'.$_POST['BusOrg'].'", 
            p.RDOrgID="'.$_POST['RDOrg'].'", 
            p.CustomerID="'.$_POST['Customer'].'", 
            p.PlatformID="'.$_POST['Platform'].'", 
            p.Type="'.$_POST['Type'].'", 
            p.Archived="'.$_POST['Archived'].'" 
            WHERE   p.ProjectID="'.$_POST['Project'].'"';

**Davboc** · 01/03/2006, 14h45

ok donc j'ai remis comme tu l'as dit et séparé les requete

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
 
<?php
	if(isset($_POST['Project'])){
		mysql_connect("localhost","root","t4r4t4t4");
		mysql_select_db("pmodatabase");
		$query="UPDATE project p, prgmgr prg, prjmgr prj, busorg b, rdorg ro, customer c, platform pt
				SET p.Name='".$_POST['ProjectName']."',
				p.PrgMgrID=".$_POST['ProgramManager'].",
				p.PrjMgrID=".$_POST['ProjectManager'].",
				p.BusOrgID=".$_POST['BusOrg'].",
				p.RDOrgID=".$_POST['RDOrg'].",
				p.CustomerID=".$_POST['Customer'].",
				p.PlatformID=".$_POST['Platform'].",
				p.Type='".$_POST['Type']."',
				p.Archived='".$_POST['Archived']."'
15 -->			WHERE	p.ProjectID=."$_POST['Project']".";
		mysql_query($query);
		$query="DELETE FROM siteprj WHERE ProjectID=."$_POST['Project']".";
		mysql_query($query);
		$query="INSERT INTO siteprj	VALUES(."$_POST['RDSite']".,." $_POST['Project]".);";
		mysql_query($query);
	}
?>

Et ça a evolué. L'erreur est maintenant la suivante

Code :

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

1
2
 
Parse error: parse error in d:\interfaceadminbdd\modifyproject-sendtodb.php on line 15

Il me semblait manquer une " et un . et meme en les ajoutant ça n'a rien changé. A moins que je ne les aie mal mis..

Au cas ou vous ne l'ayez pas compris je fais mon autoformation alors n'ayez pas peur de ma signaler des trucs qui vous semblent évidents !

[edit] Doublé par Yobs !

**Yobs** · 01/03/2006, 14h46

essaie sa

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
$query='UPDATE project p, prgmgr prg, prjmgr prj, busorg b, rdorg ro, customer c, platform pt 
            SET p.Name="'.$_POST['ProjectName'].'", 
            p.PrgMgrID="'.$_POST['ProgramManager'].'", 
            p.PrjMgrID="'.$_POST['ProjectManager'].'", 
            p.BusOrgID="'.$_POST['BusOrg'].'", 
            p.RDOrgID="'.$_POST['RDOrg'].'", 
            p.CustomerID="'.$_POST['Customer'].'", 
            p.PlatformID="'.$_POST['Platform'].'", 
            p.Type="'.$_POST['Type'].'", 
            p.Archived="'.$_POST['Archived'].'" 
            WHERE   p.ProjectID="'.$_POST['Project'].'"';

**Davboc** · 01/03/2006, 14h55

Merci ! ça fonctionne !! merci merci merci !
reste un dernier ptit bug

Code :

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

Notice: Undefined index: Archived in d:\thomson\interfaceadminbdd\modifyproject-sendtodb.php on line 14

Archived désigne un checkbox. Quel sont les valeurs renvoyées par une checkbox ? Dans ma base de donnée j'ai mis 1 ou 0 mais c'est vrai que je ne sais pas si la checkbox renvoie ça...

**Bluespike62** · 01/03/2006, 15h09

Envoyé par Yobs

essaie sa

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
$query='UPDATE project p, prgmgr prg, prjmgr prj, busorg b, rdorg ro, customer c, platform pt 
            SET p.Name="'.$_POST['ProjectName'].'", 
            p.PrgMgrID="'.$_POST['ProgramManager'].'", 
            p.PrjMgrID="'.$_POST['ProjectManager'].'", 
            p.BusOrgID="'.$_POST['BusOrg'].'", 
            p.RDOrgID="'.$_POST['RDOrg'].'", 
            p.CustomerID="'.$_POST['Customer'].'", 
            p.PlatformID="'.$_POST['Platform'].'", 
            p.Type="'.$_POST['Type'].'", 
            p.Archived="'.$_POST['Archived'].'" 
            WHERE   p.ProjectID="'.$_POST['Project'].'"';

Normalement on ne doit pas mettre de " ou ' pour des caractères numériques.

Bizarre que tu ai une erreur avec ce que je t'ai donné, il n'y avait pas besoin de ." à la fin car il n'y a plus besoin de faire de concaténation.

PS : @Davboc : Quand tu parle d'ID, ce sont des numériques ou alpha-numériques ?

**zulkifli** · 01/03/2006, 15h14

un checkbox renvoi "on" quand il est coché
si tu veux renvoyé 1 ou 0 suivant qu'il est coché ou pas tu peut faire:

Code :

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

1
2
3
4
5
6
 
if ( isset($_POST['checkbox'])  ){
echo "1"; //coché
}else{
echo "0"; //pas coché
}

**Davboc** · 01/03/2006, 15h15

les id sont numeriques ce sont les clés de mes tables, de type int(10)

[edit] Merci zulkifli ! je viens à l'instant de tomber sur un article là dessus
http://www.globalissa.com/articles/articleCheckbox.php

Voilà ca marche !
Le code est là pour ceux que ça interessera :

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
<?php
	if(isset($_POST['Project'])){
		if ( isset($_POST['Archived'])  ){
			$Archive=1; //checked
		}
		else{
			$Archive=0; //not checked
		} 
		mysql_connect("localhost","root","votremotdepasse");
		mysql_select_db("votredatabase");
		$query='UPDATE project p, prgmgr prg, prjmgr prj, busorg b, rdorg ro, customer c, platform pt
            SET p.Name="'.$_POST['ProjectName'].'",
            p.PrgMgrID="'.$_POST['ProgramManager'].'",
            p.PrjMgrID="'.$_POST['ProjectManager'].'",
            p.BusOrgID="'.$_POST['BusOrg'].'",
            p.RDOrgID="'.$_POST['RDOrg'].'",
            p.CustomerID="'.$_POST['Customer'].'",
            p.PlatformID="'.$_POST['Platform'].'",
            p.Type="'.$_POST['Type'].'",
            p.Archived="'.$Archive.'"
            WHERE   p.ProjectID="'.$_POST['Project'].'"'; 
		mysql_query($query);
		$query='DELETE FROM siteprj WHERE ProjectID="'.$_POST['Project'].'"';
		mysql_query($query);
		$query='INSERT INTO siteprj	VALUES("'.$_POST['RDSite'].'","'.$_POST['Project'].'");';
		mysql_query($query);
	}
?>

Merci à tout ceux qui ont répondu de l'avoir fait si vite et si gentiment !

Maintenant il faut que je trouve comment stocker des chemins informatiques dans ma database et c'est bon pour ce formulaire !

Bug Formulaire PHP-MySQL

PHP & Base de données

Discussions similaires

Partager

Partager