Uploaded image for project: '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
          • 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
          • 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

                Attachments

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