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.

Sign up for GitHub

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

Jump to bottom

Expressions do not support associativity as expected #1072

Closed
JHawk opened this issue May 29, 2020 · 0 comments · Fixed by #1090
Closed

Expressions do not support associativity as expected #1072

JHawk opened this issue May 29, 2020 · 0 comments · Fixed by #1090
Assignees
@sc1f
Labels
bug Concrete, reproducible bugs JS

Comments

@JHawk
Copy link
Contributor

JHawk commented May 29, 2020 •
edited
Loading

Bug Report

Steps to Reproduce:

  1. On superstore.html, open the config
  2. Open the New Column editor and enter:
"Discount" + "Quantity" + "Profit" as "broken sum"
  1. Save the New Column
  2. Remove unnecessary pivots and put the columns in order Discount, Quantity, Profit, broken sum

Expected Result:

The column should show the sum of all 3 of the selected column.
Correct behavior can be seen by creating a new column with the following:

("Discount" + "Quantity") + "Profit" as "correct sum"

Screen Shot 2020-05-29 at 4 09 07 PM

Actual Result:

The sum presented is the sum of only the first two columns Discount + Quantity
Screen Shot 2020-05-29 at 4 11 30 PM

Screen Shot 2020-05-29 at 4 03 32 PM
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sc1f sc1f

Labels
bug Concrete, reproducible bugs JS
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Computed expressions respect left-to-right associativity and operator precedence finos/perspective
2 participants
@JHawk @sc1f