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

Error: Uncaught (in promise): [object Undefined] #11991

Closed
lmarcelocc opened this issue May 16, 2024 · 3 comments
Closed

Error: Uncaught (in promise): [object Undefined] #11991

lmarcelocc opened this issue May 16, 2024 · 3 comments
Labels
needs feedback On hold until additional info is supplied needs triage type - bug

Comments

@lmarcelocc
Copy link

What happened?

Same as discussed here.

Error: Uncaught (in promise): [object Undefined]
scheduleMacroTaskWithCurrentZone | @ | zone.umd.js:724
  | (anonymous) | @ | zone.umd.js:2465
  | proto. | @ | zone.umd.js:1017
  | render_ | @ | OLCesium.js:209
  | onAnimationFrame_ | @ | OLCesium.js:250
  | timer | @ | zone.umd.js:2441
  | _ZoneDelegate.invokeTask | @ | zone.umd.js:445
  | (anonymous) | @ | core.mjs:10757
  | onInvokeTask | @ | core.mjs:10757
  | _ZoneDelegate.invokeTask | @ | zone.umd.js:444
  | onInvokeTask | @ | core.mjs:11070
  | _ZoneDelegate.invokeTask | @ | zone.umd.js:444
  | Zone.runTask | @ | zone.umd.js:210
  | ZoneTask.invokeTask | @ | zone.umd.js:527
  | ZoneTask.invoke | @ | zone.umd.js:516
  | data.args. | @ | zone.umd.js:2421

Was able to workaround it, but we really should try to see where this unhandled promise is being thrown

Reproduction steps

  1. 3D Tileset
  2. Zoom in and move a little bit fast to perform a lot of requests

Sandcastle example

No response

Environment

Browser: Android
CesiumJS Version: 1.117.0
Operating System: Tried with Android 11, 13 and 14

@ggetz
Copy link
Contributor

ggetz commented May 16, 2024

Thanks for the report @lmarcelocc! Would you be able to include a Sandcastle example that replicates the issue? That would help us give us more context to determine the issue.

@ggetz ggetz added the needs feedback On hold until additional info is supplied label May 16, 2024
@lmarcelocc
Copy link
Author

Hi @ggetz ,

Unfortunately I tried last week to replicate on Sandcastle, with same 3D data, but I was not able to. Probably because, like mentioned here, could be Angular related (zone.js).

@ggetz
Copy link
Contributor

ggetz commented May 16, 2024

Got it, thanks @lmarcelocc. From the discussion, it looks like zone.js is wrapping native promises leading to the adverse behavior. I'm not sure there is much we can do in that situation.

We are generally trying to track down instances of undefined being thrown, as this can cause issues in our unit tests. Hopefully that will fix the issue your seeing as well.

I'm going to close this issue as am not sure there is actionable path beyond what we would do to fix #11958. Let me know if you think otherwise.

@ggetz ggetz closed this as completed May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs feedback On hold until additional info is supplied needs triage type - bug
Projects
None yet
Development

No branches or pull requests

2 participants