use subject scope in :id/users

since we're loading all the members anyways

use subject scope in :id/users
since we're loading all the members anyways
59e7c39f · http://jneen.net/ · e895b49f · 59e7c39f
Commit 59e7c39f authored Apr 11, 2017 by http://jneen.net/
Show whitespace changes
Inline Side-by-side

Showing with 3 additions and 1 deletion

lib/api/projects.rb lib/api/projects.rb +3 -1

No files found.
--- a/lib/api/projects.rb
+++ b/lib/api/projects.rb
+require 'declarative_policy'
+
 module API
  # Projects API
  class Projects < Grape::API
@@ -396,7 +398,7 @@ module API
        use :pagination
      end
      get ':id/users' do
-        users = user_project.team.users
+        users = DeclarativePolicy.subject_scope { user_project.team.users }
        users = users.search(params[:search]) if params[:search].present?

        present paginate(users), with: Entities::UserBasic