Mark some more scratchpad document types as currently being read-only

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1753618 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Nick Burch 2016-07-20 22:18:25 +00:00
parent 9830fd3dca
commit 1ccced289d
3 changed files with 14 additions and 18 deletions

View File

@ -44,14 +44,23 @@ public abstract class POIReadOnlyDocument extends POIDocument {
super(fs); super(fs);
} }
/**
* Note - writing is not yet supported for this file format, sorry.
*/
// @Override // @Override
// public void write() throws IOException { // public void write() throws IOException {
// throw new IllegalStateException("Writing is not yet implemented for this Document Format"); // throw new IllegalStateException("Writing is not yet implemented for this Document Format");
// } // }
/**
* Note - writing is not yet supported for this file format, sorry.
*/
// @Override // @Override
// public void write(File file) throws IOException { // public void write(File file) throws IOException {
// throw new IllegalStateException("Writing is not yet implemented for this Document Format"); // throw new IllegalStateException("Writing is not yet implemented for this Document Format");
// } // }
/**
* Note - writing is not yet supported for this file format, sorry.
*/
@Override @Override
public void write(OutputStream out) throws IOException { public void write(OutputStream out) throws IOException {
throw new IllegalStateException("Writing is not yet implemented for this Document Format"); throw new IllegalStateException("Writing is not yet implemented for this Document Format");

View File

@ -20,9 +20,8 @@ package org.apache.poi.hdgf;
import java.io.File; import java.io.File;
import java.io.IOException; import java.io.IOException;
import java.io.InputStream; import java.io.InputStream;
import java.io.OutputStream;
import org.apache.poi.POIDocument; import org.apache.poi.POIReadOnlyDocument;
import org.apache.poi.hdgf.chunks.ChunkFactory; import org.apache.poi.hdgf.chunks.ChunkFactory;
import org.apache.poi.hdgf.pointers.Pointer; import org.apache.poi.hdgf.pointers.Pointer;
import org.apache.poi.hdgf.pointers.PointerFactory; import org.apache.poi.hdgf.pointers.PointerFactory;
@ -44,7 +43,7 @@ import org.apache.poi.util.LocaleUtil;
* http://www.gnome.ru/projects/docs/slide1.png * http://www.gnome.ru/projects/docs/slide1.png
* http://www.gnome.ru/projects/docs/slide2.png * http://www.gnome.ru/projects/docs/slide2.png
*/ */
public final class HDGFDiagram extends POIDocument { public final class HDGFDiagram extends POIReadOnlyDocument {
private static final String VISIO_HEADER = "Visio (TM) Drawing\r\n"; private static final String VISIO_HEADER = "Visio (TM) Drawing\r\n";
private byte[] _docstream; private byte[] _docstream;
@ -156,10 +155,6 @@ public final class HDGFDiagram extends POIDocument {
} }
} }
public void write(OutputStream out) {
throw new IllegalStateException("Writing is not yet implemented, see http://poi.apache.org/hdgf/");
}
/** /**
* For testing only * For testing only
*/ */
@ -167,6 +162,7 @@ public final class HDGFDiagram extends POIDocument {
NPOIFSFileSystem pfs = new NPOIFSFileSystem(new File(args[0])); NPOIFSFileSystem pfs = new NPOIFSFileSystem(new File(args[0]));
HDGFDiagram hdgf = new HDGFDiagram(pfs); HDGFDiagram hdgf = new HDGFDiagram(pfs);
hdgf.debug(); hdgf.debug();
hdgf.close();
pfs.close(); pfs.close();
} }
} }

View File

@ -20,7 +20,6 @@ package org.apache.poi.hsmf;
import java.io.File; import java.io.File;
import java.io.IOException; import java.io.IOException;
import java.io.InputStream; import java.io.InputStream;
import java.io.OutputStream;
import java.io.UnsupportedEncodingException; import java.io.UnsupportedEncodingException;
import java.util.ArrayList; import java.util.ArrayList;
import java.util.Arrays; import java.util.Arrays;
@ -29,7 +28,7 @@ import java.util.List;
import java.util.regex.Matcher; import java.util.regex.Matcher;
import java.util.regex.Pattern; import java.util.regex.Pattern;
import org.apache.poi.POIDocument; import org.apache.poi.POIReadOnlyDocument;
import org.apache.poi.hmef.attribute.MAPIRtfAttribute; import org.apache.poi.hmef.attribute.MAPIRtfAttribute;
import org.apache.poi.hsmf.datatypes.AttachmentChunks; import org.apache.poi.hsmf.datatypes.AttachmentChunks;
import org.apache.poi.hsmf.datatypes.AttachmentChunks.AttachmentChunksSorter; import org.apache.poi.hsmf.datatypes.AttachmentChunks.AttachmentChunksSorter;
@ -63,7 +62,7 @@ import org.apache.poi.util.POILogger;
* *
* [MS-OXCMSG]: Message and Attachment Object Protocol Specification * [MS-OXCMSG]: Message and Attachment Object Protocol Specification
*/ */
public class MAPIMessage extends POIDocument { public class MAPIMessage extends POIReadOnlyDocument {
/** For logging problems we spot with the file */ /** For logging problems we spot with the file */
private POILogger logger = POILogFactory.getLogger(MAPIMessage.class); private POILogger logger = POILogFactory.getLogger(MAPIMessage.class);
@ -603,14 +602,6 @@ public class MAPIMessage extends POIDocument {
} }
/**
* Note - not yet supported, sorry.
*/
public void write(OutputStream out) throws IOException {
throw new UnsupportedOperationException("Writing isn't yet supported for HSMF, sorry");
}
/** /**
* Will you get a null on a missing chunk, or a * Will you get a null on a missing chunk, or a
* {@link ChunkNotFoundException} (default is the * {@link ChunkNotFoundException} (default is the