Problem/Motivation

As a Drupal 7 user, I want to have my youtube fields migrated to media fields and the corresponding field values to media entities.

API changes

  • New migration: d7_youtube_field which migrates the field values to media entities.
  • New migrate source plugin: youtube which provides the source field values.
  • An improved media_internet_field_value migrate process plugin.
  • New field migration plugin: youtube which changes Drupal 7 youtube fields to Drupal 9 media fields.
CommentFileSizeAuthor
#10 Migrate_Youtube_field_from_D7_to_D9-3272705-10.patch379 bytesyash.rode
#6 interdiff-3272705-4-5.txt1.29 KByash.rode
#6 Migrate_Youtube_field_from_D7_to_D9-3272705-5.patch81.42 KByash.rode
#5 interdiff-3272705-3-4.txt1.63 KByash.rode
#5 Migrate_Youtube_field_from_D7_to_D9-3272705-4.patch81.37 KByash.rode
#3 Migrate_Youtube_field_from_D7_to_D9-3272705-3.patch81.88 KByash.rode

Issue fork media_migration-3272705

Command icon Show commands

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:

About issue forks

Comments

yash.rode created an issue. See original summary.

yash.rode’s picture

yash.rode commented
Title: Migrate Youtube Field module from D7 to D9. » Migrate Youtube Field from D7 to D9.
yash.rode’s picture

yash.rode commented
Status: Active » Needs review
StatusFileSize
new Migrate_Youtube_field_from_D7_to_D9-3272705-3.patch81.88 KB

Youtube field to media migration.

Status: Needs review » Needs work

The last submitted patch, 3: Migrate_Youtube_field_from_D7_to_D9-3272705-3.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

yash.rode’s picture

yash.rode commented
StatusFileSize
new Migrate_Youtube_field_from_D7_to_D9-3272705-4.patch81.37 KB
new interdiff-3272705-3-4.txt1.63 KB
yash.rode’s picture

yash.rode commented
Status: Needs work » Needs review
StatusFileSize
new Migrate_Youtube_field_from_D7_to_D9-3272705-5.patch81.42 KB
new interdiff-3272705-4-5.txt1.29 KB

Source query changed->added orderBy() to get the same result on local as well as DGO.

huzooka’s picture

huzooka
Primary language Hungarian
Location Hungary 🇭🇺🇪🇺
commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

#6 is perfect!

Thank you, @yash.rode!

  • huzooka committed 35cd13f on 8.x-1.x authored by yash.rode 
    Issue #3272705 by yash.rode: Migrate Youtube Field from D7 to D9
huzooka’s picture

huzooka
Primary language Hungarian
Location Hungary 🇭🇺🇪🇺
commented
Status: Reviewed & tested by the community » Needs work

...and I just noticed what's wrong:

We have to make sure that these youtube field migrations are executed after the other media from plain files and media from media migrations: because if d7_youtube_field is executed before them, then we will have a media entity ID conflict!

I won't revert the commit, but this should be addressed.

yash.rode’s picture

yash.rode commented
Status: Needs work » Needs review
StatusFileSize
new Migrate_Youtube_field_from_D7_to_D9-3272705-10.patch379 bytes

Added migration dependencies to make sure youtube field migration is executed after the other media from plain files and media from media migrations are executed.

huzooka’s picture

huzooka
Primary language Hungarian
Location Hungary 🇭🇺🇪🇺
commented
Status: Needs review » Fixed

The underlying issue (the assumption on source fid === destination mid) was addressed in #3281055: Media entity migrations shouldn't migrate file IDs to media IDs.

@yash.rode, thank you again!

Status: Fixed » Closed (fixed)

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

emerham made their first commit to this issue’s fork.