Problem/Motivation

Per #2533886: [meta] Move module-specific migration support into the particular modules supported, move support for taxonomy migration into the taxonomy module.

Proposed resolution

For the most part, this is a matter of moving files around and updating namespaces. Also, explicit destination module dependencies can be removed.

Migration templates to be moved:

  • d6_taxonomy_settings
  • d6_taxonomy_term
  • d6_taxonomy_vocabulary
  • d6_term_node
  • d6_term_node_revision
  • d6_vocabulary_entity_display
  • d6_vocabulary_entity_form_display
  • d6_vocabulary_field
  • d6_vocabulary_field_instance

Remaining tasks

Submit a patch.

User interface changes

N/A

API changes

Namespaces of affected migration plugins will change.

Data model changes

N/A

CommentFileSizeAuthor
#7 fix-directory-name.patch363 bytesmikeryan
#1 move_module_specific-2534034-1.patch27.81 KBmikeryan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Murphysboro, IL, USA
CreditAttribution: mikeryan commented
Status: Active » Postponed
Related issues: +#2534158: MigrateFullDrupalTestBase must use dynamic test discovery
FileSize
move_module_specific-2534034-1.patch27.81 KB

Here's the patch, postponing on #2534158: MigrateFullDrupalTestBase must use dynamic test discovery.

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
CreditAttribution: phenaproxima at Acquia commented
Status: Postponed » Needs review

Unblocked.

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
CreditAttribution: phenaproxima at Acquia commented
Status: Needs review » Reviewed & tested by the community

Thumbs up!

alexpott’s picture

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

Committed 4930307 and pushed to 8.0.x. Thanks!

  • alexpott committed 4930307 on 8.0.x 
    Issue #2534034 by mikeryan: Move module-specific migration support into...
mikeryan’s picture

mikeryan
he/him
Primary language English
Location Murphysboro, IL, USA
CreditAttribution: mikeryan commented
Status: Fixed » Needs work

A little oops, source plugin directory should be "migrate" rather than "Migrate".

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Murphysboro, IL, USA
CreditAttribution: mikeryan commented
Status: Needs work » Needs review
FileSize
fix-directory-name.patch363 bytes

Here's the fix (thanks to http://stackoverflow.com/questions/3011625/git-mv-and-only-change-case-o...).

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
CreditAttribution: phenaproxima at Acquia commented
Status: Needs review » Reviewed & tested by the community
alexpott’s picture

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

Committed 46f69a5 and pushed to 8.0.x. Thanks!

  • alexpott committed 46f69a5 on 8.0.x 
    Issue #2534034 followup by mikeryan: Move module-specific migration...

Status: Fixed » Closed (fixed)

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