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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

phenaproxima’s picture

Issue tags: +Barcelona2015

A migration of RDF's configuration variables would be a reasonable target for Barcelona.

quietone’s picture

FileSize
3.04 KB

Just a source plugin and test.

quietone’s picture

phenaproxima’s picture

Status: Active » Needs work

Good start, but needs to be finished :)

quietone’s picture

Yes, 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.

quietone’s picture

Issue tags: +migrate-d7-d8

Version: 8.0.x-dev » 8.1.x-dev

Drupal 8.0.6 was released on April 6 and is the final bugfix release for the Drupal 8.0.x series. Drupal 8.0.x will not receive any further development aside from security fixes. Drupal 8.1.0-rc1 is now available and sites should prepare to update to 8.1.0.

Bug reports should be targeted against the 8.1.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.2.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Version: 8.1.x-dev » 8.2.x-dev

Drupal 8.1.9 was released on September 7 and is the final bugfix release for the Drupal 8.1.x series. Drupal 8.1.x will not receive any further development aside from security fixes. Drupal 8.2.0-rc1 is now available and sites should prepare to upgrade to 8.2.0.

Bug reports should be targeted against the 8.2.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.3.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Version: 8.2.x-dev » 8.3.x-dev

Drupal 8.2.6 was released on February 1, 2017 and is the final full bugfix release for the Drupal 8.2.x series. Drupal 8.2.x will not receive any further development aside from critical and security fixes. Sites should prepare to update to 8.3.0 on April 5, 2017. (Drupal 8.3.0-alpha1 is available for testing.)

Bug reports should be targeted against the 8.3.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.4.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

heddn’s picture

Version: 8.3.x-dev » 8.4.x-dev

Reviewed 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.

Version: 8.4.x-dev » 8.5.x-dev

Drupal 8.4.0-alpha1 will be released the week of July 31, 2017, which means new developments and disruptive changes should now be targeted against the 8.5.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Version: 8.5.x-dev » 8.6.x-dev

Drupal 8.5.0-alpha1 will be released the week of January 17, 2018, which means new developments and disruptive changes should now be targeted against the 8.6.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

heddn’s picture

Assigned: Unassigned » heddn
Priority: Minor » Normal

Let'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.

heddn’s picture

Assigned: heddn » Unassigned
Status: Needs work » Needs review
Issue tags: +Needs tests
FileSize
2.97 KB

Here's a reboot. No interdiff, because this needed a reroll and a restart. Still needs tests.

Status: Needs review » Needs work

The last submitted patch, 14: 2500509-14.patch, failed testing. View results

Version: 8.6.x-dev » 8.7.x-dev

Drupal 8.6.0-alpha1 will be released the week of July 16, 2018, which means new developments and disruptive changes should now be targeted against the 8.7.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

maxocub’s picture

FileSize
11.81 KB
12.75 KB

Here's a continuation of @heddn's last patch.

I still need to complete the source plugin test.

maxocub’s picture

Status: Needs work » Needs review
Issue tags: -Needs tests
FileSize
13.87 KB
22.41 KB

And here's the completed source plugin test.

maxocub’s picture

Issue summary: View changes

IS update.

Status: Needs review » Needs work

The last submitted patch, 18: 2500509-18.patch, failed testing. View results

maxocub’s picture

Status: Needs work » Needs review
FileSize
580 bytes
22.41 KB

Oups, wrong namespace.

heddn’s picture

Status: Needs review » Reviewed & tested by the community

Since my patch in #14 was basically a re-roll, I think I can still mark this RTBC. Tests are very thorough. Good work here.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

@phenaproxima

good target for Barcelona

... 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.

  • alexpott committed 0e0105f on 8.7.x
    Issue #2500509 by maxocub, quietone, heddn: Upgrade path for RDF 7.x
    

  • alexpott committed 73d11b2 on 8.6.x
    Issue #2500509 by maxocub, quietone, heddn: Upgrade path for RDF 7.x
    
    (...
heddn’s picture

Someone 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!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.