Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

missing header for 404 status in the following pages #764

Closed
stan-bg opened this issue Jul 11, 2013 · 1 comment
Closed

missing header for 404 status in the following pages #764

stan-bg opened this issue Jul 11, 2013 · 1 comment

Comments

@stan-bg
Copy link
Contributor

stan-bg commented Jul 11, 2013

In Google Webmaster Tools i found large amounts of duplicate title tags, the reason for which was pages that existed, but they doesn't anymore(category pages,product pages and manufacturer pages).
Googlebot still indexed this pages with dublicate content(the content generated in not_found.tpl on category,product and manufacturer page),maybe because response header is with http status 200(OK).

The header with 404 status need to be added in category,product,manufacturer and information controller page:

//..
else {
//...

$this->response->addHeader($this->request->server['SERVER_PROTOCOL'] . '/1.1 404 Not Found');

   if (file_exists(DIR_TEMPLATE . $this->config->get('config_template') . '/template/error/not_found.tpl')) {
        $this->template = $this->config->get('config_template') . '/template/error/not_found.tpl';
   } else {
        $this->template = 'default/template/error/not_found.tpl';
   }


}
//..

P.S. Sorry for my bad english.

@danielkerr
Copy link
Member

fixed in the next commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants