Commit f3d52f82 authored by Robert Speicher's avatar Robert Speicher

Merge branch '17628-remove-unused-methods' into 'master'

Remove User#tm_in_authorized_projects and User#tm_of

These methods seems to be unused.

Closes #17628.
Signed-off-by: default avatarRémy Coutable <remy@rymai.me>

See merge request !4198
parents 57fde5a8 b340ca35
...@@ -397,11 +397,6 @@ class User < ActiveRecord::Base ...@@ -397,11 +397,6 @@ class User < ActiveRecord::Base
owned_groups.select(:id), namespace.id).joins(:namespace) owned_groups.select(:id), namespace.id).joins(:namespace)
end end
# Team membership in authorized projects
def tm_in_authorized_projects
ProjectMember.where(source_id: authorized_projects.map(&:id), user_id: self.id)
end
def is_admin? def is_admin?
admin admin
end end
...@@ -491,10 +486,6 @@ class User < ActiveRecord::Base ...@@ -491,10 +486,6 @@ class User < ActiveRecord::Base
"#{name} (#{username})" "#{name} (#{username})"
end end
def tm_of(project)
project.project_member_by_id(self.id)
end
def already_forked?(project) def already_forked?(project)
!!fork_of(project) !!fork_of(project)
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