Administrator emails not being received 2

kitchin
(@kitchin)

5 years, 11 months ago
The bug in Ninja on BlueHost, etc.., can be fixed like this:

Version: 2.3.8
File: ninja-forms/includes/display/processing/email-admin.php
Line: 26

Change
```
if ( $email_from_name AND $email_reply ) {
	$email_reply = $email_from_name . ' <' . $email_reply . '>';
}
```
to
```
// commented out, breaks sending on BlueHost
// if ( $email_from_name AND $email_reply ) {
//	$email_reply = $email_from_name . ' <' . $email_reply . '>';
// }
```
Email will still have the reply-to, it just will be in a simpler format.

For more about this bug and settings you may need, see: http://wordpress.org/support/topic/administrator-emails-not-being-received

Developers, as long as you’re fixing that, it would be nice to fix this to stop a WP_DEBUG notice about an undefined variable:
File: ninja-forms/includes/display/fields/display-fields.php
Line: 316
Add:
```
$address_class = '';
```
And the unrelated fix for the_content filter I put here:
http://wordpress.org/support/topic/duplicate-forms-shown-on-page-bug-with-slideshows

http://wordpress.org/plugins/ninja-forms/

Viewing 6 replies - 1 through 6 (of 6 total)

kitchin
(@kitchin)

5 years, 11 months ago

I have some more debugging info about what the WP phpmailer class sees at Bluehost if you want it. But the simplest thing is just to temporarily insert wp_die(“sent=$sent”) at the end of email-admin.php when in WP_DEBUG.
kitchin
(@kitchin)

5 years, 11 months ago
Looking at a similar bug in JetPack, http://wordpress.org/support/topic/contact-form-possible-bug , Developers may need an additional fix for “From:”.

It refers to a BlueHost KB article https://my.bluehost.com/cgi/help/206 which states
```
Examples of headers that should work would be:
From: user@domain.com will work
        From: "user" <user@domain.com>
Examples of headers that will NOT work:
From: "user@domain.com"
        From: user @ domain.com
        From: user@domain.com <user@domain.com>
Our servers will not accept the name for the email address and the email address to be the same. It will not accept a double declaration of the email address.
```
This means Ninja should also be more careful at Line 44 in email-admin.php $email_from = $email_from_name.' <'.$email_from.'>'; and Line 56 of email-user.php.

I’d suggest:
1. Reply-to: only use a plain email address, like user@example.com.
2. From: only use Name <user@example.com> when Name is *not* an email address. Otherwise use a plain email address.
Plugin Author James Laws
(@jameslaws)

5 years, 10 months ago

As we’ve stated in this sticky thread, we don’t offer support on these forums but of you would like support from the developers you can follow the support link it provides and we’d be happy to assist you.

If you are happy waiting here to see if someone else can assist you, you are welcome to do that as well.

kitchin
(@kitchin)

5 years, 10 months ago

That’s fine, but maybe these posts will help other users.

Plugin Author James Laws
(@jameslaws)

5 years, 10 months ago

@kitchin Absolutely! Nothings being deleted. We just want to make sure people reading these threads are aware that we fully support Ninja Forms on our own site if they aren’t getting answers here.

We very much appreciate when users can post here and the community is able to help them.

Ian Dunn
(@iandunn)

5 years, 9 months ago

This appears to be a Core bug, #21659.

http://wordpress.org/support/topic/contact-form-possible-bug
https://github.com/PHPMailer/PHPMailer/issues/193

Viewing 6 replies - 1 through 6 (of 6 total)

The topic ‘Administrator emails not being received 2’ is closed to new replies.

WordPress.org

Administrator emails not being received 2

Topic Tags

Views