Problem/Motivation

A major leap forward from the Drupal 7 Migrate will be the ability to create migrations from scratch through the UI. This will entail a lot of UI work, since many pieces need to be put together to form a usable migration (source plugins, destination plugins, field mappings, ...).

Proposed resolution

TBD.

Remaining tasks

Everything.

User interface changes

Ayup.

API changes

N/A

Data model changes

N/A

Original issue summary

I got this error when I tried to add a new migration in the UI.
Strict warning: Declaration of Drupal\migrate_tools\Controller\MigrationGroupListBuilder::buildRow() should be compatible with Drupal\Core\Entity\EntityListBuilder::buildRow(Drupal\Core\Entity\EntityInterface $entity) in require() (line 21 of modules/migrate_plus/migrate_tools/src/Controller/MigrationGroupListBuilder.php).

Comments

adooo created an issue. See original summary.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous as a volunteer commented

I get the following 5 log messages when trying to add a new migration. Presume it's in heavy development at the moment but seen as similar issue posted I thought I'd add further details:

Strict warning: Declaration of Drupal\migrate_tools\Controller\MigrationGroupListBuilder::buildRow() should be compatible with Drupal\Core\Entity\EntityListBuilder::buildRow(Drupal\Core\Entity\EntityInterface $entity) in require() (line 21 of /Users/steve/Sites/d8dev/modules/migrate_plus/migrate_tools/src/Controller/MigrationGroupListBuilder.php).

Strict warning: Declaration of Drupal\migrate_tools\Controller\MigrationListBuilder::getDefaultOperations() should be compatible with Drupal\Core\Config\Entity\ConfigEntityListBuilder::getDefaultOperations(Drupal\Core\Entity\EntityInterface $entity) in require() (line 27 of /Users/steve/Sites/d8dev/modules/migrate_plus/migrate_tools/src/Controller/MigrationListBuilder.php).

Strict warning: Declaration of Drupal\migrate_tools\Controller\MigrationListBuilder::buildRow() should be compatible with Drupal\Core\Entity\EntityListBuilder::buildRow(Drupal\Core\Entity\EntityInterface $entity) in require() (line 27 of /Users/steve/Sites/d8dev/modules/migrate_plus/migrate_tools/src/Controller/MigrationListBuilder.php).

Recoverable fatal error: Argument 2 passed to Drupal\migrate\Plugin\MigratePluginManager::createInstance() must be of the type array, null given, called in /Users/steve/Sites/d8dev/core/modules/migrate/src/Entity/Migration.php on line 266 and defined in Drupal\migrate\Plugin\MigratePluginManager->createInstance() (line 60 of /Users/steve/Sites/d8dev/core/modules/migrate/src/Plugin/MigratePluginManager.php).

Drupal\Component\Plugin\Exception\PluginNotFoundException: The "" plugin does not exist. in Drupal\Core\Plugin\DefaultPluginManager->doGetDefinition() (line 57 of /Users/steve/Sites/d8dev/core/lib/Drupal/Component/Plugin/Discovery/DiscoveryTrait.php).

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous as a volunteer commented

Just chatted with mikeryan on irc and these errors are due to the ui still in heavy dev so using drush instead. Suggest postponing this issue.

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Murphysboro, IL, USA
CreditAttribution: mikeryan at Acquia commented
Title: White screen when adding new migration » Support adding migrations through the UI
Category: Bug report » Feature request
Parent issue: » #2470855: [meta] Implement a general-purpose UI for migration

"Add migration" is not yet implemented, and won't be for a while - a general-purpose migration creator will be a big task, since there are so many pieces to put together (source plugin, destination plugin, field mappings...). I'll remove the button for now so no one is misled.

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Murphysboro, IL, USA
CreditAttribution: mikeryan at Acquia commented
Issue summary: View changes

  • mikeryan committed 07383e5 on 8.x-1.x 
    Issue #2573241 by mikeryan: Disable unimplemented features for now
mikeryan’s picture

mikeryan
he/him
Primary language English
Location Murphysboro, IL, USA
CreditAttribution: mikeryan at Acquia commented
Status: Active » Postponed

I've removed the Edit and Delete operations from the migration listing, since those aren't implemented either. I simply could not find where the "Add migration" button gets added to the form, and I have other things to work on, so for now I made the add form unsubmittable with a note that it's not yet implemented.

This work isn't going to commence any time soon - we've still got plenty of core migration work to do, and on the tools side the plan is to implement all the capabilities we can from the Drupal 7 migrate tools before tackling new features.

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Murphysboro, IL, USA
CreditAttribution: mikeryan at Acquia commented
Project: Migrate Plus » Migrate Tools
Version: 8.x-1.x-dev »
mikeryan’s picture

mikeryan
he/him
Primary language English
Location Murphysboro, IL, USA
CreditAttribution: mikeryan commented
Version: » 8.x-2.x-dev
andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost as a volunteer and at Skilld commented
Version: 8.x-2.x-dev » 8.x-4.x-dev

The link to add migration still listed but leads to 404 with white screen

better to provide stubs like if uncomment "add" form for migrate entity pointing to the issue

heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC, Acro Commerce commented
Status: Postponed » Closed (outdated)

Let's continue to watch the work that feeds_migrate is doing. I think they are building out a decent UI based on a time tested UI and UX that has made Feeds popular. While using migrate under the hood.