In the vud_node module's hook_link() implementation, there is a call to an undefined theme function "theme_vud_votes". This leads to an empty list item which produces empty bullets in the UI. See the attached screenshot for the markup that is produced.

CommentFileSizeAuthor
empty-list-item.png168.11 KBcpliakas

Comments

marvil07’s picture

marvil07 commented
Status: Active » Postponed (maintainer needs more info)

Actually, the call follow the pattern theme('vud_votes', ...). The theme hook defines the real used function, vud_votes_proxy().

So, the problem should be another.

@cpliakas: Can you please provide details on what widget you are using and in what conditions(maybe number of votes) you can reproduce it?

cpliakas’s picture

cpliakas commented

marvil07,

Thanks for the response, and I missed the proxy function. I am using both the "Default" and the "Up and down" widgets, both yield the same result. This bug exists both in the 6.x-3.x-alpha1 version and in HEAD of the 6.x-3.x branch. As a temporary fix, I had to add the following code in a hook_link() implementation.


/**
 * Implementation of hook_node_link_alter().
 */
function mymodule_link_alter(&$links, $node) {
  if (empty($links['vud_node_votes_count']['title'])) {
    unset($links['vud_node_votes_count']);
  }
}

I am using Commons, so I will do some more research as to whether or not something is conflicting in the distribution. However my feeling is that it doesn't because Vote Up/Down isn't in the core distribution.

Thanks,
Chris

marvil07’s picture

marvil07 commented
Issue summary: View changes
Status: Postponed (maintainer needs more info) » Closed (outdated)

Closing D6 issues, D6 releases are not supported now.
Please reopen as D7 if needed.