From da2b520e69bac778a52e45e18008147312f6b24a Mon Sep 17 00:00:00 2001
From: Kun Yan <kyan@redhat.com>
Date: Mon, 10 Jun 2019 10:59:40 +0800
Subject: [PATCH] fix(react-table): Fix error Unknown event handler property
 `onCollapse`. It will be ignored.

---
 .../src/components/Table/BodyWrapper.js       |  2 +-
 .../Table/__snapshots__/Table.test.js.snap    | 80 -------------------
 2 files changed, 1 insertion(+), 81 deletions(-)

diff --git a/packages/patternfly-4/react-table/src/components/Table/BodyWrapper.js b/packages/patternfly-4/react-table/src/components/Table/BodyWrapper.js
index 9b3c17cc0db..6ffdc21ec7e 100644
--- a/packages/patternfly-4/react-table/src/components/Table/BodyWrapper.js
+++ b/packages/patternfly-4/react-table/src/components/Table/BodyWrapper.js
@@ -7,7 +7,7 @@ import { mapOpenedRows } from './utils/headerUtils';
 // eslint-disable-next-line react/prefer-stateless-function
 class BodyWrapper extends React.Component {
   render() {
-    const { mappedRows, tbodyRef, ...props } = this.props;
+    const { mappedRows, tbodyRef, rows, onCollapse, ...props } = this.props;
     if (mappedRows && mappedRows.some(row => row.hasOwnProperty('parent'))) {
       return (
         <React.Fragment>
diff --git a/packages/patternfly-4/react-table/src/components/Table/__snapshots__/Table.test.js.snap b/packages/patternfly-4/react-table/src/components/Table/__snapshots__/Table.test.js.snap
index 847654317e7..d7ad140c2b9 100644
--- a/packages/patternfly-4/react-table/src/components/Table/__snapshots__/Table.test.js.snap
+++ b/packages/patternfly-4/react-table/src/components/Table/__snapshots__/Table.test.js.snap
@@ -2406,8 +2406,6 @@ exports[`Actions table 1`] = `
             >
               <tbody
                 className=""
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -11464,8 +11462,6 @@ exports[`Cell header table 1`] = `
             >
               <tbody
                 className=""
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -17731,8 +17727,6 @@ exports[`Collapsible nested table 1`] = `
               <tbody
                 className=""
                 key="tbody-0"
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -19293,8 +19287,6 @@ exports[`Collapsible nested table 1`] = `
               <tbody
                 className=""
                 key="tbody-1"
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -20324,8 +20316,6 @@ exports[`Collapsible nested table 1`] = `
               <tbody
                 className=""
                 key="tbody-2"
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -20824,8 +20814,6 @@ exports[`Collapsible nested table 1`] = `
               <tbody
                 className=""
                 key="tbody-3"
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -21324,8 +21312,6 @@ exports[`Collapsible nested table 1`] = `
               <tbody
                 className=""
                 key="tbody-4"
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -21824,8 +21810,6 @@ exports[`Collapsible nested table 1`] = `
               <tbody
                 className=""
                 key="tbody-5"
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -24785,8 +24769,6 @@ exports[`Collapsible table 1`] = `
               <tbody
                 className="pf-m-expanded"
                 key="tbody-0"
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -25816,8 +25798,6 @@ exports[`Collapsible table 1`] = `
               <tbody
                 className=""
                 key="tbody-1"
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -26316,8 +26296,6 @@ exports[`Collapsible table 1`] = `
               <tbody
                 className=""
                 key="tbody-2"
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -27347,8 +27325,6 @@ exports[`Collapsible table 1`] = `
               <tbody
                 className=""
                 key="tbody-3"
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -27847,8 +27823,6 @@ exports[`Collapsible table 1`] = `
               <tbody
                 className=""
                 key="tbody-4"
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -28347,8 +28321,6 @@ exports[`Collapsible table 1`] = `
               <tbody
                 className=""
                 key="tbody-5"
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -28847,8 +28819,6 @@ exports[`Collapsible table 1`] = `
               <tbody
                 className=""
                 key="tbody-6"
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -30271,8 +30241,6 @@ exports[`Compound Expandable table 1`] = `
               <tbody
                 className="pf-m-expanded"
                 key="tbody-0"
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -30748,8 +30716,6 @@ exports[`Compound Expandable table 1`] = `
               <tbody
                 className=""
                 key="tbody-1"
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -33507,8 +33473,6 @@ exports[`Header width table 1`] = `
               <tbody
                 className=""
                 key="tbody-0"
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -34806,8 +34770,6 @@ exports[`Header width table 1`] = `
               <tbody
                 className=""
                 key="tbody-1"
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -35671,8 +35633,6 @@ exports[`Header width table 1`] = `
               <tbody
                 className=""
                 key="tbody-2"
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -36102,8 +36062,6 @@ exports[`Header width table 1`] = `
               <tbody
                 className=""
                 key="tbody-3"
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -36533,8 +36491,6 @@ exports[`Header width table 1`] = `
               <tbody
                 className=""
                 key="tbody-4"
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -36964,8 +36920,6 @@ exports[`Header width table 1`] = `
               <tbody
                 className=""
                 key="tbody-5"
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -39867,8 +39821,6 @@ exports[`Selectable table 1`] = `
               <tbody
                 className=""
                 key="tbody-0"
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -41354,8 +41306,6 @@ exports[`Selectable table 1`] = `
               <tbody
                 className=""
                 key="tbody-1"
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -42350,8 +42300,6 @@ exports[`Selectable table 1`] = `
               <tbody
                 className=""
                 key="tbody-2"
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -42855,8 +42803,6 @@ exports[`Selectable table 1`] = `
               <tbody
                 className=""
                 key="tbody-3"
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -43360,8 +43306,6 @@ exports[`Selectable table 1`] = `
               <tbody
                 className=""
                 key="tbody-4"
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -43865,8 +43809,6 @@ exports[`Selectable table 1`] = `
               <tbody
                 className=""
                 key="tbody-5"
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -47283,8 +47225,6 @@ exports[`Simple Actions table 1`] = `
             >
               <tbody
                 className=""
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -58301,8 +58241,6 @@ exports[`Simple table aria-label 1`] = `
             >
               <tbody
                 className=""
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -64298,8 +64236,6 @@ exports[`Simple table caption 1`] = `
             >
               <tbody
                 className=""
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -70299,8 +70235,6 @@ exports[`Simple table header 1`] = `
             >
               <tbody
                 className=""
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -76347,8 +76281,6 @@ exports[`Sortable table 1`] = `
             >
               <tbody
                 className=""
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -82402,8 +82334,6 @@ exports[`Table variants Breakpoint - grid 1`] = `
             >
               <tbody
                 className=""
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -88457,8 +88387,6 @@ exports[`Table variants Breakpoint - grid-lg 1`] = `
             >
               <tbody
                 className=""
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -94512,8 +94440,6 @@ exports[`Table variants Breakpoint - grid-md 1`] = `
             >
               <tbody
                 className=""
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -100567,8 +100493,6 @@ exports[`Table variants Breakpoint - grid-xl 1`] = `
             >
               <tbody
                 className=""
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -106622,8 +106546,6 @@ exports[`Table variants Breakpoint - null 1`] = `
             >
               <tbody
                 className=""
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={
@@ -112677,8 +112599,6 @@ exports[`Table variants Size - compact 1`] = `
             >
               <tbody
                 className=""
-                onCollapse={null}
-                rows={Array []}
               >
                 <BodyRow
                   columns={