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

tool_privacy: export request email comes from user without a name

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 3.3.5, 3.4.2, 3.5
    • 3.5
    • Privacy
    • MOODLE_33_STABLE, MOODLE_34_STABLE, MOODLE_35_STABLE
    • MOODLE_35_STABLE
    • wip-MDL-62445-master
    • Hide
      1. You need a test site with working emails
      2. Create DPO role and assign to somebody, map DPO role in privacy settings
      3. Login as any user and create an export request
      4. Run cron
      5. Make sure cron has no notices about maildisplay
      6. Check email sent to DPO, make sure it comes from the user (there is a user name in the "from" field)
      Show
      You need a test site with working emails Create DPO role and assign to somebody, map DPO role in privacy settings Login as any user and create an export request Run cron Make sure cron has no notices about maildisplay Check email sent to DPO, make sure it comes from the user (there is a user name in the "from" field)

    Description

      I noticed the error in the cron:

      Notice: Undefined property: stdClass::$maildisplay in /home/marina/repositories/mdk_master/moodle/lib/moodlelib.php on line 6091
       
      Notice: Undefined property: stdClass::$maildisplay in /home/marina/repositories/mdk_master/moodle/lib/moodlelib.php on line 6092
      

      Also the emails to DPO come from user without a name.
      This happens because we pass exported user object to send_message() instead of a db record from 'users' table (or userid)

      Attachments

        Issue Links

          Activity

            People

              marina Marina Glancy
              marina Marina Glancy
              Simey Lameze Simey Lameze
              David Monllaó David Monllaó
              Carlos Escobedo Carlos Escobedo
              Andrew Lyons, Adrian Greeve, David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                17/May/18