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

API: Cleaner messaging when attempting to delete a user that has "history" #2825

Closed
pdurbin opened this issue Dec 14, 2015 · 6 comments
Closed

Comments

@pdurbin
Copy link
Member

pdurbin commented Dec 14, 2015

Related to #2824 about deleting users, it's impossible to delete a user if that user had published the root dataverse:

org.postgresql.util.PSQLException: ERROR: update or delete on table "authenticateduser" violates foreign key constraint "fk_dvobject_releaseuser_id" on table "dvobject"
Detail: Key (id)=(8) is still referenced from table "dvobject".

I'm not sure what the right fix for this should be. Would we change the "releaseuser" for the root dataverse to some other existing user?

@pdurbin
Copy link
Member Author

pdurbin commented Jan 6, 2016

One is unable to delete a user with a published dataset either: #2824 (comment)

@scolapasta scolapasta changed the title API: unable to delete the user who published the root dataverse API: Cleaner messaging when attempting to delete a user that has "history" Jan 8, 2016
@scolapasta
Copy link
Contributor

This is the desired behavior, as we want to track ho did what.

However, we should make the API call cleaner, I think, by checking for these things first and providing a friendlier message.

@bencomp
Copy link
Contributor

bencomp commented Jan 12, 2016

In DVN 3.6 we don't delete users, we deactivate their accounts when necessary. It doesn't seem to be possible though: #2419

@pdurbin
Copy link
Member Author

pdurbin commented Jan 12, 2016

In DVN 3.6 we don't delete users, we deactivate their accounts when necessary. It doesn't seem to be possible though: #2419

Yep. We haven't forgotten about that issue ( @scolapasta and I discussed it last week at #2824 (comment) and below) but thanks for linking it here, @bencomp

@pdurbin
Copy link
Member Author

pdurbin commented Apr 1, 2016

This issue may be a duplicate of #1929.

@kcondon
Copy link
Contributor

kcondon commented Apr 14, 2016

Combined with #1929

@kcondon kcondon closed this as completed Apr 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants