Since Drupal 8.5.0, Drupal to Drupal migrations require source_module and/or destination_module property.
Related change record: https://www.drupal.org/node/2831566
Error message: The d6_d7_acl_table plugin must define the source_module property.

CommentFileSizeAuthor
#3 2951661-fix_migration_properties.patch1.8 KBnevergone

Comments

nevergone created an issue. See original summary.

nevergone’s picture

nevergone
Primary language Hungarian
commented
Issue summary: View changes
nevergone’s picture

nevergone
Primary language Hungarian
commented
Status: Active » Needs work
StatusFileSize
new 2951661-fix_migration_properties.patch1.8 KB

  • nevergone authored 74db41b on 8.x-1.x 
    Issue #2951661 by nevergone: Add source_module and destination_module...
salvis’s picture

salvis
he/his
commented

Thank you for the patch!

There's still something missing apparently:
https://www.drupal.org/node/87985/qa
https://www.drupal.org/pift-ci-job/921238

1) Drupal\acl\Tests\Migrate\d6\MigrateAclListTest::testMigration
Migration d6_d7_acl did not meet the requirements. The module acl is not enabled in the source site. source_module: acl.
Failed asserting that false is true.

salvis’s picture

salvis
he/his
commented

Just how exactly do migration tests work? Do they try to instantiate a D6 site? Where does the D6 code come from? And where would D6 contribs (like D6 acl) come from?

Is there any documentation for this? Pointers, please!

salvis’s picture

salvis
he/his
commented

Well, D6 or D7...

igonzalez’s picture

igonzalez commented

Works for me

salvis’s picture

salvis
he/his
commented
Status: Needs work » Fixed

Never mind #5.

This patch was committed in #4, so closing now.

Status: Fixed » Closed (fixed)

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