zydronium/internetee-registry
1
0
Fork 0
mirror of https://github.com/internetee/registry.git synced 2025-08-12 04:29:33 +02:00
Code Issues Projects Releases Packages Wiki Activity

fixed legal attributes

Browse source
This commit is contained in:
olegphenomenon 2022-11-28 11:43:00 +02:00
parent ea2fc01a60
commit 86fad85d03
3 changed files with 29 additions and 22 deletions
Download patch file Download diff file Expand all files Collapse all files

5
app/controllers/api/v1/registrant/contacts_controller.rb
View file

@ -55,7 +55,10 @@ module Api
attributes_not_exist_error(extra_attrs) and return if extra_attrs.present? attributes_not_exist_error(extra_attrs) and return if extra_attrs.present?
end end
# render_disclosed_attributes_error and return if disclosed_attributes.present? && contact.org? if disclosed_attributes.present? && contact.org?
extra_attrs = disclosed_attributes - Contact::OPEN_LEGAL_ATTRIBUTES
render_disclosed_attributes_error and return if extra_attrs.present?
end
contact.disclosed_attributes = disclosed_attributes if disclosed_attributes contact.disclosed_attributes = disclosed_attributes if disclosed_attributes
publishable = reparsed_request[:registrant_publishable] publishable = reparsed_request[:registrant_publishable]

4
app/models/contact.rb
View file

@ -156,6 +156,10 @@ class Contact < ApplicationRecord
fax fax
].freeze ].freeze
OPEN_LEGAL_ATTRIBUTES = %w[
phone
].freeze
STATUSES = [ STATUSES = [
CLIENT_DELETE_PROHIBITED, SERVER_DELETE_PROHIBITED, CLIENT_DELETE_PROHIBITED, SERVER_DELETE_PROHIBITED,
CLIENT_TRANSFER_PROHIBITED, CLIENT_TRANSFER_PROHIBITED,

42
test/integration/api/v1/registrant/contacts/update_test.rb
View file

@ -175,31 +175,31 @@ class RegistrantApiV1ContactUpdateTest < ActionDispatch::IntegrationTest
assert_empty @contact.disclosed_attributes assert_empty @contact.disclosed_attributes
end end
# def test_legal_persons_disclosed_attributes_cannot_be_changed def test_legal_persons_disclosed_attributes_cannot_be_changed
# @contact = contacts(:acme_ltd) @contact = contacts(:acme_ltd)
# contacts(:acme_ltd).ident contacts(:acme_ltd).ident
# assert_equal '1234567', @contact.ident assert_equal '1234567', @contact.ident
# assert_equal Contact::ORG, @contact.ident_type assert_equal Contact::ORG, @contact.ident_type
# assert_equal 'US', @contact.ident_country_code assert_equal 'US', @contact.ident_country_code
# @contact.update!(disclosed_attributes: %w[]) @contact.update!(disclosed_attributes: %w[])
# assert_equal 'US-1234', @user.registrant_ident assert_equal 'US-1234', @user.registrant_ident
# assert_no_changes -> { @contact.disclosed_attributes } do assert_no_changes -> { @contact.disclosed_attributes } do
# patch api_v1_registrant_contact_path(@contact.uuid), patch api_v1_registrant_contact_path(@contact.uuid),
# params: { disclosed_attributes: %w[name] }, params: { disclosed_attributes: %w[name] },
# as: :json, as: :json,
# headers: { 'HTTP_AUTHORIZATION' => auth_token } headers: { 'HTTP_AUTHORIZATION' => auth_token }
# @contact.reload @contact.reload
# end end
# assert_response :bad_request assert_response :bad_request
# error_msg = "Legal person's data is visible by default and cannot be concealed." \ error_msg = "Legal person's data is visible by default and cannot be concealed." \
# ' Please remove this parameter.' ' Please remove this parameter.'
# assert_equal ({ errors: [{ disclosed_attributes: [error_msg] }] }), assert_equal ({ errors: [{ disclosed_attributes: [error_msg] }] }),
# JSON.parse(response.body, symbolize_names: true) JSON.parse(response.body, symbolize_names: true)
# end end
def test_legal_persons_disclosed_attributes_change_when_phone def test_legal_persons_disclosed_attributes_change_when_phone
@contact = contacts(:acme_ltd) @contact = contacts(:acme_ltd)