MDL-62564 tool_dataprivacy: Do not delete deleted user
This also ensures that we do not call delete_user on an already deleted user.
This commit is contained in:
@@ -93,6 +93,7 @@ class process_data_request_task extends adhoc_task {
|
||||
|
||||
mtrace('Processing request...');
|
||||
$completestatus = api::DATAREQUEST_STATUS_COMPLETE;
|
||||
$deleteuser = false;
|
||||
|
||||
if ($request->type == api::DATAREQUEST_TYPE_EXPORT) {
|
||||
// Get the user context.
|
||||
@@ -127,6 +128,7 @@ class process_data_request_task extends adhoc_task {
|
||||
|
||||
$manager->delete_data_for_user($approvedclcollection);
|
||||
$completestatus = api::DATAREQUEST_STATUS_DELETED;
|
||||
$deleteuser = !$foruser->deleted;
|
||||
}
|
||||
|
||||
// When the preparation of the metadata finishes, update the request status to awaiting approval.
|
||||
@@ -259,7 +261,7 @@ class process_data_request_task extends adhoc_task {
|
||||
}
|
||||
}
|
||||
|
||||
if ($request->type == api::DATAREQUEST_TYPE_DELETE) {
|
||||
if ($deleteuser) {
|
||||
// Delete the user.
|
||||
delete_user($foruser);
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user