Skip to content

Commit 107fb44

Browse files
authored
MDEE-966: Add commerce data export logging to the mview update plugin
1 parent f1b1f66 commit 107fb44

File tree

1 file changed

+12
-8
lines changed

1 file changed

+12
-8
lines changed

DataExporter/Plugin/MviewUpdatePlugin.php

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@
2020

2121
use Magento\DataExporter\Model\Indexer\ViewMaterializer;
2222
use Magento\DataExporter\Model\Indexer\FeedIndexer;
23+
use Magento\DataExporter\Model\Logging\CommerceDataExportLoggerInterface;
2324
use Magento\Framework\Exception\BulkException;
2425
use Magento\Framework\Mview\Processor;
2526
use Magento\Framework\Mview\View\CollectionFactory;
@@ -30,26 +31,23 @@
3031

3132
class MviewUpdatePlugin
3233
{
33-
/**
34-
* @var CollectionFactory
35-
*/
3634
private CollectionFactory $viewsFactory;
37-
38-
/**
39-
* @var ViewMaterializer
40-
*/
4135
private ViewMaterializer $viewMaterializer;
36+
private CommerceDataExportLoggerInterface $logger;
4237

4338
/**
4439
* @param CollectionFactory $viewsFactory
4540
* @param ViewMaterializer $viewMaterializer
41+
* @param CommerceDataExportLoggerInterface $logger
4642
*/
4743
public function __construct(
4844
CollectionFactory $viewsFactory,
49-
ViewMaterializer $viewMaterializer
45+
ViewMaterializer $viewMaterializer,
46+
CommerceDataExportLoggerInterface $logger
5047
) {
5148
$this->viewsFactory = $viewsFactory;
5249
$this->viewMaterializer = $viewMaterializer;
50+
$this->logger = $logger;
5351
}
5452

5553
/**
@@ -83,6 +81,12 @@ public function aroundUpdate(Processor $subject, callable $proceed, $group = '')
8381
}
8482

8583
if ($exception->wasErrorAdded()) {
84+
foreach ($exception->getErrors() as $e) {
85+
$this->logger->error(
86+
'Data Exporter exception has occurred: ' . $e->getMessage(),
87+
['exception' => $e]
88+
);
89+
}
8690
throw $exception;
8791
}
8892
}

0 commit comments

Comments
 (0)