D6 migration fails on user_profile_entity display. This is on a test site with 2 users, using profiles and the 'link' profile field.

Importing user_profile_entity_display
The "link_default" plugin does not exist. (/opt/sites/md82/core/lib/Drupal/Component/Plugin/Discovery/DiscoveryTrait.php:57)                                                                            [error]
CommentFileSizeAuthor
#10 interdiff-2558931-6-10.txt1.68 KBquietone
#10 2558931-10.patch6.44 KBquietone
#6 interdiff-2558931-6.txt524 bytesphenaproxima
#6 2558931-6.patch5.2 KBphenaproxima
#6 2558931-6-FAIL.patch4.98 KBphenaproxima
#3 2558931-3.patch566 bytesphenaproxima

Comments

quietone created an issue. See original summary.

quietone’s picture

quietone commented
Title: link_default plugin does not exist » link_default plugin does not exist (d6)
phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Status: Active » Needs review
StatusFileSize
new 2558931-3.patch566 bytes

Does this help?

quietone’s picture

quietone commented

I swear I tried that! Didn't work then but seems to be now.

quietone’s picture

quietone commented
Status: Needs review » Needs work
Issue tags: +Needs tests
phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Status: Needs work » Needs review
Issue tags: -Needs tests
StatusFileSize
new 2558931-6-FAIL.patch4.98 KB
new 2558931-6.patch5.2 KB
new interdiff-2558931-6.txt524 bytes

Behold -- tests! And a fix!

The last submitted patch, 6: 2558931-6-FAIL.patch, failed testing.

The last submitted patch, 6: 2558931-6-FAIL.patch, failed testing.

quietone’s picture

quietone commented 
+++ b/core/modules/user/migration_templates/user_profile_entity_display.yml
@@ -39,3 +39,6 @@ process:
+migration_dependencies:
+  required:
+    - user_profile_field_instance

Why is this needed?

/*
    // Migrated URL field.
    $field = FieldConfig::load('user.user.profile_blog');
    $this->assertIdentical('Your blog', $field->label());
    $this->assertIdentical("Paste the full URL, $field->getDescription(), including http://, of your personal blog.");
*/

Shouldn't this test, in MigrateUserProfileFieldInstanceTest.php, be restored as part of this issue?

quietone’s picture

quietone commented
StatusFileSize
new 2558931-10.patch6.44 KB
new interdiff-2558931-6-10.txt1.68 KB

Something like this.

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Pittsfield, MA, USA
commented
Status: Needs review » Reviewed & tested by the community

Looks good!

  • webchick committed 560db5b on 8.0.x 
    Issue #2558931 by phenaproxima, quietone: link_default plugin does not...
webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.0.x. Thanks!

Status: Fixed » Closed (fixed)

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

giuliovale’s picture

giuliovale commented

.