Update Spreadsheet to POI 4.0 #676
Description
The release vote passed, POI 4.0.0 will be available next week. Most breaking changes are in these quick and easy categories:
- Switch from using int values where Enum is a better fit
- Some method names changed, mostly due to original typos
- If NPOIFS classes were referenced directly, the package has been replaced by POIFS. Won't affect most consumers.
This release also contains some bug fixes, especially one that affected use cases where the same Workbook object underwent multiple edit>save>edit>save cycles [1]. In those cases the saved files after the first save are incorrect/invalid in previous releases. This didn't affect most consumers, who only write or only read a file one time, but projects based on Vaadin-Spreadsheet could easily be affected as it enables real-time editing.
If Vaadin is willing to accept it, I'm willing to take a stab at a compatibility PR. If that works, then I could re-try my major updates to fix all the conditional format bugs/shortcomings and add in full table formatting and document style theme support. I also have a slew of changes that add parsing/support for a much wider range of Excel chart formatting options. These have all been working for a year in our production code, which is starting to become a pure fork at this point, but I'd rather work toward re-unification, if Vaadin plans to continue supporting this product.