Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Problem/Motivation
We need an upgrade path from D7 for the core RDF module.
Proposed resolution
Migrate the D7 rdf_mapping
table entries to D8 RdfMapping
config entities.
Remaining tasks
Write the patch.- Review.
- Commit.
User interface changes
None.
API changes
None.
Data model changes
None.
Comment | File | Size | Author |
---|---|---|---|
#21 | 2500509-21.patch | 22.41 KB | maxocub |
#21 | interdiff-2500509-18-21.txt | 580 bytes | maxocub |
Comments
Comment #1
phenaproximaA migration of RDF's configuration variables would be a reasonable target for Barcelona.
Comment #2
quietone CreditAttribution: quietone commentedJust a source plugin and test.
Comment #3
quietone CreditAttribution: quietone commentedComment #4
phenaproximaGood start, but needs to be finished :)
Comment #5
quietone CreditAttribution: quietone commentedYes, it needs to be finished. This isn't straight forward, particularly for the fieldMappings and types. And I've yet to find documentation useful to me.
Comment #6
quietone CreditAttribution: quietone commentedComment #10
heddnReviewed this in our weekly migrate meeting. We should re-think this and revisit this from the beginning again. If someone feels this a higher priority, speak up... but we felt this was OK leaving as a minor priority.
Comment #13
heddnLet's see what we can do to knock this module off the list of missing upgrade paths. Assigning to myself for review and work this week.
Comment #14
heddnHere's a reboot. No interdiff, because this needed a reroll and a restart. Still needs tests.
Comment #17
maxocub CreditAttribution: maxocub commentedHere's a continuation of @heddn's last patch.
I still need to complete the source plugin test.
Comment #18
maxocub CreditAttribution: maxocub commentedAnd here's the completed source plugin test.
Comment #19
maxocub CreditAttribution: maxocub commentedIS update.
Comment #21
maxocub CreditAttribution: maxocub commentedOups, wrong namespace.
Comment #22
heddnSince my patch in #14 was basically a re-roll, I think I can still mark this RTBC. Tests are very thorough. Good work here.
Comment #23
alexpott@phenaproxima
... things take time in Drupal :)
Committed and pushed 0e0105fd93 to 8.7.x and 73d11b2894 to 8.6.x. Thanks!
Backported to 8.6.x because having a more complete migration from 7 to 8 is worth having for 8.6.0.
Comment #26
heddnSomeone will correct me, but I think this was the last fully incomplete D7 module? We still have other gaps in some things like #2763637: D7 taxonomy term fields are not migrated with allowed vocabularies #2587063: Variable to config: search_active_modules [d7]. But now there are no core modules in D7 that don't have at least /something/. Woohoo! Yeah to us!