search

mjul / docjure

Read and write Office documents from Clojure
MIT License
622 stars 129 forks source link

unable to read cells with formula referring to string value #34

Closed lambda-ai closed 9 years ago

lambda-ai commented 9 years ago

The code for read-cell first checks the date format, but during that check, POI throws because it's assuming already a numeric cell type. It's probably rather a POI bug, but I was wondering why the date check is needed in the first place.

mjul commented 9 years ago

Thanks for identifying this. I was not aware of it and from you description it seems like it is a bug indeed. If you would investigate it a bit more and submit pull request with a test-case and a fix I would be happy to include your contribution. Thanks for taking the time to helping improve the project. Many people rely on it for their software, so you effort will be widely appreciated. All the best, Martin

lambda-ai commented 9 years ago

Hi, it turns out that @oholworthy already fixed it a month ago. Thanks Oliver and all for the nice lib! Closing the bug report...

mjul commented 9 years ago

Excellent, thank you for investigating the issue.