The attached patch makes it possible to link directly to a user's Content Profile from the RealName. It creates a new option "Link to content profile" on the RealName administration page, which defaults to off. Unless this option is set, everything should work as before. The order of preference for the link destination is homepage, Content Profile, user page.

I rewrote most of realname_username(), so this patch should be tested extensively before being used in production. Feedback would be appreciated.

Note: this patch depends on #1635842: realname_content_profile_get_types() should return types instead of names also being applied.

CommentFileSizeAuthor
#3 realname-link_to_content_profile-1649690-3.patch18.04 KBChrisLaFrancis
#2 realname-link_to_content_profile-1649690-2.patch15.99 KBChrisLaFrancis
#1 realname-link_to_content_profile-1649690-1.patch16.06 KBChrisLaFrancis
realname-link_to_content_profile.patch16.06 KBChrisLaFrancis

Comments

ChrisLaFrancis’s picture

ChrisLaFrancis commented
StatusFileSize
new realname-link_to_content_profile-1649690-1.patch16.06 KB

Removed a couple of extraneous newlines.

ChrisLaFrancis’s picture

ChrisLaFrancis commented
StatusFileSize
new realname-link_to_content_profile-1649690-2.patch15.99 KB

Changed a variable name.

ChrisLaFrancis’s picture

ChrisLaFrancis commented
StatusFileSize
new realname-link_to_content_profile-1649690-3.patch18.04 KB

Corrected an error in the logic for whether an access check was performed on the Content Profile when returning the path. Also, created a separate function [realname_get_name()] for the code from realname_username() that actually determined what value to use as the user's name, as this could be useful in other places.

hass’s picture

hass commented
Issue summary: View changes
Status: Needs review » Closed (outdated)