Closed (fixed)
Project:
Advanced Forum
Version:
6.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
30 Jun 2008 at 05:39 UTC
Updated:
29 Jul 2009 at 18:00 UTC
Jump to comment: Most recent file
Comments
Comment #1
michelleI was going to won't fix this since this is really something that would need to be done in your theme. But it occured to me that I could probably hook into the taxonomy image module and do something there. This isn't anything I'll get to in the near future but I'll put it on the to do list.
Michelle
Comment #2
bdtushar commentedthank u. we are waiting for this feature
Comment #3
psicomante commentedNice feature. I think it'll be useful to much users. If i'll have a bit of spare time, i'll make a patch.
Comment #4
bdtushar commentedI tried taxonomy image. But image is not showing in forum. What is the problem i dont know
Comment #5
michelle@bdtushar - Why would you think it would? This is an active feature request.
Michelle
Comment #6
chasz commentedcan u add a image URL field in the create forum section?
that should be an easy way?? can i add a cck field into forum?
Comment #7
johnblade commentedHere's a patch which applied to advanced_forum-6.x-1.0-alpha13.tar.gz will add Taxonomy Image support.
You need to create a valid Taxonomy Image configuration as per (http://drupal.org/node/101682).
When you have a valid Taxonomy Image per Taxonomy, it will replace the default Post Icon with the Taxonomy Image.
It's not perfect though, as it doesn't allow for icons with 'new posts' that have Taxonomy Images. Probably best to change your theme to have a bold Topic or something like that instead.
Comment #8
michelleThanks, I'll have a look at it soon as I can.
Michelle
Comment #9
michelleI had a look at it and I don't get why you go through all the work to find the vid when you only use the already existing tid? This also, as you mentioned on IRC, doesn't address the main forum listing images, which is what this issue is about.
I'll come back to this at some point but probably not for this next alpha.
Thanks,
Michelle
Comment #10
johnblade commentedRetrieving the TID is necessary to send the taxonomy_node_get_terms function a Node array that has the correct TID.
The forum_get_topics doesn't retrieve the TID information, and would require to either override that function or change it in Forum Core.
I've started looking into adding it for the forum icons but I'm having trouble retrieving all the terms for each Forum. Anyhow I'll keep working on it.
Also checkout an example of what the above patch can do: http://lansmash.com/forum
Comment #11
michelleWill revisit this in 2.x.
Michelle
Comment #12
michelleComment #13
michelleOk, commited something that works with minimal testing. I did not use the patch in #7. Forgot there even was a patch on this issue and I still don't see why it goes through all that work. At any rate, the feature is there but I didn't thoroughly test it including checking what happens if you add an image to a container and stuff like that. So leaving it open for the moment until I'm sure it's solid.
Michelle
Comment #14
michelleOk, I poked around a bit more and it seems pretty solid. Finally marking this fixed after a year. LOL
Michelle