deepAssign does not support updating arrays using objects with numeric keys

[RyanMilligan]

How often can you reproduce it?

• [ x ] Always
• Sometimes
• Rarely
• Unable
• I didn’t try

Description: deepAssign() does not support updating specific items in arrays by passing objects with an index of, for example, 1. Instead, it ignores the keys in the source object and updates the first N items in the array.

1. JSBin: https://jsbin.com/suyowov/edit?js,console,output
2. Open the console and run the script
3. The third object output should show that "second" has changed to 2, but instead the 1 from the previous output has changed to 2

Actual results: Only the first item in the array can be edited in this way

Environment:

Software	Version
can-define version	1.5.4
Browser	Chrome Update 63
Operating system	Windows 10 # #

[RyanMilligan]

A proposed solution can be found here but I logged it here at the request of @phillipskevin.

[justinbmeyer]

Issue moved to canjs/can-reflect #99 via ZenHub

[justinbmeyer]

thanks for reporting, moving to can-reflect.

[justinbmeyer]

I'm not sure the proposed solution is the right one. What should toArray({0: "a"}) return?