Browse Source

feat: prohibit users being added to merged orgs

pull/1787/head
natdeanlewissoftwire 3 years ago
parent
commit
f8adb2be09
  1. 7
      app/models/user.rb
  2. 1
      config/locales/en.yml

7
app/models/user.rb

@ -24,6 +24,7 @@ class User < ApplicationRecord
after_validation :send_data_protection_confirmation_reminder, if: :is_dpo_changed?
validates :organisation_id, presence: true
validate :organisation_not_merged
has_paper_trail ignore: %w[last_sign_in_at
current_sign_in_at
@ -185,6 +186,12 @@ protected
private
def organisation_not_merged
if organisation&.merge_date.present? && organisation.merge_date < Time.zone.now
errors.add :organisation_id, I18n.t("validations.organisation.merged")
end
end
def send_data_protection_confirmation_reminder
return unless persisted?
return unless is_dpo?

1
config/locales/en.yml

@ -198,6 +198,7 @@ en:
managing_agent:
blank: "You must choose a managing agent"
already_added: "You have already added this managing agent"
merged: "Select an organisation that has not been merged"
not_answered: "You must answer %{question}"
invalid_option: "Enter a valid value for %{question}"
invalid_number: "Enter a number for %{question}"

Loading…
Cancel
Save