While I was reviewing the patch for #2048395-14: Several field hooks are in the wrong topics/groups, I thought "Hmmm... what about Drupal 8?". It suffers from the same problem as Drupal 7 -- there are a few hooks in the core/modules/field/field.api.php file that are not properly grouped into the groups/topics.

This can be fixed by using @ingroup or @addtogroup -- see #2048395-14: Several field hooks are in the wrong topics/groups for a list of the relevant topics to use, and https://drupal.org/node/1354#defgroup for information on the syntax of ingroup/addtogroup.

The hooks that need attention in this file:
hook_field_info_max_weight - needs to be in field_info topic

OK, it's just the one hook -- just needs an @ingroup field_info added to that hook documentation. Definitely a Novice project!

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

blueminds’s picture

Status: Active » Needs review
FileSize
690 bytes

please see the patch.

jhodgdon’s picture

Since there is only one hook that needs adding to this topic, it would be a lot simpler just to add one line saying

 * @ingroup field_info

to the doc block of this one hook, rather than an @addtogroup?

blueminds’s picture

Thanks for feedback, updated the patch.

jhodgdon’s picture

Status: Needs review » Reviewed & tested by the community

Thanks!

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

  • Commit cfbbef3 on 8.x by webchick:
    Issue #2260035 by blueminds | jhodgdon: Hook_field_info_max_weight in...

Status: Fixed » Closed (fixed)

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