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

            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

              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

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: