- Issue created by @spokje
- last update
over 1 year ago 29,884 pass - @spokje opened merge request.
- Status changed to Needs review
over 1 year ago 6:03am 27 July 2023 - Status changed to RTBC
over 1 year ago 8:31am 27 July 2023 - 🇬🇧United Kingdom longwave UK
This is the only instances of
cacheKeyPrefixin core.Looking at history we actually added this property to fix #3274474: Fix 'Access to an undefined property' PHPStan L0 errors → - we should have looked closer and realised it was unused.
$cacheKeyPrefixwas originally part of DefaultPluginManager and then was removed in #2281905: Stop caching plugin discovery/info hooks by language → , but this line was missed at the time. - last update
over 1 year ago 29,887 pass - last update
over 1 year ago 29,910 pass - last update
over 1 year ago 29,942 pass, 1 fail - Status changed to Needs work
over 1 year ago 2:54am 4 August 2023 - last update
over 1 year ago 29,939 pass, 1 fail - Status changed to RTBC
over 1 year ago 5:35am 4 August 2023 - 🇳🇱Netherlands spokje
Answered the question, resolved thread (@znerol, please reopen if you don't agree) and rebased, back to RTBC.
- last update
over 1 year ago 29,953 pass - last update
over 1 year ago 29,953 pass - last update
over 1 year ago 29,958 pass - last update
over 1 year ago 29,958 pass - last update
over 1 year ago 29,959 pass - last update
over 1 year ago 29,962 pass - Status changed to Fixed
over 1 year ago 8:48pm 16 August 2023 - 🇬🇧United Kingdom longwave UK
Backported to 10.1.x as a low risk cleanup to keep code in sync.
Committed and pushed dc249e5f70 to 11.x and 9293b10abc to 10.1.x. Thanks!
-
longwave →
committed 9293b10a on 10.1.x
Issue #3377318 by Spokje, longwave, znerol: Remove cruft from...
-
longwave →
committed 9293b10a on 10.1.x
-
longwave →
committed dc249e5f on 11.x
Issue #3377318 by Spokje, longwave, znerol: Remove cruft from...
-
longwave →
committed dc249e5f on 11.x
Automatically closed - issue fixed for 2 weeks with no activity.