Skip to content

Commit 44274f6

Browse files
authored
Merge branch 'main' into 1.2.4
2 parents d551c98 + f56670a commit 44274f6

File tree

5 files changed

+7
-54
lines changed

5 files changed

+7
-54
lines changed

.github/CODEOWNERS

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,3 +6,7 @@
66
# @global-owner1 and @global-owner2 will be requested for
77
# review when someone opens a pull request.
88
* @zlik @dabramovici @sol-loup @camcuz97 @agmurari @yang17 @nrostrow-meta @dalongi @palak-ana @n-nayak @singhramanpreett @luizmramosmeta @cristiannancumeta @singhalavi @jczhuoMeta @swinard-meta @strygo @fabik-meta @hsimsandhu
9+
10+
# These owners will be owning Catalog Directory, one would require to have
11+
# approval for making catalog related changes
12+
/app/code/Meta/Catalog/ @singhramanpreett @luizmramosmeta @cristiannancumeta @singhalavi @fabik-meta @ankurvjym @alexchertovmeta @somnotsam @danigirimeta @nanaasiedu

app/code/Meta/BusinessExtension/Helper/FBEHelper.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -263,7 +263,7 @@ public function getFBEExternalBusinessId($storeId)
263263
*/
264264
public function log($info, array $context = [])
265265
{
266-
if (!isset($context['log_type']) || !$this->systemConfig->isMetaTelemetryLoggingEnabled()) {
266+
if (!isset($context['log_type'])) {
267267
$this->logger->info($info);
268268
return;
269269
}
@@ -311,8 +311,8 @@ public function logException(Throwable $e, array $context = [])
311311

312312
public function logExceptionDetails($code, $message, $traceAsString, array $context = [])
313313
{
314-
// If the log type is not set or Meta extension logging is not enabled just log the error message and trace.
315-
if (!isset($context['log_type']) || !$this->systemConfig->isMetaExceptionLoggingEnabled()) {
314+
// If the log type is not set just log the error message and trace.
315+
if (!isset($context['log_type'])) {
316316
$this->logger->error($message);
317317
$this->logger->error($traceAsString);
318318
return;

app/code/Meta/BusinessExtension/Model/System/Config.php

Lines changed: 0 additions & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -117,11 +117,6 @@ class Config
117117
private const XML_PATH_FACEBOOK_BUSINESS_EXTENSION_ENABLE_COMMERCE_EXTENSION_BASE_URL =
118118
'facebook/internal/extension_base_url';
119119

120-
private const XML_PATH_FACEBOOK_BUSINESS_EXTENSION_ENABLE_META_EXCEPTION_LOGGING =
121-
'facebook/business_extension/meta_exception_logging_enabled';
122-
private const XML_PATH_FACEBOOK_BUSINESS_EXTENSION_ENABLE_META_TELEMETRY_LOGGING =
123-
'facebook/business_extension/meta_telemetry_logging_enabled';
124-
125120
/**
126121
* @var StoreManagerInterface
127122
*/
@@ -978,38 +973,6 @@ public function getServerTestCode(int $scopeId = null, string $scope = null): ?s
978973
);
979974
}
980975

981-
/**
982-
* Check if persisting exception logs to Meta is enabled
983-
*
984-
* @param int|null $scopeId
985-
* @param string|null $scope
986-
* @return string|null
987-
*/
988-
public function isMetaExceptionLoggingEnabled(int $scopeId = null, string $scope = null): ?string
989-
{
990-
return $this->getConfig(
991-
self::XML_PATH_FACEBOOK_BUSINESS_EXTENSION_ENABLE_META_EXCEPTION_LOGGING,
992-
$scopeId,
993-
$scope
994-
);
995-
}
996-
997-
/**
998-
* Check if persisting telemetry logs to Meta is enabled
999-
*
1000-
* @param int|null $scopeId
1001-
* @param string|null $scope
1002-
* @return string|null
1003-
*/
1004-
public function isMetaTelemetryLoggingEnabled(int $scopeId = null, string $scope = null): ?string
1005-
{
1006-
return $this->getConfig(
1007-
self::XML_PATH_FACEBOOK_BUSINESS_EXTENSION_ENABLE_META_TELEMETRY_LOGGING,
1008-
$scopeId,
1009-
$scope
1010-
);
1011-
}
1012-
1013976
/**
1014977
* Get store weight unit
1015978
*

app/code/Meta/BusinessExtension/etc/adminhtml/system.xml

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -45,18 +45,6 @@
4545
<config_path>facebook/business_extension/pixel_id</config_path>
4646
<frontend_model>Meta\BusinessExtension\Block\Adminhtml\System\Config\Field\Hidden</frontend_model>
4747
</field>
48-
<field id="meta_exception_logging_enabled" type="select" sortOrder="40" showInDefault="1"
49-
showInWebsite="0" showInStore="1">
50-
<config_path>facebook/business_extension/meta_exception_logging_enabled</config_path>
51-
<source_model>Magento\Config\Model\Config\Source\Yesno</source_model>
52-
<frontend_model>Meta\BusinessExtension\Block\Adminhtml\System\Config\Field\Hidden</frontend_model>
53-
</field>
54-
<field id="meta_telemetry_logging_enabled" type="select" sortOrder="50" showInDefault="1"
55-
showInWebsite="0" showInStore="1">
56-
<config_path>facebook/business_extension/meta_telemetry_logging_enabled</config_path>
57-
<source_model>Magento\Config\Model\Config\Source\Yesno</source_model>
58-
<frontend_model>Meta\BusinessExtension\Block\Adminhtml\System\Config\Field\Hidden</frontend_model>
59-
</field>
6048
<field id="installed" type="select" sortOrder="50" showInDefault="1"
6149
showInWebsite="0" showInStore="1">
6250
<config_path>facebook/business_extension/installed</config_path>

app/code/Meta/BusinessExtension/etc/config.xml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,6 @@
66
<business_extension>
77
<active>1</active>
88
<onsite>0</onsite>
9-
<meta_exception_logging_enabled>1</meta_exception_logging_enabled>
10-
<meta_telemetry_logging_enabled>0</meta_telemetry_logging_enabled>
119
<client_access_token>195311308289826|52dcd04d6c7ed113121b5eb4be23b4a7</client_access_token>
1210
</business_extension>
1311
<catalog_management>

0 commit comments

Comments
 (0)