Lige et lille spørgsmål ?

Den burde vel at ændre id i db til 1 hvis man changelayout.php?id=1 men den gør ingenting ved mysql

<a href="changelayout.php?id=1"><img src="pix/layouts/standard.gif"> altså hvis linket ser sådan her ud.

Husk først at afslutte dine links med </a>. Altså: <a href="changelayout.php?id=1"><img src="pix/layouts/standard.gif"></a>

Er du sikker på at SESSION[userID] indeholder noget?

Kan du evt. tjekke ved: echo $_SESSION['userID'];

Ja $_SESSION[userId]; inholder det userid som jeg har når jeg er logget ind :)
Så den virker som den skal ? Og linksne er rettet :)

Har tjekket session med echo så det virker

Damn jeg er stavespasser når jeg er træt :P

Det er fra en form ikke også? Har du husket at skrive method="get" og ikke method="post" i din form?

//mcardle

Prøv det her og husk at oprette forbindelse til din mysql-database:


<?
include("head.php");

if($_GET["id"])
{
$id = mysql_escape_string($_GET["id"]);
$userId = $_SESSION["userId"];
mysql_query("UPDATE brugere SET layout='$id' WHERE userId='$userId'")or die(mysql_error());
}
else if($_GET["reset"])
{
mysql_query("UPDATE brugere SET layout=0 WHERE userId='$userId'")or die(mysql_error());
}
?>
<br />
<h3>Ændre Layout på blabla.dk:</h3>
<br /><center>
<table width="500" id="layout">
    <tr>
        <td class=layout><a href="changelayout.php?reset"><img src="pix/layouts/standard.gif"></td>
        <td class=layout><a href="changelayout.php?id=1"><img src="pix/layouts/standard.gif"></td>
        <td class=layout>huhu</td>
        <td class=layout>huhu</td>
        <td class=layout>huhu</td>
    </tr>
</table>
</center>
<? include("foot.php"); ?>

//mcarlde

Det kan forøvrigt godt være det er nødtil at stå før din if:


<?
include("head.php");

$id = mysql_escape_string($_GET["id"]);
$userId = $_SESSION["userId"];

if($_GET["id"])
{
mysql_query("UPDATE brugere SET layout='$id' WHERE userId='$userId'")or die(mysql_error());
}
else if($_GET["reset"])
{
mysql_query("UPDATE brugere SET layout=0 WHERE userId='$userId'")or die(mysql_error());
}
?>
<br />
<h3>Ændre Layout på blabla.dk:</h3>
<br /><center>
<table width="500" id="layout">
    <tr>
        <td class=layout><a href="changelayout.php?reset"><img src="pix/layouts/standard.gif"></td>
        <td class=layout><a href="changelayout.php?id=1"><img src="pix/layouts/standard.gif"></td>
        <td class=layout>huhu</td>
        <td class=layout>huhu</td>
        <td class=layout>huhu</td>
    </tr>
</table>
</center>
<? include("foot.php"); ?>

Så skulle den være der.

I head.php bliver der oprettet forbindelse til mysql :)

Sådan der mcardle ? :)

Så virkede det sQ.
Men kan du fortælle mig hvorfor der lige skulle det der på ? :S

Oki reset virker ikke :S

det kunne muligvis også have virket hvis du havde brugt " i stedet for ' i din update-query:
eks:
mysql_query("UPDATE brugere SET layout=".$_GET[id]." WHERE userId=".$_SESSION[userId]."")or die(mysql_error());

Det skulle forresten være lige meget om $id = mysql_escape_string($_GET["id"]); $userId = $_SESSION["userId"]; står indenfor eller udenfor if'en

prøv at trække det sammen så der står elseif($_GET["reset"]) i stedet for else if($_GET["reset"])

