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

Prevent (once) live eprint items from being removed #145

Closed
drn05r opened this issue Apr 15, 2021 · 3 comments
Closed

Prevent (once) live eprint items from being removed #145

drn05r opened this issue Apr 15, 2021 · 3 comments
Assignees
Milestone

Comments

@drn05r
Copy link
Collaborator

@drn05r drn05r commented Apr 15, 2021

Once an item is live it should not be removable (only retireable). If you remove an item that have been live even if it has previously been retired it can effect third-party application trying to sync metadata user OAI-PMH. It may also have impact on other aspects of EPrints, especially involving export.

It maybe should still be possible to remove (once) live items but this should be very difficult. E.g. require a permission that even admin users do not have by default. Also when an admin with this advanced permission tries to remove an item (once) live they should receive a much more strict warning about the consequences of removing such an item.

@drn05r drn05r added this to the 3.4.4 milestone Apr 15, 2021
@drn05r drn05r self-assigned this Apr 15, 2021
@jesusbagpuss
Copy link
Contributor

@jesusbagpuss jesusbagpuss commented Apr 19, 2021

I think an additional priv that works alongside existing ones might be best, as it could be re-used in:

  • EPrints::Plugin::Screen::EPrint::Remove
  • EPrints::Plugin::Screen::EPrint::RemoveWithEmail

The EPrints::Plugin::Screen::EPrint::RequestRemoval screen could also be updated to indicate that an item may be removed or moved to deletion.

Loading

@drn05r
Copy link
Collaborator Author

@drn05r drn05r commented Apr 25, 2021

RequestRemoval already includes in its intro "Please note that we strongly discourage removal of items from the repository". So I don't think there is a need to add anything further to this page.

Loading

drn05r added a commit that referenced this issue Apr 25, 2021
@drn05r
Copy link
Collaborator Author

@drn05r drn05r commented Apr 25, 2021

I think my latest commit meets the requirements discussed. I have added info about the new permission being required to explain why "Destroy Item" will not be an option for some items on the review page, if they have previously been live. However, I cannot think of anywhere appropriate to make repository admins more generally aware of this and how they might re-enable such a permission. I can already imagine frantic emails on the tech list saying: "I cannot remove item X that has confidential information in it."

Loading

@drn05r drn05r removed this from the 3.4.4 milestone Apr 27, 2021
@drn05r drn05r added this to the 3.4.3 milestone Apr 27, 2021
@drn05r drn05r closed this Apr 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants