diff options
author | Nick Thomas <nick@gitlab.com> | 2019-02-05 19:53:04 +0300 |
---|---|---|
committer | Nick Thomas <nick@gitlab.com> | 2019-02-05 19:53:04 +0300 |
commit | 22caeb58a49a7ed2ccef7e5191e5a8c3f9ff7d10 (patch) | |
tree | 160d91d05ba01da1270ad39ad89497117b58ed53 | |
parent | 699806ba51851afc78537b2743947b1bfde430c5 (diff) | |
parent | 4e7dd94d785c201ddb5a3247293fadf174158f9d (diff) |
Merge branch 'ce-reduce-diff-in-groups-update_service' into 'master'
Reduce diff with EE in Groups::UpdateService
See merge request gitlab-org/gitlab-ce!24912
-rw-r--r-- | app/services/groups/update_service.rb | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/app/services/groups/update_service.rb b/app/services/groups/update_service.rb index de78a3f7b27..9ff1da270e2 100644 --- a/app/services/groups/update_service.rb +++ b/app/services/groups/update_service.rb @@ -11,6 +11,8 @@ module Groups return false unless valid_share_with_group_lock_change? + before_assignment_hook(group, params) + group.assign_attributes(params) begin @@ -28,6 +30,10 @@ module Groups private + def before_assignment_hook(group, params) + # overriden in EE + end + def after_update if group.previous_changes.include?(:visibility_level) && group.private? # don't enqueue immediately to prevent todos removal in case of a mistake |