Mark functions in field.info.inc deprecated.

The functions that relate to plugins (field_info_[widget|formatter]_types(), field_info_[widget|formatter]_settings()...) could move directly to the plugin managers. If needed, helper methods on those to return "just the settings" could be added.

Oh, and field_behaviors_widget() needs to die a slow and painful death. There should be no use for it anymore.

Yeah, it's just a leftover - at some point in D6 there were more stuff in there, but even in D7 it's an aberration.
Now it's just reading the "default value" of the widget plugin info...

Note that field_info_[widget|formatter]_settings() is already added as getDefaultSettings() to each plugin manager in #2014821: Introduce form modes UI and their configuration entity, but I didn't deprecate anything to keep the patch size down.

Heh. Code to steal from there then :-)

I think the Field:: should be fully namespaced in the docs.

Well, we're not system, so our services can't go in \Drupal, that's the current standard...
Views has a similar class too, not sure if there are other examples in core right now.

#8: I think you lost me :-)

+++ b/core/modules/field/lib/Drupal/field/FieldInfo.phpundefined
@@ -481,6 +481,31 @@ public function getBundleInstances($entity_type, $bundle) {
+   * @param $entity_type
+   *   The entity type for the instance.
+   * @param $field_name
+   *   The field name for the instance.
+   * @param $bundle_name
+   *   The bundle name for the instance.

maybe better to swap bundle and field?

#10 : oh, *extremely* good call !
I've been slapping myself in the face for years because of the silly order in field_info_instance()

Also, actually, shouldn't it be getInstance() instead of getBundleInstance() ?

"Also wondering whether to mark all the functions that I'm trying to kill in ..."
No need IMO

So fixed patch

EDIT I think it should be getBundleInstance() to mirror getBundleInstances() so $bundle argument have the same name

I still vote for getInstance().
You use this function when you know exactly what instance you want. getBundleInstance() has no meaning, instances are not "bundle instances".

+++ b/core/modules/field/lib/Drupal/field/FieldInfo.php
@@ -481,6 +481,31 @@ public function getBundleInstances($entity_type, $bundle) {
+   * the bundle, allowing fast retrieval of field_info_field() or
+   * field_info_instance() later in the request.

We should reference the new methods for getting the info here, as the wrappers are deprecated by this patch :)

+++ b/core/modules/field/lib/Drupal/field/FieldInfo.php
@@ -481,6 +481,31 @@ public function getBundleInstances($entity_type, $bundle) {
+   * @param $entity_type
+   *   The entity type for the instance.
+   * @param $bundle
+   *   The bundle name for the instance.
+   * @param $field_name
+   *   The field name for the instance.
+   *
+   * @return

Missing types :P

Other than that, looks good to me.

@return array .. :-s

Great, thanks!

Committed to 8.x. Thanks.