Skip to content
This repository has been archived by the owner on Mar 9, 2020. It is now read-only.

Fix sheet reference in ToR1C1 formula translation #480

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Fix sheet reference in ToR1C1 formula translation #480

wants to merge 12 commits into from

Conversation

roelvandersteen
Copy link

If an ExcelAddress token satisfies the following conditions:

  • contains a worksheet name (followed by "!")
  • contains a reference to a single row or column (so it will also have a ":")

then the worksheet name is not returned when the address is translated from A1-style to R1C1-style.

This is now fixed. Closes #479

Roel van der Steen and others added 12 commits December 5, 2018 00:16
…or addresses, available via ParsingContext. ExcelRanges are now always compiled in ExcelAddressExpression, the SkipArgumentEvaluation parameter is removed, and addresses can be accessed via cache references
# Conflicts:
#	EPPlusTest/ExcelCellBaseTests.cs
If an ExcelAddress token satisfies the following conditions:

- contains a worksheet name (followed by "!")
- contains a reference to a single row or column (so it will also have a ":")

then the worksheet name is not returned when the address is translated from A1-style to R1C1-style.

This is now fixed. Closes #479
@roelvandersteen
Copy link
Author

The only relevant commit to fix this issue is 386dbef
The other commits are from #411, because this PR builds on fixes made there.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: formula loses reference to other worksheet when translating from A1-style to R1C1-style
4 participants