Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

Version 0.8.13 crashes with "ReferenceError: _global is not defined" #835

Closed
adrienverge opened this issue Jul 13, 2017 · 15 comments · Fixed by #834
Closed

Version 0.8.13 crashes with "ReferenceError: _global is not defined" #835

adrienverge opened this issue Jul 13, 2017 · 15 comments · Fixed by #834

Comments

@adrienverge
Copy link

Updating to 0.8.13 (released yesterday) prevents our Angular application from booting:

Uncaught ReferenceError: _global is not defined
    at zone.js:1476
    at attachBrowserEvents (index-browser.js:500)
    at new Changes (index-browser.js:515)
    at Object.<anonymous> (index-browser.js:4804)
    at Object.<anonymous> (index-browser.js:12125)
    at __webpack_require__ (bootstrap 6dc1844…:52)
    at Object.44 (main.ts:31)
    at __webpack_require__ (bootstrap 6dc1844…:52)
    at Object.470 (ckeditor.module.ts:22)
    at __webpack_require__ (bootstrap 6dc1844…:52)
@JiaLiPassion
Copy link
Collaborator

thank you for posting the issue, I will check it.

JiaLiPassion added a commit to JiaLiPassion/zone.js that referenced this issue Jul 13, 2017
JiaLiPassion added a commit to JiaLiPassion/zone.js that referenced this issue Jul 13, 2017
JiaLiPassion added a commit to JiaLiPassion/zone.js that referenced this issue Jul 13, 2017
sguiheux added a commit to ovh/cds that referenced this issue Jul 13, 2017
@nerumo
Copy link

nerumo commented Jul 13, 2017

I'm having the same issue, it occurs in my app when a pouchdb lib tries to register an event handler without a window or global

addEventListener("storage", function (e) { self.emit(e.key); });

@JiaLiPassion
Copy link
Collaborator

It have been fixed in #834, please wait for the next version, sorry for the breaking error.

@adrienverge
Copy link
Author

Thanks for this fast fix!

mhevery pushed a commit that referenced this issue Jul 14, 2017
* fix(task): fix #832, task.data should be an object

* fix #835, add reference to _global

* refactor to use _global from _load_patch
@riddla
Copy link

riddla commented Jul 14, 2017

@JiaLiPassion: should I be able to get the fix with "zone.js": "git://github.com/angular/zone.js.git#8a15c660da6b178601c306cf7c63d62ae36bf3f6"? With this I'm still getting _global is not defined ... (with the context of PouchDB taht @nerumo explained).

@JiaLiPassion
Copy link
Collaborator

@riddla , the zone.js will not updated until new version is released, so you can try attached zone.js.zip for now.
zone.js.zip

@taxilian
Copy link

taxilian commented Jul 15, 2017

Thanks for the quick work; not trying to rock the boat, but do you have an ETA for the release? Just so I know when to check back, not to rush you.

fsamin pushed a commit to ovh/cds that referenced this issue Jul 17, 2017
* fix (api): change get commits algo for github
* fix (api): fix date
* fix (api): Get commit from new branch
* fix angular/zone.js#835
pubkey added a commit to pubkey/rxdb that referenced this issue Jul 17, 2017
Known bug with zone.js: [_global is not defined](angular/zone.js#835)
@endyjasmi
Copy link

@JiaLiPassion Can you tell me how to substitute the zone.js with the one installed?

@JiaLiPassion
Copy link
Collaborator

@taxilian , It should be released soon, sorry for inconvenience.

@JiaLiPassion
Copy link
Collaborator

@endyjasmi , you can just run npm install zone.js@0.8.12 for walk around.

@taxilian
Copy link

We're all coders here, we know that it happens sometimes despite the best you can do =] Thanks for your work on the project, just trying to plan is all.

@JiaLiPassion
Copy link
Collaborator

@taxilian , thank you:-)

@endyjasmi
Copy link

@JiaLiPassion Thanks for the work around. It is working great.

@JiaLiPassion
Copy link
Collaborator

@endyjasmi , zone.js@0.8.14 have been released, you can also try the new version.

@endyjasmi
Copy link

@JiaLiPassion So far so good. Thanks for the great work.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants