The documentation comment for editor_field_formatter_info_alter() is the following one.

/**
 * Implements hook_form_FORM_ID_alter() for \Drupal\filter\FilterFormatListBuilder.
 *
 * Implements hook_field_formatter_info_alter().
 *
 * @see quickedit_field_formatter_info_alter()
 */

That documentation comment is wrong, as the function is not an implementation of the first hook, but the second hook. The fist line needs to be removed, as that is an implementation of hook_field_formatter_info_alter() done from editor.module.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kiamlaluno created an issue. See original summary.

Deepthi kumari’s picture

Assigned: Unassigned » Deepthi kumari
Deepthi kumari’s picture

Status: Active » Needs review
FileSize
464 bytes

Applied Patch. Please review.

rpayanm’s picture

Assigned: Deepthi kumari » Unassigned
Status: Needs review » Reviewed & tested by the community

Thanks.

LGTM

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 3: documentation-field-formatter-3111658-3.patch, failed testing. View results

rpayanm’s picture

Status: Needs work » Needs review
rpayanm’s picture

Status: Needs review » Reviewed & tested by the community

I'm changing it back to RTBC.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed bf2240bfd8 to 9.0.x and 990e3666b3 to 8.9.x and 421269ba52 to 8.8.x. Thanks!

Backported to 8.8.x as this is a docs fix.

  • alexpott committed bf2240b on 9.0.x
    Issue #3111658 by Deepthi kumari, kiamlaluno:...

  • alexpott committed 990e366 on 8.9.x
    Issue #3111658 by Deepthi kumari, kiamlaluno:...

  • alexpott committed 421269b on 8.8.x
    Issue #3111658 by Deepthi kumari, kiamlaluno:...

Status: Fixed » Closed (fixed)

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