Details

    • Type: Sub-task
    • Status: Closed
    • Priority: 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

      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

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              stronk7 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
              stronk7 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
              stronk7 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
              stronk7 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
              skodak Petr Skoda added a comment -

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

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

              Worked as described. Passing \o/

              Show
              fred Frédéric Massart added a comment - Worked as described. Passing \o/
              Hide
              stronk7 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
              stronk7 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
              tsala 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
              tsala 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:
                    Fix Release Date:
                    3/Dec/12