sutoiku / formula.js

JavaScript implementation of most Microsoft Excel formula functions
Other
2.14k stars 293 forks source link

make SUM support multi dimensional arrays #16

Closed joaojeronimo closed 10 years ago

joaojeronimo commented 10 years ago

As of now, SUM(A1:A3) could be translated as SUM([A1, A2, A3]) and that works, but if you SUM a range that spreads for multiple lines and columns, like SUM(A1:B3), that would be translated to SUM([[A1,B1], [A2, B2], [A3, B3]]) and that wouldn't work.

This PR makes that work.

0x333333 commented 10 years ago

Thanks for the patch!