Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Found a couple of issues with the image.module hooks_ documentation.
1. Documentation for hook_image_style_flush() makes reference to a non-existent UI for flushing a style.
2. Documentation for hook_image_style_alter() argument should be passed by reference.
3. Example function for hook_image_style_alter() is missing code. The example needs to include an 'effect callback' key in the array. It is required, and without it Drupal throws errors. The php docs for the function should probably clarify this as well.
Comment | File | Size | Author |
---|---|---|---|
#1 | 658056-theunraveler.patch | 1.45 KB | theunraveler |
Comments
Comment #1
theunraveler CreditAttribution: theunraveler commentedHere's a patch.
Regarding issue #1, I took the reference to the nonexistent UI out of the documentation, but while we were working on this issue, we came up with some good arguments to reintroduce the UI for flushing Image presets. See #700696: Add 'flush' back into image module for more info.
Comment #2
eojthebraveI like it. Thanks!
Comment #3
webchickNice clean-up, thanks! Committed to HEAD.