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.This patch adds a checkbox on admin/themes/settings to toggle display of taxonomy images, only if this module is installed and activated. I've also sent a patch to phptemplate theme engine that it uses this checkbox to fill a new variable usable for templates.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | system_module_taxonomy_images.patch | 507 bytes | grohk |
| #5 | phptemplate_taxonomy_images_4.5.0.patch | 1.13 KB | carlosparamio |
| #4 | phptemplate_taxonomy_images_0.patch | 1.31 KB | carlosparamio |
| system_taxonomy_images.patch | 727 bytes | Anonymous (not verified) | |
Comments
Comment #1
(not verified) CreditAttribution: commentedPHPtemplate patch that uses this new checkbox: http://drupal.org/node/14028
Comment #2
(not verified) CreditAttribution: commentedPatch for box_grey theme: http://drupal.org/node/14029
Comment #3
jxs2151 CreditAttribution: jxs2151 commentedI am unable to get this working after applying all three patches. When patching the phptemplate engine I get this error:
patching file phptemplate.engine
Hunk #1 succeeded at 73 (offset 1 line).
Hunk #2 FAILED at 239.
Hunk #3 succeeded at 237 (offset -23 lines).
1 out of 3 hunks FAILED -- saving rejects to file phptemplate.engine.rej
The checkbox for allowing Taxonomy images in posts appears, I created the category, uploaded the image. I can view the image by direct reference in a url, it just does not show up on the page.
Comment #4
carlosparamio CreditAttribution: carlosparamio commentedIt seems that phptemplate.engine file has changed a little. Here is a new patch.
When this patch was applied, then you can use the array $taxonomy_images[] in your node.tpl.php theme template to access to the image files for correspondant terms. For example, $taxonomy_images[0] to access to the image file for the first term. This is what box_grey patch does.
Comment #5
carlosparamio CreditAttribution: carlosparamio commentedAnd here is a patch for Drupal 4.5.0 release also.
Comment #6
grohk CreditAttribution: grohk commentedTrying to keep this idea alive by resectioning to the correct place (I think) and freshening the patch against the latest CVS system.module.
Please forgive me it this is inappropriate, but I think this patch is important for many users.
Comment #7
Bèr Kessels CreditAttribution: Bèr Kessels commented-1 I dislike any if(module_exist) for contrib modules in core. Core should not go calling /any/ contributions
Comment #8
killes@www.drop.org CreditAttribution: killes@www.drop.org commentedI agree with Ber.