Skip to content

Commit 9ca5930

Browse files
committed
Merge branch 'develop' of github.com:magento/magento2-page-builder into PB-493
2 parents 40d4c7a + f6cf95b commit 9ca5930

File tree

7 files changed

+48
-37
lines changed

7 files changed

+48
-37
lines changed

app/code/Magento/PageBuilder/Model/Config/ContentType/AdditionalData/Provider/Uploader/OpenDialogUrl.php

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

1111
use Magento\PageBuilder\Model\Config\ContentType\AdditionalData\ProviderInterface;
1212
use Magento\Backend\Model\Url;
13+
use Magento\Ui\Component\Form\Element\DataType\Media\OpenDialogUrl as OpenDialogConfig;
1314

1415
/**
1516
* Provides open dialog URL for media gallery slideout
@@ -21,12 +22,21 @@ class OpenDialogUrl implements ProviderInterface
2122
*/
2223
private $urlBuilder;
2324

25+
/**
26+
* @var OpenDialogConfig
27+
*/
28+
private $openDialogConfig;
29+
2430
/**
2531
* @param Url $urlBuilder
32+
* @param OpenDialogConfig $openDialogConfig
2633
*/
27-
public function __construct(Url $urlBuilder)
28-
{
34+
public function __construct(
35+
Url $urlBuilder,
36+
OpenDialogConfig $openDialogConfig // phpstan:ignore
37+
) {
2938
$this->urlBuilder = $urlBuilder;
39+
$this->openDialogConfig = $openDialogConfig;
3040
}
3141

3242
/**
@@ -35,10 +45,7 @@ public function __construct(Url $urlBuilder)
3545
public function getData(string $itemName) : array
3646
{
3747
return [
38-
$itemName => $this->urlBuilder->getUrl(
39-
'cms/wysiwyg_images/index',
40-
['_secure' => true]
41-
)
48+
$itemName => $this->urlBuilder->getUrl($this->openDialogConfig->get(), ['_secure' => true])
4249
];
4350
}
4451
}
Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
<?xml version="1.0" encoding="UTF-8"?>
2+
<!--
3+
/**
4+
* Copyright © Magento, Inc. All rights reserved.
5+
* See COPYING.txt for license details.
6+
*/
7+
-->
8+
9+
<actionGroups xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
10+
xsi:noNamespaceSchemaLocation="urn:magento:mftf:Test/etc/actionGroupSchema.xsd">
11+
<actionGroup name="AdminDisableWYSIWYGActionGroup">
12+
<magentoCLI command="config:set cms/pagebuilder/enabled 0" stepKey="disablePageBuilder"/>
13+
</actionGroup>
14+
</actionGroups>
Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
<?xml version="1.0" encoding="UTF-8"?>
2+
<!--
3+
/**
4+
* Copyright © Magento, Inc. All rights reserved.
5+
* See COPYING.txt for license details.
6+
*/
7+
-->
8+
9+
<actionGroups xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
10+
xsi:noNamespaceSchemaLocation="urn:magento:mftf:Test/etc/actionGroupSchema.xsd">
11+
<actionGroup name="AdminEnableWYSIWYGActionGroup">
12+
<magentoCLI command="config:set cms/pagebuilder/enabled 1" stepKey="enablePageBuilder"/>
13+
</actionGroup>
14+
</actionGroups>

app/code/Magento/PageBuilder/Test/Mftf/Suite/PageBuilderDisabledSuite.xml

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

app/code/Magento/PageBuilder/Test/Mftf/Test/AdminPageBuilderSlideItemCommonTest/SlideItemDeleteImageTest.xml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -72,6 +72,8 @@
7272
</actionGroup>
7373
<!-- Delete image A -->
7474
<comment stepKey="deleteImageA" userInput="Delete image A"/>
75+
<waitForElementVisible selector="{{MediaGallerySection.lastImageOrImageCopy(PageBuilderBackgroundImage_GIF.fileName, PageBuilderBackgroundImage_GIF.extension)}}" stepKey="waitForLastImage"/>
76+
<click selector="{{MediaGallerySection.lastImageOrImageCopy(PageBuilderBackgroundImage_GIF.fileName, PageBuilderBackgroundImage_GIF.extension)}}" stepKey="selectImage"/>
7577
<actionGroup ref="DeleteImageFromStorageActionGroup" stepKey="DeleteImageFromStorage1">
7678
<argument name="Image" value="PageBuilderBackgroundImage_GIF"/>
7779
</actionGroup>

app/code/Magento/PageBuilder/Test/Unit/Model/Catalog/SortingTest.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,11 +8,11 @@
88
namespace Magento\PageBuilder\Test\Unit\Model\Catalog;
99

1010
use Magento\Framework\TestFramework\Unit\Helper\ObjectManager;
11+
use Magento\PageBuilder\Model\Catalog\Sorting;
12+
use Magento\PageBuilder\Model\Catalog\Sorting\Factory;
13+
use Magento\PageBuilder\Model\Catalog\Sorting\OptionInterface;
1114
use PHPUnit\Framework\MockObject\MockObject;
1215
use PHPUnit\Framework\MockObject\Stub\ReturnValueMap;
13-
use Magento\PageBuilder\Model\Catalog\Sorting\OptionInterface;
14-
use Magento\PageBuilder\Model\Catalog\Sorting\Factory;
15-
use Magento\PageBuilder\Model\Catalog\Sorting;
1616
use PHPUnit\Framework\TestCase;
1717

1818
/**

app/code/Magento/PageBuilder/view/adminhtml/web/js/form/element/image-uploader.js

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -10,9 +10,8 @@ define([
1010
'Magento_Ui/js/form/element/image-uploader',
1111
'Magento_PageBuilder/js/resource/resize-observer/ResizeObserver',
1212
'Magento_PageBuilder/js/events',
13-
'mage/translate',
14-
'mage/adminhtml/browser'
15-
], function ($, _, uiRegistry, Uploader, ResizeObserver, events, $t, browser) {
13+
'mage/translate'
14+
], function ($, _, uiRegistry, Uploader, ResizeObserver, events, $t) {
1615
'use strict';
1716

1817
var initializedOnce = false;
@@ -72,14 +71,6 @@ define([
7271
}
7372
},
7473

75-
/**
76-
* {@inheritDoc}
77-
*/
78-
openMediaBrowserDialog: function (imageUploader, e) {
79-
browser.modalLoaded = false;
80-
this._super(imageUploader, e);
81-
},
82-
8374
/**
8475
* {@inheritDoc}
8576
*/

0 commit comments

Comments
 (0)