Skip to content

Commit 5626885

Browse files
author
Joan He
committed
Merge remote-tracking branch 'arcticfoxes/MAGETWO-98765' into 2.1.18-develop-pr
2 parents 3684da4 + 3ef7442 commit 5626885

File tree

1 file changed

+7
-7
lines changed
  • app/code/Magento/Newsletter/view/adminhtml/templates/template

1 file changed

+7
-7
lines changed

app/code/Magento/Newsletter/view/adminhtml/templates/template/edit.phtml

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ use Magento\Framework\App\TemplateTypesInterface;
2121
<form action="<?= $block->escapeUrl($block->getPreviewUrl()) ?>" method="post" id="newsletter_template_preview_form" target="_blank">
2222
<?= $block->getBlockHtml('formkey') ?>
2323
<div class="no-display">
24-
<input type="hidden" id="preview_type" name="type" value="<?php /* @escapeNotVerified */ echo $block->isTextType() ? 1 : 2 ?>" />
24+
<input type="hidden" id="preview_type" name="type" value="<?= /* @noEscape */ $block->isTextType() ? 1 : 2 ?>" />
2525
<input type="hidden" id="preview_text" name="text" value="" />
2626
<input type="hidden" id="preview_styles" name="styles" value="" />
2727
<input type="hidden" id="preview_id" name="id" value="" />
@@ -87,7 +87,7 @@ require([
8787
var self = this;
8888

8989
confirm({
90-
content: "<?php /* @escapeNotVerified */ echo __('Are you sure that you want to strip all tags?') ?>",
90+
content: "<?= $block->escapeJsQuote($block->escapeHtml(__('Are you sure that you want to strip all tags?'))) ?>",
9191
actions: {
9292
confirm: function(){
9393
if(self.isEditor()) {
@@ -141,8 +141,8 @@ require([
141141
if($F('code').blank() || $F('code')==templateControl.templateName) {
142142

143143
prompt({
144-
content: '<?php /* @escapeNotVerified */ echo __('Please enter a new template name.') ?>',
145-
value: templateControl.templateName + '<?php /* @escapeNotVerified */ echo __(' Copy') ?>',
144+
content: '<?= $block->escapeJsQuote($block->escapeHtml(__('Please enter a new template name.'))) ?>',
145+
value: templateControl.templateName + '<?= $block->escapeJsQuote(__(' Copy')) ?>',
146146
actions: {
147147
confirm: function(value) {
148148
$('code').value = value;
@@ -176,9 +176,9 @@ require([
176176

177177
preview: function() {
178178
if (this.typeChange) {
179-
$('preview_type').value = <?php /* @escapeNotVerified */ echo TemplateTypesInterface::TYPE_TEXT ?>;
179+
$('preview_type').value = <?= $block->escapeJsQuote(TemplateTypesInterface::TYPE_TEXT) ?>;
180180
} else {
181-
$('preview_type').value = <?php /* @escapeNotVerified */ echo $block->getTemplateType() ?>;
181+
$('preview_type').value = <?= $block->escapeJsQuote($block->getTemplateType()) ?>;
182182
}
183183
if (this.isEditor() && tinyMCE.get(this.id)) {
184184
tinyMCE.triggerSave();
@@ -222,7 +222,7 @@ require([
222222
};
223223

224224
templateControl.init();
225-
templateControl.templateName = "<?php /* @escapeNotVerified */ echo $block->getJsTemplateName() ?>";
225+
templateControl.templateName = "<?= $block->escapeJsQuote($block->getJsTemplateName()) ?>";
226226
//]]>
227227

228228
});

0 commit comments

Comments
 (0)