Skip to content
This repository has been archived by the owner on May 29, 2019. It is now read-only.

fixes TypeError: Cannot set property 'isOpen' of null #5267

Closed
wants to merge 1 commit into from
Closed

fixes TypeError: Cannot set property 'isOpen' of null #5267

wants to merge 1 commit into from

Conversation

kwiateusz
Copy link
Contributor

Sometimes $evalAsync is fired when scope is null and I get error in console. This fixes that specificc situation

Sometimes $evalAsync is fired when scope is null and I get error in console. This fixes that specificc situation
@wesleycho
Copy link
Contributor

@RobJacobs is this possible when using $evalAsync? Seems strange to me...but not necessarily against this fix. Thoughts?

@RobJacobs
Copy link
Contributor

@wesleycho I agree, would be nice to see a plunk reproducing the problem or a description of how this occurred. I'm not against the change but there may be a better approach.

@wesleycho
Copy link
Contributor

I'll merge this for now, and perhaps an investigation could be done to figure out what is happening here - this seems like a perpetual problem :( .

@wesleycho wesleycho added this to the 1.1.0 milestone Jan 16, 2016
@wesleycho wesleycho closed this in e105e85 Jan 16, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants