Skip to content

[System Settings] Add additional system settings to provider#1644

Open
Corepex wants to merge 4 commits into1.xfrom
add-missing-settings-fields
Open

[System Settings] Add additional system settings to provider#1644
Corepex wants to merge 4 commits into1.xfrom
add-missing-settings-fields

Conversation

@Corepex
Copy link
Contributor

@Corepex Corepex commented Feb 6, 2026

Changes in this pull request

Additional info

This pull request expands the settings returned by the getSettings() method in SystemSettingsProvider.php to include additional system configuration options. The new settings provide more comprehensive access to various system features such as documents, objects, assets, error pages, domain redirection, and email configuration.

Additional system settings exposed:

  • Added documents, objects, and assets keys to the settings array, allowing access to their respective system configurations.
  • Added errorPages key for error page configuration.
  • Added redirectToMainDomain key for main domain redirection settings.
  • Added email key for email-related system settings.

@Corepex Corepex self-assigned this Feb 6, 2026
@Corepex Corepex marked this pull request as ready for review February 6, 2026 09:04
@Corepex Corepex requested a review from mcop1 February 6, 2026 09:21
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 6, 2026

'documents' => $this->systemSettings['documents'] ?? [],
'objects' => $this->systemSettings['objects'] ?? [],
'assets' => $this->systemSettings['assets'] ?? [],
'errorPages' => $this->systemSettings['error_pages'] ?? [],
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@mcop1, please return all relation-related elements as RelatedElementData to have the same data structure everywhere. This should be documents.error_pages.default and documents.error_pages.localized.<locale>

@Corepex Corepex changed the title Add additional system settings to provider [System Settings] Add additional system settings to provider Feb 6, 2026
@Corepex Corepex added this to the next milestone Feb 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant