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 2009/11/20 01:43:13 UTC
DO NOT REPLY [Bug 48245] New: TableCell.text() returns the texts in
itself and the next cell
https://issues.apache.org/bugzilla/show_bug.cgi?id=48245
Summary: TableCell.text() returns the texts in itself and the
next cell
Product: POI
Version: 3.5-FINAL
Platform: All
OS/Version: All
Status: NEW
Severity: normal
Priority: P2
Component: HWPF
AssignedTo: dev@poi.apache.org
ReportedBy: jerry_soung@yahoo.com
Created an attachment (id=24569)
--> (https://issues.apache.org/bugzilla/attachment.cgi?id=24569)
The test document which contains a test table
Steps to Reproduce:
With the attached Word document, use the TableIterator to build the table.
Then walk through the table to print out the cell content (TableCell.text),
yields the following result:
Actual Results:
0,0: Row 1/Cell 1Row 1/Cell 2
0,1: Row 1/Cell 2Row 1/Cell 3
0,2: Row 1/Cell 3
1,0: Row 2/Cell 1Row 2/Cell 2
1,1: Row 2/Cell 2Row 2/Cell 3
1,2: Row 2/Cell 3
2,0: Row 3/Cell 1Row 3/Cell 2
2,1: Row 3/Cell 2Row 3/Cell 3
2,2: Row 3/Cell 3
Expected Results:
0,0: Row 1/Cell 1
0,1: Row 1/Cell 2
0,2: Row 1/Cell 3
1,0: Row 2/Cell 1
1,1: Row 2/Cell 2
1,2: Row 2/Cell 3
2,0: Row 3/Cell 1
2,1: Row 3/Cell 2
2,2: Row 3/Cell 3
--
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
DO NOT REPLY [Bug 48245] [PATCH] TableCell.text() returns the texts
in itself and the next cell
Posted by bu...@apache.org.
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
DO NOT REPLY [Bug 48245] [PATCH] TableCell.text() returns the texts
in itself and the next cell
Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=48245
--- Comment #1 from jerry_soung@yahoo.com 2009-11-19 16:47:41 UTC ---
Created an attachment (id=24570)
--> (https://issues.apache.org/bugzilla/attachment.cgi?id=24570)
Proposed patch
--
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
DO NOT REPLY [Bug 48245] [PATCH] TableCell.text() returns the texts
in itself and the next cell
Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=48245
jerry_soung@yahoo.com changed:
What |Removed |Added
----------------------------------------------------------------------------
Summary|TableCell.text() returns |[PATCH] TableCell.text()
|the texts in itself and the |returns the texts in itself
|next cell |and the next cell
--
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