[MDL-62289] Expired users deletion process should include a call to core_privacy\manager::delete_data_for_user - Moodle Tracker

XML

Word

Printable

Details

Type: Bug
Status: Closed
Priority: Critical
Resolution: Fixed
Affects Version/s: 3.5
Fix Version/s: 3.5
Component/s: Privacy
Labels:
- ci

Affected Branches:
MOODLE_35_STABLE
Fixed Branches:
MOODLE_35_STABLE
Epic Link:
GDPR Subject Access Request System
Pull from Repository:
git://github.com/abgreeve/moodle.git
Pull Master Branch:
wip-~~MDL-62289~~-master
Pull Master Diff URL:
https://github.com/abgreeve/moodle/compare/446d8483a3...wip-MDL-62289-master
Testing Instructions:
- This is covered by unit tests.

Description

Reading some feedback after a reply I've noticed that we are calling delete_data_for_all_users_in_context instead of delete_data_for_user. It should be ok to call delete_data_for_all_users_in_context providing a context_user, but we should also call delete_data_for_user.

Attachments

Activity

People

Assignee:: Adrian Greeve

Reporter:: David Monllaó

Peer reviewer:: Jake Dallimore

Integrator:: Jun Pataleta

Tester:: CiBoT

Participants:: Adrian Greeve, CiBoT, David Monllaó, Jake Dallimore, Jun Pataleta

Component watchers:: 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:: 3 Start watching this issue

Dates

Created:: 02/May/18 5:02 AM

Updated:: 18/May/18 3:58 PM

Resolved:: 13/May/18 5:57 AM

Fix Release Date:: 17/May/18