React lifecycle method warnings

[denis-sokolov]

React deprecates componentWillReceiveProps in favor of getDerivedStateFromProps. Documentation. If we don’t fix this, the component will be broken in React 17.

react-image-fallback/src/index.js

Lines 39 to 47 in 6adce1a

	componentWillReceiveProps(nextProps) {
	if (nextProps.src !== this.props.src) {
	this.isLoaded = false;
	if (nextProps.initialImage) {
	this.handleInitialTimeout();
	}
	this.setDisplayImage({ image: nextProps.src, fallbacks: nextProps.fallbackImage });
	}
	}

[elquimista]

While we wait for this package to be updated, this works for me for now:

import ReactImageFallback from 'react-image-fallback'

const UNSAFE_LIFECYCLE_METHOD_NAMES = [
  'componentWillMount',
  'componentWillReceiveProps',
  'componentWillUpdate',
]

function renameUnsafeLifecycleMethods (Component) {
  const { prototype } = Component

  UNSAFE_LIFECYCLE_METHOD_NAMES.forEach((methodName) => {
    if (!prototype[methodName]) return

    Object.defineProperty(prototype, `UNSAFE_${methodName}`, {
      configurable: true,
      value: prototype[methodName],
      writable: true,
    })
    Reflect.deleteProperty(prototype, methodName)
  })

  return Component
}

const ImageFallback = renameUnsafeLifecycleMethods(ReactImageFallback)

[martinfoakes]

Just created a PR for this issue: #59