Commit b0501c34 authored by Valery Sizov's avatar Valery Sizov

BB importer: address review comment

parent 3b4e81ee
...@@ -20,7 +20,7 @@ It takes just a few steps to import your existing Bitbucket projects to GitLab. ...@@ -20,7 +20,7 @@ It takes just a few steps to import your existing Bitbucket projects to GitLab.
![Import projects](bitbucket_importer/bitbucket_import_select_project.png) ![Import projects](bitbucket_importer/bitbucket_import_select_project.png)
A new GitLab project will be created with your imported data. Keep in mind that if you want to Bitbucket users A new GitLab project will be created with your imported data. Keep in mind that if you want Bitbucket users
to be linked to GitLab user you have to have all of them in GitLab in advance. They will be matched by their BitBucket username. to be linked to GitLab user you have to have all of them in GitLab in advance. They will be matched by their BitBucket username.
### Note ### Note
......
...@@ -28,6 +28,7 @@ module Gitlab ...@@ -28,6 +28,7 @@ module Gitlab
def handle_errors def handle_errors
return unless errors.any? return unless errors.any?
project.update_column(:import_error, { project.update_column(:import_error, {
message: 'The remote data could not be fully imported.', message: 'The remote data could not be fully imported.',
errors: errors errors: errors
...@@ -35,15 +36,12 @@ module Gitlab ...@@ -35,15 +36,12 @@ module Gitlab
end end
def gitlab_user_id(project, username) def gitlab_user_id(project, username)
if username user = find_user(username)
user = find_user(username) user.try(:id) || project.creator_id
(user && user.id) || project.creator_id
else
project.creator_id
end
end end
def find_user(username) def find_user(username)
return nil unless username
User.joins(:identities).find_by("identities.extern_uid = ? AND identities.provider = 'bitbucket'", username) User.joins(:identities).find_by("identities.extern_uid = ? AND identities.provider = 'bitbucket'", username)
end end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment