You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@poi.apache.org by bu...@apache.org on 2010/06/11 15:31:44 UTC

DO NOT REPLY [Bug 48245] [PATCH] TableCell.text() returns the texts in itself and the next cell

https://issues.apache.org/bugzilla/show_bug.cgi?id=48245

Nick Burch <ni...@alfresco.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #2 from Nick Burch <ni...@alfresco.com> 2010-06-11 09:31:40 EDT ---
Alas your patch broke lots of other unit tests :(

Having looked through it in great detail, I can confirm that there was
something wrong with the cell detection, which was worse on some files than
others

The fix that seems to work best, even though it shouldn't be needed, is to
create the cell from start->end paragraphs, then increment the end paragraph
count on the cell later on. This seems to give the right text and offsets,
along with the right counts

Fix applied to svn in r953694, along with some fairly exhaustive unit tests of
the issue, which seems to show it now behaves better

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org