Commit 6153b8c1 authored by Nikola Milojevic's avatar Nikola Milojevic

Merge branch 'mmj-values-list' into 'master'

Use `Arel::Nodes::ValuesList`  in `EffectiveAccessLevelFinder`

See merge request gitlab-org/gitlab!77089
parents c7f4d8c3 a980f2a5
......@@ -27,13 +27,9 @@ module Projects
attr_reader :project
def generate_from_statement(user_ids_and_access_levels)
"(VALUES #{generate_values_expression(user_ids_and_access_levels)}) members (user_id, access_level)"
end
values_list = Arel::Nodes::ValuesList.new(user_ids_and_access_levels).to_sql
def generate_values_expression(user_ids_and_access_levels)
user_ids_and_access_levels.map do |user_id, access_level|
"(#{user_id}, #{access_level})"
end.join(",")
"(#{values_list}) members (user_id, access_level)"
end
def no_members?
......
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