Skip to content

Commit 7b337ae

Browse files
committed
MAGETWO-66675: Fix relative template references in individual Magento modules #1895
- fix references - update static test
1 parent 4ac06e9 commit 7b337ae

File tree

9 files changed

+23
-15
lines changed

9 files changed

+23
-15
lines changed

app/code/Magento/AdminNotification/view/adminhtml/layout/default.xml

+1-1
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
name="notification_window"
1818
as="notification_window"
1919
acl="Magento_AdminNotification::show_toolbar"
20-
template="notification/window.phtml"/>
20+
template="Magento_AdminNotification::notification/window.phtml"/>
2121
</referenceContainer>
2222
<referenceContainer name="header">
2323
<block class="Magento\AdminNotification\Block\ToolbarEntry" name="notification.messages" before="user" template="Magento_AdminNotification::toolbar_entry.phtml"/>

app/code/Magento/Catalog/view/adminhtml/layout/catalog_product_set_edit.xml

+1-1
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
<container name="adminhtml.catalog.product.set.edit.wrapper" htmlTag="div" htmlClass="admin__scope-old"><!-- @todo ui: remove arguments within .admin__scope-old -->
1212
<block class="Magento\Catalog\Block\Adminhtml\Product\Attribute\Set\Main"
1313
name="adminhtml.catalog.product.set.edit"
14-
template="catalog/product/attribute/set/main.phtml"/>
14+
template="Magento_Catalog::catalog/product/attribute/set/main.phtml"/>
1515
</container>
1616
</referenceContainer>
1717
</body>

app/code/Magento/Customer/view/frontend/layout/default.xml

+1-1
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
</arguments>
2121
</block>
2222
<block class="Magento\Customer\Block\Account\AuthorizationLink" name="authorization-link"
23-
template="account/link/authorization.phtml"/>
23+
template="Magento_Customer::account/link/authorization.phtml"/>
2424
</referenceBlock>
2525
<referenceContainer name="content">
2626
<block class="Magento\Customer\Block\Account\AuthenticationPopup" name="authentication-popup" as="authentication-popup" template="Magento_Customer::account/authentication-popup.phtml">

app/code/Magento/Msrp/view/frontend/layout/checkout_cart_index.xml

+1-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
<update handle="msrp_popup"/>
1010
<body>
1111
<referenceContainer name="checkout.cart.totals.container">
12-
<block name="checkout.cart.totals.msrp" before="checkout.cart.totals" class="Magento\Msrp\Block\Total" template="cart/totals.phtml">
12+
<block name="checkout.cart.totals.msrp" before="checkout.cart.totals" class="Magento\Msrp\Block\Total" template="Magento_Msrp::cart/totals.phtml">
1313
<arguments>
1414
<argument name="original_block_name" xsi:type="string">checkout.cart.totals</argument>
1515
</arguments>

app/code/Magento/Theme/view/adminhtml/layout/theme_design_config_edit.xml

+1-1
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
<referenceContainer name="page.main.actions">
1313
<block class="Magento\Theme\Block\Adminhtml\Design\Config\Edit\Scope"
1414
name="design.config.edit.scope"
15-
template="design/config/edit/scope.phtml"/>
15+
template="Magento_Theme::design/config/edit/scope.phtml"/>
1616
</referenceContainer>
1717
<referenceContainer name="content">
1818
<uiComponent name="design_config_form"/>

app/code/Magento/Wishlist/view/frontend/layout/wishlist_index_configure.xml

+1-1
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
<body>
1111
<referenceBlock name="product.info.addto">
1212
<block class="Magento\Wishlist\Block\Item\Configure" name="view.addto.wishlist"
13-
template="item/configure/addto/wishlist.phtml" />
13+
template="Magento_Wishlist::item/configure/addto/wishlist.phtml" />
1414
</referenceBlock>
1515
</body>
1616
</page>

app/code/Magento/Wishlist/view/frontend/layout/wishlist_index_configure_type_bundle.xml

