Problem/Motivation

The configuration schema for the Glossify module’s settings is not correctly registered
in the system for later versions of Drupal (ie, 10.3+) which now validates configuration schema.

This issue prevents proper validation and recognition of configuration settings leading to errors such as "missing schema" or "missing keys" during Drupal updates. This also prevents resaving any Text Format that uses Glossify.

Steps to reproduce

  • Upgrade Drupal to version 10.3.0
  • Save the configuration for the Full HTML text format or any configuration form where Glossify settings are used.

Proposed resolution

Update the config schema file (glossify_taxonomy.schema.yml and glossify_node.schema.yml) as per https://www.drupal.org/node/3419181

Issue fork glossify-3457599

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

ewehbe created an issue. See original summary.

ewehbe opened merge request !21

ewehbe’s picture

ewehbe commented
Status: Active » Needs review
ewehbe’s picture

ewehbe commented
Title: Missing Schema » Missing Keys
scott_euser’s picture

scott_euser commented
Status: Needs review » Needs work

According to https://www.drupal.org/node/3419181 the type should be 'mapping' not 'filter' as well.

scott_euser’s picture

scott_euser commented
Assigned: Unassigned » scott_euser
scott_euser’s picture

scott_euser commented
Title: Missing Keys » Missing Keys preventing saving in Drupal 10.3+
Assigned: scott_euser » Unassigned
Issue summary: View changes
Status: Needs work » Needs review

Otherwise this looks good now, but I won't change to RTBC myself since I made a change here. If someone else could please do a final review. Thanks!

ewehbe’s picture

ewehbe commented
Status: Needs review » Reviewed & tested by the community

I have tested the changes, the module works as expected.
I will change the status to RTBC.

Anybody made their first commit to this issue’s fork.

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented

Great work, thank you both!! Merging!

  • Anybody committed 603df707 on 3.x authored by ewehbe 
    Issue #3457599: Missing Keys preventing saving in Drupal 10.3+
anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented
Status: Reviewed & tested by the community » Fixed

I'll tag a new release with this fix!

Status: Fixed » Closed (fixed)

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