[RESOLVED] Creating a 404 page

eseelke

Hello all,

I have a script that determines which sql page to show according to the url I am on, ie, if I am at www.domain.com/mypage.html, it assigns the variable $id as mypage. It then matches that to a sql field called urlname and then displays the info from that sql table.

I am trying to create a 404 page where the sql field urlname is 404. I want any page that is visited and does not have a matching urlname to display the 404 page instead.

I have tried everything that I could find using Google, but nothing seems to work. I would greatly appreciate any help given.

Thank you,
Eddie Seelke

Below is the entire script:

<?php
define("CLIENTAREA",true);
require("dbconnect.php");
require("includes/functions.php");
require("includes/clientareafunctions.php");

$id = $HTTP_GET_VARS['page'];
if($id == NULL){
$id = "index";
}

$query = 'SELECT * FROM tblpages WHERE urlname = "'.$id.'"';
$result = mysql_query ($query);
while ($data = @mysql_fetch_array ($result)){
$pagetitle = $data['title'];
$urlname = $data['urlname'];
$pageicon = $data['icon'];
$breadcrumb = $data['breadcrumb'];
$loggedincode = $data['loggedincode'];
$smartvalues = $data['smartvalues'];
$body = $data['body'];
}

if($id == NULL OR $id == "index")
{
$breadcrumbnav = '<a href="index.php">'.$_LANG['globalsystemname'].'</a>'; 
}else{
$breadcrumbnav = '<a href="index.php">'.$_LANG['globalsystemname'].'</a> > <a href="'.$urlname.'.php">'.$breadcrumb.'</a>'; 
}
initialiseClientArea($pagetitle,$pageicon,$breadcrumbnav);

if ($_SESSION['uid']) {
eval($loggedincode);
}

eval($smartvalues);

$smartyvalues["body"] = $body; 

$templatefile = "index";  

outputClientArea($templatefile);
?>

Shrike

header( 'HTTP/1.0 404 Not Found' );
die( 'Page not found' );

If you want to display the standard Apache error page, you can find it in the Apache installation.

eseelke

I appreciate the advice, but I was looking for a way to modify the above php script. If I use the apache directive, it won't use my system template.

The script above is the index.php file. It uses the system template for the layout, header, footer, menu, etc. It gets the body from the database.

It also uses the .htaccess file to change index.php?page=page1 into page1.html

#Page Manager
RewriteRule ^([^/\.]+).html?$ index.php?page=$1 [L]

If you visit http://www.domain.com/page1.html, it will look for urlname in the database called page1. It will then display the contents ($body) from the urlname of page1.

If you go to http://www.domain.com without a page, it will automatically use the urlname of index. It will then display the contents ($body) from the urlname of index.

What I am trying to do is create a table with the urlname 404 in the database. So, if someone goes to http://www.domain.com/page45.html and page45 does not exist as a urlname in the database, it will use the $body of urlname 404 instead.

I hope this makes sense.

Thank you in advance for any help provided.

Eddie

Shrike

You could change your SQL to be

SELECT * FROM tblpages WHERE urlname = "'.$id.'" OR urlname = '404'

So you would either get the requested page or 404 if it did not exist.

eseelke

Shrike;10930489 wrote:
You could change your SQL to be
SELECT * FROM tblpages WHERE urlname = "'.$id.'" OR urlname = '404'
So you would either get the requested page or 404 if it did not exist.

I tried this and it made all of the pages use the 404 urlname. Even the ones that were working correctly.

Shrike

Ah this is because you will be selecting both the required page and the 404 page, then looping over both in the while(). Either remove the while loop or add a LIMIT on the end of the SQL.

eseelke

Thanks for the reply.

Would you know how I can fix this?

Do I need the while statement for this to work? Or is there a better way?

Shrike

SELECT * FROM tblpages WHERE urlname = "'.$id.'" OR urlname = '404' LIMIT 1

eseelke

Thank you so much. That worked perfectly.

Eddie

eseelke

Apparently I was premature. It seemed to work because I only had two entries in the database.

With:

$query = 'SELECT * FROM tblpages WHERE urlname = "'.$id.'" OR urlname = "404" LIMIT 1';

It will show the 404 on all pages except for the first database entry.

With:

$query = 'SELECT * FROM tblpages WHERE urlname = "'.$id.'" OR urlname = "404"';

It will show the 404 on all pages except for the last database entry.

Shrike

I was thinking that the 404 would be the last entry, but I guess that was a mistake. Perhaps it's better to just do the queries separately.

$query = 'SELECT * FROM tblpages WHERE urlname = "'.$id.'"';
$result = mysql_query ($query); 
if( mysql_num_rows( $result ) == 0 )
{
  $query = 'SELECT * FROM tblpages WHERE urlname = "404"';
  $result = mysql_query ($query); 
}
...

Or perhaps with a UNION if your database supports it?

eseelke

Thank you Shrike,

That seemed to do it. I checked several pages this time.