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.

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

Bug Fix: Wrong calculation of IFERROR and Match function #408

Merged
merged 1 commit into from
Dec 8, 2020

Conversation

Yvees
Copy link
Contributor

@Yvees Yvees commented Sep 1, 2020

Wrong calculation of IFERROR and Match function when missing argument

@tonyqus tonyqus added the bug label Oct 6, 2020
@tonyqus tonyqus added this to the NPOI 2.5.3 milestone Oct 6, 2020
@tonyqus
Copy link
Member

tonyqus commented Oct 11, 2020

It looks the second args must be there.
image

In which case should it return string.empty?

Reference: https://exceljet.net/excel-functions/excel-iferror-function

@Yvees
Copy link
Contributor Author

Yvees commented Oct 25, 2020

It looks the second args must be there.
image

In which case should it return string.empty?

Reference: https://exceljet.net/excel-functions/excel-iferror-function

image

I mean this kind of scene. Note that the second parameter of this function is empty.

@tonyqus
Copy link
Member

tonyqus commented Oct 25, 2020

这是怎么输入进去的?怎么感觉在变戏法?我不得不开母语了。。。。这什么鬼功能.....

@Yvees
Copy link
Contributor Author

Yvees commented Oct 26, 2020

这是怎么输入进去的?怎么感觉在变戏法?我不得不开母语了。。。。这什么鬼功能.....

Note that the second parameter has a comma before it, so I guess Excel may think that the second parameter is there, but it is empty。 Just try yourself 😃

@tonyqus tonyqus modified the milestones: NPOI 2.5.5, NPOI 2.5.3 Dec 8, 2020
@tonyqus tonyqus merged commit 317da2b into nissl-lab:master Dec 8, 2020
@tonyqus tonyqus mentioned this pull request Apr 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants