Problem/Motivation

There is no reason why the picture migrations need to be required to run the d6_user migration. This causes a lot of extra setup when trying to test anything that needs the user migration.

Proposed resolution

Make the dependencies optional so they're only run if in the list of migrations.

Remaining tasks

Review

User interface changes

n/a

API changes

n/a

CommentFileSizeAuthor
#1 2422301-1.patch651 bytesbenjy
d6-user-depend.patch1.33 KBbenjy
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

benjy’s picture

benjy CreditAttribution: benjy commented
FileSize
2422301-1.patch651 bytes

Had some other stuff in the patch.

chx’s picture

chx CreditAttribution: chx commented
Status: Needs review » Reviewed & tested by the community

Yeah, if we moved d6_user_picture_file then move the rest too. Makes sense.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Fixed

Committed 7d7114e and pushed to 8.0.x. Thanks!

  • alexpott committed 7d7114e on 8.0.x 
    Issue #2422301 by benjy: Fix D6 user migration dependencies

Status: Fixed » Closed (fixed)

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