Commit 9afa80bb authored by Dmitriy Zaporozhets's avatar Dmitriy Zaporozhets

Fix EE code after membership refactoring

Signed-off-by: default avatarDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
parent 616e05ef
...@@ -187,7 +187,7 @@ class ProjectTeam ...@@ -187,7 +187,7 @@ class ProjectTeam
# group access is developers we need to provide # group access is developers we need to provide
# both group master, developers as devs # both group master, developers as devs
if int_level == group_link.group_access if int_level == group_link.group_access
im.where("group_access >= ?)", group_link.group_access) im.where("access_level >= ?)", group_link.group_access)
else else
im.send(level) im.send(level)
end end
......
...@@ -8,7 +8,7 @@ class LdapGroupResetService ...@@ -8,7 +8,7 @@ class LdapGroupResetService
a = group.members.with_ldap_dn.map do |member| a = group.members.with_ldap_dn.map do |member|
# don't unauthorize the current user # don't unauthorize the current user
next if current_user == member.user next if current_user == member.user
member.update_attribute :group_access, Gitlab::Access::GUEST member.update_attribute :access_level, Gitlab::Access::GUEST
end end
group.users.ldap.update_all last_credential_check_at: nil group.users.ldap.update_all last_credential_check_at: nil
......
...@@ -151,7 +151,7 @@ module Gitlab ...@@ -151,7 +151,7 @@ module Gitlab
# Get the group_access for a give user. # Get the group_access for a give user.
# Always respect the current level, never downgrade it. # Always respect the current level, never downgrade it.
def fetch_group_access(group, user, active_group_links) def fetch_group_access(group, user, active_group_links)
current_access_level = group.users_groups.where(user_id: user).maximum(:group_access) current_access_level = group.group_members.where(user_id: user).maximum(:access_level)
max_group_access_level = active_group_links.maximum(:group_access) max_group_access_level = active_group_links.maximum(:group_access)
# TODO: Test if nil value of current_access_level in handled properly # TODO: Test if nil value of current_access_level in handled properly
......
...@@ -258,7 +258,7 @@ objectclass: posixGroup ...@@ -258,7 +258,7 @@ objectclass: posixGroup
context "existing access as guest for group-1, allowed via ldap-group1 as DEVELOPER" do context "existing access as guest for group-1, allowed via ldap-group1 as DEVELOPER" do
before do before do
gitlab_group_1.users_groups.guests.create(user_id: user.id) gitlab_group_1.group_members.guests.create(user_id: user.id)
gitlab_group_1.ldap_group_links.create cn: 'ldap-group1', group_access: Gitlab::Access::MASTER gitlab_group_1.ldap_group_links.create cn: 'ldap-group1', group_access: Gitlab::Access::MASTER
end end
...@@ -270,7 +270,7 @@ objectclass: posixGroup ...@@ -270,7 +270,7 @@ objectclass: posixGroup
context "existing access as MASTER for group-1, allowed via ldap-group1 as DEVELOPER" do context "existing access as MASTER for group-1, allowed via ldap-group1 as DEVELOPER" do
before do before do
gitlab_group_1.users_groups.masters.create(user_id: user.id) gitlab_group_1.group_members.masters.create(user_id: user.id)
gitlab_group_1.ldap_group_links.create cn: 'ldap-group1', group_access: Gitlab::Access::DEVELOPER gitlab_group_1.ldap_group_links.create cn: 'ldap-group1', group_access: Gitlab::Access::DEVELOPER
end end
...@@ -282,7 +282,7 @@ objectclass: posixGroup ...@@ -282,7 +282,7 @@ objectclass: posixGroup
context "existing access as master for group-1, not allowed" do context "existing access as master for group-1, not allowed" do
before do before do
gitlab_group_1.users_groups.masters.create(user_id: user.id) gitlab_group_1.group_members.masters.create(user_id: user.id)
gitlab_group_1.ldap_group_links.create cn: 'ldap-group1', group_access: Gitlab::Access::MASTER gitlab_group_1.ldap_group_links.create cn: 'ldap-group1', group_access: Gitlab::Access::MASTER
access.stub(cns_with_access: ['ldap-group2']) access.stub(cns_with_access: ['ldap-group2'])
end end
......
...@@ -37,6 +37,6 @@ describe LdapGroupResetService do ...@@ -37,6 +37,6 @@ describe LdapGroupResetService do
end end
def member_access(user) def member_access(user)
group.members.find_by(user_id: user).group_access group.members.find_by(user_id: user).access_level
end end
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