Conformation newsletter fails

[rsantifort]

When a user want to confirm subscription to a newsletter, a slash (/) is missing in the path.

[frold]

can you give us the code - then we can change it ourself!

[Jenpet]

Code: Select all

<?php
/*************************************************************************************
   Copyright notice
   
   (c) 2002-2003 Oliver Georgi (oliver@phpwcms.de) // All rights reserved.
 
   This script is part of PHPWCMS. The PHPWCMS web content management system is
   free software; you can redistribute it and/or modify it under the terms of
   the GNU General Public License as published by the Free Software Foundation;
   either version 2 of the License, or (at your option) any later version.
  
   The GNU General Public License can be found at http://www.gnu.org/copyleft/gpl.html
   A copy is found in the textfile GPL.txt and important notices to the license 
   from the author is found in LICENSE.txt distributed with these scripts.
  
   This script is distributed in the hope that it will be useful, but WITHOUT ANY 
   WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR A
   PARTICULAR PURPOSE.  See the GNU General Public License for more details.
 
   This copyright notice MUST APPEAR in all copies of the script!
*************************************************************************************/

// redirect old article.php to the new index.php
header("Location: http://".$_SERVER['HTTP_HOST'].str_replace("\\", "/", dirname($_SERVER['PHP_SELF']))."/include/inc_act/act_newsletter.php?".$_SERVER['QUERY_STRING']);
exit();


?>

This is the correct code in the verify.php you have to insert a / before include/inc_act/act_newsletter.php.......

[colech]

My download of RC-4 on May 5 still needs this fixed. After adding the "/" it worked beautifully.

Just wanted to pass this along in case the newer vesions of RC-4 don't have it yet.

[DeXXus]

... in case the newer vesions of RC-4 don't have it yet.

The newer version looks like this:

Code: Select all

<?php
/*************************************************************************************
   Copyright notice
   
   (c) 2002-2004 Oliver Georgi (oliver@phpwcms.de) // All rights reserved.
 
   This script is part of PHPWCMS. The PHPWCMS web content management system is
   free software; you can redistribute it and/or modify it under the terms of
   the GNU General Public License as published by the Free Software Foundation;
   either version 2 of the License, or (at your option) any later version.
  
   The GNU General Public License can be found at http://www.gnu.org/copyleft/gpl.html
   A copy is found in the textfile GPL.txt and important notices to the license 
   from the author is found in LICENSE.txt distributed with these scripts.
  
   This script is distributed in the hope that it will be useful, but WITHOUT ANY 
   WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR A
   PARTICULAR PURPOSE.  See the GNU General Public License for more details.
 
   This copyright notice MUST APPEAR in all copies of the script!
*************************************************************************************/

// redirect old article.php to the new index.php
$curdir  = dirname($_SERVER['PHP_SELF']);
$curdir  = str_replace("\", "/", $curdir).'/';
$curdir  = preg_replace('/(.*)\/\/$/i', '$1/', $curdir);
$curdir  = $_SERVER['HTTP_HOST'].$curdir;
$curdir .= 'include/inc_act/act_newsletter.php?';
$curdir .= $_SERVER['QUERY_STRING'];
header("Location: http://".$curdir);
exit();

?>

[colech]

How could I doubt. I am ashamed. Oliver is always one step ahead... oh wait... two step ahead of me. Thanks for the update DeXXus!