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

collections breaks Array#find behavior #165

Closed
erossignon opened this issue Oct 28, 2016 · 6 comments
Closed

collections breaks Array#find behavior #165

erossignon opened this issue Oct 28, 2016 · 6 comments

Comments

@erossignon
Copy link

erossignon commented Oct 28, 2016

Spotted here node-opcua/node-opcua#248

Collections v5.0.5 seems to break and replace some Array methods, introducing breaking changes.

How to reproduce:

var a = ["e1","e2","e3"];
console.log(" before   ", a.find(function(e) { return e=="e1";}));
var collections = require("collections/shim-array");
console.log(" after    ", a.find(function(e) { return e=="e1";}));
@codebling
Copy link

See also #162, #116, #95 and #94

@stevenvachon
Copy link

stevenvachon commented Apr 5, 2017

Seriously, this stupid module is breaking my code. Please unpublish.

@kriskowal
Copy link
Member

For what it's worth, I'm sorry.

@kriskowal
Copy link
Member

It's worth noting that this issue is fixed in version 2, but walked backward in version 3 and I think 4. It might be good for the collections community for the version of this library with the "latest" tag in npm is one that fixes this problem and that legacy applications should pin to the legacy versions.

@hthetiot
Copy link
Contributor

hthetiot commented Dec 7, 2017

Duplicate #185

@codebling
Copy link

See also issues #36 #70 #94 #95 #116 #139 #145 #162 #165 #169 #178 #182 #185 #197 #215 #220 and PRs #94 #95 #116 #173 #189 #212. Branch v2 fixes these issues by avoiding global object modification.

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

No branches or pull requests

5 participants