diff --git a/app/controllers/directory_items_controller.rb b/app/controllers/directory_items_controller.rb index 041d8276852..1a66c740034 100644 --- a/app/controllers/directory_items_controller.rb +++ b/app/controllers/directory_items_controller.rb @@ -11,11 +11,11 @@ class DirectoryItemsController < ApplicationController result = DirectoryItem.where(period_type: period_type).includes(:user) if params[:group] - result = result.includes(user: :groups).where(users: {groups: {name: params[:group]}}) + result = result.includes(user: :groups).where(users: { groups: { name: params[:group] } }) end if params[:exclude_usernames] - result = result.references(:user).where.not(users: {username: params[:exclude_usernames].split(",")}) + result = result.references(:user).where.not(users: { username: params[:exclude_usernames].split(",") }) end order = params[:order] || DirectoryItem.headings.first diff --git a/spec/requests/directory_items_controller_spec.rb b/spec/requests/directory_items_controller_spec.rb index 3c365ba00fc..c9126347032 100644 --- a/spec/requests/directory_items_controller_spec.rb +++ b/spec/requests/directory_items_controller_spec.rb @@ -5,7 +5,7 @@ describe DirectoryItemsController do let!(:evil_trout) { Fabricate(:evil_trout) } let!(:walter_white) { Fabricate(:walter_white) } let!(:stage_user) { Fabricate(:staged, username: 'stage_user') } - let!(:group) { Fabricate(:group, users: [evil_trout,stage_user]) } + let!(:group) { Fabricate(:group, users: [evil_trout, stage_user]) } it "requires a `period` param" do get '/directory_items.json'