While working on http://drupal.org/project/commerce_price_history, I found out the hard way (well, not really that hard) that hook_field_purge_field_instance() is actuallly hook_field_purge_instance().

Relevant part from field.crud.inc:

function field_purge_instance($instance) {
  ...
  // Invoke external hooks after the cache is cleared for API consistency.
  module_invoke_all('field_purge_instance', $instance);
}

Confusing part in field.api.php:

function hook_field_purge_field_instance($instance) {
  ...
}

Patch coming up in #1, `nuff said :)

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

amateescu’s picture

Assigned: amateescu » Unassigned
Status: Active » Needs review
FileSize
500 bytes

And the patch.

Deciphered’s picture

Status: Needs review » Reviewed & tested by the community

Patch is sane, research checks out.

Marking as RTBC.

xjm’s picture

I grepped and confirmed this is the only reference to the fake hook name:

[milankovitch:drupal | Tue 00:51:09] $ grep -r "field_purge_field_instance" *
modules/field/field.api.php:function hook_field_purge_field_instance($instance) {
catch’s picture

Version: 8.x-dev » 7.x-dev

Committed and pushed to 8.x. Moving to 7.x

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed to 7.x. Thanks!

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