Right now running drush help migrate-import outputs the following:

Examples:
 migrate-import --all                      Perform all migrations
 migrate-import --group=beer               Import all migrations in the beer group
 migrate-import BeerTerm,BeerNode          Import new terms and nodes
 migrate-import BeerUser --limit=2         Import no more than 2 users
 migrate-import BeerUser --idlist=5        Import the user record with source ID 5

The example migration names in this case are somewhat misleading in that they don't match the examples in migrate_plus, and are likely not reflective of what someone would actually use for a name. The name in this case refers to the id: portion of the migration YAML file. This looks like it is probably just a carry over from Drupal 7 where migration names where also the class name. Anyway, we should update this so the examples reflect reality.

BeerNode = beer_node
BeerTerm = beer_term
BeerUser = beer_user

Maybe we should also change the help text for the migration argument which currently reads, "Name of migration(s) to import. Delimit multiple using commas." to something like "ID of migration(s) to import. Delimit multiple using commas." as well. Just to be super clear.

CommentFileSizeAuthor
#2 2777971-1-migrate_import_help_text.patch4.27 KBeojthebrave
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

eojthebrave created an issue. See original summary.

eojthebrave’s picture

eojthebrave
he/him
Primary language English
Location Minneapolis, MN
CreditAttribution: eojthebrave at Drupalize.Me commented
Status: Active » Needs review
FileSize
2777971-1-migrate_import_help_text.patch4.27 KB

Here's a patch that makes these updates. It also updates the examples in other commands that exhibit this same issue. Example output:

Perform one or more migration processes.

Examples:
 migrate-import --all                      Perform all migrations
 migrate-import --group=beer               Import all migrations in the beer group
 migrate-import beer_term,beer_node        Import new terms and nodes
 migrate-import beer_user --limit=2        Import no more than 2 users
 migrate-import beer_user --idlist=5       Import the user record with source ID 5

Arguments:
 migration                                 ID of migration(s) to import. Delimit multiple using commas.

  • mikeryan committed e308121 on 8.x-2.x authored by eojthebrave 
    Issue #2777971 by eojthebrave: Improve help text for migrate-import...
mikeryan’s picture

mikeryan
he/him
Primary language English
Location Murphysboro, IL, USA
CreditAttribution: mikeryan as a volunteer and at Virtuoso Performance commented
Status: Needs review » Fixed

Committed, thanks!

Status: Fixed » Closed (fixed)

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