Closed (fixed)
Project:
Blue Zinfandel
Version:
5.x-1.x-dev
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
10 May 2007 at 23:29 UTC
Updated:
26 May 2007 at 02:00 UTC
This is a nice theme, very clear for new users and administration. However, I've found that the blue_zinfandel_links() function in template.php which overrides theme_links() completely empties the $links array for any node for which the viewer has comment access. As a result this kills links to stuff like 'Printer Friendly Version', 'Contact Author' etc. Same goes for links related to polls in the poll block.
As a workaround I have just commented this function out.
Comments
Comment #1
webchickWell, that's a bug. ;)
Fixed in template.php 1.4.
Comment #2
tinthing commentedFast response! Well, that revived the other links, but the 'Add Comment' link only shows on the front page teaser view (sorry, not familiar with the correct drupalisms yet), not when the node is viewed in full, and never for polls (probably other node types too) with their own blocks. Perhaps the mods to the links array in blue_zinfandel_links() should only be performed for a teaser view (dunno how you'd test for that)?
Comment #3
webchickHm. Interesting... I can't fix it that way, because I don't have the $node in theme_links. :(
I'll try a node-story.tpl.php way instead.
Comment #4
tinthing commentedHow about this - remove blue_zinfandel_links() from template.php, and amend node.tpl.php like this...
Replace
with
and then
with
Works for me.
Comment #5
webchickHm. Are you sure commenting out that code fixes the poll block? I can't get links to show up there no matter what I do... also, the CSS is hosing the location of the radio buttons. So I'll go make a separate "poll module needs love" bug.
In the meantime though, I found a much better way to change those links... I'm doing it now in _phptemplate_variables so it only affects node links rather than firing every single time a link list is displayed. D'oh!
Comment #6
webchickOops. we cross-posted. ;)
That && $comment_link is a good fix... but not really related to the current bug. I don't mind the links showing up in both teaser and full view; I just don't want two "Add comment" links... that's what that theme_links code was doing.
Comment #7
(not verified) commented