diff --git a/app/controllers/moderation/organizations_controller.rb b/app/controllers/moderation/organizations_controller.rb index 4788d3aa2..1528d0ea2 100644 --- a/app/controllers/moderation/organizations_controller.rb +++ b/app/controllers/moderation/organizations_controller.rb @@ -1,25 +1,19 @@ class Moderation::OrganizationsController < Moderation::BaseController - before_filter :load_organizations, only: :index - load_and_authorize_resource class: 'User' + load_and_authorize_resource def index + @organizations = @organizations.includes(:user).order(:name, 'users.email') end - def verify_organization - @organization.update(organization_verified_at: Time.now) + def verify + @organization.verify redirect_to action: :index end - def reject_organization - @organization.update(organization_rejected_at: Time.now) + def reject + @organization.reject redirect_to action: :index end - private - - def load_organizations - @organizations = User.organizations.order(:organization_name, :email) - end - end diff --git a/app/views/moderation/organizations/index.html.erb b/app/views/moderation/organizations/index.html.erb index 3f1d833b6..06549a9b3 100644 --- a/app/views/moderation/organizations/index.html.erb +++ b/app/views/moderation/organizations/index.html.erb @@ -1,27 +1,27 @@
| <%= organization.organization_name %> | +<%= organization.name %> | <%= organization.email %> | <%= organization.phone_number %> | - <% if organization.verified_organization? %> + <% if organization.verified? %><%= t('moderation.organizations.index.verified') %> | <% end %> - <% if can? :verify_organization, organization %> + <% if can? :verify, organization %><%= link_to t('moderation.organizations.index.verify'), - verify_organization_moderation_organization_path(organization), + verify_moderation_organization_path(organization), method: :put %> | <% end %> - <% if organization.rejected_organization? %> + <% if organization.rejected? %><%= t('moderation.organizations.index.rejected') %> | <% end %> - <% if can? :reject_organization, organization %> + <% if can? :reject, organization %><%= link_to t('moderation.organizations.index.reject'), - reject_organization_moderation_organization_path(organization), + reject_moderation_organization_path(organization), method: :put %> | diff --git a/config/routes.rb b/config/routes.rb index 87eceda55..fa1f6624b 100644 --- a/config/routes.rb +++ b/config/routes.rb @@ -29,8 +29,8 @@ Rails.application.routes.draw do root to: "dashboard#index" resources :organizations, only: :index do member do - put :verify_organization - put :reject_organization + put :verify + put :reject end end end diff --git a/spec/features/moderation/organizations_spec.rb b/spec/features/moderation/organizations_spec.rb index 9c07a2b9c..22cf8d2db 100644 --- a/spec/features/moderation/organizations_spec.rb +++ b/spec/features/moderation/organizations_spec.rb @@ -21,11 +21,11 @@ feature 'Moderations::Organizations' do expect(current_path).to eq(moderation_organizations_path) expect(page).to have_content ('Verified') - expect(organization.reload.verified_organization?).to eq(true) + expect(organization.reload.verified?).to eq(true) end scenario "verified organizations have link to reject" do - organization = create(:organization, organization_verified_at: Time.now) + organization = create(:verified_organization) visit moderation_organizations_path expect(page).to have_content ('Verified') @@ -36,11 +36,11 @@ feature 'Moderations::Organizations' do expect(current_path).to eq(moderation_organizations_path) expect(page).to have_content ('Rejected') - expect(organization.reload.rejected_organization?).to eq(true) + expect(organization.reload.rejected?).to eq(true) end scenario "rejected organizations have link to verify" do - organization = create(:organization, organization_rejected_at: Time.now) + organization = create(:rejected_organization) visit moderation_organizations_path expect(page).to have_content ('Rejected') @@ -51,7 +51,7 @@ feature 'Moderations::Organizations' do expect(current_path).to eq(moderation_organizations_path) expect(page).to have_content ('Verified') - expect(organization.reload.verified_organization?).to eq(true) + expect(organization.reload.verified?).to eq(true) end end