Page MenuHomePhabricator

Use WeakMap instead of an array in RestrictedEntityLookupFactory
Closed, ResolvedPublic

Description

1091251: Refactor: Use WeakMap in RestrictedEntityLookupFactory WeakMap implementation can be merged when the support for PHP 7.4 is dropped. The original array implementation will then be removed.

Related Objects

StatusSubtypeAssignedTask
Resolvedseanleong-WMDE
ResolvedJdforrester-WMF
ResolvedKrinkle
Resolvedtstarling
ResolvedJdforrester-WMF
ResolvedJdforrester-WMF
Resolvedtstarling
ResolvedReedy
ResolvedBUG REPORTtstarling
Resolvedtstarling
ResolvedDaimona
ResolvedDaimona
ResolvedNone
ResolvedJdforrester-WMF
ResolvedBUG REPORTNone
Resolvedtstarling
ResolvedJdforrester-WMF
Resolvedssastry
Resolvedkostajh
Resolvedkostajh
Resolvedthiemowmde
Resolvedtstarling
Resolvedtstarling
ResolvedBUG REPORTLucas_Werkmeister_WMDE
Resolvedhoo
Resolvedhoo
ResolvedJdforrester-WMF
Resolvedthiemowmde
Resolvedkostajh
ResolvedUmherirrender
ResolvedPRODUCTION ERROR brooke
ResolvedTheresNoTime
Resolvedtstarling
ResolvedJdforrester-WMF
Resolved larissagaulia
ResolvedJMeybohm
ResolvedMoritzMuehlenhoff
ResolvedNone
DuplicateNone
ResolvedNone
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORNone
ResolvedPRODUCTION ERROR mszabo
ResolvedPRODUCTION ERROR mszabo
ResolvedPRODUCTION ERROR mszabo
ResolvedPRODUCTION ERRORNone
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERROR mszabo
ResolvedPRODUCTION ERRORNone
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORhashar
ResolvedPRODUCTION ERRORUmherirrender
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORthiemowmde
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORTacsipacsi
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORJdforrester-WMF
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORdcausse
ResolvedPRODUCTION ERROR mszabo
ResolvedPRODUCTION ERROR mszabo
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORUmherirrender
ResolvedPRODUCTION ERRORUmherirrender
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORjijiki
ResolvedScott_French
ResolvedPRODUCTION ERRORABreault-WMF
ResolvedPRODUCTION ERRORdaniel
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORihurbain
ResolvedPRODUCTION ERRORJdforrester-WMF
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORJdforrester-WMF
ResolvedPRODUCTION ERRORJdforrester-WMF
DuplicatePRODUCTION ERRORNone
ResolvedPRODUCTION ERRORJdforrester-WMF
ResolvedPRODUCTION ERRORUmherirrender
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORReedy
DuplicatePRODUCTION ERRORNone
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORJdforrester-WMF
ResolvedPRODUCTION ERRORPaladox
ResolvedPRODUCTION ERRORJdforrester-WMF
ResolvedBUG REPORTWargo
ResolvedPRODUCTION ERRORJdforrester-WMF
ResolvedBUG REPORTScott_French
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORssastry
ResolvedPRODUCTION ERRORReedy
ResolvedPRODUCTION ERRORJdforrester-WMF
ResolvedPRODUCTION ERRORUmherirrender
ResolvedPRODUCTION ERRORJdforrester-WMF
ResolvedPRODUCTION ERRORUmherirrender
ResolvedLadsgroup
ResolvedPRODUCTION ERRORUmherirrender
ResolvedFeature bd808
ResolvedScott_French
ResolvedScott_French
ResolvedScott_French
ResolvedKrinkle
ResolvedMSantos
Resolved Tgr
ResolvedScott_French
ResolvedScott_French
Resolveddduvall
ResolvedClement_Goubert
ResolvedScott_French
ResolvedScott_French
ResolvedScott_French
ResolvedScott_French
Resolvedori
ResolvedClement_Goubert
ResolvedScott_French
In ProgressScott_French
ResolvedScott_French
ResolvedScott_French
ResolvedScott_French
ResolvedScott_French
ResolvedScott_French
ResolvedScott_French
ResolvedJdforrester-WMF

Event Timeline

@Lucas_Werkmeister_WMDE – did you mean to block on dropping 8.1? FWICS WeakMap is available from PHP 8.0, so this should nominally be blocked on T319432 instead?

Sorry, I mixed that up – T328921: Drop PHP 7.4 and PHP 8.0 support from MediaWiki is probably the right task?

Yup - just wanted to make sure that I wasn't missing something. :-) Phabricator is quite the thicket of gnarly tasks in this area. :-(