Page MenuHomePhabricator

Consider storing real name in user properties
Open, Needs TriagePublic

Description

In my understanding the user table is mostly there to store user authentication data. But then it also stores a users email and a users real name (email is a complicated thing and can't be tackled that easy).

The users real name is something a user can optionally set as preference (when allowed). And user properties seems to be the more natural place where that belongs.

It would also allow us to convert quite a lot of code to use UserIdentity instead of full user objects, since we no longer rely on User::getRealName().

Details

Event Timeline

Change 806239 had a related patch set uploaded (by Zabe; author: Zabe):

[mediawiki/core@master] Migrate the users real name to be a user property

https://gerrit.wikimedia.org/r/806239

Not sure the permissions/grants for managing user preferences are ideal for reading/writing a user's real name.

Real name should intend to be nothing sensitive, cf T154177: Change "Real name" to "Display name".

Agreed, before we fully blank it, it was fully visible in WMCS replicas.

Change #806239 abandoned by Hashar:

[mediawiki/core@master] Migrate the users real name to be a user property

https://gerrit.wikimedia.org/r/806239

Change #806239 restored by Thcipriani:

[mediawiki/core@master] Migrate the users real name to be a user property

https://gerrit.wikimedia.org/r/806239

Pppery renamed this task from Consider storing a users real name in user properties to Consider storing real name in user properties.Tue, Mar 31, 1:56 AM
Pppery edited projects, added Patch-Needs-Improvement; removed Patch-For-Review.

Change #806239 abandoned by Zabe:

[mediawiki/core@master] Migrate the users real name to be a user property

Reason:

currently not working on this

https://gerrit.wikimedia.org/r/806239