#2380391: Fix storage settings StringLongItem fixed the storage_settings for string_long field type to no longer have a 'max_length' setting.
The config schema needs to be adjusted accordingly.
We should wait for #2370305: Refactor field type configuration schemas for DX, easier to find errors to land though
Here's a patch that applies on top of #2370305: Refactor field type configuration schemas for DX, easier to find errors
(thus, keeping at "Postponed" rather NR)
#2370305: Refactor field type configuration schemas for DX, easier to find errors got in
This is a correct update as of now, string does not have any other settings but max_length. If string is to get more settings though, this may need updates again. File vs. image field does a similar thing because both have unique items but also common items (they inherit from each other). So they have a common base type that both extend in schema. While that would be technically correct here, defining a base type that is an empty mapping does not sound very compelling, so I think this fix is fine.
Regarding inheriting the schema : yeah I was not too sure whether it was a good thing or not. I went for explicit schema.
Issue #2381079 by yched: Adjust storage_settings schema for string_long...
Committed to 8.x. Test coverage looks to be part of #2183983: Find hidden configuration schema issues. Thanks yched.
@Gábor - regarding the setting's inheritance and schema: actually #2383277: StringLongItem should not extend StringItem is removing the inheritance on the classes, which IMO is a good move.
Automatically closed - issue fixed for 2 weeks with no activity.
Drupal is a registered trademark of Dries Buytaert.