Comments (10)
Can you please try the edge repo version of Underscore. This sounds like it could be a sparse array issue, which the edge repo version of Underscore doesn't support (but doesn't support inconsistently so for it to fail you would need to check it in IE8 or less).
from lodash.
Upgrading to edge repo version of underscore in IE8 does show the same error I'm currently seeing in every browser with lodash.
So it does sound like the sparse array issue indeed ;)
from lodash.
Upgrading to edge repo version of underscore in IE8 does show the same error I'm currently seeing in every browser with lodash.
Bummer about the sparse array use, but also high five because this underscores why I forked.
One of the big reasons I forked and started Lo-Dash was to have better cross-browser consistency in the utility belt's API. In this case it's consistently reproducing an error and in others it's consistently iterating over object properties.
So it does sound like the sparse array issue indeed ;)
Lo-Dash doesn't support sparse arrays because we don't use native ES5 array methods and so don't have to follow spec to the letter. This allows Lo-Dash to optimize for the common case, non-sparse, and increase performance in all browsers.
from lodash.
Well this does mean that lodash is no longer a drop-in replacement for underscore as it does break code that was written against underscore.js
from lodash.
Well this does mean that lodash is no longer a drop-in replacement for underscore as it does break code that was written against underscore.js
Lo-Dash has some small differences which usually aren't a problem as I side on documented/tested/popular usage.
Your issue falls into the edge case category and as luck has it will have the same issue once Underscore version bumps.
from lodash.
This issues and edge cases should probably be documented somewhere so people are actually aware of the subtile but potential destructive differences between lodash and underscore.
Also edge-case or not.. It does mean that lodash is not a drop-in replacement of underscore.
from lodash.
This issues and edge cases should probably be documented somewhere so people are actually aware of the subtile but potential destructive differences between lodash and underscore.
Sure, that's a good idea.
Also edge-case or not.. It does mean that lodash is not a drop-in replacement of underscore.
No, for most devs Lo-Dash is a drop-in replacement for Underscore.
from lodash.
I agree with 3rd-Eden, it shouldnt be called 100% drop-in replacement for underscore. It simply behaves differently in some cases
from lodash.
Well this does mean that lodash is no longer a drop-in replacement for underscore as it does break code that was written against underscore.js.
Well...yes and no. Your code will still break when the next version of Underscore is officially released, as they have removed most of their sparse array handling code.
from lodash.
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
from lodash.
Related Issues (20)
- Typescript compiler error in Lodash types
- RE: [FR] Third argument to _.memoize - CacheConstructor #3319
- Feature Request: multiple variables in _.template()
- _.defaultsDeeps return and mutate behaviour is inconsistent with null object
- Description of `compact` is incomplete.
- [5 years old bug] _.debounce().flush() fix never made it into production
- Sonatype shows a new critical security violation. HOT 3
- Would It Be Possible to Update the Typing of `max([0, anyNumber])` Such That It Returns Number? HOT 3
- Command injection vuln HOT 3
- Add "isWeakRef" function.
- sortBy nor orderBy not working properly
- Cannot import lodash single function in Node HOT 3
- keyBy not support index HOT 2
- lodash.groupby npm package version is still 4.6.0
- Strange _.rearg / _.reduce behaviour
- isEqual returns incorrect result
- bun build commands not working as expected
- orderBy orders field boolean dosent work HOT 1
- Please republish lodash.forin HOT 4
- Wrong version in lodash.pick vulnerability report CVE-2020-8203 HOT 8
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from lodash.