Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

componentWillUnmount() not being called when refreshing the current page

Tags:

reactjs

I've been having this problem where my code in the componentDidMount() method wasn't firing properly when refreshing the current page (and subsequently, the component). However, it works perfectly fine just navigating and routing through my website by clicking links. Refresh the current page? Not a chance.

I found out that the problem is that componentWillUnmount() doesn't trigger when I refresh the page and triggers fine clicking links and navigating my website/app.

The triggering of the componentWillUnmount() is crucial for my app, since the data that I load and process in the componentDidMount() method is very important in displaying information to users.

I need the componentWillUnmount() to be called when refreshing the page because in my componentWillMount() function (which needs to re-render after every refresh) I do some simple filtering and store that variable in a state value, which needs to be present in the logos state variable in order for the rest of the component to work. This does not change or receive new values at any time during the component's life cycle.

componentWillMount(){   if(dataReady.get(true)){     let logos = this.props.questions[0].data.logos.length > 0 ? this.props.questions[0].data.logos.filter((item) => {       if(item.logo === true && item.location !== ""){         return item;       }     }) : [];     this.setState({logos: logos});   } }; 

Cliffs:

  • I do DB filtering in componentWillMount()method
  • Need it to be present in the component after refresh
  • But I have a problem where the componentWillUnmount() doesn't trigger when the page is refreshed
  • Need help
  • Please
like image 610
Mabeh Al-Zuq Yadeek Avatar asked Aug 22 '16 16:08

Mabeh Al-Zuq Yadeek


People also ask

Is componentWillUnmount called on refresh?

If you load all your data in the higher level component, you won't have to rely on the componentWillMount() method to trigger after every refresh. The data will be ready in the upper level component, so you can use it however you want in the child component.

How do I contact componentWillUnmount?

On clicking the 'Delete User' button, the User component will get unmounted from the DOM tree and before destroying the User component, the componentWillUnmount method will be called. Our first component in the following example is App.

When componentWillUnmount is called in React?

The componentWillUnmount() method allows us to execute the React code when the component gets destroyed or unmounted from the DOM (Document Object Model). This method is called during the Unmounting phase of the React Life-cycle i.e before the component gets unmounted.


2 Answers

When the page refreshes react doesn't have the chance to unmount the components as normal. Use the window.onbeforeunload event to set a handler for refresh (read the comments in the code):

class Demo extends React.Component {     constructor(props, context) {         super(props, context);          this.componentCleanup = this.componentCleanup.bind(this);     }      componentCleanup() { // this will hold the cleanup code         // whatever you want to do when the component is unmounted or page refreshes     }      componentWillMount(){       if(dataReady.get(true)){         let logos = this.props.questions[0].data.logos.length > 0 ? this.props.questions[0].data.logos.filter((item) => {           if(item.logo === true && item.location !== ""){             return item;           }         }) : [];          this.setState({ logos });       }     }      componentDidMount(){       window.addEventListener('beforeunload', this.componentCleanup);     }      componentWillUnmount() {         this.componentCleanup();         window.removeEventListener('beforeunload', this.componentCleanup); // remove the event handler for normal unmounting     }  } 

useWindowUnloadEffect Hook

I've extracted the code to a reusable hook based on useEffect:

// The hook  const { useEffect, useRef, useState } = React  const useWindowUnloadEffect = (handler, callOnCleanup) => {   const cb = useRef()      cb.current = handler      useEffect(() => {     const handler = () => cb.current()        window.addEventListener('beforeunload', handler)          return () => {       if(callOnCleanup) handler()            window.removeEventListener('beforeunload', handler)     }   }, [callOnCleanup]) }   // Usage example    const Child = () => {   useWindowUnloadEffect(() => console.log('unloaded'), true)      return <div>example</div> }  const Demo = () => {   const [show, changeShow] = useState(true)    return (     <div>       <button onClick={() => changeShow(!show)}>{show ? 'hide' : 'show'}</button>       {show ? <Child /> : null}     </div>   ) }  ReactDOM.render(   <Demo />,   root )
<script crossorigin src="https://unpkg.com/react@16/umd/react.development.js"></script> <script crossorigin src="https://unpkg.com/react-dom@16/umd/react-dom.development.js"></script>  <div id="root"></div>
like image 98
Ori Drori Avatar answered Sep 24 '22 15:09

Ori Drori


I also run into this problem and realised that I needed to make sure that at least 2 components will always gracefully unmount. So I finally did a High Order Component that ensures the wrapped component is always unmounted

import React, {Component} from 'react'    // this high order component will ensure that the Wrapped Component  // will always be unmounted, even if React does not have the time to  // call componentWillUnmount function  export default function withGracefulUnmount(WrappedComponent) {        return class extends Component {            constructor(props){              super(props);              this.state = { mounted: false };              this.componentGracefulUnmount = this.componentGracefulUnmount.bind(this)          }              componentGracefulUnmount(){              this.setState({mounted: false});                window.removeEventListener('beforeunload', this.componentGracefulUnmount);          }            componentWillMount(){              this.setState({mounted: true})          }            componentDidMount(){              // make sure the componentWillUnmount of the wrapped instance is executed even if React              // does not have the time to unmount properly. we achieve that by              // * hooking on beforeunload for normal page browsing              // * hooking on turbolinks:before-render for turbolinks page browsing              window.addEventListener('beforeunload', this.componentGracefulUnmount);          }            componentWillUnmount(){              this.componentGracefulUnmount()          }            render(){                let { mounted }  = this.state;                if (mounted) {                  return <WrappedComponent {...this.props} />              } else {                  return null // force the unmount              }          }      }    }

Note: If like me, you are using turbolinks and rails, you might wanna hook on both beforeunload and turbolinks:before-render events.

like image 44
Dorian Avatar answered Sep 22 '22 15:09

Dorian