Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IE11, findParent fails with 'Unable to get property '__reactInternalInstance$...' of undefined or null reference #8031

Closed
Ragnaroek opened this issue Oct 21, 2016 · 4 comments

Comments

@Ragnaroek
Copy link

Do you want to request a feature or report a bug?
Bug

What is the current behavior?
Exception is thrown, rendering stops.

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via https://jsfiddle.net or similar (template: https://jsfiddle.net/reactjs/69z2wepo/).
Doesn't seem to be determinstic, but it happens only in IE (at least in IE11).

What is the expected behavior?
Should not throw an exception and continue rendering.

Which versions of React, and which browser / OS are affected by this issue? Did this work in previous versions of React?
React 15.2.1, IE 11 on Windows 8.1

@Ragnaroek
Copy link
Author

This PR tries to fix the symptoms: #8032

@aweary
Copy link
Contributor

aweary commented Oct 21, 2016

@Ragnaroek thanks for the report!

Doesn't seem to be determinstic, but it happens only in IE (at least in IE11).

Can you share a test case that reproduces it nondeterministically? Even if it isn't consistent, it's better than nothing. I would say fixing the symptoms and not the issue isn't a good idea here, as it may just hide a deeper issue or cause hard to debug problems somewhere else in the code path.

@aweary aweary added the Status: Unconfirmed A potential issue that we haven't yet confirmed as a bug label Oct 21, 2016
@Ragnaroek
Copy link
Author

I agree that fixing the symptom this way was a bad idea.
As it turned out the problem was not React, but an exception in one of our promise handlers.

@aweary
Copy link
Contributor

aweary commented Oct 24, 2016

@Ragnaroek thanks for the update, glad you figured it out 👍!

@aweary aweary added Resolution: Invalid and removed Status: Unconfirmed A potential issue that we haven't yet confirmed as a bug labels Oct 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants