Skip to content

INTGRTNS-371 Openprovider does not filter all the additional fields in dist.additionalfield.php#543

Open
YumeChaan wants to merge 1 commit into
masterfrom
INTGRTNS-371-WHMCS-Openprovider-does-not-filter-all-the-additional-fields-in-dist.additionalfield.php
Open

INTGRTNS-371 Openprovider does not filter all the additional fields in dist.additionalfield.php#543
YumeChaan wants to merge 1 commit into
masterfrom
INTGRTNS-371-WHMCS-Openprovider-does-not-filter-all-the-additional-fields-in-dist.additionalfield.php

Conversation

@YumeChaan

@YumeChaan YumeChaan commented May 12, 2026

Copy link
Copy Markdown
Collaborator

Changes:

  • Updated getFilteredAdditionalFields() to iterate all registrar TLDs and use our custom fields if present.
  • For TLDs with no custom fields, add ['Name' => ..., 'Remove' => true] for each dist field, hiding them from UI.

@YumeChaan
Iterate over all registrarTlds and, for any TLD lacking our custom fi…
6f4581f 
…elds, add Remove entries for each dist.additionalfields.php field so only OP-mapped fields remain.
Copilot AI review requested due to automatic review settings May 12, 2026 07:04
Copilot AI reviewed May 12, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@YumeChaan YumeChaan requested a review from Copilot May 12, 2026 07:06
Copilot AI reviewed May 12, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.

@YumeChaan YumeChaan requested a review from Copilot May 13, 2026 05:05
Copilot AI reviewed May 13, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.

@YumeChaan YumeChaan requested a review from avishka-ra May 13, 2026 05:05
@prasad-fernando-74 prasad-fernando-74 self-requested a review May 19, 2026 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments
@prasad-fernando-74 prasad-fernando-74 prasad-fernando-74 approved these changes
@avishka-ra avishka-ra avishka-ra approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@YumeChaan @prasad-fernando-74 @avishka-ra