- Issue created by @joachim
- π¬π§United Kingdom joachim
The destination property is added to the exception message in import():
catch (MigrateException $e) { $this->getIdMap()->saveIdMapping($row, [], $e->getStatus()); $msg = sprintf("%s:%s:%s", $this->migration->getPluginId(), $destination_property_name, $e->getMessage()); $this->saveMessage($msg, $e->getLevel()); $save = FALSE; }
But SubProcess calls processRow().
I *think* we can fix this by adding the same handling in processRow() -- that is not called by the main import process AFAICT.
- @joachim opened merge request.
- Status changed to Needs review
7 months ago 12:02pm 8 November 2023 - Status changed to Needs work
7 months ago 1:55pm 8 November 2023 - πΊπΈUnited States smustgrave
Thanks for reporting! Could we get a test case showing the issue or something testing the new exception.
- π¬π§United Kingdom joachim
I'm adding tests here: π Add kernel tests for prefixing of migrate process errors Needs review .