Welcome to mirror list, hosted at ThFree Co, Russian Federation.

gitlab.com/gitlab-org/gitlab-foss.git - Unnamed repository; edit this file 'description' to name the repository.
summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
Diffstat (limited to 'app/services/groups/update_service.rb')
-rw-r--r--app/services/groups/update_service.rb16
1 files changed, 16 insertions, 0 deletions
diff --git a/app/services/groups/update_service.rb b/app/services/groups/update_service.rb
index 948540619ae..81393681dc0 100644
--- a/app/services/groups/update_service.rb
+++ b/app/services/groups/update_service.rb
@@ -17,6 +17,8 @@ module Groups
return false unless valid_share_with_group_lock_change?
+ return false unless valid_path_change_with_npm_packages?
+
before_assignment_hook(group, params)
group.assign_attributes(params)
@@ -36,6 +38,20 @@ module Groups
private
+ def valid_path_change_with_npm_packages?
+ return true unless group.packages_feature_enabled?
+ return true if params[:path].blank?
+ return true if !group.has_parent? && group.path == params[:path]
+
+ npm_packages = ::Packages::GroupPackagesFinder.new(current_user, group, package_type: :npm).execute
+ if npm_packages.exists?
+ group.errors.add(:path, s_('GroupSettings|cannot change when group contains projects with NPM packages'))
+ return
+ end
+
+ true
+ end
+
def before_assignment_hook(group, params)
# overridden in EE
end