Skip to content

Commit 62efce0

Browse files
author
Joan He
committed
MAGETWO-31484: Resolved merge conflicts and static test failure
1 parent 18ca285 commit 62efce0

File tree

4 files changed

+36
-12
lines changed

4 files changed

+36
-12
lines changed

app/code/Magento/Variable/Controller/Adminhtml/System/Variable/WysiwygPlugin.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,8 @@ class WysiwygPlugin extends \Magento\Variable\Controller\Adminhtml\System\Variab
1515
*/
1616
public function execute()
1717
{
18-
$customVariables = $this->_objectManager->create('Magento\Variable\Model\Variable')->getVariablesOptionArray(true);
18+
$customVariables = $this->_objectManager->create('Magento\Variable\Model\Variable')
19+
->getVariablesOptionArray(true);
1920
$storeContactVariabls = $this->_objectManager->create(
2021
'Magento\Email\Model\Source\Variables'
2122
)->toOptionArray(

app/code/Magento/Variable/etc/acl.xml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,9 @@
88
<config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="../../../../../lib/internal/Magento/Framework/Acl/etc/acl.xsd">
99
<acl>
1010
<resources>
11-
<resource id="Magento_Adminhtml::admin">
12-
<resource id="Magento_Adminhtml::system">
13-
<resource id="Magento_Adminhtml::system_other_settings">
11+
<resource id="Magento_Backend::admin">
12+
<resource id="Magento_Backend::system">
13+
<resource id="Magento_Backend::system_other_settings">
1414
<resource id="Magento_Variable::variable" title="Custom Variables" sortOrder="20" />
1515
</resource>
1616
</resource>

dev/tests/integration/testsuite/Magento/Variable/Model/VariableTest.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,9 @@ public function testLoadByCode()
3030
$this->_model->setData(['code' => 'test_code', 'name' => 'test_name']);
3131
$this->_model->save();
3232

33-
$variable = \Magento\TestFramework\Helper\Bootstrap::getObjectManager()->create('Magento\Variable\Model\Variable');
33+
$variable = \Magento\TestFramework\Helper\Bootstrap::getObjectManager()->create(
34+
'Magento\Variable\Model\Variable'
35+
);
3436
$variable->loadByCode('test_code');
3537
$this->assertEquals($this->_model->getName(), $variable->getName());
3638
$this->_model->delete();

dev/tests/static/testsuite/Magento/Test/Legacy/_files/obsolete_classes.php

Lines changed: 28 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1751,13 +1751,34 @@
17511751
['Magento\Adminhtml\Block\Report\Wishlist', 'Magento\Reports\Block\Adminhtml\Wishlist'],
17521752
['Magento\Backend\Helper\Addresses'],
17531753
['Magento\Backend\Controller\Adminhtml\System\Variable', 'Magento\Variable\Controller\Adminhtml\System\Variable'],
1754-
['Magento\Backend\Controller\Adminhtml\System\Variable\Delete', 'Magento\Variable\Controller\Adminhtml\System\Variable\Delete'],
1755-
['Magento\Backend\Controller\Adminhtml\System\Variable\Edit', 'Magento\Variable\Controller\Adminhtml\System\Variable\Edit'],
1756-
['Magento\Backend\Controller\Adminhtml\System\Variable\Index', 'Magento\Variable\Controller\Adminhtml\System\Variable\Index'],
1757-
['Magento\Backend\Controller\Adminhtml\System\Variable\NewAction', 'Magento\Variable\Controller\Adminhtml\System\Variable\NewAction'],
1758-
['Magento\Backend\Controller\Adminhtml\System\Variable\Save', 'Magento\Variable\Controller\Adminhtml\System\Variable\Save'],
1759-
['Magento\Backend\Controller\Adminhtml\System\Variable\Validate', 'Magento\Variable\Controller\Adminhtml\System\Variable\Validate'],
1760-
['Magento\Backend\Controller\Adminhtml\System\Variable\WysiwygPlugin', 'Magento\Variable\Controller\Adminhtml\System\Variable\WysiwygPlugin'],
1754+
[
1755+
'Magento\Backend\Controller\Adminhtml\System\Variable\Delete',
1756+
'Magento\Variable\Controller\Adminhtml\System\Variable\Delete'
1757+
],
1758+
[
1759+
'Magento\Backend\Controller\Adminhtml\System\Variable\Edit',
1760+
'Magento\Variable\Controller\Adminhtml\System\Variable\Edit'
1761+
],
1762+
[
1763+
'Magento\Backend\Controller\Adminhtml\System\Variable\Index',
1764+
'Magento\Variable\Controller\Adminhtml\System\Variable\Index'
1765+
],
1766+
[
1767+
'Magento\Backend\Controller\Adminhtml\System\Variable\NewAction',
1768+
'Magento\Variable\Controller\Adminhtml\System\Variable\NewAction'
1769+
],
1770+
[
1771+
'Magento\Backend\Controller\Adminhtml\System\Variable\Save',
1772+
'Magento\Variable\Controller\Adminhtml\System\Variable\Save'
1773+
],
1774+
[
1775+
'Magento\Backend\Controller\Adminhtml\System\Variable\Validate',
1776+
'Magento\Variable\Controller\Adminhtml\System\Variable\Validate'
1777+
],
1778+
[
1779+
'Magento\Backend\Controller\Adminhtml\System\Variable\WysiwygPlugin',
1780+
'Magento\Variable\Controller\Adminhtml\System\Variable\WysiwygPlugin'
1781+
],
17611782
['Magento\Core\Model\Cookie', 'Magento\Framework\Stdlib\Cookie'],
17621783
['Magento\Core\Model\Logger', 'Psr\Log\LoggerInterface'],
17631784
['Magento\Core\Block\Template\Context', 'Magento\Framework\View\Element\Template\Context'],

0 commit comments

Comments
 (0)