Bug 50795

Summary:	Moving cell comment crashes Excel file
Product:	POI	Reporter:	andrei <andrei.astrouski>
Component:	XSSF	Assignee:	POI Developers List <dev>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	P2
Version:	3.7-FINAL
Target Milestone:	---
Hardware:	PC
OS:	Windows XP
Attachments:	CellCommentTest.xlsx CellCommentTest.java

Description andrei 2011-02-16 09:30:02 UTC

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

Comment 1 andrei 2011-02-16 09:30:42 UTC

Created attachment 26669 [details]
CellCommentTest.java

Comment 2 Nick Burch 2011-02-18 10:31:05 UTC

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.