summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorArsenev Vladislav <nobodywithbody@gmail.com>2016-12-18 21:56:13 +0000
committerArsenev Vladislav <nobodywithbody@gmail.com>2016-12-18 23:26:29 +0000
commit3cc334eae5abf65b0ecb4159e4e9712dab81c33c (patch)
treee99bf0fb80e83e986ff8fd90d93e7f94c37a5d11
parent546fa165ff728bc2d25ed9b55b95dd1d48139d4a (diff)
downloadgitlab-ce-3cc334eae5abf65b0ecb4159e4e9712dab81c33c.tar.gz
remove build_user from model User
-rw-r--r--app/models/user.rb4
-rw-r--r--changelogs/unreleased/25678-remove-user-build.yml4
-rw-r--r--lib/api/users.rb2
3 files changed, 5 insertions, 5 deletions
diff --git a/app/models/user.rb b/app/models/user.rb
index 3a17c98eff6..d771eaff472 100644
--- a/app/models/user.rb
+++ b/app/models/user.rb
@@ -311,10 +311,6 @@ class User < ActiveRecord::Base
find_by(id: Key.unscoped.select(:user_id).where(id: key_id))
end
- def build_user(attrs = {})
- User.new(attrs)
- end
-
def reference_prefix
'@'
end
diff --git a/changelogs/unreleased/25678-remove-user-build.yml b/changelogs/unreleased/25678-remove-user-build.yml
new file mode 100644
index 00000000000..873e637d670
--- /dev/null
+++ b/changelogs/unreleased/25678-remove-user-build.yml
@@ -0,0 +1,4 @@
+---
+title: remove build_user
+merge_request: 8162
+author: Arsenev Vladislav
diff --git a/lib/api/users.rb b/lib/api/users.rb
index 0842c3874c5..4c22287b5c6 100644
--- a/lib/api/users.rb
+++ b/lib/api/users.rb
@@ -94,7 +94,7 @@ module API
identity_attrs = params.slice(:provider, :extern_uid)
confirm = params.delete(:confirm)
- user = User.build_user(declared_params(include_missing: false))
+ user = User.new(declared_params(include_missing: false))
user.skip_confirmation! unless confirm
if identity_attrs.any?