Can’t resolve page references that point to other pages in the same import batch

[hissy]

When two pages in the same import batch link to each other using the {ccm:export:page:/path} token, the validator reports:

Referenced page at path /test-page-b cannot be found in the site or in the current content batch.

…even though /test-page-b (and /test-page) are included in that batch.

Also, the link in the imported page will be empty.

Since importing pages is a core feature of the Migration Tool, this bug significantly impacts its reliability and makes it difficult to use for real-world site migrations.

Steps to Reproduce

1. Create two pages: Test Page A (/test-page) and Test Page B (/test-page-b).
2. Add a Content block to each page containing a link to the other page.
3. Export both pages to CIF using the Migration Tool. Here's the xml: export.xml
4. Delete the two pages from the site.
5. Install/enable the Migration Tool add-on.
6. Create a new Import Batch and upload the XML from step 3.

Actual Result

The batch shows a validation warning for Test Page A:

Referenced page at path /test-page-b cannot be found in the site or in the current content batch.

…and similarly for Test Page B.

[hissy]

@aembler @mlocati Anyway, how do you run the tests in this package?

[mlocati]

@aembler @mlocati Anyway, how do you run the tests in this package?

It seems nobody has been taking care of tests for a long time - see the history of the tests folder.

[mlocati]

What about #66?

[hissy]

Reference errors have disappeared,

but the links in the imported pages are still empty.

[mlocati]

Which ConcreteCMS version are you using?

[hissy]

@mlocati Recent 9.4.x branch

[mlocati]

The same for me, but in my case it works.
Are you using multisite?

[hissy]

@mlocati No, clean installed one to write this issue

[mlocati]

What's your configuration for the "Destination of publication"?

[hissy]

@mlocati

Import Batches.

[mlocati]

Import Batches.

So I guess that NormalizePagePathsCommandHandler.php doesn't work as expected when importing to Import Batches.

I'll fix it.

[mlocati]

What about #68?

[hissy]

@mlocati Failed with this error message:

Messenger Worker Message Failed: /path/to/concrete/vendor/symfony/messenger/Middleware/HandleMessageMiddleware.php:130 Handling "Concrete\Core\Command\Batch\Command\HandleBatchMessageCommand" failed: Handling "PortlandLabs\Concrete5\MigrationTool\Publisher\Command\PublishPageContentCommand" failed: Concrete\Core\Block\BlockController::import(): Argument #3 ($blockNode) must be of type SimpleXMLElement, bool given, called in /path/to/packages/migration_tool/src/PortlandLabs/Concrete5/MigrationTool/Publisher/Block/CIFPublisher.php on line 17