- 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
over 1 year ago 12:02pm 8 November 2023 - Status changed to Needs work
over 1 year 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 .
- πΊπΈUnited States dcam
π Migrate exception message should give the index of the process plugin in the pipeline Active adds indexes to subprocess pipeline exception messages to indicate which plugin had the problem. Is this a duplicate of that issue?