From c50895310587cdfd3e12495c8b763e234a8f7707 Mon Sep 17 00:00:00 2001
From: George Sobolev <gsobolev@users.noreply.github.com>
Date: Tue, 12 Dec 2017 11:34:39 +0300
Subject: [PATCH] fix(react-grid): get rid of console error during column
 dragging (#570)

---
 .../src/templates/drag-drop.jsx               |  6 ++----
 .../src/templates/drag-drop.test.jsx          | 16 ---------------
 .../src/templates/drag-drop.jsx               |  6 ++----
 .../src/templates/drag-drop.test.jsx          | 20 -------------------
 4 files changed, 4 insertions(+), 44 deletions(-)

diff --git a/packages/dx-react-grid-bootstrap3/src/templates/drag-drop.jsx b/packages/dx-react-grid-bootstrap3/src/templates/drag-drop.jsx
index 76970abf8b..94e7a15c2c 100644
--- a/packages/dx-react-grid-bootstrap3/src/templates/drag-drop.jsx
+++ b/packages/dx-react-grid-bootstrap3/src/templates/drag-drop.jsx
@@ -3,8 +3,8 @@ import PropTypes from 'prop-types';
 import classNames from 'classnames';
 
 export const Container = ({
-  clientOffset, columns, columnTemplate,
-  style, className, children, ...restProps
+  clientOffset, style, className, children,
+  ...restProps
 }) => (
   <ul
     className={classNames('list-group', className)}
@@ -29,8 +29,6 @@ Container.propTypes = {
     x: PropTypes.number.isRequired,
     y: PropTypes.number.isRequired,
   }).isRequired,
-  columns: PropTypes.array.isRequired,
-  columnTemplate: PropTypes.func.isRequired,
   style: PropTypes.object,
   className: PropTypes.string,
   children: PropTypes.oneOfType([
diff --git a/packages/dx-react-grid-bootstrap3/src/templates/drag-drop.test.jsx b/packages/dx-react-grid-bootstrap3/src/templates/drag-drop.test.jsx
index 76c7eb53ce..3bf5fd1536 100644
--- a/packages/dx-react-grid-bootstrap3/src/templates/drag-drop.test.jsx
+++ b/packages/dx-react-grid-bootstrap3/src/templates/drag-drop.test.jsx
@@ -7,10 +7,6 @@ describe('Container', () => {
     const tree = shallow((
       <Container
         clientOffset={{ x: 10, y: 20 }}
-        columns={[{
-          name: 'Test',
-        }]}
-        columnTemplate={() => <div />}
       />
     ));
 
@@ -24,11 +20,7 @@ describe('Container', () => {
   it('should apply custom styles', () => {
     const tree = shallow((
       <Container
-        columns={[{
-          name: 'Test',
-        }]}
         clientOffset={{ x: 10, y: 20 }}
-        columnTemplate={() => <div />}
         style={{ color: 'red' }}
       />
     ));
@@ -43,11 +35,7 @@ describe('Container', () => {
   it('should pass the className prop to the root element', () => {
     const tree = shallow((
       <Container
-        columns={[{
-          name: 'Test',
-        }]}
         clientOffset={{ x: 10, y: 20 }}
-        columnTemplate={() => <div />}
         className="custom-class"
       />
     ));
@@ -61,11 +49,7 @@ describe('Container', () => {
   it('should pass rest props to the root element', () => {
     const tree = shallow((
       <Container
-        columns={[{
-          name: 'Test',
-        }]}
         clientOffset={{ x: 10, y: 20 }}
-        columnTemplate={() => <div />}
         data={{ a: 1 }}
       />
     ));
diff --git a/packages/dx-react-grid-material-ui/src/templates/drag-drop.jsx b/packages/dx-react-grid-material-ui/src/templates/drag-drop.jsx
index 5268ee8e00..5d8c14f038 100644
--- a/packages/dx-react-grid-material-ui/src/templates/drag-drop.jsx
+++ b/packages/dx-react-grid-material-ui/src/templates/drag-drop.jsx
@@ -23,8 +23,8 @@ const styles = theme => ({
 });
 
 const ContainerBase = ({
-  clientOffset, columns, columnTemplate, classes,
-  style, className, children, ...restProps
+  clientOffset, classes, style, className, children,
+  ...restProps
 }) => (
   <Paper
     className={classNames(classes.container, className)}
@@ -47,8 +47,6 @@ ContainerBase.propTypes = {
     PropTypes.node,
     PropTypes.arrayOf(PropTypes.node),
   ]),
-  columns: PropTypes.array.isRequired,
-  columnTemplate: PropTypes.func.isRequired,
   classes: PropTypes.object.isRequired,
   style: PropTypes.object,
   className: PropTypes.string,
diff --git a/packages/dx-react-grid-material-ui/src/templates/drag-drop.test.jsx b/packages/dx-react-grid-material-ui/src/templates/drag-drop.test.jsx
index df62d9695f..3f7240e01c 100644
--- a/packages/dx-react-grid-material-ui/src/templates/drag-drop.test.jsx
+++ b/packages/dx-react-grid-material-ui/src/templates/drag-drop.test.jsx
@@ -11,11 +11,7 @@ describe('DragDrop', () => {
       shallow = createShallow({ dive: true });
       classes = getClasses((
         <Container
-          columns={[{
-            name: 'Test',
-          }]}
           clientOffset={{ x: 10, y: 20 }}
-          columnTemplate={() => <div />}
         />
       ));
     });
@@ -23,11 +19,7 @@ describe('DragDrop', () => {
     it('should have correct styles', () => {
       const tree = shallow((
         <Container
-          columns={[{
-            name: 'Test',
-          }]}
           clientOffset={{ x: 10, y: 20 }}
-          columnTemplate={() => <div />}
         />
       ));
 
@@ -38,11 +30,7 @@ describe('DragDrop', () => {
     it('should apply custom styles', () => {
       const tree = shallow((
         <Container
-          columns={[{
-            name: 'Test',
-          }]}
           clientOffset={{ x: 10, y: 20 }}
-          columnTemplate={() => <div />}
           style={{ color: 'red' }}
         />
       ));
@@ -57,11 +45,7 @@ describe('DragDrop', () => {
     it('should pass the className prop to the root element', () => {
       const tree = shallow((
         <Container
-          columns={[{
-            name: 'Test',
-          }]}
           clientOffset={{ x: 10, y: 20 }}
-          columnTemplate={() => <div />}
           className="custom-class"
         />
       ));
@@ -75,11 +59,7 @@ describe('DragDrop', () => {
     it('should pass rest props to the root element', () => {
       const tree = shallow((
         <Container
-          columns={[{
-            name: 'Test',
-          }]}
           clientOffset={{ x: 10, y: 20 }}
-          columnTemplate={() => <div />}
           data={{ a: 1 }}
         />
       ));