Moodle

import phpmailer 5.2.1

Details

  • Type: Sub-task Sub-task
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 2.4
  • Fix Version/s: 2.4
  • Component/s: Libraries
  • Labels:
  • Testing Instructions:
    Hide

    1/ set up email sending on your test server
    2/ do some actions that results in sending of emails (forgotten password, forum post, etc.) Try with sender/destination/subject being looong (>100 chars) and containing some UTF8 chars (accents, €, whatever).
    3/ verify received emails and their contents.

    Show
    1/ set up email sending on your test server 2/ do some actions that results in sending of emails (forgotten password, forum post, etc.) Try with sender/destination/subject being looong (>100 chars) and containing some UTF8 chars (accents, €, whatever). 3/ verify received emails and their contents.
  • Affected Branches:
    MOODLE_24_STABLE
  • Fixed Branches:
    MOODLE_24_STABLE
  • Pull from Repository:
  • Pull Master Branch:
    w36_MDL-35186_m24_phpmailer
  • Pull Master Diff URL:

Description

http://code.google.com/a/apache-extras.org/p/phpmailer/downloads/detail?name=PHPMailer_5.2.1.tgz

Version 5.2.1 (January 16, 2012)

  • Closed several bugs
  • Performance improvements
  • MsgHTML() now returns the message as required.
  • New method: GetSentMIMEMessage() (returns full copy of sent message)

Version 5.2 (July 19, 2011)

  • protected MIME body and header
  • better DKIM DNS Resource Record support
  • better aly handling
  • htmlfilter class added to extras
  • moved to Apache Extras

Issue Links

has been marked as being related by

Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-21835 get_mailer: no mail sent to CCed people when using IsMail i.e. no smpthosts defined

  • Minor - Minor loss of function where workaround is possible
  • Closed - The issue is considered finished, the resolution is correct. Issues which are closed should not be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Eloy Lafuente (stronk7) added a comment -

Uhm... perhaps we could, some day, verify if their EncodeHeader() is already ok to take out our one? Uhm... seems to rely on "mbstring" installed...

Show
Eloy Lafuente (stronk7) added a comment - Uhm... perhaps we could, some day, verify if their EncodeHeader() is already ok to take out our one? Uhm... seems to rely on "mbstring" installed...
Hide
Permalink
Eloy Lafuente (stronk7) added a comment -

Integrated (master only), thanks!

PS: I've tidied a bit the testing instructions to control specifically the encoding of headers and also marked this as candidate for QA testing.

Show
Eloy Lafuente (stronk7) added a comment - Integrated (master only), thanks! PS: I've tidied a bit the testing instructions to control specifically the encoding of headers and also marked this as candidate for QA testing.
Hide
Permalink
Petr Škoda added a comment -

thanks, I remember the header was a bit messy, let's try it after 2.4 release...

Show
Petr Škoda added a comment - thanks, I remember the header was a bit messy, let's try it after 2.4 release...
Hide
Permalink
Frédéric Massart added a comment -

Worked as described. Passing \o/

Show
Frédéric Massart added a comment - Worked as described. Passing \o/
Hide
Permalink
Eloy Lafuente (stronk7) added a comment -

Many thanks for the hard work.

These changes have been spread upstream and are already available in the git and cvs repositories.

Ciao

Show
Eloy Lafuente (stronk7) added a comment - Many thanks for the hard work. These changes have been spread upstream and are already available in the git and cvs repositories. Ciao
Hide
Permalink
Helen Foster added a comment -

Removing qa_test_required label as MDLQA-937 amended to include instructions about test forum post "Make the subject of the forum post very long i.e. more than 100 characters and include some UTF8 characters e.g. accents, €."

If further QA tests are required, please comment and re-add the label.

Show
Helen Foster added a comment - Removing qa_test_required label as MDLQA-937 amended to include instructions about test forum post "Make the subject of the forum post very long i.e. more than 100 characters and include some UTF8 characters e.g. accents, €." If further QA tests are required, please comment and re-add the label.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: