Problem/Motivation

A disposable Drupal test-site installation with strict configuration schema checking fails while installing MCP Sentinel 2.23.1. The checker reports that mcp_sentinel.settings must specify a language code because it contains translatable values. The dashboard_broadcast.message schema uses the translatable label type, but the default settings do not declare langcode.

Steps to reproduce

Install MCP Sentinel 2.23.1 and its dependencies using Drupal’s test-site installer with strict schema checking enabled. Installation fails in ConfigSchemaChecker before the site is usable.

Proposed resolution

Add an isolated strict default-configuration regression and correct the default language metadata. Preserve the broadcast message’s translation semantics. Do not disable schema checking.

Remaining tasks

Reproduce in an isolated module test, implement the metadata correction, and verify supported Drupal/PHP versions.

User interface and API changes

None intended.

Data model changes

Default configuration language metadata only. No existing site records are changed by this proposal.

Issue fork mcp_sentinel-3624679

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

jmcerda created an issue. See original summary.

jmcerda opened merge request !5

jmcerda’s picture

jmcerda
Location Boca Raton, FL
commented
Status: Active » Needs review

The correction and strict-schema regression are ready for review in MR !5. The regression reproduces the failure before the change and passes on Drupal 10.6.17/PHP 8.3, 11.3.17/PHP 8.3 and 11.4.7/PHP 8.4 (one test, three assertions per run). Existing dependency deprecation notices remain visible; coding standards pass. CI is running.

  • jmcerda committed e58ab5d2 on 1.x 
    Issue #3624679: Type nested configuration mappings in regression

  • jmcerda committed 24e3e587 on 1.x 
    Issue #3624679: Declare the default settings language and verify strict...
jmcerda’s picture

jmcerda
Location Boca Raton, FL
commented
Status: Needs review » Fixed

Committed to 1.x. Default settings now declare langcode so strict configuration validation accepts the translatable dashboard broadcast message. This ships in 2.23.2.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.