Er prøvet. Gav intet resultat :(

lav dit link om så der står reset=1

burde det være nødvendig hvis jeg kalder reset i toppen hvor der står i mysql_query(UPDATE brugere SET layout=0 ? :S

Ikke for at modsætte mig men bare for at hører ?

Så er det prøvet og det virker heller ikke

du kan også sætte den til 0, jeg mener bare at for at if($_GET["reset"]) skal blive true skal der stå noget i $_GET["reset"] og det gør der ikke nu

du kunne også prøve at lave det samme som mcardle har gjort med id

Som hvad foreksempel ? :S

Altså jeg kunne jo bare ændre linket til id=0 men kunne bare godt tænke mig at få reset til at virke :)

det jeg mente var at du prøve det her:
$reset = mysql_escape_string($_GET["reset"]);

men kan du ikke prøve det her:
echo $_GET["reset"];

evt. sætte den ind lige før den update-query'en

det vil faktisk sige at ikke engang med ID kan jeg få den til at ændre sig til 0 som er standard :(

jojo det med at bruge id=0 kan sagtens lade sig gøre

Nu har jeg sat en echo ind foran mysql_query og når jeg trykker på den sender den ikke echoet :S

hmm okay den ændre ihvertfald bare ingenting hvis jeg laver linket så den hedder ?id=0 til sidst. i db bliver bare ved med at ha 1 som layout id

hvilken mysql_query har du sat den foran?

den med reset ?

Det mærkelige er at hvis jeg ændre id=2 så der står 2 i min layout i db og så ændre linket til 1 igen så går den ned på 1. men ændre jeg linket til id=0 sker der ikke en skid ? Har det nået at gøre med at 0 er en speciel værdi eller ? :S

hmm det kan være at den validere til false hvis der står 0...

Det gør den jo nok ? men kan man ikke gøre nået ved det ?

Burde den egentligtalt ikke komme ud med en mysql fejl om at det id man prøver at sætte den til er en fejl ? :S

det tror jeg ikke. problemet er at den ikke kan blive true i if'en hvis den er sat til 0
prøv at lave det om til if(isset($_GET["reset"]) && $_GET["reset"] != '') {

Ja og så imellem de 2 "" skal der vel stå 0 eller ?

SÅDAN DER :P

Oki men kan du forklare mig hvorfor der i else if skulle bruges det som du lige har givet mig og hvorfor der i det som mcardle har brugt skulle sættes det på ? Før det virkede ? :S

øhh jeg kan forsøge.. men jeg er elendig til at forklare ting, så er du advaret ;o)

if($_GET["reset"]) { kommer jo til at være if() { eller if(0) { og så bliver den jo false og derfor udføres koden mellem { } aldrig.

rettelsen som mcardle laver er egentlig ikke nødvendig som jeg skrev 29/11-2006 10:03:04, men den er ret god da den beskytter mod SQL-injection

Desuden kunne det måske også være at du skal ændre if($_GET["id"]) til if($id != '')

hmm okay ? :S Men hva så hvis den id som de prøver at finde ikke findes. Kan man så lave så de får en besked hvor der står den ikke findes ?? altså ud fra det jeg har lavet nu ? f.eks en else ? Er nemlig ikke så go til PHP endnu

jeg kan godt prøve at lave noget til dig, men jeg er nødt til at vide mere om hvordan du har lavet det med layout.
Har du en tabel med layout-id'er eller hvordan vælger du de forskellige layouts ud fra det tal der står i tabellen brugere?

Nå jeg er desværre nødt til at smutte. Skal til Doktormand skrives senere og tak for hjælpen indtil videre mcardle og sw red 6

Så er jeg her igen sw. Jeg vil gerne selv lave det :P Skal jo lærer det, men skal bare vide hvordan :P

Men nu har jeg siddet og strikket nået sammen, men det giver sQ ikke rigtig resultat :S

hmm en af jer der vil ha nogle point :) Har fikset det med fejl også :)

Points er altid gode ;-)

du kan da også få et svar fra mig

Tak til jer begge :)