Skip to content

Commit 2e2b9bb

Browse files
committed
Run simpleSaveChangeHistory in transaction
- now it sees user_id, too - if updateAuth0UserFromDb fails, we don't end up with an inconsistent DB state
1 parent 2941495 commit 2e2b9bb

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

library/ajax/deleteprofile.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,9 @@
77
// Pattern checks for .deleted.<digits> after the @ symbol (e.g., user@domain.com.deleted.123)
88
db_query('UPDATE cms_users SET deleted = NOW(), email = CONCAT(email,".deleted.",id) WHERE id = :id AND (NOT deleted OR deleted IS NULL) AND email NOT REGEXP "@.*\.deleted\.[0-9]+$"', ['id' => $user_id]);
99
updateAuth0UserFromDb($user_id);
10+
simpleSaveChangeHistory('cms_users', $user_id, 'Record deleted without undelete');
1011
});
1112

12-
simpleSaveChangeHistory('cms_users', $user_id, 'Record deleted without undelete');
1313
// when a user deactive its account we need to ensure that user logged out immediately and then redirected to Auth0 login page
1414
global $settings;
1515
logout();

0 commit comments

Comments
 (0)