Moodle
  1. Moodle
  2. MDL-35183 import latest 3rd party libs into 2.4dev META
  3. MDL-35186

import phpmailer 5.2.1

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

            Gliffy Diagrams

              Issue Links

              has been marked as being related by

              Sub-task - Issues are sometimes broken into multiple sub-tasks. MDL-47299 Upgrade PHPMailer to 5.2.9 for Moodle 2.8 release

              • Minor - Minor loss of function where workaround is possible
              • Closed

              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

                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 Skoda added a comment -

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

                Show
                Petr Skoda 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

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

                    Dates

                    • Created:
                      Updated:
                      Resolved: