[pLog-svn] r7119 - plog/branches/lifetype-1.2/class/view

jondaley at devel.lifetype.net jondaley at devel.lifetype.net
Mon Feb 28 11:14:59 EST 2011


Author: jondaley
Date: 2011-02-28 11:14:59 -0500 (Mon, 28 Feb 2011)
New Revision: 7119

Modified:
   plog/branches/lifetype-1.2/class/view/errorview.class.php
Log:
add an 404 status on ALL error pages.  I know there was a discussion a while back about not returning 404s on dynamic URLs (index.php?asd=blah) but I can't find any references on the internet of if that is a bad thing.  Does anyone have a link to someone saying that is bad?  (I have an error_fetching_album page that is indexed higher than other pages, so I'm trying to get that out of the search engine index)

Modified: plog/branches/lifetype-1.2/class/view/errorview.class.php
===================================================================
--- plog/branches/lifetype-1.2/class/view/errorview.class.php	2011-02-28 15:47:09 UTC (rev 7118)
+++ plog/branches/lifetype-1.2/class/view/errorview.class.php	2011-02-28 16:14:59 UTC (rev 7119)
@@ -17,6 +17,7 @@
 		function ErrorView( $blogInfo, $message = null )
         {
         	$this->BlogView( $blogInfo, ERROR_TEMPLATE, SMARTY_VIEW_CACHE_DISABLED );
+			$this->addHeaderResponse( "HTTP/1.0 404" );
 
             $this->_message = $message;
         }



More information about the pLog-svn mailing list