Skip to content

Implement ability to read formulas#504

Open
nh43de wants to merge 2 commits into
ExcelDataReader:developfrom
nh43de:develop
Open

Implement ability to read formulas#504
nh43de wants to merge 2 commits into
ExcelDataReader:developfrom
nh43de:develop

Conversation

@nh43de

@nh43de nh43de commented Dec 18, 2020

Copy link
Copy Markdown

Changes

  • Implement ability to read formulas via the IExcelDataReader interface (OpenXML only, though pseudocode provided for biff implementation).
  • Unit tests

Closes #332

@victor-gutemberg

Copy link
Copy Markdown

hello @nh43de,

do you still plan to merge this PR? I was looking for a way to read formulas using ExcelDataReader when I found your changes.

@giogama

giogama commented Sep 19, 2024

Copy link
Copy Markdown

I found bug. There are some new .xlsx files that are coming with xl/workbook2.xml insted of xl/workbook.xml. The ExcelDataReader lib only looks for xl/workbook.xml. it should look for both files. I made the fix locally and it worked fine.

@wizardgsz

Copy link
Copy Markdown
Collaborator

I found bug. There are some new .xlsx files that are coming with xl/workbook2.xml insted of xl/workbook.xml. The ExcelDataReader lib only looks for xl/workbook.xml. it should look for both files. I made the fix locally and it worked fine.

Maybe you may open a new issue... anynow, which program was used to create such files?

@giogama

giogama commented Sep 20, 2024

Copy link
Copy Markdown

Microsoft Excel, but I don't know in which version from Microsoft Excel was created.

@hughbe

hughbe commented Oct 30, 2025

Copy link
Copy Markdown
Contributor

I'd be happy to help with the pseudocode implementation, but would love to see this reviewed

@appel1

appel1 commented Oct 31, 2025

Copy link
Copy Markdown
Collaborator

I'm fine with exposing formulas as a string like this. So if you'd like to complete this task it'd be great @hughbe .

@hughbe

hughbe commented Oct 31, 2025

Copy link
Copy Markdown
Contributor

Sure - I'm happy to start on something and send in a PR, assuming this will be merged :)

return RowCells[i]?.Value;
}

public string GetFormula(int i)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would prefer GetCellFormula

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Reading Formulas?

6 participants