Use taxonomy image
Dudditz - June 30, 2008 - 05:39
| Project: | Advanced Forum |
| Version: | 6.x-2.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed |
Description
I understand you are working on a way to change the status of a post icon
but I was curious of another approach.
It would be nice to be able to assign a particular icon(s) to a particular forum.
Such as:
Ford forum would have Ford logo
Chevy forum would have Chevy logo
Toyota forum would have Toyota logo
etc
If I had a bit more php experience, I would attempt it myself.
| Attachment | Size |
|---|---|
| example.jpg | 94.72 KB |
#1
I 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
#2
thank u. we are waiting for this feature
#3
Nice feature. I think it'll be useful to much users. If i'll have a bit of spare time, i'll make a patch.
#4
I tried taxonomy image. But image is not showing in forum. What is the problem i dont know
#5
@bdtushar - Why would you think it would? This is an active feature request.
Michelle
#6
can 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?
#7
Here'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.
#8
Thanks, I'll have a look at it soon as I can.
Michelle
#9
I 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
#10
Retrieving 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
#11
Will revisit this in 2.x.
Michelle
#12
#13
Ok, 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
#14
Ok, I poked around a bit more and it seems pretty solid. Finally marking this fixed after a year. LOL
Michelle
#15
Automatically closed -- issue fixed for 2 weeks with no activity.