Page MenuHomePhabricator

User::isLoggedIn() is confusing
Closed, ResolvedPublic

Description

/**
 * Get whether the user is logged in
 * @return bool
 */
public function isLoggedIn() {
        return $this->isRegistered();
}

isLoggedIn being the same as isRegistered does not make any sense semantically; it dates from when a User object was only part of the current request, rather than a real object that can occur in other circumstances.

Details

SubjectRepoBranchLines +/-
mediawiki/skins/BlueSkymaster+0 -0
mediawiki/extensions/DisplayTitleREL1_36+2 -2
mediawiki/extensions/DisplayTitleREL1_35+2 -2
mediawiki/extensions/URNamesmaster+1 -1
mediawiki/extensions/OnlineStatusBarmaster+2 -2
mediawiki/extensions/Commentboxmaster+3 -3
mediawiki/extensions/OfflineImportLexiconmaster+1 -1
mediawiki/extensions/JsonDatamaster+2 -2
mediawiki/extensions/ContributionScoresmaster+2 -2
mediawiki/extensions/ConfirmAccountmaster+2 -2
mediawiki/extensions/CollapsibleVectormaster+1 -1
mediawiki/extensions/BreadCrumbs2master+1 -1
mediawiki/extensions/AccountInfomaster+1 -1
mediawiki/extensions/AutoCreateCategoryPagesmaster+1 -1
mediawiki/extensions/TwitterLoginmaster+3 -3
mediawiki/extensions/PageFormsmaster+5 -11
mediawiki/extensions/HSTSmaster+4 -1
mediawiki/extensions/ImportFreeImagesmaster+2 -2
mediawiki/extensions/Inboxmaster+2 -2
mediawiki/extensions/MultiLanguageManagermaster+3 -3
mediawiki/extensions/NetworkAuthmaster+1 -1
mediawiki/extensions/NewUserActionsmaster+4 -1
mediawiki/coremaster+7 -0
mediawiki/extensions/DisplayTitlemaster+2 -2
mediawiki/extensions/OpenIDmaster+2 -2
mediawiki/extensions/Favoritesmaster+2 -2
mediawiki/extensions/OnlineStatusmaster+2 -2
mediawiki/extensions/PronunciationRecordingmaster+2 -2
mediawiki/extensions/SecureSessionsmaster+1 -1
mediawiki/extensions/SkinPerNamespacemaster+2 -2
mediawiki/extensions/SocialLoginmaster+4 -4
mediawiki/extensions/SSLClientAuthenticationmaster+2 -2
mediawiki/extensions/Surveymaster+1 -1
mediawiki/extensions/TwnMainPagemaster+3 -3
mediawiki/extensions/WebChatmaster+2 -2
mediawiki/skins/Metrolookmaster+1 -1
mediawiki/skins/Ponchomaster+5 -2
mediawiki/extensions/BlueSpicePageAssignmentsmaster+1 -1
mediawiki/extensions/BlueSpiceDashboardsmaster+1 -1
mediawiki/extensions/BlueSpiceRemindermaster+2 -2
mediawiki/extensions/BlueSpicePrivacymaster+2 -2
mediawiki/extensions/BlueSpiceExpirymaster+2 -2
mediawiki/extensions/BlueSpiceExtendedSearchmaster+3 -3
mediawiki/skins/BlueSpiceCalummamaster+10 -10
mediawiki/extensions/EditSubpagesmaster+4 -1
mediawiki/extensions/CreateAPagemaster+3 -3
mediawiki/extensions/SportsTeamsmaster+20 -20
mediawiki/extensions/FanBoxesmaster+9 -9
mediawiki/extensions/Commentsmaster+5 -5
mediawiki/extensions/MiniInvitemaster+3 -3
mediawiki/extensions/PollNYmaster+4 -4
mediawiki/extensions/QuizGamemaster+8 -8
mediawiki/extensions/UserStatusmaster+8 -8
mediawiki/extensions/Challengemaster+4 -4
mediawiki/extensions/EditSimilarmaster+2 -2
mediawiki/extensions/PictureGamemaster+13 -13
mediawiki/extensions/WikiForummaster+4 -4
mediawiki/extensions/LinkFiltermaster+1 -1
mediawiki/extensions/OOJsUIAjaxLoginmaster+4 -1
mediawiki/extensions/Draftsmaster+2 -2
mediawiki/extensions/NewsBoxmaster+2 -2
mediawiki/skins/Tempomaster+2 -2
mediawiki/skins/Nimbusmaster+3 -3
mediawiki/skins/Anisamaster+1 -1
mediawiki/extensions/WikiTextLoggedInOutmaster+2 -2
mediawiki/extensions/VoteNYmaster+1 -1
mediawiki/extensions/PrivateDomainsmaster+2 -2
mediawiki/extensions/SocialProfilemaster+27 -27
mediawiki/extensions/StickToThatLanguagemaster+1 -1
mediawiki/coremaster+133 -133
mediawiki/extensions/MobileFrontendmaster+1 -1
mediawiki/coreREL1_35+5 -1
mediawiki/skins/Vectormaster+9 -9
mediawiki/extensions/UniversalLanguageSelectormaster+2 -2
mediawiki/extensions/Wikibasemaster+5 -5
mediawiki/extensions/GlobalWatchlistmaster+5 -5
mediawiki/extensions/NewUserMessagemaster+2 -2
mediawiki/extensions/DismissableSiteNoticemaster+2 -2
mediawiki/extensions/WikibaseQualityConstraintsmaster+2 -2
mediawiki/extensions/Translatemaster+3 -3
mediawiki/extensions/VisualEditormaster+3 -3
mediawiki/extensions/GlobalCssJsmaster+3 -3
mediawiki/extensions/LdapAuthenticationmaster+3 -3
mediawiki/extensions/ContactPagemaster+3 -3
mediawiki/extensions/MobileFrontendmaster+4 -4
mediawiki/extensions/GrowthExperimentsmaster+4 -4
mediawiki/extensions/ContentTranslationmaster+5 -5
mediawiki/extensions/Flowmaster+2 -2
mediawiki/extensions/LoginNotifymaster+1 -1
mediawiki/extensions/BetaFeaturesmaster+1 -1
mediawiki/extensions/CentralNoticemaster+5 -5
mediawiki/skins/MinervaNeuemaster+9 -9
mediawiki/extensions/Echomaster+4 -4
mediawiki/extensions/Newslettermaster+4 -4
mediawiki/extensions/CentralAuthmaster+6 -6
mediawiki/extensions/AbuseFiltermaster+1 -1
mediawiki/extensions/OAuthmaster+4 -4
mediawiki/extensions/Thanksmaster+3 -3
mediawiki/extensions/Collectionmaster+2 -2
mediawiki/extensions/PageTriagemaster+2 -2
mediawiki/extensions/SecurePollmaster+2 -2
mediawiki/extensions/WikimediaMaintenancemaster+2 -2
mediawiki/extensions/WikimediaMessagesmaster+3 -3
mediawiki/extensions/FlaggedRevsmaster+1 -1
mediawiki/extensions/MultimediaViewermaster+1 -1
mediawiki/extensions/WikimediaEventsmaster+1 -1
mediawiki/extensions/OpenStackManagermaster+1 -1
mediawiki/extensions/WikiLovemaster+1 -1
mediawiki/extensions/WebAuthnmaster+1 -1
mediawiki/extensions/WikimediaIncubatormaster+2 -2
mediawiki/skins/Nostalgiamaster+1 -1
mediawiki/skins/WikimediaApiPortalmaster+1 -1
mediawiki/skins/Timelessmaster+1 -1
mediawiki/coremaster+5 -1
Show related patches Customize query in gerrit

