Details

    • Type: Sub-task Sub-task
    • Status: 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
    • Rank:
      43823

      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

          Activity

          Hide
          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
          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
          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
          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
          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
          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

            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: