Those are just wrappers around the corresponding plugin managers.

Comments

yched’s picture

Status: Active » Needs review
StatusFileSize
new3.6 KB

Patch.

yched’s picture

Sister issue to actually remove the current usages in core : #2047993: Remove current uses of field_info_*_types() / field_info_*_settings()

swentel’s picture

Status: Needs review » Reviewed & tested by the community

Yes, let's do this, should still be ok in this stage - I think ..

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 94b3946 and pushed to 8.x. Thanks!

plopesc’s picture

Status: Fixed » Needs review
StatusFileSize
new1021 bytes

Hello,
Patch committed does not point to the right services in field_info_widget_settings() and field_info_formatter_settings().

+++ b/core/modules/field/field.info.incundefined
@@ -518,6 +541,10 @@ function field_info_instance_settings($type) {
+ *
+ * @deprecated as of Drupal 8.0. Use the 'settings' property in the definition
+ *   array returned by
+ *   Drupal::service('plugin.manager.field.formatter')->getDefinition()
  */
 function field_info_widget_settings($type) {

Comment should point to plugin.manager.field.widget service.

+++ b/core/modules/field/field.info.incundefined
@@ -534,6 +561,10 @@ function field_info_widget_settings($type) {
+ *
+ * @deprecated as of Drupal 8.0. Use the 'settings' property in the definition
+ *   array returned by
+ *   Drupal::service('plugin.manager.field.widget')->getDefinition()
  */

Comment should point to plugin.manager.field.widget service.

This is not a major issue, given that these functions will be removed in #2047993: Remove current uses of field_info_*_types() / field_info_*_settings(). We could live with this, but just attaching new patch fixing those doc blocks if you consider it.

Regards

yched’s picture

Status: Needs review » Reviewed & tested by the community

Doh, copy / paste.
yched--

alexpott’s picture

alexpott-- too as I specifically tried to look for a copy / paste here

yched’s picture

Followup is still RTBC ;-)

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed f1ce57c and pushed to 8.x. Thanks!

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