Related to http://drupal.org/node/155254.

Add an optional $link param to theme_username to control whether name should be displayed as a link or just text. This will allow us to use theme('username', $object, FALSE) in core if we want to just show the textual name.

CommentFileSizeAuthor
username.link_.patch1.52 KBRobRoy
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Status: Needs review » Needs work

no longer applies. Small enough to leave in D6 issue queue though I think.

dvessel’s picture

dvessel CreditAttribution: dvessel commented
Status: Needs work » Postponed (maintainer needs more info)

It gets linked based on permission to view profiles.

In what situation would you force it as text even for users with view permissions on profiles.

geodaniel’s picture

geodaniel CreditAttribution: geodaniel commented
Version: 6.x-dev » 7.x-dev
Status: Postponed (maintainer needs more info) » Needs work

The situation would be, for example, if you wanted to theme a user's username as part of a page title (e.g in user module, contact module, etc). Instead of showing their username, you may want to theme it to display information from their profile data instead, and without this patch you'd be forced to use the generated HTML link.

geodaniel’s picture

geodaniel CreditAttribution: geodaniel commented

See also #192056: User's raw login name should not be output directly

Arto’s picture

Arto CreditAttribution: Arto commented

This would be a useful feature. Subscribing.

geodaniel’s picture

geodaniel CreditAttribution: geodaniel commented
Status: Needs work » Closed (duplicate)

I think the other way (#192056) of doing this would be a better route to go in the long-run, so I'm going to mark this as a duplicate.