Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Implement basic tests for all field_attach hooks. I just discovered this little gem which has been around since 12/29:
foreach (module_implements('field_attach_update') as $module) {
$function = $module . '_field_attach_update';
$function($output, $obj_type, $object);
}
Notice the incorrect $output.
Comments
Comment #1
bjaspan CreditAttribution: bjaspan commentedThe approach for doing this is laid out in #489438: hook_field_create_field() is missing. It is now a fairly easy task, given field_test_memorize().
Comment #2
catchStatus change after: #520620: Delete revision and prepare translation field hooks are called incorrectly (or not at all)
Comment #3
sun.core CreditAttribution: sun.core commentedIs this still the case? Aren't field_attach* hooks already tested by plenty of code in the meantime?
At least not critical, I think.
Comment #4
Anonymous (not verified) CreditAttribution: Anonymous commentedDo we still need to write tests for field_attach* hooks?
Comment #12
pameeela CreditAttribution: pameeela commentedAs part of the Bug smash initiative, we are triaging issues that are marked 'Postponed (maintainer needs more info)'. Based on this change record, we believe this issue is no longer relevant so I am marking it 'Closed (outdated)'.
Comment #13
pameeela CreditAttribution: pameeela commented