summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRobert Speicher <rspeicher@gmail.com>2016-09-20 21:48:58 +0300
committerRobert Speicher <rspeicher@gmail.com>2016-09-20 21:48:58 +0300
commit6f558121b45f705624b76c2ba39491dc72810c13 (patch)
tree50306de202c113cb5601766fb6c89cf64e9bd3f1
parentb7b41d1f68bf30f9d115bc6687f6762a78a546af (diff)
downloadgitlab-ce-6f558121b45f705624b76c2ba39491dc72810c13.tar.gz
Fix a logic error in ProjectTeam#fetch_invited_members
We were calling `.where` and `.send` on the relation, but never doing anything with the return value, resulting in proper access-level filtering never being of any consequence.
-rw-r--r--app/models/project_team.rb4
-rw-r--r--spec/models/project_team_spec.rb1
2 files changed, 3 insertions, 2 deletions
diff --git a/app/models/project_team.rb b/app/models/project_team.rb
index e8606c8c2c6..04f4fa849c4 100644
--- a/app/models/project_team.rb
+++ b/app/models/project_team.rb
@@ -248,9 +248,9 @@ class ProjectTeam
# group access is developers we need to provide
# both group master, developers as devs
if int_level == group_link.group_access
- im.where("access_level >= ?)", group_link.group_access)
+ im = im.where("access_level >= ?", group_link.group_access)
else
- im.send(level)
+ im = im.send(level)
end
end
diff --git a/spec/models/project_team_spec.rb b/spec/models/project_team_spec.rb
index 469e306044b..f979d66c88c 100644
--- a/spec/models/project_team_spec.rb
+++ b/spec/models/project_team_spec.rb
@@ -111,6 +111,7 @@ describe ProjectTeam, models: true do
group_access: Gitlab::Access::REPORTER
)
+ expect(project.team.guests).to be_empty
expect(project.team.reporters).to contain_exactly(group_member.user)
end
end