Skip to content

Commit 3ee3962

Browse files
committed
Merge branch 'AC-9246' of https://github.com/magento-l3/magento2ce into PR-VK-2023-09-29
2 parents bc1c7ba + d094e42 commit 3ee3962

File tree

11 files changed

+47
-13
lines changed

11 files changed

+47
-13
lines changed

app/code/Magento/ReleaseNotification/Controller/Adminhtml/Notification/MarkUserNotified.php

Lines changed: 5 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -7,16 +7,19 @@
77
namespace Magento\ReleaseNotification\Controller\Adminhtml\Notification;
88

99
use Magento\Backend\App\Action;
10+
use Magento\Framework\App\Action\HttpPostActionInterface;
1011
use Magento\Framework\Exception\LocalizedException;
1112
use Magento\Framework\Controller\ResultFactory;
1213
use Magento\ReleaseNotification\Model\ResourceModel\Viewer\Logger as NotificationLogger;
1314
use Magento\Framework\App\ProductMetadataInterface;
1415
use Psr\Log\LoggerInterface;
1516

1617
/**
17-
* Controller to record that the current admin user has seen the release notification content
18+
* @deprecated Starting from Magento OS 2.4.7 Magento_ReleaseNotification module is deprecated
19+
* in favor of another in-product messaging mechanism
20+
* @see Current in-product messaging mechanism
1821
*/
19-
class MarkUserNotified extends Action
22+
class MarkUserNotified extends Action implements HttpPostActionInterface
2023
{
2124
/**
2225
* @var ProductMetadataInterface
@@ -84,12 +87,4 @@ public function execute()
8487
$resultJson = $this->resultFactory->create(ResultFactory::TYPE_JSON);
8588
return $resultJson->setData($responseContent);
8689
}
87-
88-
/**
89-
* @return bool
90-
*/
91-
protected function _isAllowed()
92-
{
93-
return parent::_isAllowed();
94-
}
9590
}