Related Objects

Mentioned In
rSPIVcfee790ea755: isLogged in was removed in favor of isRegistered
rEDPT639a04ef89c0: Use User->isRegistered(), not deprecated isLoggedIn()
rEDPTaaa88bc19cdc: Use User->isRegistered(), not deprecated isLoggedIn()
rEOILa03782b5f79e: Use User->isRegistered(), not deprecated isLoggedIn()
rENAUf966a3a0e3e9: Use User->isRegistered(), not deprecated isLoggedIn()
rECNSbb3429ecffd7: Use User->isRegistered(), not deprecated isLoggedIn()
rEACP19452131d94c: Use User->isRegistered(), not deprecated isLoggedIn()
rECAC96439acd49e5: Use User->isRegistered(), not deprecated isLoggedIn()
rENUACb000125d5442: Use User->isRegistered(), not deprecated isLoggedIn()
rEIBXbe6413f7b9d9: Use User->isRegistered(), not deprecated isLoggedIn()
rEDPT12c445d758cf: Use User->isRegistered(), not deprecated isLoggedIn()
rEPRRdd1b668f8a6f: Use User->isRegistered(), not deprecated isLoggedIn()
rMSPOd9842623781c: Use User->isRegistered(), not deprecated isLoggedIn()
rEBUFb0afe78749af: Use User->isRegistered(), not deprecated isLoggedIn()
rEBSESdce991efde9e: Use User->isRegistered(), not deprecated isLoggedIn()
rEBUC4aae71d6a608: Use User->isRegistered(), not deprecated isLoggedIn()
rEBUHdda8f6165492: Use User->isRegistered(), not deprecated isLoggedIn()
rMSBCa8440481d5b1: Use User->isRegistered(), not deprecated isLoggedIn()
rEBTDd8ed31fa735d: Use User->isRegistered(), not deprecated isLoggedIn()
rEOAL842a51ada7f5: Use User->isRegistered(), not deprecated isLoggedIn()
rEPGA852654d5ace9: Use User->isRegistered(), not deprecated isLoggedIn()
rECPG5fa8c81e03a2: Use User->isRegistered(), not deprecated isLoggedIn()
rEPRDbfe1aaacc153: Use User->isRegistered(), not deprecated isLoggedIn()
rENBX0be1cc468a89: Use User->isRegistered(), not deprecated isLoggedIn()
rSANIfa6646a8a45e: Use User->isRegistered(), not deprecated isLoggedIn()
rESPR726a4e9e34b3: Use User->isRegistered(), not deprecated isLoggedIn()
rEGLWf6e5c240ffb0: Use User->isRegistered(), not deprecated isLoggedIn()
rECPAa447576202f7: Use User->isRegistered(), not deprecated isLoggedIn()
rEWAT6d2024f17f14: Use User->isRegistered(), not deprecated isLoggedIn()
Mentioned Here
rEMFR8632bf0305e0: Use User->isRegistered(), not deprecated isLoggedIn()
rMW388d95d7b8a5: De-cruftification continues. * Replacing Namespace:: functions with NS_…
rMWf7201e3b9b81: Convert WatchedItem and friends to UserIdentity

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes

