Skip to content

Commit 7bd81ec

Browse files
author
Oleksandr Gorkun
committed
MC-34764: Fix performance degradation cause by CSP
1 parent 376a0ad commit 7bd81ec

File tree

2 files changed

+8
-6
lines changed
  • app/code/Magento

2 files changed

+8
-6
lines changed

app/code/Magento/Catalog/view/adminhtml/templates/catalog/product/composite/configure.phtml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,10 @@
4646
prodCompConfIframe.style.top = "-1000px";
4747
prodCompConfIframe.style.left = "-1000px";
4848
document.querySelectorAll(".product_composite_configure_messages").forEach((e) => e.style.display = "none");
49-
document.querySelectorAll(".product_composite_configure_form_additional").forEach((e) => e.style.display = "none");
50-
document.querySelectorAll(".product_composite_configure_form_confirmed").forEach((e) => e.style.display = "none");
49+
document.querySelectorAll(".product_composite_configure_form_additional")
50+
.forEach((e) => e.style.display = "none");
51+
document.querySelectorAll(".product_composite_configure_form_confirmed")
52+
.forEach((e) => e.style.display = "none");
5153
document.querySelectorAll(".product_composite_configure_confirmed").forEach((e) => e.style.display = "none");
5254
document.querySelectorAll(".product-configure-popup").forEach((e) => e.style.display = "none");
5355

app/code/Magento/Swatches/view/frontend/templates/product/layered/renderer.phtml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -58,11 +58,11 @@
5858
$script = 'let ' .$element
5959
.' = document.querySelector(\'div[data-option-id="' .$escaper->escapeJs($option)
6060
.'"]\');' .PHP_EOL;
61-
$script .= $element .'.style.background = "background: url(\''
61+
$script .= $element .'.style.background = "url(\''
6262
.$escapedUrl .'\') no-repeat center";' .PHP_EOL;
6363
$script .= $element .'.style.backgroundSize = "initial";';
64-
echo /* @noEscape*/ $secureRenderer->renderTag('script', [], $script, false);
6564
?>
65+
<?= /* @noEscape*/ $secureRenderer->renderTag('script', [], $script, false); ?>
6666
<?php break;
6767
case '1':
6868
?>
@@ -84,11 +84,11 @@
8484
$script = 'let ' .$element
8585
.' = document.querySelector(\'div[data-option-id="' .$escaper->escapeJs($option)
8686
.'"]\');' .PHP_EOL;
87-
$script .= $element .'.style.background = "background: ' .$backgroundValue
87+
$script .= $element .'.style.background = "' .$backgroundValue
8888
.' no-repeat center";' .PHP_EOL;
8989
$script .= $element .'.style.backgroundSize = "initial";';
90-
echo /* @noEscape*/ $secureRenderer->renderTag('script', [], $script, false);
9190
?>
91+
<?= /* @noEscape*/ $secureRenderer->renderTag('script', [], $script, false); ?>
9292
<?php break;
9393
case '0':
9494
default:

0 commit comments

Comments
 (0)