app/code/Magento/ReleaseNotification/Model/Condition/CanViewNotification.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,10 @@
1515
* Dynamic validator for UI release notification, manage UI component visibility.
1616
* Return true if the logged in user has not seen the notification.
1717
* @SuppressWarnings(PHPMD.CookieAndSessionMisuse)
18+
*
19+
* @deprecated Starting from Magento OS 2.4.7 Magento_ReleaseNotification module is deprecated
20+
* in favor of another in-product messaging mechanism
21+
* @see Current in-product messaging mechanism
1822
*/
1923
class CanViewNotification implements VisibilityConditionInterface
2024
{

app/code/Magento/ReleaseNotification/Model/ContentProvider/Http/HttpContentProvider.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,10 @@
1313

1414
/**
1515
* Requests the release notification content data via an HTTP call to a REST API
16+
*
17+
* @deprecated Starting from Magento OS 2.4.7 Magento_ReleaseNotification module is deprecated
18+
* in favor of another in-product messaging mechanism
19+
* @see Current in-product messaging mechanism
1620
*/
1721
class HttpContentProvider implements ContentProviderInterface
1822
{

app/code/Magento/ReleaseNotification/Model/ContentProvider/Http/UrlBuilder.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,10 @@
99

1010
/**
1111
* Builder to build Url to retrieve the notification content.
12+
*
13+
* @deprecated Starting from Magento OS 2.4.7 Magento_ReleaseNotification module is deprecated
14+
* in favor of aMagento OS 2.4.7
15+
* @see Current in-product messaging mechanism
1216
*/
1317
class UrlBuilder
1418
{

app/code/Magento/ReleaseNotification/Model/ContentProviderInterface.php

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,9 @@
99
/**
1010
* Requests the release notification content data from a defined service
1111
* @api
12+
* @deprecated Starting from Magento OS 2.4.7 Magento_ReleaseNotification module is deprecated
13+
* in favor of another in-product messaging mechanism
14+
* @see Current in-product messaging mechanism
1215
*/
1316
interface ContentProviderInterface
1417
{

app/code/Magento/ReleaseNotification/Model/ResourceModel/Viewer/Logger.php

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,13 +15,17 @@
1515
* Release notification viewer log data logger.
1616
*
1717
* Saves and retrieves release notification viewer log data.
18+
*
19+
* @deprecated Starting from Magento OS 2.4.7 Magento_ReleaseNotification module is deprecated
20+
* in favor of another in-product messaging mechanism
21+
* @see Current in-product messaging mechanism
1822
*/
1923
class Logger
2024
{
2125
/**
22-
* Log table name
26+
* Release notification log table name
2327
*/
24-
const LOG_TABLE_NAME = 'release_notification_viewer_log';
28+
public const LOG_TABLE_NAME = 'release_notification_viewer_log';
2529

2630
/**
2731
* @var Resource

app/code/Magento/ReleaseNotification/Model/Viewer/Log.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,10 @@
99

1010
/**
1111
* Release notification viewer log resource
12+
*
13+
* @deprecated Starting from Magento OS 2.4.7 Magento_ReleaseNotification module is deprecated
14+
* in favor of another in-product messaging mechanism
15+
* @see Current in-product messaging mechanism
1216
*/
1317
class Log extends DataObject
1418
{

app/code/Magento/ReleaseNotification/Ui/DataProvider/Modifier/Notifications.php

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,9 @@
2020
* Modifies the metadata returning to the Release Notification data provider
2121
*
2222
* @SuppressWarnings(PHPMD.CookieAndSessionMisuse)
23+
* @deprecated Starting from Magento OS 2.4.7 Magento_ReleaseNotification module is deprecated
24+
* in favor of another in-product messaging mechanism
25+
* @see Current in-product messaging mechanism
2326
*/
2427
class Notifications implements ModifierInterface
2528
{

app/code/Magento/ReleaseNotification/Ui/DataProvider/NotificationDataProvider.php

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,10 @@
1414

1515
/**
1616
* Data Provider for the Release Notifications UI component.
17+
*
18+
* @deprecated Starting from Magento OS 2.4.7 Magento_ReleaseNotification module is deprecated
19+
* in favor of another in-product messaging mechanism
20+
* @see Current in-product messaging mechanism
1721
*/
1822
class NotificationDataProvider implements DataProviderInterface
1923
{
@@ -177,20 +181,25 @@ public function getRequestFieldName()
177181

178182
/**
179183
* @inheritdoc
184+
* phpcs:disable Magento2.CodeAnalysis.EmptyBlock
180185
*/
181186
public function addFilter(\Magento\Framework\Api\Filter $filter)
182187
{
188+
// phpcs:ignore Squiz.PHP.NonExecutableCode.ReturnNotRequired
189+
return;
183190
}
184191

185192
/**
186193
* @inheritdoc
194+
* phpcs:disable Magento2.CodeAnalysis.EmptyBlock
187195
*/
188196
public function addOrder($field, $direction)
189197
{
190198
}
191199

192200
/**
193201
* @inheritdoc
202+
* phpcs:disable Magento2.CodeAnalysis.EmptyBlock
194203
*/
195204
public function setLimit($offset, $size)
196205
{

app/code/Magento/ReleaseNotification/Ui/Renderer/NotificationRenderer.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,10 @@
1010

1111
/**
1212
* Builds the HTML for the release notification modals
13+
*
14+
* @deprecated Starting from Magento OS 2.4.7 Magento_ReleaseNotification module is deprecated
15+
* in favor of another in-product messaging mechanism
16+
* @see Current in-product messaging mechanism
1317
*/
1418
class NotificationRenderer
1519
{

app/code/Magento/ReleaseNotification/etc/config.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
<default>
1010
<system>
1111
<release_notification>
12-
<content_url>magento.com/release_notifications</content_url>
12+
<content_url/>
1313
<use_https>1</use_https>
1414
</release_notification>
1515
</system>

0 commit comments

Comments
 (0)