search

jspreadsheet / ce

Jspreadsheet is a lightweight vanilla javascript plugin to create amazing web-based interactive tables and spreadsheets compatible with other spreadsheet software.
https://bossanova.uk/jspreadsheet/v4
MIT License
6.66k stars 820 forks source link

JExcel Status and JExcel Formula Evaluation Bug #1628

Closed jumafuse closed 11 months ago

jumafuse commented 1 year ago

Dear Paul,

Posting here since I didn't get feedback from email.

I hope you are doing well, I can see you have done quite a bit of work in JExcel since the last time we exchange an email.

From our side we continue supporting JExcel in our product even if we have not move yet to the latest version. But this will happen during this year when we do our forms rewrite. I hope you continue supporting the same APIs!!

One of our users reported a bug evaluating formulas that I could reproduce in your web example, I think this is worth looking into because it affects a core feature, formulas evaluation. I hope the screenshot is self explanatory but let me know if clarifications are needed.

Under any case let me know if continue taking sponsorships, I would expect we may need some work during the year.

Have a good day, Juan

cascading-simple-formula-original.xlsx cascading-simple-formula-web-jspreadsheet-error cascading-simple-formula-web-jspreadsheet-save.csv

hodeware commented 11 months ago

There were some updates yestarday that should be improve this. Please upgrade your JSS version.

jumafuse commented 11 months ago

We will test and let you know. Thanks!

jumafuse commented 11 months ago

Did you test the example provided? The problem still persists for me, I still got a NaN.

When calculating the formula: =(G4+G5+G6+G7)/(G2+G3+G4+G5+G6+G7+G9)

It seems like the problem lies on the second part, if I substitute it, calculating that part in another cell it works. =(G4+G5+G6+G7)/H14

But of course this will mean than in complex spreadsheets is required to have lots of additional artificial cells that is a big NONO.

Hope this helps to find what is wrong.

hodeware commented 11 months ago

Here is a working example: https://jsfiddle.net/wqo3r1hx/3/

jumafuse commented 11 months ago

I'm very sorry to have reopened the issue twice, I didn't realise I was testing the wrong version.

Thanks for taking care of this.