Token for "First name or username" [#1849112]

There are situations such as in Commons email notifications where it would be ideal to address the user by her first name, but be able to fall back on the username if the first name field isn't populated or isn't defined on the site.

This token should probably live in Commons_Profile_base, but could potentially be sufficient in Commons Activity streams.

Comment	File	Size	Author
#4	1849112.commons_profile_base.short_name_token.patch	1.75 KB	jpontani
#2	commons_add-short-name-token-1849112.patch	1.68 KB	pierrepaul

Comments

Comment #1

pierrepaul commented 19 December 2012 at 21:05

Assigned:

Unassigned

» pierrepaul

Comment #2

pierrepaul commented 19 December 2012 at 23:17

Status:

Active

» Needs review

Status	File	Size
new	commons_add-short-name-token-1849112.patch	1.68 KB

I'm not sure I got the patch right...

Comment #3

ezra-g commented 16 February 2013 at 00:40

Priority:

Major

» Normal

Moving to "normal" priority sine this is not a release blocker.

Comment #4

jpontani commented 16 February 2013 at 01:02

Status	File	Size
new	1849112.commons_profile_base.short_name_token.patch	1.75 KB

This patch doesn't actually apply to commons_profile_base, it applies to a full build of commons (using drush make with --working-copy).

Re-rolled against commons_profile_base repo, and added a fix, since the original patch only checks for the exact token of "[user:short-name]", whereas tokens could exist such as "[message:user:short-name]" and then not be replaced properly.