Page MenuHomePhabricator

Add accessor for current user to EchoEventPresentationModel
Closed, ResolvedPublic


We keep finding cases where the recipient's gender is needed (primarily for French messages). For Thanks notifications the recipient is also in the extra event data, but for mentions it's not. There should be an accessor exposing $this->user (or the recipient for gender purposes, if we want to support that being different from the user for permission checks purposes for some reason).


Related Gerrit Patches:

Event Timeline

Catrope created this task.Nov 7 2015, 1:28 AM
Catrope raised the priority of this task from to Needs Triage.
Catrope updated the task description. (Show Details)
Catrope added a subscriber: Catrope.
Restricted Application added subscribers: StudiesWorld, Aklapper. · View Herald TranscriptNov 7 2015, 1:28 AM
Catrope assigned this task to Legoktm.Nov 7 2015, 1:30 AM

Change 252855 had a related patch set uploaded (by Legoktm):
EventPresentationModel: Add accessor for current user's username

Change 252855 merged by jenkins-bot:
EventPresentationModel: Add accessor for current user's username

Legoktm closed this task as Resolved.Nov 13 2015, 12:59 AM

Checked in betalabs with uselang ru, fr.