Skip to content

Commit d362138

Browse files
authored
MDEE-78: Improve logging (#108)
* MDEE-78: Improve logging
1 parent c945112 commit d362138

File tree

48 files changed

+186
-86
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

48 files changed

+186
-86
lines changed

BundleProductDataExporter/Model/Provider/Product/BundleProductOptions.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
use Magento\CatalogDataExporter\Model\Provider\Product\OptionProviderInterface;
1414
use Magento\DataExporter\Exception\UnableRetrieveData;
1515
use Magento\Framework\App\ResourceConnection;
16-
use Psr\Log\LoggerInterface;
16+
use Magento\DataExporter\Model\Logging\CommerceDataExportLoggerInterface as LoggerInterface;
1717
use Throwable;
1818

1919
/**
@@ -93,7 +93,7 @@ public function get(array $values) : array
9393
}
9494
}
9595
} catch (Throwable $exception) {
96-
$this->logger->error($exception->getMessage());
96+
$this->logger->error($exception->getMessage(), ['exception' => $exception]);
9797
throw new UnableRetrieveData('Unable to retrieve bundle product options data');
9898
}
9999

CatalogDataExporter/Model/Provider/Categories.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
use Magento\DataExporter\Exception\UnableRetrieveData;
1515
use Magento\Framework\App\ResourceConnection;
1616
use Magento\Store\Model\Store;
17-
use Psr\Log\LoggerInterface;
17+
use Magento\DataExporter\Model\Logging\CommerceDataExportLoggerInterface as LoggerInterface;
1818

1919
/**
2020
* Categories main data provider
@@ -110,7 +110,7 @@ public function get(array $values) : array
110110
));
111111
}
112112
} catch (\Throwable $exception) {
113-
$this->logger->error($exception->getMessage());
113+
$this->logger->error($exception->getMessage(), ['exception' => $exception]);
114114
throw new UnableRetrieveData('Unable to retrieve category data');
115115
}
116116

CatalogDataExporter/Model/Provider/Category/AvailableSortBy.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
namespace Magento\CatalogDataExporter\Model\Provider\Category;
1010

1111
use Magento\DataExporter\Exception\UnableRetrieveData;
12-
use Psr\Log\LoggerInterface;
12+
use Magento\DataExporter\Model\Logging\CommerceDataExportLoggerInterface as LoggerInterface;
1313

1414
/**
1515
* Category available sort by data provider
@@ -57,7 +57,7 @@ public function get(array $values) : array
5757
}
5858
}
5959
} catch (\Throwable $exception) {
60-
$this->logger->error($exception->getMessage());
60+
$this->logger->error($exception->getMessage(), ['exception' => $exception]);
6161
throw new UnableRetrieveData('Unable to retrieve category available sort by data');
6262
}
6363

CatalogDataExporter/Model/Provider/Category/Breadcrumbs.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
use Magento\CatalogDataExporter\Model\Query\Eav\CategoryAttributeQueryBuilder;
1212
use Magento\DataExporter\Exception\UnableRetrieveData;
1313
use Magento\Framework\App\ResourceConnection;
14-
use Psr\Log\LoggerInterface;
14+
use Magento\DataExporter\Model\Logging\CommerceDataExportLoggerInterface as LoggerInterface;
1515

1616
/**
1717
* Category breadcrumbs data provider
@@ -90,7 +90,7 @@ public function get(array $values): array
9090
);
9191
}
9292
} catch (\Throwable $exception) {
93-
$this->logger->error($exception->getMessage());
93+
$this->logger->error($exception->getMessage(), ['exception' => $exception]);
9494
throw new UnableRetrieveData('Unable to retrieve category breadcrumbs data');
9595
}
9696

CatalogDataExporter/Model/Provider/Category/Children.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
namespace Magento\CatalogDataExporter\Model\Provider\Category;
1010

1111
use Magento\DataExporter\Exception\UnableRetrieveData;
12-
use Psr\Log\LoggerInterface;
12+
use Magento\DataExporter\Model\Logging\CommerceDataExportLoggerInterface as LoggerInterface;
1313
use Magento\Catalog\Model\ResourceModel\Category\CollectionFactory;
1414

1515
/**
@@ -63,7 +63,7 @@ public function get(array $values) : array
6363
}
6464
}
6565
} catch (\Throwable $exception) {
66-
$this->logger->error($exception->getMessage());
66+
$this->logger->error($exception->getMessage(), ['exception' => $exception]);
6767
throw new UnableRetrieveData('Unable to retrieve category children');
6868
}
6969

CatalogDataExporter/Model/Provider/Category/DefaultSortBy.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010

1111
use Magento\Catalog\Model\Config as CatalogConfig;
1212
use Magento\DataExporter\Exception\UnableRetrieveData;
13-
use Psr\Log\LoggerInterface;
13+
use Magento\DataExporter\Model\Logging\CommerceDataExportLoggerInterface as LoggerInterface;
1414

1515
/**
1616
* Category default sort by data provider
@@ -79,7 +79,7 @@ public function get(array $values) : array
7979
];
8080
}
8181
} catch (\Throwable $exception) {
82-
$this->logger->error($exception->getMessage());
82+
$this->logger->error($exception->getMessage(), ['exception' => $exception]);
8383
throw new UnableRetrieveData('Unable to retrieve category default sort by data');
8484
}
8585

CatalogDataExporter/Model/Provider/Category/Formatter/Output.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010

1111
use Magento\Catalog\Helper\Output as OutputFormatter;
1212
use Magento\DataExporter\Exception\UnableRetrieveData;
13-
use Psr\Log\LoggerInterface;
13+
use Magento\DataExporter\Model\Logging\CommerceDataExportLoggerInterface as LoggerInterface;
1414

1515
/**
1616
* Prepare category attribute html output
@@ -69,7 +69,7 @@ public function format(array $row) : array
6969
}
7070
}
7171
} catch (\Throwable $exception) {
72-
$this->logger->error($exception->getMessage());
72+
$this->logger->error($exception->getMessage(), ['exception' => $exception]);
7373
throw new UnableRetrieveData('Unable to retrieve category formatted attribute data');
7474
}
7575

CatalogDataExporter/Model/Provider/Category/Image.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
use Magento\Framework\Filesystem\Directory\WriteInterface;
1818
use Magento\Framework\UrlInterface;
1919
use Magento\Store\Model\StoreManagerInterface;
20-
use Psr\Log\LoggerInterface;
20+
use Magento\DataExporter\Model\Logging\CommerceDataExportLoggerInterface as LoggerInterface;
2121

2222
/**
2323
* Category image data provider
@@ -135,7 +135,7 @@ public function get(array $values) : array
135135
];
136136
}
137137
} catch (\Throwable $exception) {
138-
$this->logger->error($exception->getMessage());
138+
$this->logger->error($exception->getMessage(), ['exception' => $exception]);
139139
throw new UnableRetrieveData('Unable to retrieve category image');
140140
} finally {
141141
$this->storeManager->setCurrentStore($actualStoreCode);

CatalogDataExporter/Model/Provider/Category/PathInStore.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010

1111
use Magento\DataExporter\Exception\UnableRetrieveData;
1212
use Magento\Store\Model\StoreManagerInterface;
13-
use Psr\Log\LoggerInterface;
13+
use Magento\DataExporter\Model\Logging\CommerceDataExportLoggerInterface as LoggerInterface;
1414

1515
/**
1616
* Category store path data provider
@@ -76,7 +76,7 @@ public function get(array $values) : array
7676
];
7777
}
7878
} catch (\Throwable $exception) {
79-
$this->logger->error($exception->getMessage());
79+
$this->logger->error($exception->getMessage(), ['exception' => $exception]);
8080
throw new UnableRetrieveData('Unable to retrieve category store path data');
8181
}
8282

CatalogDataExporter/Model/Provider/EavAttributes/EavAttributesProvider.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
use Magento\DataExporter\Exception\UnableRetrieveData;
1313
use Magento\DataExporter\Sql\FieldToPropertyNameConverter;
1414
use Magento\Framework\App\ResourceConnection;
15-
use Psr\Log\LoggerInterface;
15+
use Magento\DataExporter\Model\Logging\CommerceDataExportLoggerInterface as LoggerInterface;
1616

1717
/**
1818
* Eav attributes data provider
@@ -97,7 +97,7 @@ public function getEavAttributesData(array $entityIds, string $storeCode, array
9797
return $this->formatEavAttributesArray($data, $attributeCodes);
9898
}, $this->attributesDataConverter->convert($attributes));
9999
} catch (\Throwable $exception) {
100-
$this->logger->error($exception->getMessage());
100+
$this->logger->error($exception->getMessage(), ['exception' => $exception]);
101101
throw new UnableRetrieveData('Unable to retrieve category eav attributes');
102102
}
103103
}

0 commit comments

Comments
 (0)