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.We noticed that the module was missing a hook associated with getting supported field types.
Have included a patch to fix this.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | field_type_hook-2390767-4.patch | 992 bytes | nnevill |
Comments
Comment #1
simonmc CreditAttribution: simonmc commentedChanging issue
Comment #2
nnevillHere is updated patch which adds new hook and fixes two other issues (with saving asset on "override asset" form).
Comment #3
imyaro CreditAttribution: imyaro as a volunteer and at DrupalJedi commentedI think it should be separated patches.
+ please describe your new hook in the asset.api.php file.
Comment #4
nnevillHere is a new version with hook described in asset.api.php file.
And yes, I agree separate patches should be done for other issues - will indeed create them recently.
Comment #5
imyaro CreditAttribution: imyaro as a volunteer and at DrupalJedi commentedOk for me.
Thanks.
Comment #6
IRuslan CreditAttribution: IRuslan as a volunteer and at DrupalJedi commentedHi guys,
it's great idea to complete this functionality,
but current hook in patch is not enough.
Because we also need to add a way to hook into handling of each type, see assets_get_placeholder().
Because of this, real overridden values will not be stored within tag.
Comment #8
IRuslan CreditAttribution: IRuslan as a volunteer and at DrupalJedi commentedGuys, thanks for work.
Accordingly to the last version of Asset module logic of this function was improved and the hook was added.
Pushed to dev version.