[PHP] Waarom werkt deze update niet??

[Verwijderd]

Hi,
Ik heb alles gecontrolleerd. Alle variabelen, Tabelnamen en colomnamen.
Ik begrijp er geen *** van.

De waarde van $adres komt uit een URL, en die werkt ook, want de $query komt goed uit.

PHP:

<?php 
$db = mysql_pconnect("127.0.0.1","xxxxx","xxxxxx") or die ("Error while connection to Database");
$database = flashpoint;
mysql_select_db ("$database");
$adres = $_GET['Emailadres'];
echo $adres;
$query = "UPDATE temp SET MailingList='n' WHERE Emailadres=$adres";
$results = mysql_query ($query, $db);
echo $query;
echo "<br>\n<br>\nYou're email adres (".$_GET['Emailadres'].") has been removed from the Mailinglist.";
?>

Iemand een idee.

[blackd]

Welke PHP versie?
$_GET[] gebruiken misschien? Lezen blackd, lezen

Oh, en maak die invoer maar wat veiliger..

[ Voor 48% gewijzigd door blackd op 01-05-2003 20:40 ]

[Verwijderd]

Laatste versie, maar als je goed kijkt gebuik ik $_GET[].
Alleen uit moedeloosheid geef ik een variable die waarde omdat ik het ff niet meer weet.
En alles variabelen krijgen de juiste waarden, want dat heb ik gecontrolleerd.

[ Voor 22% gewijzigd door Verwijderd op 01-05-2003 20:38 ]

[Verwijderd]

blackd schreef op 01 May 2003 @ 20:35:

Oh, en maak die invoer maar wat veiliger..

Het is een locale testomgeving

[D2k]

or die(mysql_error());
uitvoer hier inleveren onder de stippellijn
..................................

[blackd]

Verwijderd schreef op 01 May 2003 @ 20:40:
Het is een locale testomgeving

Nog wel..

Zet er eens een 'or die()' achter, en kijk eens wat de fout is.

PHP:

<?php
$result = mysql_query("SELECT my_col FROM my_tbl")
    or die("Invalid query: " . mysql_error());
?>

Bron

[ Voor 6% gewijzigd door blackd op 01-05-2003 20:43 ]

[Verwijderd]

......................................
You have an error in your SQL syntax near '@wanadoo.nl' at line 1

[gorgi_19]

En wat is het resultaat van

PHP:

echo $query;

?

[D2k]

nou ja zeg
der gaat wat fout.......

[Patrickvs]

code:

$database = flashpoint; 
mysql_select_db ("$database");

Zet de toekenning van de variabele database even als string d.m.v. aanhalingstekens:

code:

$database = "flashpoint"; 
mysql_select_db ( $database );

Zoiets moet volgens mij beter zijn....

[ Voor 8% gewijzigd door Patrickvs op 01-05-2003 20:51 ]

[Verwijderd]

Probeer de query eens zelf in mysql te goeien, via een mooi frontend, verder lijkt zoiets me ok wel een goed idee:

PHP:

$result = mysql_query($query)
           or die (mysql_error());

/edit:
ok dit was dubbel, maar nu een echte tip: doe eens haakjes om je mailadres in de query...

[ Voor 21% gewijzigd door Verwijderd op 01-05-2003 20:48 ]

[Verwijderd]

$query = UPDATE temp SET MailingList='n' WHERE Emailadres=[e-mail]

[Verwijderd]

PHP:

$query = "UPDATE temp SET MailingList='n' WHERE Emailadres=$adres";

moet het niet met aanhalings tekens tussen $adres? dus zo

PHP:

$query = "UPDATE temp SET MailingList='n' WHERE Emailadres='$adres'";

[ Voor 21% gewijzigd door Verwijderd op 01-05-2003 20:50 ]

[gorgi_19]

Mooi, en voer deze nu eens direct uit tegen je database zonder PHP.

UPDATE temp SET MailingList='n' WHERE Emailadres=[e-mail]

Verwijderd schreef op 01 May 2003 @ 20:48:

