TextualSummaries: fix rendering of individual parts.

Author: martinpovolny

app/helpers/textual_listview.rb

@@ -13,4 +13,8 @@ def locals
   def self.new_from_hash(h)
     new(*h.values_at(*members))
   end
+
+  def self.data(h)
+    self.new_from_hash(h).locals
+  end
 end

app/javascript/packs/component-definitions-common.js

@@ -1,6 +1,8 @@
 import React from 'react';
 import { TagGroup, TableListView, GenericGroup } from '@manageiq/react-ui-components/dist/textual_summary';
 import TextualSummaryWrapper from '../react/textual_summary_wrapper';
+import TableListViewWrapper from '../react/table_list_view_wrapper';
+import GenericGroupWrapper from '../react/generic_group_wrapper';
 import { addReact } from '../miq-component/helpers';
 
 /**
@@ -11,6 +13,8 @@ import { addReact } from '../miq-component/helpers';
 
 addReact('TagGroup', props => <TagGroup {...props} />);
 addReact('TableListView', TableListView);
+addReact('TableListViewWrapper', TableListViewWrapper);
 addReact('GenericGroup', GenericGroup);
+addReact('GenericGroupWrapper', GenericGroupWrapper);
 addReact('TextualSummaryWrapper', TextualSummaryWrapper);
 

app/javascript/react/generic_group_wrapper.jsx

@@ -0,0 +1,8 @@
+import React from 'react';
+import { GenericGroup } from '@manageiq/react-ui-components/dist/textual_summary';
+import textualSummaryGenericClick from './textual_summary_click';
+
+export default (props) => {
+  const onClick = props.onClick || textualSummaryGenericClick;
+  return <GenericGroup onClick={onClick} {...props} />;
+}

app/javascript/react/table_list_view_wrapper.js

@@ -0,0 +1,8 @@
+import React from 'react';
+import { TableListView } from '@manageiq/react-ui-components/dist/textual_summary';
+import textualSummaryGenericClick from './textual_summary_click';
+
+export default (props) => {
+  const onClick = props.onClick || textualSummaryGenericClick;
+  return <TableListView onClick={onClick} {...props} />;
+}

app/views/automation_manager/_configuration_script.html.haml

@@ -1,9 +1,9 @@
 = render :partial => "layouts/flash_msg"
 .row
   .col-md-12.col-lg-8
-    = react 'GenericGroup', expand_generic_group(TextualGroup.new(_('Properties'), textual_configuration_script_group_properties), @record)
+    = react 'GenericGroupWrapper', expand_generic_group(TextualGroup.new(_('Properties'), textual_configuration_script_group_properties), @record)
   .col-md-12.col-lg-8
-    = react 'TableListView', textual_configuration_script_variables
+    = react 'TableListViewWrapper', TextualListview.data(textual_configuration_script_variables)
   .col-md-12.col-lg-8
     - if @record.survey_spec['spec']
-      = react 'TableListView', textual_configuration_script_survey
+      = react 'TableListViewWrapper', TextualListview.data(textual_configuration_script_survey)

app/views/automation_manager/_main_inventory_group.html.haml

@@ -1,4 +1,4 @@
 = render :partial => "layouts/flash_msg"
 .row
   .col-md-12.col-lg-6
-    = react 'GenericGroup', expand_generic_group(TextualGroup.new(_('Properties'), textual_inventory_group_properties), @record)
+    = react 'GenericGroupWrapper', expand_generic_group(TextualGroup.new(_('Properties'), textual_inventory_group_properties), @record)

app/views/provider_foreman/_configuration_script.html.haml

@@ -1,9 +1,9 @@
 = render :partial => "layouts/flash_msg"
 .row
   .col-md-12.col-lg-8
-    = react 'GenericGroup', expand_generic_group(TextualGroup.new(_('Properties'), textual_configuration_script_group_properties), @record)
+    = react 'GenericGroupWrapper', expand_generic_group(TextualGroup.new(_('Properties'), textual_configuration_script_group_properties), @record)
   .col-md-12.col-lg-8
-    = react 'TableListView', textual_configuration_script_variables
+    = react 'TableListViewWrapper', TextualListview.data(textual_configuration_script_variables)
   .col-md-12.col-lg-8
     - if @record.survey_spec['spec']
-      = react 'TableListView', textual_configuration_script_survey
+      = react 'TableListViewWrapper', TextualListview.data(textual_configuration_script_survey)

app/views/provider_foreman/_main_configuration_profile.html.haml

@@ -1,8 +1,8 @@
 = render :partial => "layouts/flash_msg"
 .row
   .col-md-12.col-lg-6
-    = react 'GenericGroup', expand_generic_group(TextualGroup.new(_('Properties'), textual_configuration_profile_group_properties), @record)
-    = react 'GenericGroup', expand_generic_group(TextualGroup.new(_('Environment'), textual_configuration_profile_group_environment), @record)
-    = react 'GenericGroup', expand_generic_group(TextualGroup.new(_('Operating System'), textual_configuration_profile_group_os), @record)
+    = react 'GenericGroupWrapper', expand_generic_group(TextualGroup.new(_('Properties'), textual_configuration_profile_group_properties), @record)
+    = react 'GenericGroupWrapper', expand_generic_group(TextualGroup.new(_('Environment'), textual_configuration_profile_group_environment), @record)
+    = react 'GenericGroupWrapper', expand_generic_group(TextualGroup.new(_('Operating System'), textual_configuration_profile_group_os), @record)
   .col-md-12.col-lg-6
-    = react 'GenericGroup', expand_generic_group(TextualGroup.new(_('Tenancy'), textual_configuration_profile_group_tenancy), @record)
+    = react 'GenericGroupWrapper', expand_generic_group(TextualGroup.new(_('Tenancy'), textual_configuration_profile_group_tenancy), @record)

app/views/provider_foreman/_main_inventory_group.html.haml

@@ -1,4 +1,4 @@
 = render :partial => "layouts/flash_msg"
 .row
   .col-md-12.col-lg-6
-    = react 'GenericGroup', expand_generic_group(TextualGroup.new(_('Properties'), textual_inventory_group_properties), @record)
+    = react 'GenericGroupWrapper', expand_generic_group(TextualGroup.new(_('Properties'), textual_inventory_group_properties), @record)

app/views/pxe/_iso_datastore_details.html.haml

@@ -7,7 +7,7 @@
         = render :partial => "iso_datastore_form"
       - else
         = render :partial => "layouts/flash_msg"
-        = react 'GenericGroup', expand_generic_group(TextualGroup.new(binf, textual_iso_datastore_basicinfo), @record)
+        = react 'GenericGroupWrapper', expand_generic_group(TextualGroup.new(binf, textual_iso_datastore_basicinfo), @record)
         - unless @isd.iso_images.empty?
           %h3= _('ISO Images')
           %table.table.table-bordered.table-striped.table-hover
@@ -27,4 +27,4 @@
         = render :partial => "iso_img_form"
       - else
         = render :partial => "layouts/flash_msg"
-        = react 'GenericGroup', expand_generic_group(TextualGroup.new(binf, textual_iso_img_info), @record)
+        = react 'GenericGroupWrapper', expand_generic_group(TextualGroup.new(binf, textual_iso_img_info), @record)