Bug Formulaire PHP-MySQL

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

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, 14h59

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

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

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

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

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, 15h07

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

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

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, 15h10

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, 15h11

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, 15h12

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, 15h14

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, 15h31

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, 15h45

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, 15h45

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, 15h46

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, 15h55

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, 16h09

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, 16h14

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, 16h15

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

Vue hybride

Discussions similaires

Partager

Partager