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.
It should be possible to include user-account default "content" for testing or demo purposes.
Comment | File | Size | Author |
---|---|---|---|
#8 | migrate_defaultcontent-support_user_account_migrations-2394133-8.patch | 5.87 KB | idebr |
#8 | interdiff-5-8.txt | 570 bytes | idebr |
Comments
Comment #1
Rob C CreditAttribution: Rob C commentedComment #2
Rob C CreditAttribution: Rob C commentedComment #3
Rob C CreditAttribution: Rob C commentedComment #4
FreekVR CreditAttribution: FreekVR commentedThanks, it looks good for the most part. A couple of improvements need to be made though.
The complete() and prepareRollback() should use code from the common class. Here's an example from node.inc.
The 'addReference' and 'resolveReference' methods are now handled by the common class as well and should be removed.
Comment #5
Rob C CreditAttribution: Rob C commentedIt's getting smaller and smaller.
private $references removed.
->prepareRollback() replaced.
->complete() replaced.
->addReference() removed
->resolveReferences() removed.
And updated a comment added by the patch.
Comment #6
Rob C CreditAttribution: Rob C commentedComment #7
idebr CreditAttribution: idebr at One Shoe commentedThis results in a notice on the Migrate dashboard:
Strict warning: Declaration of DefaultContentUserMigration::complete() should be compatible with DrupalUser7Migration::complete($account, $row)
Comment #8
idebr CreditAttribution: idebr at One Shoe commentedThis patch fixes the notice on the Migrate dashboard.
Comment #10
FreekVR CreditAttribution: FreekVR at One Shoe commented