Skip to content

Commit 1bc2d6d

Browse files
authored
Merge pull request #9791 from magento-gl/spartans_08052025_AC-14597
[Spartans] Bugfix Delivery
2 parents e5271c3 + 6bee163 commit 1bc2d6d

File tree

5 files changed

+21
-17
lines changed

5 files changed

+21
-17
lines changed

app/code/Magento/ProductAlert/Model/Mailing/AlertProcessor.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
<?php
22
/**
3-
* Copyright © Magento, Inc. All rights reserved.
4-
* See COPYING.txt for license details.
3+
* Copyright 2021 Adobe
4+
* All rights reserved.
55
*/
66
declare(strict_types=1);
77

@@ -128,7 +128,7 @@ public function process(string $alertType, array $customerIds, int $websiteId):
128128
if (!empty($errors)) {
129129
/** @var Website $website */
130130
$website = $this->storeManager->getWebsite($websiteId);
131-
$defaultStoreId = $website->getDefaultStore()->getId();
131+
$defaultStoreId = (int)$website->getDefaultStore()->getId();
132132
$this->errorEmailSender->execute($errors, $defaultStoreId);
133133
}
134134
}

app/code/Magento/ProductAlert/etc/crontab.xml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,13 @@
11
<?xml version="1.0"?>
22
<!--
33
/**
4-
* Copyright © Magento, Inc. All rights reserved.
5-
* See COPYING.txt for license details.
4+
* Copyright 2021 Adobe
5+
* All rights reserved.
66
*/
77
-->
88
<config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="urn:magento:module:Magento_Cron:etc/crontab.xsd">
99
<group id="default">
10-
<job name="product_alert" instance="Magento\ProductAlert\Model\Observer" method="process">
10+
<job name="catalog_product_alert" instance="Magento\ProductAlert\Model\Observer" method="process">
1111
<config_path>crontab/default/jobs/catalog_product_alert/schedule/cron_expr</config_path>
1212
</job>
1313
</group>

app/code/Magento/Theme/Ui/Component/Design/Config/DataProvider.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -126,8 +126,10 @@ public function getData()
126126
private function getCoreConfigData(): array
127127
{
128128
$connection = $this->resourceConnection->getConnection();
129+
$tableName = $this->resourceConnection->getTableName('core_config_data');
130+
129131
return $connection->fetchAll(
130-
$connection->select()->from($connection->getTableName('core_config_data'))
132+
$connection->select()->from($tableName)
131133
->where('path = ?', 'design/theme/theme_id')
132134
);
133135
}

app/code/Magento/User/view/adminhtml/web/js/roles-tree.js

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
/**
2-
* Copyright © Magento, Inc. All rights reserved.
3-
* See COPYING.txt for license details.
2+
* Copyright 2013 Adobe
3+
* All rights reserved.
44
*/
55

66
/**
@@ -12,8 +12,6 @@ define([
1212
'jquery/jstree/jquery.jstree',
1313
'mage/translate'
1414
], function ($) {
15-
'use strict';
16-
1715
// jscs:disable requireCamelCaseOrUpperCaseIdentifiers
1816
$.widget('mage.rolesTree', {
1917
options: {
@@ -97,9 +95,13 @@ define([
9795
_destroy: function () {
9896
this.element.jstree('destroy');
9997

100-
this.buttons.forEach(function (element) {
101-
element.parentNode.removeChild(element);
102-
});
98+
if (this.buttons) {
99+
this.buttons.forEach(function (element) {
100+
if (element && element.parentNode) {
101+
element.parentNode.removeChild(element);
102+
}
103+
});
104+
}
103105
},
104106

105107
/**

dev/tests/integration/testsuite/Magento/Cron/Observer/ProcessCronQueueObserverTest.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
<?php
22
/**
3-
* Copyright © Magento, Inc. All rights reserved.
4-
* See COPYING.txt for license details.
3+
* Copyright 2015 Adobe
4+
* All rights reserved.
55
*/
66
namespace Magento\Cron\Observer;
77

@@ -36,7 +36,7 @@ public function testDispatchScheduled()
3636
\Magento\Cron\Model\ResourceModel\Schedule\Collection::class
3737
);
3838
$collection->addFieldToFilter('status', \Magento\Cron\Model\Schedule::STATUS_PENDING);
39-
$collection->addFieldToFilter('job_code', 'product_alert');
39+
$collection->addFieldToFilter('job_code', 'catalog_product_alert');
4040
$this->assertGreaterThan(0, $collection->count(), 'Cron has failed to schedule tasks for itself for future.');
4141
}
4242

0 commit comments

Comments
 (0)