From 7a27756a36d41a625420867732487e7382cf9398 Mon Sep 17 00:00:00 2001 From: Stas Date: Wed, 25 May 2016 12:42:10 +0300 Subject: [PATCH 1/2] 111601356-errors_customization --- app/models/contact.rb | 2 +- app/models/epp/contact.rb | 4 +++- config/locales/en.yml | 1 + 3 files changed, 5 insertions(+), 2 deletions(-) diff --git a/app/models/contact.rb b/app/models/contact.rb index edc7bedd3..bdf0a6017 100644 --- a/app/models/contact.rb +++ b/app/models/contact.rb @@ -33,7 +33,7 @@ class Contact < ActiveRecord::Base uniqueness: { message: :epp_id_taken }, format: { with: /\A[\w\-\:\.\_]*\z/i, message: :invalid }, length: { maximum: 100, message: :too_long_contact_code } - validates :ident_type, inclusion: {in: %w( org priv birthday), message: :invalid} + validates :ident_type, inclusion: {in: %w( org priv birthday), message: :ident_type_invalid} validate :val_ident_valid_format? validate :uniq_statuses? validate :validate_html diff --git a/app/models/epp/contact.rb b/app/models/epp/contact.rb index f6bf54f74..af3cc13a9 100644 --- a/app/models/epp/contact.rb +++ b/app/models/epp/contact.rb @@ -126,13 +126,15 @@ class Epp::Contact < Contact [:ident, :invalid_birthday_format], [:ident, :invalid_country_code], [:ident_type, :missing], - [:ident_type, :invalid], [:code, :invalid], [:code, :too_long_contact_code] ], '2302' => [ # Object exists [:code, :epp_id_taken] ], + '2304' => [ + [:ident_type, :invalid, { value: { obj: 'ident_type', val: ident_type } }] + ], '2305' => [ # Association exists [:domains, :exist] ], diff --git a/config/locales/en.yml b/config/locales/en.yml index de5891c77..ea05adec3 100644 --- a/config/locales/en.yml +++ b/config/locales/en.yml @@ -266,6 +266,7 @@ en: unimplemented_command: 'Unimplemented command' domain_exists_but_belongs_to_other_registrar: 'Domain exists but belongs to other registrar' required_ident_attribute_missing: "Required ident attribute missing: %{key}" + ident_type_invalid: 'Ident type is invalid' code: 'Code' value: 'Value' From 9e3bb02dd169b6e42362d935070c823df8607856 Mon Sep 17 00:00:00 2001 From: Stas Date: Wed, 25 May 2016 17:19:50 +0300 Subject: [PATCH 2/2] 111601356-error_text_modification --- app/models/epp/contact.rb | 4 ++-- config/locales/en.yml | 4 +++- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/app/models/epp/contact.rb b/app/models/epp/contact.rb index af3cc13a9..8e1da8e4b 100644 --- a/app/models/epp/contact.rb +++ b/app/models/epp/contact.rb @@ -132,8 +132,8 @@ class Epp::Contact < Contact '2302' => [ # Object exists [:code, :epp_id_taken] ], - '2304' => [ - [:ident_type, :invalid, { value: { obj: 'ident_type', val: ident_type } }] + '2304' => [ # Object status prohibits operation + [:ident_type, :epp_ident_type_invalid, { value: { obj: 'code', val: code } }] ], '2305' => [ # Association exists [:domains, :exist] diff --git a/config/locales/en.yml b/config/locales/en.yml index ea05adec3..4e915de6a 100644 --- a/config/locales/en.yml +++ b/config/locales/en.yml @@ -52,6 +52,9 @@ en: invalid_EE_identity_format_update: "Ident not in valid Estonian identity format. Please create new contact" invalid_birthday_format: "Ident not in valid birthady format, should be YYYY-MM-DD" invalid_country_code: "Ident country code is not valid, should be in ISO_3166-1 alpha 2 format" + ident_type: + ident_type_invalid: 'Ident type is invalid' + epp_ident_type_invalid: 'Object status prohibits operation: ident_type of contact %{code} is invalid' domains: exist: 'Object association prohibits operation' statuses: @@ -266,7 +269,6 @@ en: unimplemented_command: 'Unimplemented command' domain_exists_but_belongs_to_other_registrar: 'Domain exists but belongs to other registrar' required_ident_attribute_missing: "Required ident attribute missing: %{key}" - ident_type_invalid: 'Ident type is invalid' code: 'Code' value: 'Value'