Incorrect Enumerable type when object contains length property about linq HOT 5 CLOSED

Thank you for your response
I created local module with Enumerable extension (patch adds .fromObject method) and updated linq module to have type definition for the new method. It's suitable for me

from linq.

Good find, I don't think we need to keep that logic. I would change it to Array.isArray(obj), but need to check potential problems.

from linq.

I've done some tests and it seems like this change is not a good idea. Using Array.isArray(obj) (or instanceof ) instead of typeof obj.length == Types.Number would exclude array-like objects, most notably the arguments object and return values from DOM functions like document.getElementsByTagName() that return a list of elements.

You can consider this issue a small limitation of the library, but it's probably better to change the name of your length property to avoid other problems down the line.

Also a small correction to your code, when converting to object use .toObject(i => i.key, i => i.value); (without the capitalization)

from linq.

Is it possible to create method .fromObject or something like this?
We cannot change name of length property because we have too many models with this property. So renaming will take too much time to support it across the system.

from linq.

I think it would be odd to create a separate method for such a narrow use-case.

But you can of course create your own helper method which will transform your object to an array of key-value pairs.

from linq.