| • AGueyte | |
| Aug 22 2023, 3:28 PM |
| F38167372: image.png | |
| Oct 11 2023, 7:44 PM |
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/SpecialTranslatorSignup.php
Code Search Link
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| SpecialTranslatorSignup: Require named user | mediawiki/extensions/TranslationNotifications | master | +1 -1 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | kostajh | T344276 Should SpecialPages that return true for ::requireLogin allow access to temporary users? | |||
| Resolved | abi_ | T344725 Update requireLogin() for temp users on TranslationNotifications extension |
It might be helpful to keep this special page available to temporary users, to encourage them to translate.
Temporary users do not have access to email or preferences, but they could receive notifications via their talk page. It would be nice not to show the message about setting their email in their preferences:
This page uses user preferences to store the preferences, so we need to limit it to named users.
Change 977690 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):
[mediawiki/extensions/TranslationNotifications@master] SpecialTranslatorSignup: Require named user
Change 977690 merged by jenkins-bot:
[mediawiki/extensions/TranslationNotifications@master] SpecialTranslatorSignup: Require named user