Problem/Motivation

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

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
#6 interdiff-2534014-3-6.txt1.29 KBphenaproxima
#6 2534014-6.patch3.65 KBphenaproxima
#3 2534014-3.patch3.61 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
phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Status: Active » Needs review
StatusFileSize
new 2534014-3.patch3.61 KB

Status: Needs review » Needs work

The last submitted patch, 3: 2534014-3.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 2534014-6.patch3.65 KB
new interdiff-2534014-3-6.txt1.29 KB

Moved the test into the Drupal\filter\Tests\Migrate namespace.

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented

Er...hit Save twice. That was durrrmb.

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 3a0537b and pushed to 8.0.x. Thanks!

  • alexpott committed 3a0537b on 8.0.x 
    Issue #2534014 by phenaproxima: Move module-specific migration support...

Status: Fixed » Closed (fixed)

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