- Issue created by @quietone
- Status changed to Active
2 months ago 1:37am 25 April 2025 - π³πΏNew Zealand quietone
Updated from discussion on the Migrate video call today, benjifisher, heddn, mikelutz and myself were present. π [meeting] Migrate Meeting 2025-04-24 2100Z Active
- First commit to issue fork.
- Merge request !12617Deprecate DrupalSqlBase, but skip deprecations for core implementations. β (Open) created by longwave
- π¬π§United Kingdom longwave UK
Alternative proposal in !12617 which deprecates DrupalSqlBase but then skips the deprecation notices for core extensions of that class; any other extensions in contrib will trigger the message as normal.
- π¬π§United Kingdom longwave UK
Added some test coverage by removing test modules from the skip, and then expecting the deprecation in tests that use them. Let's see if anything else will fail now I copied in the other two source plugin deprecations from the original MR.
- π¬π§United Kingdom longwave UK
Added similar dynamic migration messages for the EmptySource plugin and I18nQueryTrait trait and skipped the core implementations of them, they should still trigger in contrib/custom code.
- πΊπΈUnited States mikelutz Michigan, USA
I like that, itβs pretty slick and much fewer lines than I was expecting to have to deal with. Iβm inclined to try to get this merged in early in the 10.3 cycle so that contrib has time to deal with the fallout. It leaves the βhardβ problem of identifying each of the source plugins the legacy tests for the actual removal when the 12.x branch opens, but at least we donβt have to do that twice, once for @legacy annotations, and again for removal. And when we get around to doing it, we could just remove the suppressions to get a list of all affected tests easily enough.
- πΊπΈUnited States mikelutz Michigan, USA
mikelutz β changed the visibility of the branch 3507572-deprecate-migrate-source to hidden.
- πΊπΈUnited States mikelutz Michigan, USA
@longwave, Do we need to add the @deprecated annotation for DrupalSqlBase, or should that just be covered with the deprecation of the whole module eventually?
- π¬π§United Kingdom longwave UK
We can, because that will notify people via PHPStan as well as via tests or at runtime - but I suspect the PHPStan baseline will grow quite a bit.