Created attachment 26668 [details] CellCommentTest.xlsx Steps: 1. Create Excel file. 2. Add cell comment. 3. Process with POI. Get our created cell comment and try to set to another cell. XSSFCell cell = row.getCell(0); XSSFComment comment = cell.getCellComment(); XSSFCell cellWithoutComment = row.getCell(1); cellWithoutComment.setCellComment(comment); 4. Try to save changes FileOutputStream out = new FileOutputStream(testFilePath); wb.write(out); out.close(); Actual result: NPE appears, file size = 0 bytes Expected: comment moved successfull (in 3.6 it works) See attachments to test
Created attachment 26669 [details] CellCommentTest.java
Thanks for the test case, it made it much quicker to investigate I'm rather confused by this one, but it's either a bug in xmlbeans, or we're doing something too odd in our processing of the vml diagram part. However, by adding a .toString() call in after the update, it seems to make xmlbeans re-compute whatever was broken. I'm sure there should be a better solution, but this seems to work fine as a workaround... Test and workaround added in r1072022.