Consider replacing hook_field_type_category_info with YML based plugin discovery

Converting to plugins would make implementing DX improvements like 📌 Allow field_type_categories.yml entries to define asset libraries Fixed easier too.

Some unresolved threads.

Looking good. Glad we are doing this!

we also need a test for the new plugin manager

Left a review, also updated issue credits

This looks good to go.

Adding tag for the followup to change $val to something else per review from @tedbow

Adding 'needs change record updates' to repurpose the one about the info hook to be about the new plugin-type

Some BC considerations and a bikeshed comment about the naming of this whole thing.

Confirmed that the feedback has been addressed 👍 I also postponed 📌 Retrieve field type category information in \Drupal\Core\Field\FieldTypePluginManager::getGroupedDefinitions Fixed on this.

Filed the follow-up issue: 📌 Rename $val in \Drupal\field_ui\Form\FieldStorageAddForm::buildForm to something more descriptive Fixed .

One more naming nitpick: plugin managers are singular, e.g. plugin.manager.block or plugin.manager.action and also field plugin managers have a common namespace:

• plugin.manager.field.field_type
• plugin.manager.field.widget
• plugin.manager.field.formatter

Therefore I think the service name for this should be plugin.manager.field.field_type_category

We also should add an interface for this:

interface FieldTypeCategoryManagerInterface extends PluginManagerInterface {
}

We should typehint this interface in the FieldTypePluginManager constructor and docblock (instead of the generic PluginManagerInterface), and then also should add an interface alias so this service can be autowired:

Drupal\Core\Field\FieldTypeCategoryManagerInterface: '@plugin.manager.field.field_type_category'

Confirmed that #20 has been addressed! Thanks for the reviews @longwave and thank you for addressing all the feedback @srishtiiee! 😊

Committed and pushed f1a52222a7 to 11.x. Thanks!

Will publish the change records.

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