Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Formula does not get initially evaluated with column mapping #4066

Closed
Ubersteinn opened this issue Feb 9, 2017 · 11 comments
Closed

Formula does not get initially evaluated with column mapping #4066

Ubersteinn opened this issue Feb 9, 2017 · 11 comments
Labels
bug Formulas Plugin Part of scope This issue is a part of a larger task Reported

Comments

@Ubersteinn
Copy link

Description

Formulas are not initially evaluated when a column uses data mapping.
Editing the cell fixes the issue.

Demo

Cell A1 is not evaluated as a formula since its column is using data mapping.
The cells in other columns with formulas are not effected.
https://jsfiddle.net/Ubersteinn/xf28qLqo/

Your environment

  • Handsontable version: pro 1.9.1

Any workaround would be greatly appreciated!

@Ubersteinn Ubersteinn changed the title Formula does not work with column mapping Formula does not get initially evaluated with column mapping Feb 10, 2017
@AMBudnik
Copy link
Contributor

Thank you for sharing @Ubersteinn

@Ubersteinn
Copy link
Author

Thank you for your quick response!
I have looked a little into the code.

The recalculateFull() and recalculateOptimized() methods in sheet.js get the source data object but do not take into account the mapping.
This may be the root cause.
I hope this helps!

@gstenson
Copy link

gstenson commented Jun 6, 2017

Have noticed this too. Any update on this?

This renders formulae unusable in anything bar simple applications in the PRO version, as if you are loading and binding them to/from an external source then they need to evaluated on initialisation.

@AMBudnik
Copy link
Contributor

Hi guys,

Formula plugin is in an alpha stage. We have stopped our work on a more stable version but we will hopefully get back to it in Q3 or Q4.

@rohitjoshi1990
Copy link

Would really like an update on this ?

Evaluating formulas on data initialization is crucial, since currently Object data source with column mapping is used to render entire Handsontable Matrix & changing data representation just to use formulas does not seem right.

@AMBudnik
Copy link
Contributor

AMBudnik commented Nov 2, 2018

Hi @rohitjoshi1990

we are working on a calculating engine that will allow handling all the operations bugless and super-fast, however, this is an innovative approach and it will definitely take us a couple of weeks.

@AMBudnik
Copy link
Contributor

Updated to 7.1.1 https://jsfiddle.net/mujLfc3w/
The issue stays the same A1's formula is not evaluated.

Zrzut ekranu 2019-09-27 o 10 33 11

@AMBudnik
Copy link
Contributor

AMBudnik commented Feb 11, 2021

This issue is closed temporarily. It will be reopened for development as it became a part of New formula plugin task reported at #6466

I will make sure to inform everyone interested in this topic after the official fix.

Please feel welcome to makes any comments on this issue. And if you experience similar behavior feel free to contact me at support@handsontable.com

Note to myself - inform Zen 26962

@aninde
Copy link
Contributor

aninde commented Jun 16, 2021

This issue works different with HyperFormula.
There is an console error with v9.0.0 that we should address and the table doesn't renderer.
https://jsfiddle.net/aninde/xt9gqbpk/

handsontable.full.js:4685 Uncaught Error: Unable to parse value: {
    "value": "=$B$2"

@hihhhello
Copy link

Was it fixed?

@AMBudnik
Copy link
Contributor

Hi @hihhhello

The issue is temporarily closed but not fixed yet. I will inform you after the fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Formulas Plugin Part of scope This issue is a part of a larger task Reported
Projects
None yet
Development

No branches or pull requests

7 participants