En volgens mij wilden daar al enkele mensen hem heen schoppen... Alleen hij mag er imho zelf achterkomen; leert hij gelijk nog een goede debugprocedure..

[ Voor 51% gewijzigd door gorgi_19 op 01-05-2003 20:49 ]

[Verwijderd]

okey, als ik hem in MySQL front doe krijg ik een error, iets met het email adres voor de @.
RAAR, als ik het in de tabel doe werkt het wel, want ik het de query direct uit MySQL front naar PHP gekopieerd.

[Verwijderd]

Verwijderd schreef op 01 May 2003 @ 20:48:
$query = "UPDATE temp SET MailingList='n' WHERE Emailadres=$adres";

moet het niet met aanhalings tekens tussen $adres? dus zo

$query = "UPDATE temp SET MailingList='n' WHERE Emailadres='$adres'";

idd en mocht dat niet werken moet het zo:

$query = "UPDATE `temp` SET `MailingList`= 'n' WHERE `Emailadres` = '$adres'";

[WimB]

dat emailadres moet tussen quotes want het is een string

edit:
nu staat het al een keer of 4 in dit topic, oeps

[ Voor 36% gewijzigd door WimB op 01-05-2003 20:52 ]

[D2k]

Verwijderd schreef op 01 May 2003 @ 20:40:
[...]


Het is een locale testomgeving

rule nr1: NEVER TRUST USER INPUT

[Verwijderd]

okey, het volgende.
ik heb de query nog een keer in MySQL front geprobeerd met single quote.
DIT WERKT, dus dat is het , maar nu geeft de error dat de database niet bestaat, maar die heb ik toch echt voor me neus.

[Patrickvs]

heb je mijn aanpassing al gedaan?

[gorgi_19]

Verwijderd schreef op 01 May 2003 @ 20:53:
okey, het volgende.
ik heb de query nog een keer in MySQL front geprobeerd met single quote.
DIT WERKT, dus dat is het , maar nu geeft de error dat de database niet bestaat, maar die heb ik toch echt voor me neus.

We zijn hier trouwens dol op exacte, originele, Engelse foutmeldingen. "Hij geeft aan dat hij niet bestaat" snappen wij niet; wat zegt hij precies.

Dan nog een tip.. Er staat in de FAQ een stukje over debuggen.. Zou je iets meer info, met resultaatcode en foutmeldingen kunnen posten. Het werkt niet; het andere werkte wel kunnen we op zich niet zo veel mee...

[ Voor 20% gewijzigd door gorgi_19 op 01-05-2003 20:57 ]

[Verwijderd]

Ja, geen verschil. Maar dat werkte altijd al, ook met andere scripts.

[Verwijderd]

gorgi_19 schreef op 01 May 2003 @ 20:55:
[...]

We zijn hier trouwens dol op exacte, originele, Engelse foutmeldingen. "Hij geeft aan dat hij niet bestaat" snappen wij niet; wat zegt hij precies.

Table 'flashpoint.temp' doesn't exist

Dit maakt het zeker VEEL duidelijker.

[mulder]

Ik vind btw dit: $database = flashpoint; al heel knap

[blackd]

Verwijderd schreef op 01 May 2003 @ 20:56:
Dit maakt het zeker VEEL duidelijker.

Jazeker, jij suggereert dat de database niet bestaat, terwijl MySQL je vertelt dat de tabel niet bestaat.

[Glimi]

Owkay

Ik zou het fijn vinden als je volgende keer wat meer initiatief toont en niet bij opmerkingen van andere personen hier gelijk gilt dat het niet werkt
Nu komt het een beetje over van 'het werkt niet! snel help me' en daar zijn de mensen hier niet zo van gecharmeerd

Zie P&W als een leerschool. Wij willen je graag helpen, als je aangeeft te willen leren Die houdt oa in, zelf zoeken, zelf onderzoek doen, dingen uitpluizen ed

Owjah, je antwoord staat al redelijk aan het begin van de thread

Succes ermee

[ Voor 89% gewijzigd door Glimi op 01-05-2003 21:05 ]