continue;
}
- // Process
- switch(type) {
- // Types 0->7 = a flat at bit 0->7
- case 0: case 1: case 2: case 3: case 4: case 5: case 6: case 7:
- int val = contents[offset] & (1<<type);
- command.value = Boolean.valueOf(val > 0);
- break;
- case 8:
- command.value = Byte.valueOf(contents[offset]);
- break;
- case 9:
- command.value = new Double(
- LittleEndian.getDouble(contents, offset)
- );
- break;
- case 12:
- // A Little Endian String
- // Starts 8 bytes into the data segment
- // Ends at end of data, or 00 00
-
- // Ensure we have enough data
- if(contents.length < 8) {
- command.value = "";
+ try {
+ // Process
+ switch(type) {
+ // Types 0->7 = a flat at bit 0->7
+ case 0: case 1: case 2: case 3: case 4: case 5: case 6: case 7:
+ int val = contents[offset] & (1<<type);
+ command.value = Boolean.valueOf(val > 0);
break;
- }
-
- // Find the end point
- int startsAt = 8;
- int endsAt = startsAt;
- for(int j=startsAt; j<contents.length-1 && endsAt == startsAt; j++) {
- if(contents[j] == 0 && contents[j+1] == 0) {
- endsAt = j;
+ case 8:
+ command.value = Byte.valueOf(contents[offset]);
+ break;
+ case 9:
+ command.value = new Double(
+ LittleEndian.getDouble(contents, offset)
+ );
+ break;
+ case 12:
+ // A Little Endian String
+ // Starts 8 bytes into the data segment
+ // Ends at end of data, or 00 00
+
+ // Ensure we have enough data
+ if(contents.length < 8) {
+ command.value = "";
+ break;
}
- }
- if(endsAt == startsAt) {
- endsAt = contents.length;
- }
-
- int strLen = (endsAt-startsAt) / 2;
- command.value = StringUtil.getFromUnicodeLE(contents, startsAt, strLen);
- break;
- case 25:
- command.value = Short.valueOf(
- LittleEndian.getShort(contents, offset)
- );
- break;
- case 26:
- command.value = Integer.valueOf(
- LittleEndian.getInt(contents, offset)
- );
- break;
- // Types 11 and 21 hold the offset to the blocks
- case 11: case 21:
- if(offset < contents.length - 3) {
- int bOffset = (int)LittleEndian.getUInt(contents, offset);
- BlockOffsetCommand bcmd = (BlockOffsetCommand)command;
- bcmd.setOffset(bOffset);
- }
- break;
+ // Find the end point
+ int startsAt = 8;
+ int endsAt = startsAt;
+ for(int j=startsAt; j<contents.length-1 && endsAt == startsAt; j++) {
+ if(contents[j] == 0 && contents[j+1] == 0) {
+ endsAt = j;
+ }
+ }
+ if(endsAt == startsAt) {
+ endsAt = contents.length;
+ }
- default:
- logger.log(POILogger.INFO,
- "Command of type " + type + " not processed!");
+ int strLen = endsAt - startsAt;
+ command.value = new String(contents, startsAt, strLen, header.getChunkCharset().name());
+ break;
+ case 25:
+ command.value = Short.valueOf(
+ LittleEndian.getShort(contents, offset)
+ );
+ break;
+ case 26:
+ command.value = Integer.valueOf(
+ LittleEndian.getInt(contents, offset)
+ );
+ break;
+
+ // Types 11 and 21 hold the offset to the blocks
+ case 11: case 21:
+ if(offset < contents.length - 3) {
+ int bOffset = (int)LittleEndian.getUInt(contents, offset);
+ BlockOffsetCommand bcmd = (BlockOffsetCommand)command;
+ bcmd.setOffset(bOffset);
+ }
+ break;
+
+ default:
+ logger.log(POILogger.INFO,
+ "Command of type " + type + " not processed!");
+ }
+ }
+ catch (Exception e) {
+ logger.log(POILogger.ERROR, "Unexpected error processing command, ignoring and continuing. Command: " +
+ command, e);
}
// Add to the array
int pos = 0;
byte[] contents = getStore().getContents();
- while(pos < contents.length) {
- // Ensure we have enough data to create a chunk from
- int headerSize = ChunkHeader.getHeaderSize(chunkFactory.getVersion());
- if(pos+headerSize <= contents.length) {
- Chunk chunk = chunkFactory.createChunk(contents, pos);
- chunksA.add(chunk);
+ try {
+ while(pos < contents.length) {
+ // Ensure we have enough data to create a chunk from
+ int headerSize = ChunkHeader.getHeaderSize(chunkFactory.getVersion());
+ if(pos+headerSize <= contents.length) {
+ Chunk chunk = chunkFactory.createChunk(contents, pos);
+ chunksA.add(chunk);
- pos += chunk.getOnDiskSize();
- } else {
- System.err.println("Needed " + headerSize + " bytes to create the next chunk header, but only found " + (contents.length-pos) + " bytes, ignoring rest of data");
- pos = contents.length;
+ pos += chunk.getOnDiskSize();
+ } else {
+ System.err.println("Needed " + headerSize + " bytes to create the next chunk header, but only found " + (contents.length-pos) + " bytes, ignoring rest of data");
+ pos = contents.length;
+ }
}
}
+ catch (Exception e)
+ {
+ System.err.println("Failed to create chunk at " + pos + ", ignoring rest of data." + e);
+ }
chunks = chunksA.toArray(new Chunk[chunksA.size()]);
}
package org.apache.poi.hdgf;
+import org.apache.poi.hdgf.extractor.VisioTextExtractor;
import org.apache.poi.hdgf.streams.PointerContainingStream;
import org.apache.poi.hdgf.streams.TrailerStream;
import org.apache.poi.poifs.filesystem.POIFSFileSystem;
HDGFDiagram hdgf = new HDGFDiagram(fs);
assertNotNull(hdgf);
}
+
+ public void testV6NonUtf16LE() throws Exception {
+ fs = new POIFSFileSystem(_dgTests.openResourceAsStream("v6-non-utf16le.vsd"));
+
+ HDGFDiagram hdgf = new HDGFDiagram(fs);
+ assertNotNull(hdgf);
+
+ VisioTextExtractor textExtractor = new VisioTextExtractor(hdgf);
+ String text = textExtractor.getText().replace("\u0000", "").trim();
+
+ assertEquals("Table\n\n\nPropertySheet\n\n\n\nPropertySheetField", text);
+ }
+
+ public void testUtf16LE() throws Exception {
+ fs = new POIFSFileSystem(_dgTests.openResourceAsStream("Test_Visio-Some_Random_Text.vsd"));
+
+ HDGFDiagram hdgf = new HDGFDiagram(fs);
+ assertNotNull(hdgf);
+
+ VisioTextExtractor textExtractor = new VisioTextExtractor(hdgf);
+ String text = textExtractor.getText().trim();
+
+ assertEquals("text\nView\nTest View\nI am a test view\nSome random text, on a page", text);
+ }
}