mobx-react + injection: wrappedComponent not found

[gcattan]

Hello,
I have just finished the mobx 4 to mobx 6 migration, and I noticed a difference in behavior.
In the image below we are rendering a component called BeSelectorField and we keep a reference to this component inside a member called fieldHasPopup.

The BeSelectorField is a class component with the annotation @inject.

In Mobx 4, the reference fieldHasPopup was an Injector, and to access the component itself we had to do fieldHasPopup.wrappedComponent.

Since we migrated to Mobx 6, fieldHasPopup is directly an instance of BeSelectorField.

Changing the syntax is not a issue, but I just want to check with you that this difference is expected. Thanks!

[urugator]

Yes, we now forward the ref automatically:

mobx/packages/mobx-react/src/inject.ts

Lines 19 to 30 in f0e066f

	// Support forward refs
	let Injector: IReactComponent<any> = React.forwardRef((props, ref) => {
	const newProps = { ...props }
	const context = React.useContext(MobXProviderContext)
	Object.assign(newProps, grabStoresFn(context || {}, newProps) || {})
	if (ref) {
	newProps.ref = ref
	}
	return React.createElement(component, newProps)
	})

I assume React.forwardRef did not exist or it just wasn't possible for other reason at that time . For completeness:
https://github.com/mobxjs/mobx-react/blob/f8302d81aee6795dd4b042e7408d8f2d38912d03/src/inject.js#L36-L73

EDIT: Not sure what you mean by "wrappedComponent not found", but the component should still be also available as wrappedComponent prop:

mobx/packages/mobx-react/src/inject.ts

Line 37 in f0e066f

Injector["wrappedComponent"] = component

[gcattan]

Ok, thanks for the answer! Indeed, in our case, the ref is not of type Injector and there is no wrappedComponent property available inside.
So instead of doing fieldHasProp.wrappedComponent.myAwesomeProperty (mobx 4) we are doing fieldHasProp.myAwesomeProperty directly (since mobx 6).