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.
#1196828: add microdata support for image (core) added the theme_microdata_image_formatter()
because theme_image_formatter()
did not account for custom attributes.
However as from Drupal 7.14 this has been fixed as per #1329586: theme_image_formatter should pass along attributes.
Comment | File | Size | Author |
---|---|---|---|
#1 | 1875346-1-remove_theme_microdata_image_formatter.patch | 1.95 KB | fenstrat |
Comments
Comment #1
fenstratAttached removes everything connected with
theme_microdata_image_formatter()
. Could add a requirement to microdata.info to for minimum core 7.14 but I don't think it's worth it.Comment #2
jbrown CreditAttribution: jbrown commentedThis needs to be committed as it is preventing https://drupal.org/project/hires_images from working.
Comment #4
das-peter CreditAttribution: das-peter commentedWhy isn't it worth it?
Added.
Also added version hint on project page.
Comment #5
fenstrat@das-peter Thanks for committing. My theory was 7.14 is old enough not to worry about, but better safe than sorry.
Nice to see some movement in the microdata queue, thanks for your work.