diff options
| author | Douglas Barbosa Alexandre <dbalexandre@gmail.com> | 2017-04-24 20:45:51 -0300 |
|---|---|---|
| committer | Douglas Barbosa Alexandre <dbalexandre@gmail.com> | 2017-04-24 20:45:51 -0300 |
| commit | 22a33d8256101b0a5f348ae93623d4b189a96d47 (patch) | |
| tree | 247f40f4853914a710ac6eba276438cb57ebb250 | |
| parent | 5c72ba0ff1be82f31c292765e8d6a76277327979 (diff) | |
| download | gitlab-ce-22a33d8256101b0a5f348ae93623d4b189a96d47.tar.gz | |
Avoid unnecessary use of Arel to find users by external uid
| -rw-r--r-- | lib/github/import.rb | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/lib/github/import.rb b/lib/github/import.rb index 56b0b7a3d4f..ba30ea0e2cd 100644 --- a/lib/github/import.rb +++ b/lib/github/import.rb @@ -355,11 +355,9 @@ module Github def user_id_by_external_uid(id) return nil unless id - identities = ::Identity.arel_table - ::User.select(:id) .joins(:identities) - .where(identities[:provider].eq(:github).and(identities[:extern_uid].eq(id))) + .merge(::Identity.where(provider: :github, extern_uid: id)) .first&.id end |