Problem/Motivation

In #2936365: Migrate UI - allow modules to declare the state of their migrations the noUpgradePath arrays were removed and replaced with entries in migrate_drupal/migrations/state/migrate_drupal.migrate_drupal.yml. There entries need to be reviewed to decide if any can be moved to a more appropriate module and then move them.

At least the i18m ones need a closer look and so do any modules that provide fields.
In #3175729: Mark i18n migrations as finished all the i18n properties were moved out of migrate_drupal.migrate_drupal.yml.

Proposed resolution

Review each migration state entry in migrate_drupal and move any that can be moved.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#9 3053167-9.patch1.81 KBquietone
#9 diff-5-9.txt199 bytesquietone
#5 3053167-5.patch1.79 KBquietone

Comments

quietone created an issue. See original summary.

Version: 8.8.x-dev » 8.9.x-dev

Drupal 8.8.0-alpha1 will be released the week of October 14th, 2019, which means new developments and disruptive changes should now be targeted against the 8.9.x-dev branch. (Any changes to 8.9.x will also be committed to 9.0.x in preparation for Drupal 9’s release, but some changes like significant feature additions will be deferred to 9.1.x.). For more information see the Drupal 8 and 9 minor version schedule and the Allowed changes during the Drupal 8 and 9 release cycles.

Version: 8.9.x-dev » 9.1.x-dev

Drupal 8.9.0-beta1 was released on March 20, 2020. 8.9.x is the final, long-term support (LTS) minor release of Drupal 8, which means new developments and disruptive changes should now be targeted against the 9.1.x-dev branch. For more information see the Drupal 8 and 9 minor version schedule and the Allowed changes during the Drupal 8 and 9 release cycles.

Version: 9.1.x-dev » 9.2.x-dev

Drupal 9.1.0-alpha1 will be released the week of October 19, 2020, which means new developments and disruptive changes should now be targeted for the 9.2.x-dev branch. For more information see the Drupal 9 minor version schedule and the Allowed changes during the Drupal 9 release cycle.

quietone’s picture

quietone commented
Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new 3053167-5.patch1.79 KB

The remaining candidates for moving are

finished:
  6:
    nodereference: core
    userreference: core
    # Blog requires node.
    blog: node
  7:
    # Blog requires node.
    blog: node

I suppose node and user reference can go to the fields modules and blog to the node module. Then the remaining items are for sources that don't have an upgrade path, e.g. PHP module. And there is one outlier,

not_finished:
  7:
    references_uuid: core

Version: 9.2.x-dev » 9.3.x-dev

Drupal 9.2.0-alpha1 will be released the week of May 3, 2021, which means new developments and disruptive changes should now be targeted for the 9.3.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

marvil07’s picture

marvil07 commented
Status: Needs review » Needs work

The changes here follow the description on the issue.

I was confused about references_uuid, looking at the log, I see that line was added at #2814953: Migrate Drupal 7 node/user reference fields, and there I found that module lived at D7 references module.
I found only a few matches to the module name at core/modules/migrate_drupal/tests/fixtures/drupal7.php, apart from the modified YAML.
It sounds like that can wait till the migrations related to that module are finished, and if they are, we may have an answer.
Hint: there may be an answer on the long thread at the mentioned issue, if someone is interested.

Moving as NW for a re-roll, needed because of 21f096acfda0b2f16c3ccbd341734b3c4677b365 changing one of the context lines.
Hint: git apply fails, but patch -p1 works.

marvil07’s picture

marvil07 commented
Issue tags: +Needs reroll
quietone’s picture

quietone commented
Status: Needs work » Needs review
Issue tags: -Needs reroll
StatusFileSize
new diff-5-9.txt199 bytes
new 3053167-9.patch1.81 KB

Thanks marvil07.

marvil07’s picture

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

@quietone, thanks for the re-roll.
Moving to RTBC now.

  • catch committed ba1ff55 on 9.3.x 
    Issue #3053167 by quietone, marvil07: Move state entries out of...

  • catch committed f34a5d5 on 9.2.x 
    Issue #3053167 by quietone, marvil07: Move state entries out of...
catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 9.3.x and cherry-picked to 9.2.x, thanks!

Status: Fixed » Closed (fixed)

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