refactoring

This commit is contained in:
Oleg Hasjanov 2021-04-08 11:15:24 +03:00
parent 1cc6f2576f
commit 441a1911c4

View file

@ -559,16 +559,24 @@ class Domain < ApplicationRecord
statuses.include?(DomainStatus::FORCE_DELETE) statuses.include?(DomainStatus::FORCE_DELETE)
end end
def update_unless_locked_by_registrant(update)
update(admin_store_statuses_history: update) unless locked_by_registrant?
end
def update_not_by_locked_statuses(update)
return unless locked_by_registrant?
result = update.reject { |status| LOCK_STATUSES.include? status }
update(admin_store_statuses_history: result)
end
# special handling for admin changing status # special handling for admin changing status
def admin_status_update(update) def admin_status_update(update)
update_unless_locked_by_registrant(update)
update_not_by_locked_statuses(update)
# check for deleted status # check for deleted status
update(admin_store_statuses_history: update) unless locked_by_registrant?
if locked_by_registrant?
result = update.reject { |status| LOCK_STATUSES.include? status }
update(admin_store_statuses_history: result)
end
statuses.each do |s| statuses.each do |s|
unless update.include? s unless update.include? s
case s case s