<action dev="POI-DEVELOPERS" type="add">Created a common interface for handling PowerPoint files, irrespective of if they are .ppt or .pptx</action>
<action dev="POI-DEVELOPERS" type="add">Created a common interface for handling Excel files, irrespective of if they are .xls or .xlsx</action>
</release>
- <release version="3.1.1-alpha1" date="2008-??-??">
+ <release version="3.2-alpha1" date="2008-??-??">
+ <action dev="POI-DEVELOPERS" type="fix">45804 - Update HSMF to handle Outlook 3.0 msg files, which have a different string chunk type</action>
<action dev="POI-DEVELOPERS" type="add">Expose the name of Named Cell Styles via HSSFCellStyle (normally held on the parent style though)</action>
<action dev="POI-DEVELOPERS" type="fix">45978 - Fixed IOOBE in Ref3DPtg.toFormulaString() due eager initialisation of SheetReferences</action>
<action dev="POI-DEVELOPERS" type="add">Made HSSFFormulaEvaluator no longer require initialisation with sheet or row</action>
<action dev="POI-DEVELOPERS" type="add">Created a common interface for handling PowerPoint files, irrespective of if they are .ppt or .pptx</action>
<action dev="POI-DEVELOPERS" type="add">Created a common interface for handling Excel files, irrespective of if they are .xls or .xlsx</action>
</release>
- <release version="3.1.1-alpha1" date="2008-??-??">
+ <release version="3.2-alpha1" date="2008-??-??">
+ <action dev="POI-DEVELOPERS" type="fix">45804 - Update HSMF to handle Outlook 3.0 msg files, which have a different string chunk type</action>
<action dev="POI-DEVELOPERS" type="add">Expose the name of Named Cell Styles via HSSFCellStyle (normally held on the parent style though)</action>
<action dev="POI-DEVELOPERS" type="fix">45978 - Fixed IOOBE in Ref3DPtg.toFormulaString() due eager initialisation of SheetReferences</action>
<action dev="POI-DEVELOPERS" type="add">Made HSSFFormulaEvaluator no longer require initialisation with sheet or row</action>
public class MAPIMessage {
private POIFSChunkParser chunkParser;
private POIFSFileSystem fs;
+ private Chunks chunks;
/**
* Constructor for creating new files.
public MAPIMessage(InputStream in) throws IOException {
this.fs = new POIFSFileSystem(in);
chunkParser = new POIFSChunkParser(this.fs);
+
+ // Figure out the right string type, based on
+ // the chunks present
+ chunks = chunkParser.identifyChunks();
}
* @throws ChunkNotFoundException
*/
public String getTextBody() throws IOException, ChunkNotFoundException {
- return getStringFromChunk(Chunks.getInstance().textBodyChunk);
+ return getStringFromChunk(chunks.textBodyChunk);
}
/**
* @throws ChunkNotFoundException
*/
public String getSubject() throws ChunkNotFoundException {
- return getStringFromChunk(Chunks.getInstance().subjectChunk);
+ return getStringFromChunk(chunks.subjectChunk);
}
* @throws ChunkNotFoundException
*/
public String getDisplayTo() throws ChunkNotFoundException {
- return getStringFromChunk(Chunks.getInstance().displayToChunk);
+ return getStringFromChunk(chunks.displayToChunk);
}
/**
* @throws ChunkNotFoundException
*/
public String getDisplayFrom() throws ChunkNotFoundException {
- return getStringFromChunk(Chunks.getInstance().displayFromChunk);
+ return getStringFromChunk(chunks.displayFromChunk);
}
/**
* @throws ChunkNotFoundException
*/
public String getDisplayCC() throws ChunkNotFoundException {
- return getStringFromChunk(Chunks.getInstance().displayCCChunk);
+ return getStringFromChunk(chunks.displayCCChunk);
}
/**
* @throws ChunkNotFoundException
*/
public String getDisplayBCC() throws ChunkNotFoundException {
- return getStringFromChunk(Chunks.getInstance().displayBCCChunk);
+ return getStringFromChunk(chunks.displayBCCChunk);
}
* @throws ChunkNotFoundException
*/
public String getConversationTopic() throws ChunkNotFoundException {
- return getStringFromChunk(Chunks.getInstance().conversationTopic);
+ return getStringFromChunk(chunks.conversationTopic);
}
/**
* @throws ChunkNotFoundException
*/
public String getMessageClass() throws ChunkNotFoundException {
- return getStringFromChunk(Chunks.getInstance().messageClass);
+ return getStringFromChunk(chunks.messageClass);
}
}
*/
public class Chunks {
/* String parts of Outlook Messages that are currently known */
- public StringChunk messageClass = new StringChunk(0x001A); //Type of message that the MSG represents (ie. IPM.Note)
- public StringChunk textBodyChunk = new StringChunk(0x1000); //BODY Chunk, for plain/text messages
- public StringChunk subjectChunk = new StringChunk(0x0037); //Subject link chunk, in plain/text
- public StringChunk displayToChunk = new StringChunk(0x0E04); //Value that is in the TO field (not actually the addresses as they are stored in recip directory nodes
- public StringChunk displayFromChunk = new StringChunk(0x0C1A); //Value that is in the FROM field
- public StringChunk displayCCChunk = new StringChunk(0x0E03); //value that shows in the CC field
- public StringChunk displayBCCChunk = new StringChunk(0x0E02); //Value that shows in the BCC field
- public StringChunk conversationTopic = new StringChunk(0x0070); //Sort of like the subject line, but without the RE: and FWD: parts.
- public StringChunk sentByServerType = new StringChunk(0x0075); //Type of server that the message originated from (SMTP, etc).
+
+ /** Type of message that the MSG represents (ie. IPM.Note) */
+ public StringChunk messageClass;
+ /** BODY Chunk, for plain/text messages */
+ public StringChunk textBodyChunk;
+ /** Subject link chunk, in plain/text */
+ public StringChunk subjectChunk;
+ /** Value that is in the TO field (not actually the addresses as they are stored in recip directory nodes */
+ public StringChunk displayToChunk;
+ /** Value that is in the FROM field */
+ public StringChunk displayFromChunk;
+ /** value that shows in the CC field */
+ public StringChunk displayCCChunk;
+ /** Value that shows in the BCC field */
+ public StringChunk displayBCCChunk;
+ /** Sort of like the subject line, but without the RE: and FWD: parts. */
+ public StringChunk conversationTopic;
+ /** Type of server that the message originated from (SMTP, etc). */
+ public StringChunk sentByServerType;
+
+ private Chunks(boolean newStringType) {
+ messageClass = new StringChunk(0x001A, newStringType);
+ textBodyChunk = new StringChunk(0x1000, newStringType);
+ subjectChunk = new StringChunk(0x0037, newStringType);
+ displayToChunk = new StringChunk(0x0E04, newStringType);
+ displayFromChunk = new StringChunk(0x0C1A, newStringType);
+ displayCCChunk = new StringChunk(0x0E03, newStringType);
+ displayBCCChunk = new StringChunk(0x0E02, newStringType);
+ conversationTopic = new StringChunk(0x0070, newStringType);
+ sentByServerType = new StringChunk(0x0075, newStringType);
+ }
- public static Chunks getInstance() {
- return new Chunks();
+ public static Chunks getInstance(boolean newStringType) {
+ return new Chunks(newStringType);
}
}
private String value;
- public StringChunk(int chunkId) {
+ /**
+ * Creates a String Chunk, for either the old
+ * or new style of string chunk types.
+ */
+ public StringChunk(int chunkId, boolean newStyleString) {
+ this(chunkId, getStringType(newStyleString));
+ }
+ private static int getStringType(boolean newStyleString) {
+ if(newStyleString)
+ return Types.NEW_STRING;
+ return Types.OLD_STRING;
+ }
+
+ /**
+ * Create a String Chunk, with the specified
+ * type.
+ */
+ public StringChunk(int chunkId, int type) {
this.chunkId = chunkId;
- this.type = Types.STRING;
+ this.type = type;
}
/* (non-Javadoc)
public class Types {
public static int BINARY = 0x0102;
- public static int STRING = 0x001E;
+
+ /** A string, until Outlook 3.0 */
+ public static int OLD_STRING = 0x001E;
+ /** A string, from Outlook 3.0 onwards */
+ public static int NEW_STRING = 0x001F;
+
public static int LONG = 0x0003;
public static int TIME = 0x0040;
public static int BOOLEAN = 0x000B;
+
+ public static String asFileEnding(int type) {
+ String str = Integer.toHexString(type).toUpperCase();
+ while(str.length() < 4) {
+ str = "0" + str;
+ }
+ return str;
+ }
}
import java.util.Iterator;
import org.apache.poi.hsmf.datatypes.Chunk;
+import org.apache.poi.hsmf.datatypes.Chunks;
+import org.apache.poi.hsmf.datatypes.Types;
import org.apache.poi.hsmf.exceptions.ChunkNotFoundException;
import org.apache.poi.hsmf.exceptions.DirectoryChunkNotFoundException;
import org.apache.poi.poifs.filesystem.DirectoryEntry;
this.directoryMap = this.processPOIIterator(iter);
}
-
+
+ /**
+ * Returns a list of the standard chunk types, as
+ * appropriate for the chunks we find in the file.
+ */
+ public Chunks identifyChunks() {
+ // Are they of the old or new type of strings?
+ boolean hasOldStrings = false;
+ boolean hasNewStrings = false;
+ String oldStringEnd = Types.asFileEnding(Types.OLD_STRING);
+ String newStringEnd = Types.asFileEnding(Types.NEW_STRING);
+
+ for(Iterator i = directoryMap.keySet().iterator(); i.hasNext();) {
+ String entry = (String)i.next();
+ if(entry.endsWith( oldStringEnd )) {
+ hasOldStrings = true;
+ }
+ if(entry.endsWith( newStringEnd )) {
+ hasNewStrings = true;
+ }
+ }
+
+ if(hasOldStrings && hasNewStrings) {
+ throw new IllegalStateException("Your file contains string chunks of both the old and new types. Giving up");
+ } else if(hasNewStrings) {
+ return Chunks.getInstance(true);
+ }
+ return Chunks.getInstance(false);
+ }
+
/**
* Pull the chunk data that's stored in this object's hashmap out and return it as a HashMap.
* @param entryName
TestSuite suite = new TestSuite();
suite.addTestSuite(org.apache.poi.hsmf.model.TestBlankFileRead.class);
suite.addTestSuite(org.apache.poi.hsmf.model.TestSimpleFileRead.class);
+ suite.addTestSuite(org.apache.poi.hsmf.model.TestOutlook30FileRead.class);
suite.addTestSuite(org.apache.poi.hsmf.model.TestChunkData.class);
return suite;
*
*/
public class TestChunkData extends TestCase {
+ private Chunks chunks = Chunks.getInstance(false);
+
public void testChunkCreate() {
- StringChunk chunk = new StringChunk(0x0200);
+ StringChunk chunk = new StringChunk(0x0200, false);
TestCase.assertEquals("__substg1.0_0200001E", chunk.getEntryName());
/* test the lower and upper limits of the chunk ids */
- chunk = new StringChunk(0x0000);
+ chunk = new StringChunk(0x0000, false);
TestCase.assertEquals("__substg1.0_0000001E", chunk.getEntryName());
- chunk = new StringChunk(0xFFFF);
+ chunk = new StringChunk(0xFFFF, false);
TestCase.assertEquals("__substg1.0_FFFF001E", chunk.getEntryName());
+
+ chunk = new StringChunk(0xFFFF, true);
+ TestCase.assertEquals("__substg1.0_FFFF001F", chunk.getEntryName());
}
public void testTextBodyChunk() {
- StringChunk chunk = new StringChunk(0x1000);
- TestCase.assertEquals(chunk.getEntryName(), Chunks.getInstance().textBodyChunk.getEntryName());
+ StringChunk chunk = new StringChunk(0x1000, false);
+ TestCase.assertEquals(chunk.getEntryName(), chunks.textBodyChunk.getEntryName());
}
public void testDisplayToChunk() {
- StringChunk chunk = new StringChunk(0x0E04);
- TestCase.assertEquals(chunk.getEntryName(), Chunks.getInstance().displayToChunk.getEntryName());
+ StringChunk chunk = new StringChunk(0x0E04, false);
+ TestCase.assertEquals(chunk.getEntryName(), chunks.displayToChunk.getEntryName());
}
public void testDisplayCCChunk() {
- StringChunk chunk = new StringChunk(0x0E03);
- TestCase.assertEquals(chunk.getEntryName(), Chunks.getInstance().displayCCChunk.getEntryName());
+ StringChunk chunk = new StringChunk(0x0E03, false);
+ TestCase.assertEquals(chunk.getEntryName(), chunks.displayCCChunk.getEntryName());
}
public void testDisplayBCCChunk() {
- StringChunk chunk = new StringChunk(0x0E02);
- TestCase.assertEquals(chunk.getEntryName(), Chunks.getInstance().displayBCCChunk.getEntryName());
+ StringChunk chunk = new StringChunk(0x0E02, false);
+ TestCase.assertEquals(chunk.getEntryName(), chunks.displayBCCChunk.getEntryName());
}
public void testSubjectChunk() {
- Chunk chunk = new StringChunk(0x0037);
- TestCase.assertEquals(chunk.getEntryName(), Chunks.getInstance().subjectChunk.getEntryName());
+ Chunk chunk = new StringChunk(0x0037, false);
+ TestCase.assertEquals(chunk.getEntryName(), chunks.subjectChunk.getEntryName());
}
}
--- /dev/null
+/* ====================================================================
+ 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.hsmf.model;
+
+import java.io.IOException;
+
+import org.apache.poi.hsmf.MAPIMessage;
+import org.apache.poi.hsmf.exceptions.ChunkNotFoundException;
+
+import junit.framework.TestCase;
+
+/**
+ * Tests to verify that we can still work on the newer Outlook 3.0 files.
+ */
+public class TestOutlook30FileRead extends TestCase {
+private MAPIMessage mapiMessage;
+
+ /**
+ * Initialize this test, load up the blank.msg mapi message.
+ * @throws Exception
+ */
+ public TestOutlook30FileRead() throws IOException {
+ String dirname = System.getProperty("HSMF.testdata.path");
+ this.mapiMessage = new MAPIMessage(dirname + "/outlook_30_msg.msg");
+ }
+
+ /**
+ * Test to see if we can read the CC Chunk.
+ * @throws ChunkNotFoundException
+ *
+ */
+ public void testReadDisplayCC() throws ChunkNotFoundException {
+ String obtained = mapiMessage.getDisplayCC();
+ String expected = "";
+
+ TestCase.assertEquals(obtained, expected);
+ }
+
+ /**
+ * Test to see if we can read the CC Chunk.
+ * @throws ChunkNotFoundException
+ *
+ */
+ public void testReadDisplayTo() throws ChunkNotFoundException {
+ String obtained = mapiMessage.getDisplayTo();
+
+ assertTrue(obtained.startsWith("Bohn, Shawn"));
+ }
+
+ /**
+ * Test to see if we can read the From Chunk.
+ * @throws ChunkNotFoundException
+ *
+ */
+ public void testReadDisplayFrom() throws ChunkNotFoundException {
+ String obtained = mapiMessage.getDisplayFrom();
+ String expected = "Cramer, Nick";
+
+ TestCase.assertEquals(obtained, expected);
+ }
+
+ /**
+ * Test to see if we can read the CC Chunk.
+ * @throws ChunkNotFoundException
+ *
+ */
+ public void testReadDisplayBCC() throws ChunkNotFoundException {
+ String obtained = mapiMessage.getDisplayBCC();
+ String expected = "";
+
+ TestCase.assertEquals(obtained, expected);
+ }
+
+
+ /**
+ * Check if we can read the body of the blank message, we expect "".
+ *
+ * @throws Exception
+ */
+ public void testReadBody() throws Exception {
+ String obtained = mapiMessage.getTextBody();
+ assertTrue(obtained.startsWith("I am shutting down"));
+ }
+
+ /**
+ * Check if we can read the subject line of the blank message, we expect ""
+ *
+ * @throws Exception
+ */
+ public void testReadSubject() throws Exception {
+ String obtained = mapiMessage.getSubject();
+ String expected = "IN-SPIRE servers going down for a bit, back up around 8am";
+
+ TestCase.assertEquals(expected, obtained);
+ }
+
+ /**
+ * Check if we can read the subject line of the blank message, we expect ""
+ *
+ * @throws Exception
+ */
+ public void testReadConversationTopic() throws Exception {
+ String obtained = mapiMessage.getConversationTopic();
+ TestCase.assertEquals("IN-SPIRE servers going down for a bit, back up around 8am", obtained);
+ }
+
+
+ /**
+ * Check if we can read the subject line of the blank message, we expect ""
+ *
+ * @throws Exception
+ */
+ public void testReadMessageClass() throws Exception {
+ String obtained = mapiMessage.getMessageClass();
+ TestCase.assertEquals("IPM.Note", obtained);
+ }
+
+
+
+}