Page MenuHomePhabricator

Update requireLogin() for temp users on MobileFrontend 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/specials/SpecialMobileEditWatchlist.php
Code Search Link

Event Timeline

On the long term we're hoping to remove this page and replace it with Special:ReadingList (see T363032). If we can do that, we can decline this and save us the effort. If we can't, then we need to make some changes here. @ovasileva what do you want to do?

Jdlrobson changed the task status from Open to Stalled.May 23 2024, 10:41 PM
Jdlrobson moved this task from Incoming to Groomed on the Web-Team-Backlog-Archived board.

This task is still relevant, but it's not clear what to do here - so the task needs to be fleshed out.

Special:EditWatchlist is not available to temporary users from the mobile main menu:

Screenshot 2024-07-26 at 9.23.25 AM.png (2,276×1,420 px, 224 KB)

Currently visiting https://test.m.wikipedia.org/w/index.php?title=Special:EditWatchlist as a temporary user redirects me to the login page so this appears to be working as expected.