Problem/Motivation

Sandbox gives error "(Migrate Message) [node]: title=This value should not be null." on out-of-the-box D7 Node Source Example (d7_node) example.

Steps to reproduce

1. Install module
2. Select "D7 Node Source Example (d7_node)"
3. Click 'Save & Run' button.
4. See error message "(Migrate Message) [node]: title=This value should not be null."
5. Results show empty title, e.g. title: { }

Comments

jonathan_hunt created an issue. See original summary.

danflanagan8’s picture

Yeah, that probably shouldn't happen, huh?

I'll update the config for that example.

danflanagan8’s picture

I'm wondering if I want to make a broader change here. The issue is not so much with the node source as it is with trying to use the node destination. MigrateSandboxEntityContentBase could make life easier by giving the node a fake title to avoid validation errors. There could be a message added to alert the user that a stand-in title was used.

I think that's what I'll do.

In the meantime, adding title: title to the process pipeline will get rid of that validation error.

danflanagan8’s picture

Status: Active » Needs review
StatusFileSize
new6.32 KB

Here's a patch that makes the sandbox entity destination a bit more forgiving with the entity label validation. This also makes some related changes to the severity of status messages with updates to the corresponding tests. This was also a good opportunity to make use of the relatively new status messages assertions in the tests.

Status: Needs review » Needs work

The last submitted patch, 4: migrate_sandbox-validation-3308754-4.patch, failed testing. View results

danflanagan8’s picture

Status: Needs work » Needs review
StatusFileSize
new6.37 KB
new888 bytes

That yaml validation always causes me fits on the test runner here. Let's try this...

Status: Needs review » Needs work

The last submitted patch, 6: migrate_sandbox-validation-3308754-6.patch, failed testing. View results

danflanagan8’s picture

Status: Needs work » Needs review
StatusFileSize
new6.37 KB
new1.47 KB

oops. Wrong place to wait. Try this...

Status: Needs review » Needs work

The last submitted patch, 8: migrate_sandbox-validation-3308754-8.patch, failed testing. View results

danflanagan8’s picture

Status: Needs work » Needs review
StatusFileSize
new944 bytes
new6.31 KB

What!?! How about this?

Status: Needs review » Needs work

The last submitted patch, 10: migrate_sandbox-validation-3308754-10.patch, failed testing. View results

danflanagan8’s picture

Status: Needs work » Needs review
StatusFileSize
new6.44 KB

I give up trying to understand this. This one will pass.

  • danflanagan8 committed c2e0192 on 1.0.x
    Issue #3308754 by danflanagan8, jonathan_hunt: D7 Node Source Example (...
danflanagan8’s picture

Status: Needs review » Fixed

This will be in a new release shortly

Status: Fixed » Closed (fixed)

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