Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-66169

Undefined property notice for $debugsmtp on outgoing mail config test page

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.7, 3.8
    • Fix Version/s: 3.7.3
    • Component/s: Administration
    • Labels:
    • Testing Instructions:
      Hide

      With debugging on:

      1. Ensure that $CFG->debugsmtp is not set in your config.php
      2. Configure outbound mail (mailhog is fine)
      3. Go to the "Test outgoing mail configuration" page
      4. Send a test email to any address
      5. Verify you don't see a php notice on the page.
      Show
      With debugging on: Ensure that $CFG->debugsmtp is not set in your config.php Configure outbound mail (mailhog is fine) Go to the "Test outgoing mail configuration" page Send a test email to any address Verify you don't see a php notice on the page.
    • Affected Branches:
      MOODLE_37_STABLE, MOODLE_38_STABLE
    • Fixed Branches:
      MOODLE_37_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-66169-master

      Description

      With debugging on:

      • Configure outbound mail (mailhog is fine)
      • Go to the "Test outgoing mail configuration" page
      • Send a test email to any address
      • See the php notice:

        Undefined property: stdClass::$debugsmtp
        

      I suspect this is caused by the master-only change made in MDL-60347 (making that CFG var optional)

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              jaked Jake Dallimore
              Reporter:
              jaked Jake Dallimore
              Peer reviewer:
              Andrew Nicols
              Integrator:
              Eloy Lafuente (stronk7)
              Tester:
              Janelle Barcega
              Participants:
              Component watchers:
              Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                11/Nov/19

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 45 minutes
                  45m