Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
Builder plugins have been converted into classes or derivers they are no longer needed
Proposed resolution
- Remove migrate.migration_builder service and classes
- Remove plugin.manager.migrate.builder service and classes
- Delete all the plugins which have a namespace that contains
Plugin\migrate\builder
- there are 8 atm
Remaining tasks
User interface changes
None
API changes
The API changes occurred in #2625696: Make migrations themselves plugins instead of config entities
Data model changes
The data model changes occurred in #2625696: Make migrations themselves plugins instead of config entities
Comment | File | Size | Author |
---|---|---|---|
#22 | 2676258-23.patch | 35.35 KB | benjy |
#15 | 2676258-15.patch | 34.93 KB | benjy |
#11 | 2676258-11.patch | 34.93 KB | alexpott |
#11 | 8-11-interdiff.txt | 5.61 KB | alexpott |
#8 | 2676258-8.patch | 29.64 KB | benjy |
Comments
Comment #2
alexpottComment #3
alexpottAs
migrate.migration_builder
is used inMigrateDrupalTestBase::installMigrations()
I think we need to remove that here too so this will make #2668436: Figure out what happens to installMigrations superfluous.Comment #4
benjy CreditAttribution: benjy at PreviousNext commentedComment #5
benjy CreditAttribution: benjy at PreviousNext commentedRemove the builder interface as well.
Comment #6
benjy CreditAttribution: benjy at PreviousNext commentedRe-rolled against 8.1.x and removed the migration builder and migration template services + interfaces.
Comment #8
benjy CreditAttribution: benjy at PreviousNext commentedRemoved the template test as well.
Comment #9
XanoComment #10
chx CreditAttribution: chx at Smartsheet commentedMigrate is experimental, it's not under the BC umbrella.
Comment #11
alexpottI think there's some stuff missing.
Comment #12
benjy CreditAttribution: benjy at PreviousNext commentedThanks alexpott, I was wondering how I missed that but then I realised I had it in #4 and the merge after the plugins patch messed up.
+1 for RTBC from me but I created the first patch.
Comment #13
chx CreditAttribution: chx at Smartsheet commentedLet's try this again :)
Comment #14
catchNeeds a re-roll.
Comment #15
benjy CreditAttribution: benjy at PreviousNext commentedRe-rolled.
Comment #16
chx CreditAttribution: chx at Smartsheet commentedComment #17
catchCommitted/pushed to 8.1.x, thanks!
Comment #20
xjmHad to revert this unfortunately. See #2625696: Make migrations themselves plugins instead of config entities.
Comment #22
benjy CreditAttribution: benjy at PreviousNext commentedRe-rolled.
Comment #23
alexpottNice less code!
Comment #24
catchCommitted/pushed to 8.2.x, thanks!
Leaving RTBC against 8.1.x.
Comment #27
xjmComment #28
xjmComment #29
catchCherry-picked to 8.1.x.
Comment #33
kylebrowning CreditAttribution: kylebrowning at Acquia commentedHow does one create dynamic migrations now?