GeocoderService incompatibility between when.js Promise and browser native Promise
See original GitHub issueI have made two examples in Sandcastle, one with browser-native Promise and one with when.js calls. Please load each example, search for anything, then hit Enter. In the browser-native Promise example, the bar continues to show “Searching…” until you click the X button to clear it. In the when.js example, the first (and only) result is selected as expected.
The docs say that geocode() should return a Promise.<Array.<GeocoderResult>> – I would be very happy if native Promise were supported, but until y’all figure out what’s broken maybe at least update the docs to specify when() instead?
Ubuntu 16.04, Chromium 58.0.3029.110 (latest stable)
Issue Analytics
- State:
- Created 6 years ago
- Comments:5 (5 by maintainers)
Top Results From Across the Web
GeocoderService incompatibility between when.js Promise ...
I have made two examples in Sandcastle, one with browser-native Promise and one with when.js calls. Please load each example, ...
Read more >Promise - JavaScript - MDN Web Docs
A Promise is a proxy for a value not necessarily known when the promise is created. It allows you to associate handlers with...
Read more >Promises | Can I use... Support tables for HTML5, CSS3, etc
"Can I use" provides up-to-date browser support tables for support of front-end ... A promise represents the eventual result of an asynchronous operation....
Read more >Native JavaScript Promises and Browser APIs - Blog - fuqua.io
Additionally, because you can't catch errors that are thrown inside callbacks from outside those callbacks, error handling needs to be spread throughout every ......
Read more >Native Support for Promises in Node.js - Stack Overflow
Although Node.js added native promise in stable version 0.12 . But due to the memory leak issue, I recommend to use bluebird to...
Read more >
Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free
Top Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found

As of https://github.com/CesiumGS/cesium/pull/10149, I believe this is resolved. Let me know if I’m missing anything.
It’s in two places:
https://github.com/AnalyticalGraphicsInc/cesium/blob/master/Source/Widgets/Geocoder/GeocoderViewModel.js#L348
and
https://github.com/AnalyticalGraphicsInc/cesium/blob/master/Source/Widgets/Geocoder/GeocoderViewModel.js#L446
I think that should fix the issue.