Skip to content

Commit 0c97a6d

Browse files
committed
ACP2E-23: [Magento Cloud] Error message in PDP disappearing quickly - Code refactoring applied.
1 parent 3925fcf commit 0c97a6d

File tree

5 files changed

+6
-11
lines changed

5 files changed

+6
-11
lines changed

app/code/Magento/Catalog/Model/Theme/CustomerData/MessageProvider.php renamed to app/code/Magento/Catalog/Model/Theme/CustomerData/MessagesProvider.php

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
use Magento\Framework\Message\ManagerInterface as MessageManager;
1515
use Magento\Theme\CustomerData\MessagesProviderInterface;
1616

17-
class MessageProvider implements MessagesProviderInterface
17+
class MessagesProvider implements MessagesProviderInterface
1818
{
1919
/**
2020
*
@@ -61,12 +61,7 @@ public function getMessages(): Collection
6161
$clearSessionMessages = true;
6262

6363
if ((bool) $this->appConfig->getValue(Synchronizer::ALLOW_SYNC_WITH_BACKEND_PATH)) {
64-
65-
$forceNewSectionTimestamp = $this->request->getParam('force_new_section_timestamp') ?? null;
66-
67-
if ('true' !== $forceNewSectionTimestamp) {
68-
$clearSessionMessages = false;
69-
}
64+
$clearSessionMessages = $this->request->getParam('force_new_section_timestamp') === 'true';
7065
}
7166

7267
return $this->messageManager->getMessages($clearSessionMessages);

app/code/Magento/Catalog/etc/di.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@
7676
<preference for="Magento\Catalog\Api\Data\MassActionInterface" type="Magento\Catalog\Model\MassAction" />
7777
<preference for="Magento\Catalog\Model\ProductLink\Data\ListCriteriaInterface" type="Magento\Catalog\Model\ProductLink\Data\ListCriteria" />
7878
<preference for="Magento\Catalog\Api\CategoryListDeleteBySkuInterface" type="Magento\Catalog\Model\CategoryLinkRepository"/>
79-
<preference for="Magento\Theme\CustomerData\MessagesProviderInterface" type="Magento\Catalog\Model\Theme\CustomerData\MessageProvider"/>
79+
<preference for="Magento\Theme\CustomerData\MessagesProviderInterface" type="Magento\Catalog\Model\Theme\CustomerData\MessagesProvider"/>
8080
<type name="Magento\Customer\Model\ResourceModel\Visitor">
8181
<plugin name="catalogLog" type="Magento\Catalog\Model\Plugin\Log" />
8282
</type>

app/code/Magento/Theme/CustomerData/Messages.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ class Messages implements SectionSourceInterface
3939
*
4040
* @param MessageManager $messageManager
4141
* @param InterpretationStrategyInterface $interpretationStrategy
42-
* @param MessagesProviderInterface $messageProvider
42+
* @param MessagesProviderInterface|null $messageProvider
4343
*/
4444
public function __construct(
4545
MessageManager $messageManager,

app/code/Magento/Theme/CustomerData/MessageProvider.php renamed to app/code/Magento/Theme/CustomerData/MessagesProvider.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
use Magento\Framework\Message\Collection;
1111
use Magento\Framework\Message\ManagerInterface as MessageManager;
1212

13-
class MessageProvider implements MessagesProviderInterface
13+
class MessagesProvider implements MessagesProviderInterface
1414
{
1515
/**
1616
* Manager messages

app/code/Magento/Theme/etc/di.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919
<preference for="Magento\Framework\View\Model\PageLayout\Config\BuilderInterface" type="Magento\Theme\Model\PageLayout\Config\Builder"/>
2020
<preference for="Magento\Theme\Model\Design\Config\MetadataProviderInterface" type="Magento\Theme\Model\Design\Config\MetadataProvider"/>
2121
<preference for="Magento\Theme\Model\Theme\StoreThemesResolverInterface" type="Magento\Theme\Model\Theme\StoreThemesResolver"/>
22-
<preference for="Magento\Theme\CustomerData\MessagesProviderInterface" type="Magento\Theme\CustomerData\MessageProvider"/>
22+
<preference for="Magento\Theme\CustomerData\MessagesProviderInterface" type="Magento\Theme\CustomerData\MessagesProvider"/>
2323
<type name="Magento\Theme\Model\Config">
2424
<arguments>
2525
<argument name="configCache" xsi:type="object">Magento\Framework\App\Cache\Type\Config</argument>

0 commit comments

Comments
 (0)