mirror of
https://github.com/internetee/registry.git
synced 2025-08-25 10:33:38 +02:00
618f2f5aed
This commit refines the contact duplication checks and notification messages for domain creation and updates, ensuring consistency and addressing a bug in domain updates.
**Key Changes:**
* **Consistent Duplicate Contact Handling:**
* The `check_for_cross_role_duplicates`, `remove_duplicate_contacts`, and `duplicate_contact?` methods are now more closely aligned between `DomainCreate` and `DomainUpdate` interactors.
* `DomainUpdate` now correctly uses `admin_contact_ids=` and `tech_contact_ids=` to assign filtered contacts. This resolves a `PG::UniqueViolation` error that occurred when trying to re-associate existing contacts using `_attributes=` methods.
* The `duplicate_contact?` method in `DomainCreate` was updated to match `DomainUpdate`, primarily checking for semantic duplicates based on attributes (name, ident, email, phone) rather than also including a `contact.code` check, which is more suitable for cross-role duplication.
* **Standardized Notification Messages:**
* The `notify_about_removed_duplicates` method in both interactors now generates a more concise message: ". [Role] contact [CODE] was discarded as duplicate;" for each discarded contact.
* This message is appended to `domain.skipped_domain_contacts_validation`.
* **EPP & REPP Response Updates:**
* The `message` method in `Repp::V1::DomainsController` now correctly appends `domain.skipped_domain_contacts_validation` to the "Command completed successfully" message, ensuring it appears in REPP JSON responses for both create and update.
* The EPP XML views (`app/views/epp/domains/create.xml.builder` and `app/views/epp/domains/success.xml.builder`) are updated to dynamically include `domain.skipped_domain_contacts_validation` in the `<msg>` tag.
* **Model Attribute:**
* Added `skipped_domain_contacts_validation` as a string attribute to the `Domain` model to store these notification messages.
* **Test Refinements (Implicit):**
* The related test suite for domain updates (`test/integration/epp/domain/base_test.rb`) was being updated to reflect these logic changes and assert the correct notification messages.
These changes improve the robustness and consistency of contact management during domain operations, providing clearer feedback to users about discarded duplicate contacts.
|
||
|---|---|---|
| .. | ||
| api_log | ||
| authorization | ||
| balance_auto_reload_types | ||
| billing | ||
| concerns | ||
| contact | ||
| dns | ||
| epp | ||
| invoice | ||
| payment_orders | ||
| type | ||
| validation_event | ||
| version | ||
| whois | ||
| ability.rb | ||
| account.rb | ||
| account_activity.rb | ||
| action.rb | ||
| admin_domain_contact.rb | ||
| admin_user.rb | ||
| api_user.rb | ||
| application_record.rb | ||
| auction.rb | ||
| bank_statement.rb | ||
| bank_transaction.rb | ||
| billing.rb | ||
| blocked_domain.rb | ||
| bounced_mail_address.rb | ||
| business_registry_contact.rb | ||
| certificate.rb | ||
| certification_request.rb | ||
| contact.rb | ||
| contact_request.rb | ||
| contact_update_action.rb | ||
| csync_record.rb | ||
| deposit.rb | ||
| directo.rb | ||
| dispute.rb | ||
| dns.rb | ||
| dnskey.rb | ||
| domain.rb | ||
| domain_contact.rb | ||
| domain_cron.rb | ||
| domain_status.rb | ||
| domain_transfer.rb | ||
| epp_session.rb | ||
| feature.rb | ||
| iban.rb | ||
| inactive_contacts.rb | ||
| invoice.rb | ||
| invoice_item.rb | ||
| invoice_state_machine.rb | ||
| legal_document.rb | ||
| mass_action.rb | ||
| nameserver.rb | ||
| notification.rb | ||
| payment_order.rb | ||
| registrant.rb | ||
| registrant_user.rb | ||
| registrant_verification.rb | ||
| registrar.rb | ||
| registry.rb | ||
| repp_api.rb | ||
| reserved_domain.rb | ||
| retained_domains.rb | ||
| setting.rb | ||
| setting_entry.rb | ||
| tech_domain_contact.rb | ||
| user.rb | ||
| validation_event.rb | ||
| version.rb | ||
| white_ip.rb | ||
| whois_record.rb | ||