Mail script not working correctly

rickaclark

I have a script called thanks.php3 that is supposed to enter an email address into a text file called newsletter. It is then supposed to send a message to the individual email from a text file called autoresponder.txt.

The email address is inserted correctly into the appropriate text file. Unfortunately, the autoresponder text, which is 74 characters long, displays on the html page and sends a character count, of 74, to the email recipient.

Any ideas why?

<html><head><title>Thanks!</title></head><body>
<?

$Body =  readfile("data/autoresponder.txt");        

mail("$Email","Welcome to Heartland's Mailing list!",
          "$Body","From: Rick\nReply-To: rick@rickclark.com");

$myfile = file("data/newsletter.txt");
$fh = fopen("data/newsletter.txt","w");
for ($index=0; $index < count($myfile); $index++)
  {
  if ($Email != chop($myfile[$index]))
    {fputs($fh,$myfile[$index]);}
  }
fputs($fh,$Email."\n");
fclose($fh);

?>

Thank you!
<br><br>
<a href="index.php3">Home</a>.
</body></html>

drew010

problem is that you are using readfile....

PHP.NET wrote:
int readfile ( string filename [, bool use_include_path [, resource context]] )

Reads a file and writes it to the output buffer.

Returns the number of bytes read from the file.

use [man]file_get_contents[/man] instead.
also, it would be much easier if you were to open the newsletter.txt file in "a" mode and then you will only have to fwrite one time.

rickaclark

Thanks,

using file_get_contents worked fine. I tried using the append command, but the script duplicates the entire file while adding te new email. ie:

I cleared all names but one:
patty@domain.com

entered rick@doamin.com and the text file now holds:

patty@domain.com
patty@domain.com
rick@domain.com

I am open to suggestions.

Rick

drew010

<?php

$fp = fopen('file.txt', 'a');
fwrite($fp, "string\r\n");
fclose($fp);

?>

that should just open the file for writing and put the pointer at the end of the file. its strange how it seems to duplicate it. try that code and see what ends up happening with it.