can someone please tell me whats wrong with my code

PHP Code:

<?php
$day = 60 * 60 * 24 * 60 + time(); 
$theme = $_POST['theme'];
$dtheme = "naruto";


if(isset($_COOKIE['theme'])
{
echo "<?php include 'theme/ ";
echo "$_COOKIE['theme'];";
echo ".txt";
}
else
{
setcookie('theme', $dtheme, $day);
}



?>

You don't need to "echo" the php code to include a file. Also, do you want to set the cookie to the new theme if its passed?

if the cookie doesnt exist i want it to set the cookie to the default theme (naruto)

but if it does it should read it in from the cookie

and if someone changes the them it should change the cookie

doesnt work and the code is now

<?php
$day = 60 * 60 * 24 * 60 + time();
$theme = $_POST['theme'];
$dtheme = "naruto";


if(isset($_COOKIE['theme'])
{
include("theme/$theme.php");
}
else
{
setcookie('theme', $dtheme, $day);
include ("theme/naruto.txt");
}



?>

PHP Code:

<?php
 
  $theme = 'naruto';
  
  if (isset($_COOKIE['theme']) AND file_exists('theme/' . $_COOKIE['theme'] . '.txt'))
  {
    include('theme/' . $_COOKIE['theme'] . '.txt');
    $theme = $_COOKIE['theme'];
  }
  else
  {
    if (isset($_POST['theme']) AND file_exists('theme/' . $_POST['theme'] . '.txt'))
    {
      include('theme/' . $_POST['theme'] . '.txt');
      $theme = $_POST['theme'];
    }
    else
    {
      include('theme/' . $theme . '.txt');
    }
    
    setcookie('theme', $theme, time()+60*60*24*60);
  }
 
?>

I just realized that when you include() your theme file, you will not be able to set your new cookie because output will already be sent to the browser.

You can do this instead (set cookie before including the file)

PHP Code:

<?php
 
  $theme = 'naruto';
 
  if (isset($_COOKIE['theme']) AND file_exists('theme/' . $_COOKIE['theme'] . '.txt'))
  {
    $theme = $_COOKIE['theme'];
  }
  else
  {
    if (isset($_POST['theme']) AND file_exists('theme/' . $_POST['theme'] . '.txt'))
    {
      $theme = $_POST['theme'];
    }
 
    setcookie('theme', $theme, time()+60*60*24*60);
  }
 
  include('theme/' . $theme . '.txt');
 
?>

Good posting Mgraphic

i cant change the them with my form

<div align="center">

<form method="post" action="index.php">
<select multiple name="theme" size="4">
<option value="Naruto" selected>Naruto</option>
<option value="negima" >Negima</option>
</select>
<input type="submit" value="Change">
</form>


</div>

and my index.php code is

<?php

$theme = 'negima';

if (isset($_COOKIE['theme']) AND file_exists('theme/' . $_COOKIE['theme'] . '.txt'))
{
include('theme/' . $_COOKIE['theme'] . '.txt');
$theme = $_COOKIE['theme'];
}
else
{
if (isset($_POST['theme']) AND file_exists('theme/' . $_POST['theme'] . '.txt'))
{
include('theme/' . $_POST['theme'] . '.txt');
$theme = $_POST['theme'];
}
else
{
include('theme/' . $theme . '.txt');
}

setcookie('theme', $theme, time()+60*60*24*60);
}

?>

Firstly i think you want to remove the multiple, because you cant have more than one theme at the same time!

and i dont think you need Size

ill try that now

still doesnt do anything

code:

<div align="center">

<form method="post" action="index.php">
<select name="theme">
<option value="Naruto" selected>Naruto</option>
<option value="negima" >Negima</option>
</select>
<input type="submit" value="Change">
</form>

</div>

Alright, I made a mistake. This will allow you to set a new theme and the comments show what is going on...

PHP Code:

<?php
  
  // Set default
  $theme = 'naruto';
  
  if (isset($_POST['theme']) AND file_exists('theme/' . $_POST['theme'] . '.txt'))
  {
    // Set to new selected theme from previous form
    $theme = $_COOKIE['theme'];
  }
  else if (isset($_COOKIE['theme']) AND file_exists('theme/' . $_COOKIE['theme'] . '.txt'))
  {
    // Set to current cookie selected
    $theme = $_POST['theme'];
  }
  
  setcookie('theme', $theme, time()+60*60*24*60);
  include('theme/' . $theme . '.txt');
  
?>

im sorry it doesnt work at the moment (with the form changing it)