Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
As per #1628044-31: Implement magic for the table of attachments on issues -- instead of only passing $entity as context to hook_file_metadata_table_items_alter()
why don't we pass all the context we've got, just like we do for hook_file_metadata_table_rows_alter()
? Obviously, we wouldn't need $items inside $context, but it seems like all the rest of these might be useful/necessary in certain alter cases, and it costs us nothing to pass them all at once:
* - field: The field definition array.
* - instance: The field instance definition array.
* - entity: An object representing the entity the file field is attachted to.
* - entity_type: String with the type of entity the field is attached to.
Any objections?
Thanks,
-Derek
Comments
Comment #1
jthorson CreditAttribution: jthorson commentedNo objections ... adding now.
Comment #2
jthorson CreditAttribution: jthorson commentedAdded in #1867102-2: See if we can avoid a whole separate hook invocation for the table for hidden files.
Comment #3
jthorson CreditAttribution: jthorson commentedCommitted in 25ca17b.
Comment #5
jthorson CreditAttribution: jthorson commentedModule name change.