Make WordPress Core

Opened 16 years ago

Closed 14 years ago

#2460 closed defect (bug) (fixed)

wp-mail.php: quoted printable encoding not decoding

Reported by: Staro Owned by:
Milestone: 2.2 Priority: normal
Severity: normal Version: 2.0.1
Component: General Keywords: wp-mail.php quoted printable has-patch
Focuses: Cc:


When sending an email in this format:

Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable


the message body will not be decoded. It should use the PHP function quoted_printable_decode(),
since many mobile phones can only send emails in this encoding.

Attachments (2)

diff (814 bytes) - added by Staro 16 years ago.
2460.diff (1.8 KB) - added by Nazgul 15 years ago.

Download all attachments as: .zip

Change History (11)

#1 @Staro
16 years ago

  • Component changed from Administration to General

#2 @Staro
16 years ago

  • Milestone 2.0.1 deleted
  • Version changed from 1.2 to 2.0.1

16 years ago

#3 @Staro
16 years ago

I think this diff in attachment could fix it. (but don't know at which position exactly the conversion should take place)

15 years ago

#4 @Nazgul
15 years ago

  • Keywords has-patch added
  • Milestone set to 2.1

Updated the already present patch to the current version of trunk.

#5 @matt
15 years ago

  • Milestone changed from 2.1 to 2.2


#6 @rob1n
15 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [5188]) Add support to wp-mail.php for quoted-printable Content-Transfer-Encoding. Props Nazgul. fixes #2460

#7 @welcomb
14 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Note that function stripos is only available in PHP 5. You might want to change the condition on line 121 to strpos(strtolower($content_transfer_encoding), "quoted-printable") to ensure compatibility with providers still using PHP 4.

Also I think MIME headers are case insensitive too so the explode on line 116 might not always work.

And I'm still having the problem I posted in #3717.

#8 @rob1n
14 years ago


stripos() was added in for compatibility already.

#9 @welcomb
14 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

oops. I must have missed out updating that file. sorry.

Note: See TracTickets for help on using tickets.