[pat]

cobbled together this snippet:

 <?
if(!$page) {
include("home.php");
}
else{
include("/" . "$page" . ".php");
}
?> 

Put it in index.php, idea being that if no links are clicked, the page displays home.php where this code is palced.

If a link is clicked, it's sposed to load that page, but it doesn't!

My links look like index.php?page=background

any ideas?

[d3mcfadden]

have you defined $page?
eg.

PHP Code:

$page = isset ($_GET['page']) ? $_GET['page'] : "home"; 


most likely reg globals are off (as they should be) so you will have to define $page. This will eliminate the need for that if statement too because above we default it to "home" if none is supplied.

now just:

PHP Code:

include("/" . $page . ".php"); 


[EGONick]

PHP Code:

<?php
if($_REQUEST['page'])
{
    if(file_exists($_REQUEST['page'] . '.php'))
    {
        include($_REQUEST['page'] . '.php');
    }
    else
    {
        include('home.php');
    }
}
else
{
    include('home.php');
}
?>

Always check if the file exists.

[datahound]

Are you sorted with this now pat.

Liking that code EGONick. (borrows)

I think the / was messing you up pat.

[datahound]

Here's another trick Pat. (which I have mentioned before)

say you want to include a page stuff.htm.

the url index.php?page=stuff is not very SEO or user friendly.

So create another page stuff.php with the content

<?php

$page = "stuff";

include("index.php");

?>

[pat]

Thanks very much all, very helpful. I'm not overly worried about SEO but I'll see about adding that in DH.

[pgo]

Or you could rewrite "index.php?page=stuff" as "/stuff/" which would be better in the long run anyway, I think.