The problem looks like when the form loads, the replyto field on the form has a vaule of the user name and the user email address, when it only needs to have the email address. When the user submist the form, both the user name and the user email gets posted to the server, causing the email validity testing function to return false.
That's how I remember it. Why it
doesn't
fail sometimes, I dunno. In any case, validating against user->email only should do the trick.
OK, I switched the test email.php back to the way it was and added a debugging line. On the broken test version, the testing function is only looking at the user name, not the user name and user email. I don't yet know why.
In any case, validating against user->email only should do the trick.
Yes, but if Gus can help us fix the regex (I have a version in my inbox already) simply, we might as well go with that.
Why it doesn't fail sometimes
Betchya it's spaces or punctuation.
Betchya it's spaces or punctuation.
Betcha you're right. Although, isn't there a space between the name and the < to the left of the email? If so, it couldn't be spaces, right? Gotta be a period or something.
The regex is supposed to be able to accept "Jon. B " as a valid e-mail address. It is not. However, it likes "ita " just fine. Which is why the problem is not in what's passed, but in how it's parsed.
ita <ita@example.com>
is a valid rfc822 address.
Jon B <jonb@somewhere.com>
is also valid.
Jon B. <jonb@somewhere.com>
is not -- unquoted periods are not allowed in the "user name" portion outside the <> pair.
"Jon B." <jonb@somewhere.com>
is valid, however.
ita, try putting quotes around "Jon B." in your test, and see if it passes. If so, then your address parser is probably just obeying rfc822.
Here's a seemingly-out-of-nowhere request (which obviously is not urgent): Is there any way to make the highlighted text under whitefont any darker? I find the blue difficult to read.
Is there any way to make the highlighted text under whitefont any darker? I find the blue difficult to read.
I think that's your browser. The code just makes it white.
Thanks for that tip, Karl. Let me try.
That's a browser thing, not a code thing, Katie. No idea if there's a browser setting you could change for that...
t /not helpful
Y'know, I added a period to my username specifically to see if it would break things.... Glad to see it's still useful.
Karl, I've changed the code to quote properly, but it still breaks the regex. Gus is on the case.
Yes, but if Gus can help us fix the regex (I have a version in my inbox already) simply, we might as well go with that.
Unless I misunderstand things, the user name doesn't actually do anything. It's just part of the value that gets returned to the server if the user accepts the default email address for the "from" part of the email. (If the user does not accept the default but types in her own "from" address, the server gets the email address without the user name.)
Fixing that is easy. In the code that sets up the form, just change
<input type="radio" name="replyto" value="<? echo "$_user->name <$_user->email>" ?>" checked>
to
<input type="radio" name="replyto" value="<? echo $_user->email ?>" checked>
eta: I can send e-mail to the admins at .net just fine
Oh, I missed that. The fixed version of email.php was there when you tried.