Hello!
Please, help me with one thing I need to implement. I want to display in the block the thumbnail from ImageCache, not the teaser. Just title and the thumbnail. How I can modify the module to achieve that?
I want to implement that at http://uhaaa.com and to display under the article the related articles, but with thumbnails. I have ImageCache thumbs for every article.
Thanks!
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | similar.module.patch | 9.14 KB | jonhy81 |
| #2 | Similar Image.zip | 5.13 KB | jonhy81 |
Comments
Comment #1
cybershan commentedI got the same problem, seems this module just show the text links in a a block. can it show the image field in node. say similar image... ??
thanks.
Comment #2
jonhy81 commentedThat is a great idea and i had the same problem but i had modified the module to accept that. I could share my version here but i don't know how to do that. I 'll share here my version.
@deekayen: how can i share or contribute to this great module with my modification?
Comment #3
deekayen commentedYou can start by turning your zip into a patch.
Comment #4
jonhy81 commentedI had put in the attach the full module modified for testing. I'll try to made a patch.
Comment #5
jonhy81 commented@deekayen: i send a patch in attachment to this post. This patch is for the latest version for Drupal 6.
Comment #6
deekayen commentedYour code needs to conform to the coding standards. The Coder module can help you find most of the problems. Specifically, I see spacing problems and use of tabs.
Titles to form fields generally only capitalize the first letter, so Image Configuration needs a lowercase C.
Please specifically enumerate your #options keys in places like
array(t('No'), t('Yes')).$xdoesn't tell anyone what that variable contains, where it came from, or what it is useful for.Doing double quotes inside a single quoted query generally isn't pgsql supported. Gotta swap those.
Can you use the l() function instead of hard-coding href links?
Setting a height and width of 60 might be nice on your site, but that's definitely not something to hard-code for general use.
I also wonder if, in general, this functionality is better as a sub-module like similar_imagecache.module. It could still integrate with the forms by form_altering them, but then it could specifically require the imagecache module in its .info file. Maybe that's not possible with the integration you did in the last part of the patch, but maybe that can be resolved by adding a hook implementation.
Comment #7
jonhy81 commented@deekayen: Thanks for the review. I'll try to follow your idea of submodule.
Comment #8
jordojuice commentedImages can be added to the block via views with the newest versions of Similar entries. No further development is being done on earlier versions of the module, thus this feature will not otherwise be implemented.