Change 685929 merged by jenkins-bot:

[mediawiki/skins/Anisa@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685931 merged by jenkins-bot:

[mediawiki/skins/Nimbus@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685934 merged by jenkins-bot:

[mediawiki/skins/Tempo@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685858 merged by jenkins-bot:

[mediawiki/extensions/NewsBox@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685841 merged by jenkins-bot:

[mediawiki/extensions/Drafts@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685854 merged by jenkins-bot:

[mediawiki/extensions/LinkFilter@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685818 merged by jenkins-bot:

[mediawiki/extensions/WikiForum@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685815 merged by jenkins-bot:

[mediawiki/extensions/PictureGame@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685803 merged by jenkins-bot:

[mediawiki/extensions/EditSimilar@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685801 merged by jenkins-bot:

[mediawiki/extensions/Challenge@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685781 merged by jenkins-bot:

[mediawiki/extensions/UserStatus@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685780 merged by jenkins-bot:

[mediawiki/extensions/QuizGame@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685777 merged by jenkins-bot:

[mediawiki/extensions/PollNY@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685769 merged by jenkins-bot:

[mediawiki/extensions/MiniInvite@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685767 merged by jenkins-bot:

[mediawiki/extensions/Comments@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685764 merged by jenkins-bot:

[mediawiki/extensions/FanBoxes@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685761 merged by jenkins-bot:

[mediawiki/extensions/SportsTeams@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685768 merged by Jack Phoenix:

[mediawiki/extensions/CreateAPage@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685844 merged by jenkins-bot:

[mediawiki/extensions/EditSubpages@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

We're technically ready to hard-deprecate; is this too soon, do people feel?

I have now uploaded patches for all remaining repositories, so I think we can proceed with the hard-deprecation.

Change 685763 merged by jenkins-bot:

[mediawiki/skins/BlueSpiceCalumma@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685765 merged by jenkins-bot:

[mediawiki/extensions/BlueSpiceExtendedSearch@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685791 merged by jenkins-bot:

[mediawiki/extensions/BlueSpiceExpiry@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685793 merged by jenkins-bot:

[mediawiki/extensions/BlueSpicePrivacy@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685798 merged by jenkins-bot:

[mediawiki/extensions/BlueSpiceReminder@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685829 merged by jenkins-bot:

[mediawiki/extensions/BlueSpiceDashboards@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685830 merged by jenkins-bot:

[mediawiki/extensions/BlueSpicePageAssignments@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685932 merged by jenkins-bot:

[mediawiki/skins/Poncho@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685930 merged by jenkins-bot:

[mediawiki/skins/Metrolook@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685926 merged by jenkins-bot:

[mediawiki/extensions/WebChat@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685922 merged by jenkins-bot:

[mediawiki/extensions/TwnMainPage@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685918 merged by jenkins-bot:

[mediawiki/extensions/Survey@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685916 merged by jenkins-bot:

[mediawiki/extensions/SSLClientAuthentication@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685874 merged by jenkins-bot:

[mediawiki/extensions/SocialLogin@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685872 merged by jenkins-bot:

[mediawiki/extensions/SkinPerNamespace@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685870 merged by jenkins-bot:

[mediawiki/extensions/SecureSessions@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685868 merged by jenkins-bot:

[mediawiki/extensions/PronunciationRecording@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685862 merged by jenkins-bot:

[mediawiki/extensions/OnlineStatus@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685806 merged by jenkins-bot:

[mediawiki/extensions/Favorites@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685808 merged by jenkins-bot:

[mediawiki/extensions/OpenID@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685839 merged by jenkins-bot:

[mediawiki/extensions/DisplayTitle@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 651677 merged by jenkins-bot:

[mediawiki/core@master] User: Hard-deprecate the confusingly-named isLoggedIn() method

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

Change 685859 merged by jenkins-bot:

[mediawiki/extensions/NewUserActions@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685857 merged by jenkins-bot:

[mediawiki/extensions/NetworkAuth@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685856 merged by jenkins-bot:

[mediawiki/extensions/MultiLanguageManager@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685848 merged by jenkins-bot:

[mediawiki/extensions/Inbox@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685846 merged by jenkins-bot:

[mediawiki/extensions/ImportFreeImages@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685845 merged by jenkins-bot:

[mediawiki/extensions/HSTS@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685809 abandoned by Thiemo Kreuz (WMDE):

[mediawiki/extensions/PageForms@master] Use User->isRegistered(), not deprecated isLoggedIn()

Reason:

Obsolete via I66b50d5f632872543376aee2a849122ede9966b4.

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

Change 685816 merged by jenkins-bot:

[mediawiki/extensions/TwitterLogin@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685828 merged by jenkins-bot:

[mediawiki/extensions/AutoCreateCategoryPages@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685827 merged by jenkins-bot:

[mediawiki/extensions/AccountInfo@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685832 merged by jenkins-bot:

[mediawiki/extensions/BreadCrumbs2@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685833 merged by jenkins-bot:

[mediawiki/extensions/CollapsibleVector@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685834 merged by jenkins-bot:

[mediawiki/extensions/ConfirmAccount@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685838 merged by jenkins-bot:

[mediawiki/extensions/ContributionScores@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685850 merged by Umherirrender:

[mediawiki/extensions/JsonData@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685861 merged by jenkins-bot:

[mediawiki/extensions/OfflineImportLexicon@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685802 merged by jenkins-bot:

[mediawiki/extensions/Commentbox@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685863 merged by jenkins-bot:

[mediawiki/extensions/OnlineStatusBar@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 685924 merged by jenkins-bot:

[mediawiki/extensions/URNames@master] Use User->isRegistered(), not deprecated isLoggedIn()

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

Zabe updated the task description. (Show Details)

Change 749776 had a related patch set uploaded (by Cicalese; author: ZabeMath):

[mediawiki/extensions/DisplayTitle@REL1_35] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 749777 had a related patch set uploaded (by Cicalese; author: ZabeMath):

[mediawiki/extensions/DisplayTitle@REL1_36] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 749776 merged by jenkins-bot:

[mediawiki/extensions/DisplayTitle@REL1_35] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 749777 merged by jenkins-bot:

[mediawiki/extensions/DisplayTitle@REL1_36] Use User->isRegistered(), not deprecated isLoggedIn()

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

Change 804624 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/BlueSky@master] User::isLogged was removed, replaced with isRegistered

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

Change 804624 abandoned by Jforrester:

[mediawiki/skins/BlueSky@master] User::isLogged was removed, replaced with isRegistered

Reason:

Done in I9112002a700d3b0a9e9de53b778ff2b3f4ff555d.

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