Skip to content
New issue

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

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

Already on GitHub? # to your account

Incorrect array size error for some cases of reversed ranges with absolute addressing (previously: Partially locked, reversed range breaks setCellContents) #1106

Closed
AJNandi opened this issue Nov 22, 2022 · 2 comments · Fixed by #1159
Assignees
Labels
Bug Something isn't working Impact: High

Comments

@AJNandi
Copy link

AJNandi commented Nov 22, 2022

Description

setCellContents on hyperformula instance with the following data causes an "Incorrect array size" error.

const tableData = [
  [1, "=SUM(A1:A$1)"],
  [1, "=SUM(A2:A$1)"]
];

Demo

https://codesandbox.io/s/frosty-meadow-umnbk2?file=/src/data.js

Partially locked ranges that are in the correct order work.
Calculating a single reversed and locked formula seems to work fine. See this repo here: https://codesandbox.io/s/silly-leftpad-d38lgu?file=/src/data.js

HyperFormula version: 2.2

Links

Related #835

@AJNandi AJNandi added the Bug Something isn't working label Nov 22, 2022
@sequba
Copy link
Contributor

sequba commented Nov 22, 2022

Hi @AJNandi, thanks for your report. It seems to be a very peculiar bug, but - obviously - this is not the intended behavior of HyperFormula so we'll address it in one of the upcoming releases. I hope it's not a blocker for you ;)

@sequba sequba self-assigned this Nov 22, 2022
@sequba sequba linked a pull request Feb 14, 2023 that will close this issue
14 tasks
@sequba sequba changed the title Partially locked, reversed range breaks setCellContents Incorrect array size for some cases of reversed ranges with absolute addressing (previously: Partially locked, reversed range breaks setCellContents) Feb 15, 2023
@sequba sequba changed the title Incorrect array size for some cases of reversed ranges with absolute addressing (previously: Partially locked, reversed range breaks setCellContents) Incorrect array size error for some cases of reversed ranges with absolute addressing (previously: Partially locked, reversed range breaks setCellContents) Feb 15, 2023
@sequba
Copy link
Contributor

sequba commented Mar 3, 2023

@AJNandi,

I'm more than happy to announce that we just released HyperFormula 2.3.1 where this issue is fixed. We are closing this issue as solved. If there is anything that won't work for you after updating, please leave a comment.

@sequba sequba closed this as completed Mar 3, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Bug Something isn't working Impact: High
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants