|Summary:||[PATCH] Accept 0-based ID values|
|Product:||POI||Reporter:||Joachim Breuer <apache>|
|Component:||XWPF||Assignee:||POI Developers List <dev>|
|Attachments:||Patch extending the accepted identifier range to include 0|
Description Joachim Breuer 2013-09-17 16:21:44 UTC
Created attachment 30854 [details] Patch extending the accepted identifier range to include 0 XWPF documents using identifiers starting with 0 will currently fail with: "IllegalArgumentException: Value for parameter 'id' was out of bounds". This is due to the IdentifierManager being initialized for IDs starting at 1. Documents from LibreOffice (at least 220.127.116.11 and 4.1.1) may use IDs starting at 0. (AFAIU IDs get used for objects embedded into the documents, i.e. pictures.) See also http://stackoverflow.com/questions/18756484/getting-illegalarguementexception-value-for-parameter-id-was-out-of-bounds With the attached trivial patch, POI/XWPF worked correctly for such documents here.
Comment 1 Edu Garcia 2014-05-27 04:25:35 UTC
Is there any reason this hasn't been patched already?
Comment 2 Nick Burch 2014-05-27 09:59:35 UTC
This is pending a unit test, along with a very small file that shows the problem, which can be used to check that the problem is fixed by the bug + remains fixed into the future
Comment 3 Edu Garcia 2014-05-28 00:54:35 UTC
The following document breaks it: calibre-ebook.com/downloads/demos/demo.docx
Comment 4 Nick Burch 2014-05-28 14:18:50 UTC
Do you know what license that file is under? We can only accept test files that are either under the ASLv2, or explicitly contributed to us by the author