Shoutbox

Hey guys I wanted a way for users to select the style sheet they want to use for  my site and save their selection in a cookie so my friend Ezra wrote this up for me but it dosent work becouse the code just gives me and error "unexpected { on line 4"

so here is the code its not all of it just the part that gives the error

code:

---

<?php
//Setting the cookie

if(!isset($_COOKIE['css']){
    if(isset($_GET['style']) && $_GET['style'] != ""){
        setcookie("css", htmlspecialchars($_GET['style'], ENT_QUOTES), time()+11156);
        header("/");
    }
}
?>

---

if you need more code just request it

quote:

---

Originally posted by ValSpy
if(!isset($_COOKIE['css']){

---

There is a ) missing at the end, between the ) and the {

quote:

---

Originally posted by Leo
There is a ) missing at the end, between the ) and the {

---

Thanks leo but now I get a 500 error check it out, http://valspy.net/V2/set.php and if you click on in the header of this page it gives same error http://valspy.net/V2/index.php

quote:

---

Originally posted by ValSpy

quote:

---

Originally posted by Leo
There is a ) missing at the end, between the ) and the {

---

Thanks leo but now I get a 500 error check it out, http://valspy.net/V2/set.php and if you click on in the header of this page it gives same error http://valspy.net/V2/index.php

---

that would most likely be this line:

quote:

---

Originally posted by ValSpy
header("/");

---

I assume you're trying to do a header relocation? if you are then you need to do this:
header("Location: the place you are relocating to");
and im not sure if you can use / as the location, you will probably need to use the file, eg, index.php, although im not sure about this

yeah, I was tired when I made that

But I gave him a new working & tested version

quote:

---

Originally posted by Madman
and im not sure if you can use / as the location, you will probably need to use the file, eg, index.php, although im not sure about this

---

Yes, you can.

Edit: meh, I didn't see the header("/"); when I replied before (my first reply to this thread). Anyway, in this post I meant you can put header("Location: /");

Thanks guys for all your help and if you want to test it out go here http://valspy.net/V2/ and I have 3 colors blue, pink, orange if anyone wants to suggest a color please do so here in this thread

Red+Black would go pretty nicely!
Overall good start