- Issue created by @Grevil
- Issue was unassigned.
- 🇩🇪Germany Anybody Porta Westfalica
@Grevil: I think this can't be fixed in the module here, but must be fixed in the field providing module!
The icon field is provided by the micon module, I think?
- 🇩🇪Germany Anybody Porta Westfalica
@Grevil: Yes this is a Micon issue!
This is the track:
https://git.drupalcode.org/project/micon/-/blob/2.x/src/Plugin/Field/Fie...
class StringMiconItem extends StringItem
=>
https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Field%21P...
/** * {@inheritdoc} */ public static function defaultStorageSettings() { return [ 'max_length' => 255, 'is_ascii' => FALSE, ] + parent::defaultStorageSettings(); }
So I think it's a micon schema error and we need to create an issue and fix over there.
- Assigned to Grevil
- 🇩🇪Germany Anybody Porta Westfalica
@Grevil: Eventually we can simply unset these keys like here:
https://git.drupalcode.org/project/drupal/-/blob/11.x/core/lib/Drupal/Co... but then we'd have to be sure it has no side effects.I looked into core, where these values are being handled in the schema and it's here:
https://git.drupalcode.org/project/drupal/-/blob/11.x/core/config/schema...Now the question is, why the schema isn't inherited on Micon fields extending this.
Any I think the best way to find the solution is to find a similar case, where a contrib module (with pretty schema) also extends a core field type?
Are you on the track with this information @Grevil?
- Status changed to Postponed
8 months ago 4:28pm 15 March 2024 - 🇩🇪Germany Anybody Porta Westfalica
I created the issue: 🐛 Micon missing schema for its fields definitions Needs review
- Issue was unassigned.
- Status changed to Fixed
8 months ago 1:24pm 19 March 2024 - 🇩🇪Germany Grevil
🐛 Micon missing schema for its fields definitions Needs review is fixed any minute, and the tests are running again!
Automatically closed - issue fixed for 2 weeks with no activity.