Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.Drupal_alter is called on a function ending in the word 'alter'.
Because of this the alter hook is confusingly called hook_fillpdf_merge_fields_alter_alter instead of hook_fillpdf_merge_fields_alter, as one would expect.
Problem also mentioned in #1437620: Document hooks in fillpdf.api.php
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | fillpdf-rename-alter-hook-2140435-4.patch | 473 bytes | oenie |
| #2 | fillpdf-rename-alter-hook-2140435-2.patch | 526 bytes | oenie |
Comments
Comment #1
oenie commentedComment #2
oenie commentedComment #3
oenie commentedDue to backwards compatibility issues could only rename this on version 7.x-2.x (see #1437620: Document hooks in fillpdf.api.php , comment #13)
Switching branch + new patch on that branch
Comment #4
wizonesolutionsI'm not continuing on 7.x-2.x, so I'm simply going to add the more intuitive hook name to the module as well.
Comment #6
wizonesolutions