Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[6.0] soft delete users #45121

Open
wants to merge 2 commits into
base: 6.0-dev
Choose a base branch
from
Open

Conversation

reem-atalah
Copy link

Pull Request for Issue #45026.

Summary of Changes

Update the Delete button in the Actions dropdown list on the Users page when selecting a user and clicking on it, the user is soft deleted.

Testing Instructions

Go to Users -> Create New User
(
Go to Articles -> Create New Article
Go to the Publishing tab -> Set the Created By to the new user
Go to Users -> Select the new user -> Click on Actions dropdown list -> Click on Delete

or

Login with the new user
Go to Articles -> Create New Article
Logout and login again with an administrator user
Go to Users -> Select the new user -> Click on Actions dropdown list -> Click on Delete
)

Go to the article where the Create By is the deleted user, edit and save it, it will be saved normally.

Actual result BEFORE applying this Pull Request

Deleting a user from the system will mean you have 'orphan' articles, categories, and other items within Joomla, so you can no longer edit those records, unless the create by is changed.

Expected result AFTER applying this Pull Request

Deleting a user from the system will soft delete the user where the user won't appear in the users' list, and we can't log in with that user. However, we can edit content created by this user.

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants