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.
Reasons:
* Migrate module is much superior than OG's own migrate implementation.
* Migrate module might be in D8 core
Comment | File | Size | Author |
---|---|---|---|
#9 | 1686394-og-migrate-9.patch | 2.13 MB | amitaibu |
#7 | 1686394-og-migrate-7.patch | 2.11 MB | amitaibu |
#2 | og_membership.inc_.not-really-a.patch | 6.35 KB | gdl |
Comments
Comment #1
amitaibuSome work has been committed to
1686394
branch. So not working!Comment #2
gdl CreditAttribution: gdl commentedAttached is a Migrate destination class for OG membership entities, cribbed almost entirely from the node destination class. It's been working quite well for me, but I'm sure there are issues. The error handling could probably be better, for example, and there's a @fixme or two in there. Hopefully it'll be of some use!
Thanks for Organic Groups!
-G
Comment #3
amitaibutagging
Comment #4
Nephele CreditAttribution: Nephele commentedAdmittedly, I don't have any experience with the Migrate module. But glancing over the project description, it looks like the primary intent of the module is to import data from non-drupal sites into drupal.
What looks more relevant to og_migrate is migrate_d2d (the Drupal-to-Drupal migrate module). That's the project brought up in the D8 core discussion at #1052692: New import API for major version upgrades (was migrate module). However, migrate_d2d only exists in a sandbox version, and it emphasizes that its API is still in flux. Plus, There's no guarantee that migrate_d2d will even have a D7 version, especially if the top priority is getting a product ready for D8.
So it seems to me like any work on replacing og_migrate should wait until migrate_d2d is more stable, and might even be better for D8 than D7.
Comment #5
amitaibuMigrate is pretty robust, and I believe it can help us. For example have a look at origin/1686394 on
og/includes/migrate/7200/og_7200_roles.migrate.inc
(untested, but that the idea).Comment #6
amitaibuSorry, didn't push correctly, will do later on.My mistake, everything was pushed correctly.
Comment #7
amitaibuWork in progress, but good progress :)
Now working on tests.
Comment #8
Nephele CreditAttribution: Nephele commentedUsing the head of the 1686394 branch, I'm getting the following error message every time I view the admin/content/migrate page:
However, I seem to be able to ignore the message and proceed to do an import. (This is when migrating a database from og-1.4 to og-2)
Also, it would be good to update the link in the message: "Organic groups or one of its modules needs to migrate data. Organic groups may not work properly without it. After a successful execution you can disable it." The link needs to be changed to point to admin/content/migrate instead of admin/config/group/group-migrate. And perhaps also tweak the message to give some information about what to do on the migrate page. I assumed that I had to select the "Update" checkbox in the Options section and then execute an "Import", but I'm not sure whether that's correct.
Comment #9
amitaibuAll tests are passing locally (needs review for testbost)
Need to update OG-UI and OGUR migrtion plugins.
Comment #11
amitaibuNote All Migrate plugins are now suposed to work (branch
1686394
).Comment #12
amitaibuApart of the OGUR migrations, all is migrated and tested. Please open new issues for follow-ups.
Note that unfortunately the testbot doesn't execute the tests that have
'dependencies' => array('migrate'),
-- those need to be executed locally.