Ticket #4318 (closed defect: duplicate)

Opened 1 year ago

Last modified 1 year ago

wp-mail.php not functioning

Reported by: rob1n Assigned to: anonymous
Priority: normal Milestone:
Component: Administration Version: 2.2.2
Severity: normal Keywords:
Cc: cfactor

Description

As mentioned by dmd on IRC -- after upgrading to 2.2, wp-mail is broken (only title, no body), but substituting the old wp-mail.php works fine. Diff attached.

I'm thinking the Content-transfer-encoding part.

Attachments

wp-mail.diff (1.5 kB) - added by rob1n on 05/23/07 01:30:41.
PN_mailfix.php (2.7 kB) - added by morty on 07/08/07 23:51:11.
My Plugin for fixing the wp-mail problem

Change History

05/23/07 01:30:41 changed by rob1n

  • attachment wp-mail.diff added.

05/23/07 02:00:14 changed by rob1n

  • milestone changed from 2.2.1 to 2.3.

Okay, he narrowed it down to something in class-pop3.php.

06/15/07 08:58:45 changed by cfactor

As far as I can tell, this cannot be fixed by modifying wp-mail. I attached "what-works-for-me" patch of class-pop3.php patch to ticket #4337. Please look over it and see if it does things properly.

(follow-up: ↓ 6 ) 06/15/07 09:10:38 changed by cfactor

  • cc set to cfactor.

(in reply to: ↑ description ; follow-up: ↓ 5 ) 07/08/07 23:50:39 changed by morty

It's the encoding. I already wrote a patch for my plugin.

07/08/07 23:51:11 changed by morty

  • attachment PN_mailfix.php added.

My Plugin for fixing the wp-mail problem

(in reply to: ↑ 4 ) 07/09/07 19:22:14 changed by cfactor

Replying to morty:

It's the encoding. I already wrote a patch for my plugin.

morty, this ticket is for when posting via e-mail. I don't think it has anything to do with wordpress *sending* emails.

(in reply to: ↑ 3 ; follow-up: ↓ 7 ) 07/15/07 16:40:55 changed by lexhair

Replying to cfactor:Please consider adding a check box switch to the Blog by Email admin panel to dump emails that come from addresses not in the user database. I believe the wp-mail.php code must be user modified for this functionality. Should this be a separate enhancement request?

(in reply to: ↑ 6 ; follow-up: ↓ 8 ) 07/15/07 16:57:09 changed by Nazgul

Replying to lexhair:

Should this be a separate enhancement request?

Yes, please create a new enhancement request for that. One issue per ticket, to avoid confusion.

(in reply to: ↑ 7 ) 07/15/07 18:49:00 changed by lexhair

Replying to Nazgul:

Replying to lexhair:

Should this be a separate enhancement request?

Yes, please create a new enhancement request for that. One issue per ticket, to avoid confusion.

Done. Thank you for the quick response.

(follow-up: ↓ 10 ) 08/21/07 18:38:05 changed by dmd

  • milestone changed from 2.3 (trunk) to 2.2.3.

There's absolutely no reason why this patch can't be applied for 2.2.3.

1) This is core functionality which used to work, and got broken.

2) The patch changes only one file, wp-mail.php - a file which currently has NO functionality due to this defect.

3) wp-mail.php is used by no part of wordpress other than this functionality. The only possible effect of applying this patch is a change to this functionality - either it will start working again, or it won't. It can't break anything else.

4) It does, in fact, cause wp-mail to start working again.

(in reply to: ↑ 9 ) 08/21/07 19:44:18 changed by foolswisdom

  • version set to 2.2.2.

Replying to dmd:

There's absolutely no reason why this patch can't be applied for 2.2.3.

dmd, the milestone process isn't formalized well. Often things get committed to trunk, tested, and then committed to the maintenance branch.

08/29/07 18:01:17 changed by foolswisdom

  • milestone changed from 2.2.3 to 2.3.

(follow-up: ↓ 13 ) 08/31/07 19:31:20 changed by shadowdad

New to WP. I would be happy to provide additional testing on this patch, havejust been down the rat-hole of trying to get post by mail to work. I have found another bug (footer email activation doesn't seem to work with Firefox) which i can test with the this fix and see if it is an independent bug (likely) but might as well check before entering. please provide pointer to fixed, i would like to be moving code forward, currently am using the 2.1.3 class-pop3.php and wp-mail.php to get around problem.

(in reply to: ↑ 12 ) 08/31/07 20:13:45 changed by cfactor

Replying to shadowdad:

please provide pointer to fixed

I still believe this cannot be fixed by patching just wp-mail.php, as class-pop3.php will pass it blank body content. Please look at ticket 4337. Be sure to use only one or the other. Nazgul's 4337.diff is more elegant than my class-pop3.php.patch, but Nazgul's patch will add an extra blank line to the end of the post. This may or may not concern you. I have not had the time to fix his patch to take care of the extra blank line.

09/07/07 12:08:59 changed by Nazgul

  • status changed from new to closed.
  • resolution set to duplicate.
  • milestone deleted.

Duplicate of #4337.