refactor of sumproduct

[izulin]

Context

SUMPRODUCT supports variable #args

How has this been tested?

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature or improvement (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Additional language file or change to the existing one (translations)

Related issue(s):

1. SUMPRODUCT with +3 arrays results with N/A #810

Checklist:

• My code follows the code style of this project,
• My change requires a change to the documentation,
• I described the modification in the CHANGELOG.md file.

[codecov]

Codecov Report

Merging #826 (92e2d6d) into develop (6ee4e70) will decrease coverage by 0.00%.
The diff coverage is 79.16%.

@@             Coverage Diff             @@
##           develop     #826      +/-   ##
===========================================
- Coverage    92.43%   92.43%   -0.01%     
===========================================
  Files          167      167              
  Lines        40302    40305       +3     
  Branches      5576     5579       +3     
===========================================
+ Hits         37253    37254       +1     
- Misses        3043     3045       +2     
  Partials         6        6              

Impacted Files	Coverage Δ
src/interpreter/plugin/SumprodPlugin.ts	89.47% <79.16%> (-3.12%)	⬇️

[wojciechczerniak]

It shouldn't be in "Fixed" category? SUMPRODUCT should support more args from the start. And according to docs it was announced that it does.

[wojciechczerniak]

Thanks! 🏆