What files control the 404 product not found?

[philip937]

Hi all, I touched on this quite some time ago but can't seem to find the thread.

I want to change the behaviour of my site when products are disabled. Instead of displaying the product not found page and issuing a 404 message, I want the product to display as normal.

So essentially, all I want disabling a product to do is for it to not be listed in the front facing catalogue and thus ensuring that the original link tot the item (whilst the id exists) still works. meaning that if someone shared a link somewhere to an old product, I still want that link to work and display the product.

so as a start, in the following file:

includes/modules/pages/product_info/header_php.php

this code:

 // if specified product_id is disabled or doesn't exist, ensure that metatags and breadcrumbs don't share inappropriate information
  $sql = "select count(*) as total
          from " . TABLE_PRODUCTS . " p, " .
                   TABLE_PRODUCTS_DESCRIPTION . " pd
          where    p.products_status = '1'
          and      p.products_id = '" . (int)$_GET['products_id'] . "'
          and      pd.products_id = p.products_id
          and      pd.language_id = '" . (int)$_SESSION['languages_id'] . "'";
  $res = $db->Execute($sql);
  if ( $res->fields['total'] < 1 ) {
   unset($_GET['products_id']);
    unset($breadcrumb->_trail[sizeof($breadcrumb->_trail)-1]['title']);
    $robotsNoIndex = true;
    header('HTTP/1.1 404 Not Found');
  } 

I assume I just remove the p.products_status = '1' WHERE clause and that will remove the 404 bit?

Thats just the start, can someone point me to ALL the other places I need to make a similar change so that the page displays correctly even when the product is disabled?

Many Thanks

Phil

[mc12345678]

Hi all, I touched on this quite some time ago but can't seem to find the thread.

I want to change the behaviour of my site when products are disabled. Instead of displaying the product not found page and issuing a 404 message, I want the product to display as normal.

So essentially, all I want disabling a product to do is for it to not be listed in the front facing catalogue and thus ensuring that the original link tot the item (whilst the id exists) still works. meaning that if someone shared a link somewhere to an old product, I still want that link to work and display the product.

so as a start, in the following file:

includes/modules/pages/product_info/header_php.php

this code:

 // if specified product_id is disabled or doesn't exist, ensure that metatags and breadcrumbs don't share inappropriate information
  $sql = "select count(*) as total
          from " . TABLE_PRODUCTS . " p, " .
                   TABLE_PRODUCTS_DESCRIPTION . " pd
          where    p.products_status = '1'
          and      p.products_id = '" . (int)$_GET['products_id'] . "'
          and      pd.products_id = p.products_id
          and      pd.language_id = '" . (int)$_SESSION['languages_id'] . "'";
  $res = $db->Execute($sql);
  if ( $res->fields['total'] < 1 ) {
   unset($_GET['products_id']);
    unset($breadcrumb->_trail[sizeof($breadcrumb->_trail)-1]['title']);
    $robotsNoIndex = true;
    header('HTTP/1.1 404 Not Found');
  } 

I assume I just remove the p.products_status = '1' WHERE clause and that will remove the 404 bit?

Thats just the start, can someone point me to ALL the other places I need to make a similar change so that the page displays correctly even when the product is disabled?

Many Thanks

Phil

Maybe it's just "luck" that this one snippet of code was used, but looks like the intent is to try to game the system about what is offered on the site.

Have you ever shopped at a site where it is impossible to find the thing that got you to the site other than from some external site? If the real intent is to target customers, I hope when that product is disabled that there is some good information about the situation as it sounds like once on the site at that product that can't be purchased because it no longer exists, it can never be found on the store again either as it won't display in the category(ies) in which it assigned.

[philip937]

My question was about zen cart code. I am not sure I understand your response?

[mc12345678]

My question was about zen cart code. I am not sure I understand your response?

Sorry, the short of it is: Why? Please explain.

[philip937]

Why what? Why do I want my URL to display useful information? Because someone landing on a white 404 page (or even a generic custom 404 page) is not helpful and a potential customer will bounce 99.99% of the time.

So the reasoning is that the user will land on a page where yes the product is no longer avaiable but will have custom code referring customers to newer and existing alternatives.

So the link remains useful.

[mc12345678]

Why what? Why do I want my URL to display useful information? Because someone landing on a white 404 page (or even a generic custom 404 page) is not helpful and a potential customer will bounce 99.99% of the time.

So the reasoning is that the user will land on a page where yes the product is no longer avaiable but will have custom code referring customers to newer and existing alternatives.

So the link remains useful.

Sounds like then still have a custom '404' page, but without telling the internet machines that there is/was a '404' error. Why do that?

[philip937]

404 not found. If the page is there then it is found. To be honest the question wasn't asking advice on the logic, I was after the technical answer.

[RodG]

I assume I just remove the p.products_status = '1' WHERE clause and that will remove the 404 bit?

This assumption isn't quite correct.

Removing the p.products_status = '1' will include the disabled product in the count(*), and that will indeed cause the '404 bit' to be skipped, assuming the product is still in the DB, but if the product with the $_GET['products_id'] has been deleted the res->fields['total'] will still be < 1 and the '404 bit' will be acted upon.

This may or may not be what you are after.

Cheers
RodG

[mc12345678]

404 not found. If the page is there then it is found. To be honest the question wasn't asking advice on the logic, I was after the technical answer.

My apologies that the thought is that asking a question necessitates an answer without discourse on the reasons and intentions. Heaven forbid that someone try to do something that appears as if involve a lot of work on their side to sustain or that appears to be trying to let search engines believe that the link previously found leads to the same content and really is a valid/useful link instead of a basic redirect without making it known that it is a redirect or any other type of non-existent product.

The solution is simple, and can be performed/determined from your own store's admin and some form of ftp/file manipulation tool. I'm still not convinced that the solution is one to which I feel it is right to give an answer and will allow another to offer the answer of pointing you to ALL the other places you need to make a similar change so that the page displays correctly even when the product is disabled?

[philip937]

Yes that pretty much what I'm after. So a disabled product will:

- Not be visible in the product listings
- Accessible via external links

A product deleted from database will:
- Will return 404 not found

In addition to the above I need to find out what code I need to change so that the product info page displays normally with the content and page title etc.

Any ideas RodD?