Update med datum

Jag får inte riktigt till följande och jag misstänker att det har med de variblerna som ska in i ett datumfält i databasen. DE är $start och $slut som är datum.

$sql = mysql_query("UPDATE nyhet SET rubrik='".$_POST["rubrik"]."', text='".$_POST["text"]."', start='".$start."', slut='".$slut."' WHERE id=".$_POST["id"]"")

Om jag skriver ut sql-satsen får jag fäöjande rad.
UPDATE nyhet SET rubrik='Min rubrik', text='', start='20040827', slut='20080304' WHERE id=3
Dock kan jag tyvärr inte se vad som är fel. Nån som kan hjälpa mig?

Vad får du för felmeddelande? Får text vara null? Text kan också vara ett reserverat ord i din DBMS, testa att klamra in det.

... SET ... , [text]='".$_ ...

Text får vara null och det hjälper inte med klarar runt. DEt felmedelande som jag får är.
Parse error: parse error, unexpected T_CONSTANT_ENCAPSED_STRING in C:\Server root\int\update.php on line 6
rad 6 är den med sql-satsen.

Jag kan inte php, men avslutningen av raden ser konstig ut. Bort med de två avslutande citationstecknen, och dit med ett semikolon sist på raden.

Dessutom är text inget bra kolumnnamn, som SPiN skriver. Byt namn på fältet.

mvh

Vad jag kan se har du glömt en punk och ett semikolon.

Testa om det funkar med

$sql = mysql_query("UPDATE nyhet SET rubrik='".$_POST["rubrik"]."', text='".$_POST["text"]."', start='".$start."', slut='".$slut."' WHERE id=".$_POST["id"]."");

Uwe skrev:

Vad jag kan se har du glömt en punk och ett semikolon.

Kan du motivera varför du vill lägga till en tom sträng efter SQL-frågan med .""? Resultatet blir visserligen detsamma, men om man lägger in en massa onödiga saker som de här i koden blir det oöverskådligt.

mvh

@nders skrev:

Uwe skrev:

Vad jag kan se har du glömt en punk och ett semikolon.

Kan du motivera varför du vill lägga till en tom sträng efter SQL-frågan med .""? Resultatet blir visserligen detsamma, men om man lägger in en massa onödiga saker som de här i koden blir det oöverskådligt.

mvh

Den första punkten och dubbel-fnutten används för att sammanfoga strängen med php-koden. Men du har helt rätt i att det går att utesluta både fnuttar och punkten. Tänkte bara att det kanske var lättare att förstå när dahlgren redan använt sig av den varianten.

Hur som helst är den ursprungliga strängen inte helt rätt, därav felmeddelandet.

/Uwe

Den första punkten och dubbel-fnutten används för att sammanfoga strängen med php-koden.

Ja, självklart, det är jag helt med på. Det verkar bara som att det sprider sig som en löpeld (framför allt inom ASP/VBScript-världen) att vilja konkatenera in ingenting till sina strängar helt i onödan.

Jag står alltså fast vid det jag skrev tidigare; bort med de två citationstecknen, och på med ett semikolon. :)

@nders skrev:

Jag står alltså fast vid det jag skrev tidigare; bort med de två citationstecknen, och på med ett semikolon. :)

Måste medge att jag håller med dig till fullo. Koden blir både snyggare och snabbare. :bire

Eftersom hela raden ser ut så här

$sql = mysql_query("UPDATE nyhet SET rubrik='".$_POST["rubrik"]."', [text]='".$_POST["text"]."', start='".$start."', slut='".$slut."' WHERE id=".$_POST["id"]"")
or
exit(mysql_error());

så blir det inte så bra om jag tar bort de två citationtecknen och stopar dit ett semikolon tyvärr.

@nders skrev:

Jag kan inte php

Inte jag heller ;) Men anyhoo: Är det verkligen nödvändigt att konkatenera PHP-variabler i SQL-frågor? Bör inte följande fungera:

mysql_query("UPDATE nyhet SET rubrik='$_POST["rubrik"]', [text]='$_POST["text"]', start='$start', slut='$slut' WHERE id=$_POST["id"]")

:)

Då får jag följande fel istället.
Parse error: parse error, unexpected '"', expecting T_STRING or T_VARIABLE or T_NUM_STRING in C:\Server root\int\update.php on line 6

mysql_query(UPDATE nyhet SET rubrik='$_POST["rubrik"]', [text]='$_POST["text"]', start='$start', slut='$slut' WHERE id=$_POST["id"])

:q

Hmm det här går ju inte så bra. Nu kom det ett nytt felmedelande.

Parse error: parse error, unexpected T_STRING in C:\Server root\int\update.php on line 6

mysql_query(
"UPDATE nyhet SET rubrik= '" . $_POST["rubrik"] . "'" .
",`text`='" .$_POST["text"] .  "'" . 
",`start`= '" . $start ."'" .
",slut='" . $slut . "'" .
" WHERE id = " . $_POST["id"]) or die(mysql_error());