Problem/Motivation

Per #2533886: [meta] Move module-specific migration support into the particular modules supported, move support for block migration into the block 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_block

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 interdiff-2533986-4-7.txt4.17 KBphenaproxima
#7 2533986-7.patch8.06 KBphenaproxima
#4 2533986-4.patch3.45 KBphenaproxima

Comments

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Pittsfield, MA, USA
commented
Issue summary: View changes
phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Assigned: Unassigned » phenaproxima
mikeryan’s picture

mikeryan
he/him
Primary language English
Location Pittsfield, MA, USA
commented
Parent issue: #2533984: Move module-specific migration support into the aggregator module » #2533886: [meta] Move module-specific migration support into the particular modules supported
phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Status: Active » Needs review
StatusFileSize
new 2533986-4.patch3.45 KB

Status: Needs review » Needs work

The last submitted patch, 4: 2533986-4.patch, failed testing.

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Status: Needs work » Postponed

Blocked by #2534158: MigrateFullDrupalTestBase must use dynamic test discovery.

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
StatusFileSize
new 2533986-7.patch8.06 KB
new interdiff-2533986-4-7.txt4.17 KB

Moved the test into the Drupal\block\Tests\Migrate namespace, and moved the relevant process plugins out of Migrate Drupal

phenaproxima’s picture

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

Unblocked.

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Pittsfield, MA, USA
commented
Status: Needs review » Reviewed & tested by the community

Looks good to me.

alexpott’s picture

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

Committed 5d183c7 and pushed to 8.0.x. Thanks!

  • alexpott committed 5d183c7 on 8.0.x 
    Issue #2533986 by phenaproxima: Move module-specific migration support...

Status: Fixed » Closed (fixed)

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