hyperformula icon indicating copy to clipboard operation
hyperformula copied to clipboard

[Bug]: `simpleCellRangeToString` does not work with column ranges and row ranges

Open sandzone opened this issue 1 year ago • 4 comments
trafficstars

Description

If the range has values with entire columns or rows, simpleCellRangeToString is not giving the correct answer.

Video or screenshots

image

Demo

none

HyperFormula version

2.6.1

Your framework

No response

Your environment

node v18.7

sandzone avatar Feb 09 '24 07:02 sandzone

Hi @sandzone

Thank you for contacting us. I wasn't able to get the same result as you. Can you please send us a code demo where the issue is replicable?

adrianszymanski89 avatar Feb 09 '24 14:02 adrianszymanski89

Hi @sandzone

Thank you for contacting us. I wasn't able to get the same result as you. Can you please send us a code demo where the issue is replicable?

Thanks for the response. Here is a jsfiddle with the issue

https://jsfiddle.net/sandeep_muthangi/nsvqmbza/4/

image

sandzone avatar Feb 10 '24 23:02 sandzone

Hi @sandzone

Thank you for the example. I'm consulting our HyperFormula developer about it and should have more information by tomorrow. I'll keep you updated.

adrianszymanski89 avatar Feb 12 '24 10:02 adrianszymanski89

Hi, @sandzone, thank you for reporting this issue.

Currently simpleCellRangeToString method does not work with column ranges and row ranges. We will discuss this issue with the HyperFormula team. We need to either describe this limitation in the API reference or make it work with all types of ranges.

sequba avatar Feb 13 '24 19:02 sequba

Estimation for making simpleCellRangeToString work with all types of ranges: 3 SP.

sequba avatar Mar 07 '24 14:03 sequba

Hi @sandzone

Thank you again for your contribution to make HyperFormula better. I'm more than happy to announce that by the merge of https://github.com/handsontable/hyperformula/pull/1391 this issue made it to the 2.7.0 version, which was released today.

I'm closing this issue as done, but feel free to create a new one or write to us if you have any questions.

AMBudnik avatar Apr 10 '24 10:04 AMBudnik