+1-1
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121
<block class="Magento\Catalog\Block\Product\View" name="product.info.addto.bundle" as="addto" after="product.info.addtocart.bundle"
2222
template="Magento_Catalog::product/view/addto.phtml" cacheable="false">
2323
<block class="Magento\Wishlist\Block\Item\Configure" name="view.addto.wishlist.bundle"
24-
template="item/configure/addto/wishlist.phtml" />
24+
template="Magento_Wishlist::item/configure/addto/wishlist.phtml" />
2525
<block class="Magento\Catalog\Block\Product\View\AddTo\Compare" name="view.addto.compare.bundle" after="view.addto.wishlist"
2626
template="Magento_Catalog::product/view/addto/compare.phtml" />
2727
</block>

dev/tests/integration/testsuite/Magento/Framework/View/Layout/MergeTest.php

+6-3
Original file line numberDiff line numberDiff line change
@@ -230,10 +230,12 @@ public function testLoadFileSystem()
230230
$expectedResult = '
231231
<root>
232232
<body>
233-
<block class="Magento\Framework\View\Element\Template" template="Magento_Framework::fixture_template_one.phtml"/>
233+
<block class="Magento\Framework\View\Element\Template"
234+
template="Magento_Framework::fixture_template_one.phtml"/>
234235
</body>
235236
<body>
236-
<block class="Magento\Framework\View\Element\Template" template="Magento_Framework::fixture_template_two.phtml"/>
237+
<block class="Magento\Framework\View\Element\Template"
238+
template="Magento_Framework::fixture_template_two.phtml"/>
237239
</body>
238240
</root>
239241
';
@@ -249,7 +251,8 @@ public function testLoadFileSystemWithPageLayout()
249251
<root>
250252
<body>
251253
<referenceContainer name="main.container">
252-
<block class="Magento\Framework\View\Element\Template" template="Magento_Framework::fixture_template_one.phtml"/>
254+
<block class="Magento\Framework\View\Element\Template"
255+
template="Magento_Framework::fixture_template_one.phtml"/>
253256
</referenceContainer>
254257
</body>
255258
</root>

dev/tests/integration/testsuite/Magento/Framework/View/Model/Layout/MergeTest.php

+10-5
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,8 @@ class MergeTest extends \PHPUnit_Framework_TestCase
1010
/**
1111
* Fixture XML instruction(s) to be used in tests
1212
*/
13-
const FIXTURE_LAYOUT_XML = '<block class="Magento\Framework\View\Element\Template" template="Magento_Framework::fixture.phtml"/>';
13+
const FIXTURE_LAYOUT_XML
14+
= '<block class="Magento\Framework\View\Element\Template" template="Magento_Framework::fixture.phtml"/>';
1415

1516
/**
1617
* @var \Magento\Framework\View\Model\Layout\Merge
@@ -32,7 +33,8 @@ protected function setUp()
3233
$layoutUpdate1->setHandle('fixture_handle_one');
3334
$layoutUpdate1->setXml(
3435
'<body>
35-
<block class="Magento\Framework\View\Element\Template" template="Magento_Framework::fixture_template_one.phtml"/>
36+
<block class="Magento\Framework\View\Element\Template"
37+
template="Magento_Framework::fixture_template_one.phtml"/>
3638
</body>'
3739
);
3840
$layoutUpdate1->setHasDataChanges(true);
@@ -49,7 +51,8 @@ protected function setUp()
4951
$layoutUpdate2->setHandle('fixture_handle_two');
5052
$layoutUpdate2->setXml(
5153
'<body>
52-
<block class="Magento\Framework\View\Element\Template" template="Magento_Framework::fixture_template_two.phtml"/>
54+
<block class="Magento\Framework\View\Element\Template"
55+
template="Magento_Framework::fixture_template_two.phtml"/>
5356
</body>'
5457
);
5558
$layoutUpdate2->setHasDataChanges(true);
@@ -74,10 +77,12 @@ public function testLoadDbApp()
7477
$expectedResult = '
7578
<root>
7679
<body>
77-
<block class="Magento\Framework\View\Element\Template" template="Magento_Framework::fixture_template_one.phtml"/>
80+
<block class="Magento\Framework\View\Element\Template"
81+
template="Magento_Framework::fixture_template_one.phtml"/>
7882
</body>
7983
<body>
80-
<block class="Magento\Framework\View\Element\Template" template="Magento_Framework::fixture_template_two.phtml"/>
84+
<block class="Magento\Framework\View\Element\Template"
85+
template="Magento_Framework::fixture_template_two.phtml"/>
8186
</body>
8287
</root>
8388
';

0 commit comments

Comments
 (0)