Problem/Motivation

Over in #2314289: Track result of migrations and use it to properly define migrate_dependencies we're now tracking the migration result in the database. This means that we have to manually set the migration result on dependant migrations during test setup. We added $migration->setMigrationStatus() to prepareIdMappings() with the vision of renaming it to something more appropriate. That's what we're going to do here.

Proposed resolution

Decide on a name and then find/replace all occurrences.

Remaining tasks

Wait for #2314289: Track result of migrations and use it to properly define migrate_dependencies and then roll a patch.

User interface changes

n/a

API changes

MigrationTestBase::prepareIdMappings() will be renamed.

CommentFileSizeAuthor
#2 2315489-2.patch24.64 KBbenjy

Comments

chx’s picture

I vote prepareMigrations() simply.

benjy’s picture

Status: Active » Needs review
StatusFileSize
new24.64 KB

That works for me.

chx’s picture

Status: Needs review » Reviewed & tested by the community

As I said, that works for me. THanks.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 37df1bd and pushed to 8.0.x. Thanks!

  • alexpott committed 37df1bd on 8.0.x
    Issue #2315489 by benjy: Renamed prepareIdMappings() in MigrateTestBase.
    

Status: Fixed » Closed (fixed)

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