From: Nick Burch Date: Sat, 12 Jul 2008 16:56:55 +0000 (+0000) Subject: Support for cloning one font record onto another, plus tests X-Git-Tag: REL_3_2_FINAL~246 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=0027e4bb0f6e3f3d9255c8e398c876020ce95c60;p=poi.git Support for cloning one font record onto another, plus tests git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@676201 13f79535-47bb-0310-9956-ffa450edef68 --- diff --git a/src/java/org/apache/poi/hssf/record/FontRecord.java b/src/java/org/apache/poi/hssf/record/FontRecord.java index 42e058f472..d6a5ce859f 100644 --- a/src/java/org/apache/poi/hssf/record/FontRecord.java +++ b/src/java/org/apache/poi/hssf/record/FontRecord.java @@ -531,6 +531,8 @@ public class FontRecord public int getRecordSize() { + // Note - no matter the original, we always + // re-serialise the font name as unicode return (getFontNameLength() * 2) + 20; } @@ -538,6 +540,25 @@ public class FontRecord { return sid; } + + /** + * Clones all the font style information from another + * FontRecord, onto this one. This + * will then hold all the same font style options. + */ + public void cloneStyleFrom(FontRecord source) { + field_1_font_height = source.field_1_font_height; + field_2_attributes = source.field_2_attributes; + field_3_color_palette_index = source.field_3_color_palette_index; + field_4_bold_weight = source.field_4_bold_weight; + field_5_super_sub_script = source.field_5_super_sub_script; + field_6_underline = source.field_6_underline; + field_7_family = source.field_7_family; + field_8_charset = source.field_8_charset; + field_9_zero = source.field_9_zero; + field_10_font_name_len = source.field_10_font_name_len; + field_11_font_name = source.field_11_font_name; + } public int hashCode() { final int prime = 31; diff --git a/src/testcases/org/apache/poi/hssf/record/AllRecordTests.java b/src/testcases/org/apache/poi/hssf/record/AllRecordTests.java index 988be1dac5..e8d1cdf443 100755 --- a/src/testcases/org/apache/poi/hssf/record/AllRecordTests.java +++ b/src/testcases/org/apache/poi/hssf/record/AllRecordTests.java @@ -62,6 +62,7 @@ public final class AllRecordTests { result.addTestSuite(TestEndSubRecord.class); result.addTestSuite(TestEscherAggregate.class); result.addTestSuite(TestExternalNameRecord.class); + result.addTestSuite(TestFontRecord.class); result.addTestSuite(TestFontBasisRecord.class); result.addTestSuite(TestFontIndexRecord.class); result.addTestSuite(TestFormulaRecord.class); diff --git a/src/testcases/org/apache/poi/hssf/record/TestFontRecord.java b/src/testcases/org/apache/poi/hssf/record/TestFontRecord.java new file mode 100644 index 0000000000..6485f586f9 --- /dev/null +++ b/src/testcases/org/apache/poi/hssf/record/TestFontRecord.java @@ -0,0 +1,124 @@ + +/* ==================================================================== + Licensed to the Apache Software Foundation (ASF) under one or more + contributor license agreements. See the NOTICE file distributed with + this work for additional information regarding copyright ownership. + The ASF licenses this file to You under the Apache License, Version 2.0 + (the "License"); you may not use this file except in compliance with + the License. You may obtain a copy of the License at + + http://www.apache.org/licenses/LICENSE-2.0 + + Unless required by applicable law or agreed to in writing, software + distributed under the License is distributed on an "AS IS" BASIS, + WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + See the License for the specific language governing permissions and + limitations under the License. +==================================================================== */ + +package org.apache.poi.hssf.record; + + +import junit.framework.TestCase; + +/** + * Tests the serialization and deserialization of the FontRecord + * class works correctly. Test data taken directly from a real + * Excel file. + */ +public class TestFontRecord + extends TestCase +{ + byte[] header = new byte[] { + 0x31, 00, 0x1a, 00, // sid=31, 26 bytes long + }; + byte[] data = new byte[] { + 0xC8-256, 00, // font height = xc8 + 00, 00, // attrs = 0 + 0xFF-256, 0x7F, // colour palette = x7fff + 0x90-256, 0x01, // bold weight = x190 + 00, 00, // supersubscript + 00, 00, // underline, family + 00, 00, // charset, padding + 05, 01, // name length, unicode flag + 0x41, 0x00, 0x72, 0x00, 0x69, // Arial, as unicode + 0x00, 0x61, 0x00, 0x6C, 0x00 + }; + + public TestFontRecord(String name) + { + super(name); + } + + public void testLoad() + throws Exception + { + + FontRecord record = new FontRecord(new TestcaseRecordInputStream((short)0x31, (short)data.length, data)); + assertEquals( 0xc8, record.getFontHeight()); + assertEquals( 0x00, record.getAttributes()); + assertFalse( record.isItalic()); + assertFalse( record.isStruckout()); + assertFalse( record.isMacoutlined()); + assertFalse( record.isMacshadowed()); + assertEquals( 0x7fff, record.getColorPaletteIndex()); + assertEquals( 0x190, record.getBoldWeight()); + assertEquals( 0x00, record.getSuperSubScript()); + assertEquals( 0x00, record.getUnderline()); + assertEquals( 0x00, record.getFamily()); + assertEquals( 0x00, record.getCharset()); + assertEquals( 0x05, record.getFontNameLength()); + assertEquals( "Arial", record.getFontName()); + + + assertEquals( 26 + 4, record.getRecordSize() ); + record.validateSid((short)0x31); + } + + public void testStore() + { +// .fontheight = c8 +// .attributes = 0 +// .italic = false +// .strikout = false +// .macoutlined= false +// .macshadowed= false +// .colorpalette = 7fff +// .boldweight = 190 +// .supersubscript = 0 +// .underline = 0 +// .family = 0 +// .charset = 0 +// .namelength = 5 +// .fontname = Arial + + FontRecord record = new FontRecord(); + record.setFontHeight((short)0xc8); + record.setAttributes((short)0); + record.setColorPaletteIndex((short)0x7fff); + record.setBoldWeight((short)0x190); + record.setSuperSubScript((short)0); + record.setUnderline((byte)0); + record.setFamily((byte)0); + record.setCharset((byte)0); + record.setFontNameLength((byte)5); + record.setFontName("Arial"); + + byte [] recordBytes = record.serialize(); + assertEquals(recordBytes.length - 4, data.length); + for (int i = 0; i < data.length; i++) + assertEquals("At offset " + i, data[i], recordBytes[i+4]); + } + + public void testCloneOnto() throws Exception { + FontRecord base = new FontRecord(new TestcaseRecordInputStream((short)0x31, (short)data.length, data)); + + FontRecord other = new FontRecord(); + other.cloneStyleFrom(base); + + byte [] recordBytes = other.serialize(); + assertEquals(recordBytes.length - 4, data.length); + for (int i = 0; i < data.length; i++) + assertEquals("At offset " + i, data[i], recordBytes[i+4]); + } +}