Closed (fixed)
Project:
Drupal core
Version:
main
Component:
file system
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
2 Feb 2026 at 15:41 UTC
Updated:
23 Mar 2026 at 10:00 UTC
Jump to comment: Most recent
Please don't pick up multiple ones, want to give everyone a chance
Remove all D12 deprecated code from the mentioned libraries.
Remove all D12 deprecated code from the mentioned libraries.
Make sure tests are passing before review.
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #2
smustgrave commentedFileTransfer overlapped with one dww is working on so rescoping.
Comment #3
smustgrave commentedComment #4
smustgrave commentedComment #5
smustgrave commentedBeen a few weeks so going to pick it up and expand the scope
Comment #7
smustgrave commentedFound an issue so this is postponed on #3574721: FormBuilder service order causes failure
Comment #10
smustgrave commentedComment #11
smustgrave commentedThis should be a good chunk. Avoiding the test folders in lib for #3550268: Remove deprecated paths from the testing framework
Comment #12
dcam commentedI left comments on the MR. At least some of these need to be changed, so I'm setting the status to Needs Work.
Comment #13
smustgrave commentedAttempted to address feedback. Believe nightwatch is random but can't rerun
Comment #14
dcam commentedThank you, @smustgrave. I left a couple of additional comments that I think need to be addressed.
Comment #15
smustgrave commentedAdded the test
Comment #16
dcam commentedI think we're done here. Thank you for working with me on it, @smustgrave.
Comment #17
berdirThis has some removals in the Plugin namespace but not \Drupal\Core\Plugin\DefaultPluginManager::__construct(), but I guess it's OK to separate that out specifically because I'm not 100% sure about really going through with that in D12 (it's a disruptive, non-trivial removal affecting a ton of projects that doesn't have tooling yet). Lets open a separate issue to discuss that further?
Comment #18
catchYeah let's open a dedicated issue for
DefaultPluginManager::__construct()we can always defer that one to removal in 13.x if we think it'll be better, although the problem is that if contrib plugin managers don't support attributes, then implementations also can't support attributes, which means it's harder to guarantee everything support 13.0.0 in time, but we'd only be giving people longer to make the change, not enforcing that they can't make it... For that issue anyway.Committed/pushed to main, thanks!
Comment #22
berdirI created #3577900: Removing support for not supporting attribute-based discovery for that.