Skip to content

Commit 8aafc69

Browse files
committed
Resolve "Redirect to CMS-page if Cookies are Disabled" is "No" but it still redirect issue25148
1 parent abbfa03 commit 8aafc69

File tree

3 files changed

+28
-6
lines changed

3 files changed

+28
-6
lines changed

app/code/Magento/Cookie/Block/RequireCookie.php

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,15 +3,20 @@
33
* Copyright © Magento, Inc. All rights reserved.
44
* See COPYING.txt for license details.
55
*/
6-
6+
declare(strict_types=1);
77
/**
88
* Frontend form key content block
99
*/
1010
namespace Magento\Cookie\Block;
1111

12+
use Magento\Framework\App\ObjectManager;
13+
use Magento\Framework\App\Config\ScopeConfigInterface;
14+
1215
/**
1316
* @api
1417
* @since 100.0.2
18+
*
19+
* Class \Magento\Cookie\Block\RequireCookie
1520
*/
1621
class RequireCookie extends \Magento\Framework\View\Element\Template
1722
{
@@ -22,9 +27,12 @@ class RequireCookie extends \Magento\Framework\View\Element\Template
2227
*/
2328
public function getScriptOptions()
2429
{
30+
$isRedirectCmsPage = ObjectManager::getInstance()->get(ScopeConfigInterface::class)
31+
->getValue('web/browser_capabilities/cookies');
2532
$params = [
2633
'noCookieUrl' => $this->escapeUrl($this->getUrl('cookie/index/noCookies/')),
27-
'triggers' => $this->escapeHtml($this->getTriggers())
34+
'triggers' => $this->escapeHtml($this->getTriggers()),
35+
'isRedirectCmsPage' => (boolean)$isRedirectCmsPage
2836
];
2937
return json_encode($params);
3038
}

app/code/Magento/Cookie/i18n/en_US.csv

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,3 +11,5 @@
1111
"Cookie Domain","Cookie Domain"
1212
"Use HTTP Only","Use HTTP Only"
1313
"Cookie Restriction Mode","Cookie Restriction Mode"
14+
"Cookies are disabled in your browser.","Cookies are disabled in your browser."
15+

app/code/Magento/Cookie/view/frontend/web/js/require-cookie.js

Lines changed: 16 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,15 +8,19 @@
88
*/
99
define([
1010
'jquery',
11-
'jquery-ui-modules/widget'
12-
], function ($) {
11+
'Magento_Ui/js/modal/alert',
12+
'jquery-ui-modules/widget',
13+
'mage/mage',
14+
'mage/translate'
15+
], function ($, alert) {
1316
'use strict';
1417

1518
$.widget('mage.requireCookie', {
1619
options: {
1720
event: 'click',
1821
noCookieUrl: 'enable-cookies',
19-
triggers: ['.action.login', '.action.submit']
22+
triggers: ['.action.login', '.action.submit'],
23+
isRedirectCmsPage: true
2024
},
2125

2226
/**
@@ -49,8 +53,16 @@ define([
4953
if (navigator.cookieEnabled) {
5054
return;
5155
}
56+
5257
event.preventDefault();
53-
window.location = this.options.noCookieUrl;
58+
59+
if (this.options.isRedirectCmsPage) {
60+
window.location = this.options.noCookieUrl;
61+
} else {
62+
alert({
63+
content: $.mage.__('Cookies are disabled in your browser.')
64+
});
65+
}
5466
}
5567
});
5668

0 commit comments

Comments
 (0)