Make react-intl react 16 compatible (formatjs#1201)

ptomasroos · redonkulus · commit 1c5afd236e99 · 2018-10-23T14:52:10.000-07:00
Closes formatjs#1078 This is not a breaking change since the access of the ref is used when calling getWrappedInstance() In order to support React 16. We need to remove the legacy way of using ref's by setting a string. This is now marked as legacy. ``` We advise against it because string refs have some issues, are considered legacy, and are likely to be removed in one of the future releases. Note If you’re currently using this.refs.textInput to access refs, we recommend using either the callback pattern or the createRef API. ``` https://reactjs.org/docs/refs-and-the-dom.html#legacy-api-string-refs
diff --git a/src/inject.js b/src/inject.js
@@ -42,15 +42,15 @@ export default function injectIntl(WrappedComponent, options = {}) {
           '`injectIntl()`'
       );
 
-      return this.refs.wrappedInstance;
+      return this._wrappedInstance;
     }
 
     render() {
       return (
         <WrappedComponent
           {...this.props}
           {...{[intlPropName]: this.context.intl}}
-          ref={withRef ? 'wrappedInstance' : null}
+          ref={withRef ? /* istanbul ignore next */ (ref => this._wrappedInstance = ref) : null}
         />
       );
     }
