MAGETWO-52958: Better Semantic for UI Components

Author: Karpenko, Oleksandr

app/code/Magento/Catalog/view/adminhtml/ui_component/category_form.xml

@@ -424,7 +424,7 @@
         </settings>
         <container name="assign_products_container" sortOrder="160">
             <htmlContent name="html_content">
-                <argument name="block" xsi:type="object">Magento\Catalog\Block\Adminhtml\Category\AssignProducts</argument>
+                <block name="gallery" class="Magento\Catalog\Block\Adminhtml\Category\AssignProducts"/>
             </htmlContent>
         </container>
     </fieldset>

app/code/Magento/GoogleOptimizer/view/adminhtml/ui_component/category_form.xml

@@ -13,7 +13,7 @@
         </settings>
         <container name="google_experiment_container" sortOrder="250">
             <htmlContent name="html_content">
-                <argument name="block" xsi:type="object">Magento\GoogleOptimizer\Block\Adminhtml\Catalog\Category\Edit\Googleoptimizer</argument>
+                <block name="googleoptimizer" class="Magento\GoogleOptimizer\Block\Adminhtml\Catalog\Category\Edit\Googleoptimizer"/>
             </htmlContent>
         </container>
     </fieldset>

app/code/Magento/GoogleOptimizer/view/adminhtml/ui_component/cms_page_form.xml

@@ -13,7 +13,7 @@
         </settings>
         <container name="google_experiment_container" sortOrder="250">
             <htmlContent name="html_content">
-                <argument name="block" xsi:type="object">Magento\GoogleOptimizer\Block\Adminhtml\Form\CmsPage</argument>
+                <block name="googleOptimizerBlockAdminhtmlFormCmsPage" class="MagentoGoogleOptimizerBlockAdminhtmlFormCmsPage"/>
             </htmlContent>
         </container>
     </fieldset>

app/code/Magento/SalesRule/view/adminhtml/ui_component/sales_rule_form.xml

@@ -409,7 +409,7 @@
         </settings>
         <container name="conditions_apply_to" sortOrder="10">
             <htmlContent name="html_content">
-                <argument name="block" xsi:type="object">Magento\SalesRule\Block\Adminhtml\Promo\Quote\Edit\Tab\Conditions</argument>
+                <block name="sales_rule_conditions" class="Magento\SalesRule\Block\Adminhtml\Promo\Quote\Edit\Tab\Conditions" />
             </htmlContent>
         </container>
     </fieldset>
@@ -521,7 +521,7 @@
         </field>
         <container name="actions_apply_to" sortOrder="40">
             <htmlContent name="html_content">
-                <argument name="block" xsi:type="object">Magento\SalesRule\Block\Adminhtml\Promo\Quote\Edit\Tab\Actions</argument>
+                <block class="Magento\SalesRule\Block\Adminhtml\Promo\Quote\Edit\Tab\Actions"/>
             </htmlContent>
         </container>
     </fieldset>
@@ -544,7 +544,7 @@
         </field>
         <container name="store_view_labels" sortOrder="40">
             <htmlContent name="html_content">
-                <argument name="block" xsi:type="object">Magento\SalesRule\Block\Adminhtml\Promo\Quote\Edit\Tab\Labels</argument>
+                <block class="Magento\SalesRule\Block\Adminhtml\Promo\Quote\Edit\Tab\Labels"/>
             </htmlContent>
         </container>
     </fieldset>

dev/tests/integration/testsuite/Magento/Ui/Component/ConfigurationTest.php

@@ -53,9 +53,6 @@ class ConfigurationTest extends \PHPUnit_Framework_TestCase
         'argument[@name="data"]/item[@name="config"]/item[@name="multiple"]' => [
             '//*[@formElement="select"]',
             '//*[substring(@component, string-length(@component) - string-length("ui-group") +1) = "ui-group"]'
-        ],
-        'argument[@name="block"]' => [
-            '//*[@name="html_content"]'
         ]
     ];
 

lib/internal/Magento/Framework/View/Layout/Reader/UiComponent.php

@@ -94,7 +94,10 @@ public function interpret(Context $readerContext, Element $currentElement)
             $scheduledStructure->setElementToIfconfigList($referenceName, $configPath, $this->scopeType);
         }
 
-        foreach ($this->getLayoutElementsFromUiConfiguration($referenceName) as $layoutElement) {
+        $elements = [];
+        $config = $this->uiConfigFactory->create(['componentName' => $referenceName])->get($referenceName);
+        $this->getLayoutElementsFromUiConfiguration([$referenceName => $config], $elements);
+        foreach ($elements as $layoutElement) {
             $layoutElement = simplexml_load_string(
                 $layoutElement,
                 Element::class
@@ -108,19 +111,20 @@ public function interpret(Context $readerContext, Element $currentElement)
     /**
      * Find layout elements in UI configuration for correct layout generation
      *
-     * @param string $uiConfigName
-     * @return array
+     * @param array $config
+     * @param array $elements
+     * @return void
      */
-    private function getLayoutElementsFromUiConfiguration($uiConfigName)
+    private function getLayoutElementsFromUiConfiguration(array $config, array &$elements = [])
     {
-        $elements = [];
-        $config = $this->uiConfigFactory->create(['componentName' => $uiConfigName])->get($uiConfigName);
-        foreach ($config['children'] as $name => $data) {
+        foreach ($config as $name => $data) {
             if (isset($data['arguments']['block']['layout'])) {
-                $elements[$name] = $data['arguments']['block']['layout'];
+                $elements[] = $data['arguments']['block']['layout'];
+            }
+            if (isset($data['children']) && !empty($data['children'])) {
+                $this->getLayoutElementsFromUiConfiguration($data['children'], $elements);
             }
         }
-        return $elements;
     }
 
     /**