Page MenuHomePhabricator

Update requireLogin() for temp users on Echo extension
Closed, DeclinedPublic

Description

In T344276, we want to make sure the SpecialPage::requireLogin() function is used in accordance with the introduction of temporary users. As for now, we want to treat temporary users just like anonymous or IP users.

Request:
Should requireLogin() be available to temporary users?
If yes, no update is needed.
If not, requireNamedUser must return true instead.

Notes:
RequireLogin was found in the following files:
includes/Special/SpecialNotifications.php
includes/Special/SpecialNotificationsMarkRead.php
Code Search Link

Event Timeline

Urbanecm_WMF subscribed.

Thanks for filling the task! According to T330509: [IP Masking] Make Echo Notifications available to temporary users, we want notifications to be available for temporary accounts. Closing the task as Declined, as no updates seem to be needed as part of it.