Skip to content

Commit 858ac73

Browse files
Merge branch 'develop' into ACQE-3442-test-fix
2 parents 832ee91 + 2016d43 commit 858ac73

File tree

11 files changed

+45
-57
lines changed

11 files changed

+45
-57
lines changed

app/code/Magento/PageBuilder/Model/ResourceModel/Template/Grid/Collection.php

Lines changed: 18 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,12 @@ class Collection extends TemplateCollection implements SearchResultInterface
3131
*/
3232
private $aggregations;
3333

34+
/** @var string */
35+
private $model;
36+
37+
/** @var string */
38+
private $resourceModel;
39+
3440
/**
3541
* @param EntityFactoryInterface $entityFactory
3642
* @param LoggerInterface $logger
@@ -58,6 +64,8 @@ public function __construct(
5864
$connection = null,
5965
AbstractDb $resource = null
6066
) {
67+
$this->resourceModel = $resourceModel;
68+
$this->model = $model;
6169
parent::__construct(
6270
$entityFactory,
6371
$logger,
@@ -68,10 +76,19 @@ public function __construct(
6876
);
6977
$this->_eventPrefix = $eventPrefix;
7078
$this->_eventObject = $eventObject;
71-
$this->_init($model, $resourceModel);
79+
$this->_init($this->model, $this->resourceModel);
7280
$this->setMainTable($mainTable);
7381
}
7482

83+
/**
84+
* @inheritDoc
85+
*/
86+
public function _resetState(): void
87+
{
88+
parent::_resetState();
89+
$this->_init($this->model, $this->resourceModel);
90+
}
91+
7592
/**
7693
* Get aggregations
7794
*

app/code/Magento/PageBuilder/Test/Mftf/Section/PageBuilderTextSection/TextOnConfigurationSection.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@
1919
<element name="tinyMCEWidgetBtn" type="text" selector="#pagebuilder_text_form_content_magentowidget"/>
2020
<!-- WYSIWYG Disabled -->
2121
<element name="textArea" type="text" selector="#pagebuilder_text_form_content"/>
22+
<element name="textAreaError" type="text" selector="//*[@id='pagebuilder_text_form_content']//ancestor::*[contains(@class,'admin__field-wide')]/*[contains(@class,'admin__field-error')]"/>
2223
<element name="textAreaId" type="text" selector="pagebuilder_text_form_content"/>
2324
<element name="insertWidgetButton" type="text" selector="//button[contains(@class,'action-add-widget')][.='Insert Widget...']"/>
2425
<element name="insertImageButton" type="text" selector="//button[contains(@class,'action-add-image')][.='Insert Image...']" timeout="10"/>

app/code/Magento/PageBuilder/Test/Mftf/Section/PageBuilderTextSection/TextOnStageSection.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@
2626
<element name="widgetPlaceholderJS" type="text" selector="(document.querySelectorAll('div.pagebuilder-text')[{{indexOfText}} - 1]).querySelector('.magento-placeholder.magento-widget')" parameterized="true"/>
2727
<element name="placeholder" type="text" selector="(//div[contains(@class,'pagebuilder-text')])[{{arg1}}]//div[contains(@class, 'placeholder-text') and text()='Edit Text']" parameterized="true"/>
2828
<element name="textArea" type="text" selector="(//div[contains(@class,'pagebuilder-text')])[{{arg}}]//textarea[contains(@class,'inline-wysiwyg-textarea')]" parameterized="true"/>
29+
<element name="textAreaForExecuteJS" type="text" selector="(//div[contains(@class,&quot;pagebuilder-text&quot;)])[{{arg}}]//textarea[contains(@class,&quot;inline-wysiwyg-textarea&quot;)]" parameterized="true"/>
2930
<element name="textareaPlaceholder" type="text" selector="(//div[contains(@class,'pagebuilder-text')])[{{arg1}}]//textarea[contains(@class, 'inline-wysiwyg-textarea') and contains(@placeholder,'Edit Text')]" parameterized="true"/>
3031
<element name="textPlaceholder" type="text" selector="(//div[contains(@class,'pagebuilder-text')])[{{arg}}]//*[contains(@class, 'placeholder-text') and text()='Edit Text']" parameterized="true"/>
3132
<element name="tinymce" type="text" selector="(//div[contains(@class,'pagebuilder-text')])[{{arg}}]//div[contains(@class, 'inline-wysiwyg')]" parameterized="true"/>

app/code/Magento/PageBuilder/Test/Mftf/Test/AdminCMSPagePageBuilderTest/PageBuilderCMSPageContentValidationTest.xml

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,17 @@
7979
<argument name="text" value="\u{1F600}"/>
8080
</actionGroup>
8181

82+
<click selector="{{EditPanelForm.editFormSave}}" stepKey="savePageBuilderTextContentType"/>
83+
<waitForElementVisible selector="{{TextOnConfiguration.textAreaError}}" stepKey="waitForPageBuilderTextContentTypeError"/>
84+
<grabTextFrom selector="{{TextOnConfiguration.textAreaError}}" stepKey="grabTextFromPageBuilderTextContentTypeError"/>
85+
<assertStringContainsString stepKey="assertErrorMessageForPageBuilderTextContentType">
86+
<expectedResult type="string">Please remove invalid characters</expectedResult>
87+
<actualResult type="variable">grabTextFromPageBuilderTextContentTypeError</actualResult>
88+
</assertStringContainsString>
89+
<actionGroup ref="removeTextFromHtmlCodeWYSIWYGActionGroup" stepKey="removeTheInvalidCharacterFromPageBuilderTextContentType">
90+
<argument name="selector" value="//textarea[@id=&quot;{{TextOnConfiguration.textAreaId}}&quot;][1]"/>
91+
<argument name="text" value="\u{1F600}"/>
92+
</actionGroup>
8293
<actionGroup ref="saveEditPanelSettingsFullScreen" stepKey="saveTextEditPanelSettings"/>
8394

8495
<!-- Validate Stage -->
@@ -90,6 +101,16 @@
90101
<expectedResult type="string">{{PageBuilderHtmlPropertyButton.value}}</expectedResult>
91102
<actualResult type="variable">actualTextFromHtmlCode</actualResult>
92103
</assertStringContainsString>
104+
105+
<!-- Add the invalid character in the text element on stage -->
106+
<click selector="{{TextOnStage.textareaPlaceholder('1')}}" stepKey="focusOnEditorArea"/>
107+
<actionGroup ref="addTextToHtmlCodeWYSIWYGActionGroup" stepKey="enterWrongTextContentOnStage">
108+
<argument name="selector" value="{{TextOnStage.textAreaForExecuteJS('1')}}"/>
109+
<argument name="text" value="\u{1F600}"/>
110+
</actionGroup>
111+
<!-- Trigger keyup to dispatch changes -->
112+
<pressKey selector="{{TextOnStage.textareaPlaceholder('1')}}" parameterArray="[\Facebook\WebDriver\WebDriverKeys::END]" stepKey="pressEndKey"/>
113+
<click selector="{{PageBuilderPanel.searchPanel}}" stepKey="lostFocusFromTextarea"/>
93114
<actionGroup ref="exitPageBuilderFullScreen" stepKey="exitPageBuilderFullScreen"/>
94115

95116
<!-- Cannot save invalid text-->

app/code/Magento/PageBuilder/etc/adminhtml/di.xml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -641,9 +641,6 @@
641641
</argument>
642642
</arguments>
643643
</type>
644-
<type name="Magento\Framework\Filter\Template">
645-
<plugin name="convertBackgroundImages" type="Magento\PageBuilder\Plugin\Filter\TemplatePlugin"/>
646-
</type>
647644
<type name="Magento\Catalog\Ui\DataProvider\Product\Form\Modifier\Eav">
648645
<plugin name="pagebuilder_product_form_eav" type="Magento\PageBuilder\Plugin\Catalog\Ui\DataProvider\Product\Form\Modifier\EavPlugin" sortOrder="20"/>
649646
</type>

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

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -316,6 +316,9 @@
316316
</argument>
317317
</arguments>
318318
</type>
319+
<type name="Magento\Framework\Filter\Template">
320+
<plugin name="convertBackgroundImages" type="Magento\PageBuilder\Plugin\Filter\TemplatePlugin"/>
321+
</type>
319322
<type name="Magento\PageBuilder\Model\UpgradableEntitiesPool">
320323
<arguments>
321324
<argument name="entities" xsi:type="array">

app/code/Magento/PageBuilder/etc/frontend/di.xml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,4 @@
1212
<type name="Magento\Email\Model\Template\Filter">
1313
<plugin name="escapeCustomVarDirectives" type="Magento\PageBuilder\Plugin\Filter\CustomVarTemplate"/>
1414
</type>
15-
<type name="Magento\Framework\Filter\Template">
16-
<plugin name="convertBackgroundImages" type="Magento\PageBuilder\Plugin\Filter\TemplatePlugin"/>
17-
</type>
1815
</config>

app/code/Magento/PageBuilder/etc/crontab/di.xml renamed to app/code/Magento/PageBuilder/etc/graphql/di.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,6 @@
77
-->
88
<config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="urn:magento:framework:ObjectManager/etc/config.xsd">
99
<type name="Magento\Framework\Filter\Template">
10-
<plugin name="convertBackgroundImages" type="Magento\PageBuilder\Plugin\Filter\TemplatePlugin"/>
10+
<plugin name="convertBackgroundImages" type="Magento\PageBuilder\Plugin\Filter\TemplatePlugin" disabled="true"/>
1111
</type>
1212
</config>

app/code/Magento/PageBuilder/etc/webapi_rest/di.xml

Lines changed: 0 additions & 12 deletions
This file was deleted.

app/code/Magento/PageBuilder/etc/webapi_soap/di.xml

Lines changed: 0 additions & 12 deletions
This file was deleted.

0 commit comments

Comments
 (0)