In both the messages and user list, chatroom generates its own text and links for users. Is there any reason why theme('username', ...) was not used?

I have a site that needs to override theme('username',...) and I can't figure out how to override the text that chatroom displays.

Thanks for any help.

Comments

Dave Cohen’s picture

no? nothing?

WoozyDuck’s picture

I have the issue, this module uses JS to get username so would be great if someone helps?

Anonymous’s picture

Version: 6.x-1.0-beta2 » 6.x-1.0-beta3
Assigned: Unassigned »
Anonymous’s picture

Status: Active » Needs review

seems this was half done. i've committed fixes to finish the job. the site-wide user list is broken right now anyway...

Anonymous’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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

ball.in.th’s picture

Version: 6.x-1.0-beta3 » 6.x-2.9
Status: Closed (fixed) » Needs review

Hello,

I've just installed Chat Room 6.x-2.9, the usernames are still not displayed through the theme function. Looking at the code in theme_chatroom_message(), I can still see l() instead of theme('username', ...) .

Changing it to something like this seems to work:

    //$username = l($message->name, 'user/'. $message->uid);
    $username = theme('username', $message);
Anonymous’s picture

Status: Needs review » Fixed

thanks ball.in.th. i've committed the fix from #7, will roll a 6.x-2.10 release later today.

Status: Fixed » Closed (fixed)

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