Missing mapping for "nodereference_url" widget

This needed a reroll.

In regard to #42, 'Node Reference URL Widget', is added to the list of modules that will be upgraded and it is tested.

+++ b/core/modules/migrate_drupal_ui/tests/src/Functional/d6/MultilingualReviewPageTest.php
@@ -101,6 +101,7 @@ protected function getAvailablePaths() {
+      'Node Reference URL Widget',

+++ b/core/modules/migrate_drupal_ui/tests/src/Functional/d6/NoMultilingualReviewPageTest.php
@@ -93,6 +93,7 @@ protected function getAvailablePaths() {
+      'Node Reference URL Widget',

+++ b/core/modules/migrate_drupal_ui/tests/src/Functional/d6/Upgrade6Test.php
@@ -150,6 +150,7 @@ protected function getAvailablePaths() {
+      'Node Reference URL Widget',

To keep this moving I created a fresh D6 site, downloaded CCK and NodeReferenceUrl, and created content with a field that uses the url node reference widget. And here is a screenshot of upgrading that site from /upgrade

Thanks @quietone, know we briefly spoke in slack about this one in the #migrate channel.

Change looks good and test coverage is there so think it's good.

The last submitted patch, 44: 3000717-44.patch, failed testing. View results →

Test failure on

1) Drupal\Tests\ckeditor5\FunctionalJavascript\MediaLibraryTest::testButton
Failed asserting that a NULL is not empty.

Retesting.

Restarted the tests but that’s one of the random ones we see now

The last submitted patch, 44: 3000717-44.patch, failed testing. View results →

Random failure.

Crediting @benjifisher

Committed to 10.1.x, needs re-roll for 10.0.x and 9.5.x so marking it as patch (to be ported).

Added patch for 9.5.x

Tried fixing test fail

Committed/pushed/cherry-picked the backport to 10.0.x and 9.5.x, thanks!

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