From e91d59af540a8353e8493a077575561aeda515af Mon Sep 17 00:00:00 2001
From: Hypnosphi <Filipp.Riabchun@jetbrains.com>
Date: Mon, 9 Apr 2018 02:32:53 +0300
Subject: [PATCH] Fix lifecycle method usages in Modal

---
 src/components/Modal.js | 52 ++++++++++++++---------------------------
 1 file changed, 18 insertions(+), 34 deletions(-)

diff --git a/src/components/Modal.js b/src/components/Modal.js
index 397ca31c..6beedf12 100644
--- a/src/components/Modal.js
+++ b/src/components/Modal.js
@@ -16,12 +16,10 @@ const createPortal = isReact16
   : ReactDOM.unstable_renderSubtreeIntoContainer;
 
 function getParentElement(parentSelector) {
-  return parentSelector() || Modal.defaultProps.parentSelector;
+  return parentSelector();
 }
 
 class Modal extends Component {
-  state = Object.assign({}, Modal.defaultProps);
-
   static setAppElement(element) {
     ariaAppHider.setElement(element);
   }
@@ -122,45 +120,31 @@ class Modal extends Component {
     !isReact16 && this.renderPortal(this.props);
   }
 
-  static getDerivedStateFromProps(newProps, prevState) {
-    if (!canUseDOM) return null;
-    const { isOpen } = newProps;
-    // Stop unnecessary renders if modal is remaining closed
-    if (prevState && !prevState.isOpen && !isOpen) return null;
-
-    const currentParent = getParentElement(prevState.parentSelector);
-    const newParent = getParentElement(newProps.parentSelector);
+  getSnapshotBeforeUpdate(prevProps) {
+    const prevParent = getParentElement(prevProps.parentSelector);
+    const nextParent = getParentElement(this.props.parentSelector);
+    return { prevParent, nextParent };
+  }
 
-    const enumeratedState = {
-      isOpen: newProps.isOpen,
-      parentSelector: newProps.parentSelector,
-      portalClassName: newProps.portalClassName
-    };
+  componentDidUpdate(prevProps, _, snapshot) {
+    if (!canUseDOM) return;
+    const { isOpen, portalClassName } = this.props;
 
-    if (newParent !== currentParent) {
-      currentParent.removeChild(this.node);
-      newParent.appendChild(this.node);
+    if (prevProps.portalClassName !== portalClassName) {
+      this.node.className = portalClassName;
     }
 
-    const newState = Object.assign(prevState, enumeratedState);
-    !isReact16 && this.renderPortal(newProps);
-    return newState;
-  }
+    // Stop unnecessary renders if modal is remaining closed
+    if (!prevProps.isOpen && !isOpen) return;
 
-  getSnapshotBeforeUpdate(prevProps, prevState) {
-    if (!canUseDOM) {
-      return null;
-    } else {
-      this.node.className = prevState.portalClassName;
-      return prevState.portalClassName;
+    const { prevParent, nextParent } = snapshot;
+    if (nextParent !== prevParent) {
+      prevParent.removeChild(this.node);
+      nextParent.appendChild(this.node);
     }
-  }
 
-  /*eslint-disable no-unused-vars*/
-  componentDidUpdate(prevProps, prevState) {
-    // Just to satisfy polyfilling
+    !isReact16 && this.renderPortal(this.props);
   }
-  /*eslint-enable no-unused-vars*/
 
   componentWillUnmount() {
     if (!canUseDOM || !this.node || !this.portal) return;