This is a very useful module. I think it would be more user friendly if it welcomed the user back by real name instead of user name - if the real name module was in place.

CommentFileSizeAuthor
#7 persistent_login-realname_support-849300-7.patch629 bytesptmkenny
#3 persistent_login-849300-3.patch869 bytesgapple

Comments

achton’s picture

achton
Location Denmark
commented
Version: 5.x-1.4-beta4 » 6.x-1.x-dev

Any chance support for the RealName module is in the works?

Apparently, other modules can hook into RealName if they "[...] change their code to use theme('username', ...)" but I do not know how that interferes with exisiting code in Persistent Login.

Cheers,
Achton

YK85’s picture

YK85 commented

subscribing

gapple’s picture

gapple
he/they
Primary language English
commented
Status: Active » Needs review
StatusFileSize
new persistent_login-849300-3.patch869 bytes

This should be a quick fix. Instead of using the username directly, pass the account object through theme('username').

jwhat’s picture

jwhat commented

Can we make the Welcome Back message configurable by an administrator?

gapple’s picture

gapple
he/they
Primary language English
commented

@jwhat: yes, but open up a new feature request issue for that.

cgmonroe’s picture

cgmonroe commented

FYI - Realname support is also included with the patch in issue: #1203056: PL Bootstrap state notification

ptmkenny’s picture

ptmkenny commented
StatusFileSize
new persistent_login-realname_support-849300-7.patch629 bytes

Syntax has changed for D7. Attached is a new patch.

ptmkenny’s picture

ptmkenny commented
Version: 6.x-1.x-dev » 7.x-1.x-dev
neograph734’s picture

neograph734
he/him
Location Netherlands
commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

Great, works as expected.

jay.lee.bio’s picture

jay.lee.bio commented

#7 worked for me too. Thank you.

  • gapple committed 43eee9c on 7.x-1.x authored by ptmkenny 
    Issue #849300 by gapple, ptmkenny: Use real name for welcome back...
gapple’s picture

gapple
he/they
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed to 7.x-1.x-dev

Status: Fixed » Closed (fixed)

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