119 $user = user_load($node->uid);
120 return theme('blockify_node', $user, $node);

Potential security risk to say $user when you don't mean the global $user. It's good code style to say $author instead. The theme function should be changed too.

Comments

psynaptic’s picture

Status: Active » Needs work

Sure, or $account. No patch? Shame on you.

psynaptic’s picture

Status: Needs work » Fixed

Committed as $account to 6.x-1.x.

joachim’s picture

> No patch? Shame on you.

I was at work, and took a quick look at this because it *might* be useful in due course...

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.