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

Add cell location to exception #196

Closed
maxkoshevoi opened this issue Jun 29, 2022 · 1 comment
Closed

Add cell location to exception #196

maxkoshevoi opened this issue Jun 29, 2022 · 1 comment

Comments

@maxkoshevoi
Copy link

maxkoshevoi commented Jun 29, 2022

There're may be more cases, but here's mine.

  • I have a cell in Excel with value 55555555 and Date type . Type is incorrect, it should be Number or Currency, or something like that. Here's how Excel interprets it:
    image
  • When I try to read said Excel document via FetchAsync I get a ArgumentOutOfRangeException: Value to add was out of range. (Parameter 'value') with no additional information, which doesn't help me at all
  • It would be helpful to at least have a cell location in the exception somewhere.

Exception is thrown here:

return cell.DateCellValue;

PS: Also this should fix the issue nissl-lab/npoi#865

@mganss mganss closed this as completed in f25e34a Jun 29, 2022
@mganss
Copy link
Owner

mganss commented Jun 29, 2022

Thanks for reporting this. Fixed in 5.2.413.

mganss pushed a commit that referenced this issue Jun 30, 2022
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

No branches or pull requests

2 participants