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.

Sign up for GitHub

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

Jump to bottom

Jkmarx/data set owner transfer #2794

Merged
merged 16 commits into from
Jun 6, 2018
Merged

Jkmarx/data set owner transfer #2794

Show file tree
Hide file tree
Changes from 8 commits
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
1c50cdd
Add tranfer_ownership method to model.
jkmarx May 24, 2018
fd73d7c
Remove share for non-unioned groups.
jkmarx May 25, 2018
d322818
Add email template to notify users of transfer.
jkmarx May 29, 2018
672dcf2
Update text for clarity.
jkmarx May 30, 2018
0062fad
Move email to helper method.
jkmarx May 30, 2018
61f087d
Merge branch 'develop' of https://github.com/refinery-platform/refine…
jkmarx May 30, 2018
48e86d6
Add unit tests for view and email helper method.
jkmarx May 30, 2018
e8bfa00
Add unit tests for update_group_perms.
jkmarx May 30, 2018
c9cd15f
Jkmarx/data set owner transfer ui (#2796)
jkmarx May 31, 2018
a7f542d
Fix incorrect order.
jkmarx Jun 1, 2018
a1c79f6
Minimize verbose name.
jkmarx Jun 1, 2018
ff281c6
Refactor to remove other if.
jkmarx Jun 1, 2018
c83665b
Add exception handling.
jkmarx Jun 1, 2018
1183d58
Refactor for DRY.
jkmarx Jun 1, 2018
3b9dccf
Add unit test to check rollback.
jkmarx Jun 2, 2018
cb5bc40
Update response status for error.
jkmarx Jun 6, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
9 changes: 9 additions & 0 deletions refinery/core/models.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -316,6 +316,15 @@ def set_owner(self, user):
if self._meta.verbose_name == 'dataset':
assign_perm("read_meta_%s" % self._meta.verbose_name, user, self)

def transfer_ownership(self, user, new_owner):
remove_perm("add_%s" % self._meta.verbose_name, user, self)
remove_perm("read_%s" % self._meta.verbose_name, user, self)
remove_perm("delete_%s" % self._meta.verbose_name, user, self)
remove_perm("change_%s" % self._meta.verbose_name, user, self)
if self._meta.verbose_name == 'dataset':
remove_perm("read_meta_%s" % self._meta.verbose_name, user, self)
self.set_owner(new_owner)

def get_owner(self):
# ownership is determined by "add" permission
user_permissions = get_users_with_perms(
Expand Down