diff options
author | Robert Speicher <robert@gitlab.com> | 2017-10-02 17:34:08 +0000 |
---|---|---|
committer | Robert Speicher <robert@gitlab.com> | 2017-10-02 17:34:08 +0000 |
commit | 95f613837dca391a77eed9f948326b982c77d723 (patch) | |
tree | 579d270f764599115283e7e3d7dea17a3e26197c /app | |
parent | fd8c56008fdadf45bd2ed1e0c50fbea51424a176 (diff) | |
parent | bac29160302549c3c651991bf839b304a9e1c8b4 (diff) | |
download | gitlab-ce-95f613837dca391a77eed9f948326b982c77d723.tar.gz |
Merge branch 'sh-fix-import-repos' into 'master'
Fix gitlab-rake gitlab:import:repos task
Closes #37682
See merge request gitlab-org/gitlab-ce!14597
Diffstat (limited to 'app')
-rw-r--r-- | app/models/project.rb | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/app/models/project.rb b/app/models/project.rb index bb3f74c4b89..44d1190cc5b 100644 --- a/app/models/project.rb +++ b/app/models/project.rb @@ -72,6 +72,7 @@ class Project < ActiveRecord::Base attr_accessor :old_path_with_namespace attr_accessor :template_name attr_writer :pipeline_status + attr_accessor :skip_disk_validation alias_attribute :title, :name @@ -227,7 +228,7 @@ class Project < ActiveRecord::Base validates :import_url, importable_url: true, if: [:external_import?, :import_url_changed?] validates :star_count, numericality: { greater_than_or_equal_to: 0 } validate :check_limit, on: :create - validate :can_create_repository?, on: [:create, :update], if: ->(project) { !project.persisted? || project.renamed? } + validate :check_repository_path_availability, on: [:create, :update], if: ->(project) { !project.persisted? || project.renamed? } validate :avatar_type, if: ->(project) { project.avatar.present? && project.avatar_changed? } validates :avatar, file_size: { maximum: 200.kilobytes.to_i } @@ -1018,7 +1019,8 @@ class Project < ActiveRecord::Base end # Check if repository already exists on disk - def can_create_repository? + def check_repository_path_availability + return true if skip_disk_validation return false unless repository_storage_path expires_full_path_cache # we need to clear cache to validate renames correctly |