I have found two cases where schema files end with ".data_types.yml" instead of ".data_types.schema.yml".
Patch added to rename them.

core/modules/entity/config/schema/entity.data_types.yml
core/modules/rdf/config/schema/rdf.data_types.yml

These files were being skipped from "shipped configuration schema" extraction by potx, and were causing issues with my work on #1933988: Support for Drupal 8 shipped configuration translatables with external dependencies (in effect contrib).

CommentFileSizeAuthor
fix-schema-names.patch570 bytesherom
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs review » Reviewed & tested by the community

perfect

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented
Title: Fix 2 bad schema file names. » Quickfix: Two schema files are not named with the common naming scheme
Issue tags: +D8MI, +sprint, +language-config, +Quickfix

@herom: I think it is safe for you to consider any .yml file in schema/ a schema yml even if there is no schema.yml suffix. At least core does that. I think this is good standardisation, but should not be required per say AFAIS.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Fixed

Yeah any YAML in config/schema/ should be considered a schema. But nice to be consistent.

Committed b2c4388 and pushed to 8.x. Thanks!

  • alexpott committed b2c4388 on 8.x 
    Issue #2308971 by herom: Fixed Quickfix: Two schema files are not named...
Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented
Issue tags: -sprint

Status: Fixed » Closed (fixed)

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