Add some more paragraph property documentation and tests

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@997413 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Nick Burch 2010-09-15 17:38:50 +00:00
parent 58d1404684
commit 9647ab8976
2 changed files with 24 additions and 2 deletions

View File

@ -115,6 +115,7 @@ public final class ParagraphSprmUncompressor
} }
break; break;
case 0x3: case 0x3:
// Physical justification of the paragraph
newPAP.setJc ((byte) sprm.getOperand()); newPAP.setJc ((byte) sprm.getOperand());
break; break;
case 0x4: case 0x4:
@ -384,6 +385,9 @@ public final class ParagraphSprmUncompressor
case 0x4c: case 0x4c:
newPAP.setFTtpEmbedded((byte)sprm.getOperand()); newPAP.setFTtpEmbedded((byte)sprm.getOperand());
break; break;
case 0x61:
// Logicial justification of the paragraph, eg left, centre, right
break;
default: default:
break; break;
} }

View File

@ -29,8 +29,6 @@ import junit.framework.TestCase;
* Tests to ensure that our ranges end up with * Tests to ensure that our ranges end up with
* the right text in them, and the right font/styling * the right text in them, and the right font/styling
* properties applied to them. * properties applied to them.
*
* TODO - re-enable me when unicode paragraph stuff is fixed!
*/ */
public final class TestRangeProperties extends TestCase { public final class TestRangeProperties extends TestCase {
private static final char page_break = (char)12; private static final char page_break = (char)12;
@ -146,6 +144,16 @@ public final class TestRangeProperties extends TestCase {
assertEquals("Arial Black", c7.getFontName()); assertEquals("Arial Black", c7.getFontName());
assertEquals(22, c1.getFontSize()); assertEquals(22, c1.getFontSize());
assertEquals(32, c7.getFontSize()); assertEquals(32, c7.getFontSize());
// This document has 15 styles
assertEquals(15, a.getStyleSheet().numStyles());
// Ensure none of the paragraphs refer to one that isn't there,
// and none of their character runs either
for(int i=0; i<a.getRange().numParagraphs(); i++) {
Paragraph p = a.getRange().getParagraph(i);
assertTrue(p.getStyleIndex() < 15);
}
} }
/** /**
@ -360,5 +368,15 @@ public final class TestRangeProperties extends TestCase {
p1_parts[6].length() + 1, p1_parts[6].length() + 1,
c7b.getEndOffset() c7b.getEndOffset()
); );
// This document has 15 styles
assertEquals(15, a.getStyleSheet().numStyles());
// Ensure none of the paragraphs refer to one that isn't there,
// and none of their character runs either
for(int i=0; i<a.getRange().numParagraphs(); i++) {
Paragraph p = a.getRange().getParagraph(i);
assertTrue(p.getStyleIndex() < 15);
}
} }
} }