Setting content (Java parser)

Question

Hi, I'm using a Java parser to take in binary files.
In windows explorer if I drag and drop on to an ifs
it works fine, however if the file already exists
then it is replaced by a 0 byte file of the same name.
Heres the relevant parser code:
public LibraryObject parse(InputStream inStream, ParserCallback callback, Hashtable options)
dis = new DataInputStream(inStream);
baos = new ByteArrayOutputStream();
int b = -99;
byte[] buffer = new byte[1024];
try
while((b = dis.read(buffer)) != -1)
if(b < buffer.length)
baos.write(buffer, 0, b);
else
baos.write(buffer);
catch(EOFException e)
DocumentDefinition docDef = new DocumentDefinition (m_librarySession);
Collection allFormats = m_librarySession.getFormatExtensionCollection();
docDef.setFormat((Format) allFormats.getItems("bin"));
docDef.setClassname("BINARYFILE");
docDef.setAttribute("NAME", AttributeValue.newAttributeValue(options.get(CURRENT_NAME_OPTION)));
byte[] outbytes = baos.toByteArray();
bais = new ByteArrayInputStream(outbytes);
docDef.setContentStream(bais);
Document newDoc = (Document)
m_librarySession.createPublicObject(docDef);
if(options.get(UPDATE_OBJECT_OPTION) != null)
Document currentDoc = (Document) currentFolder.findPublicObjectByPath
(docDef.getAttribute("NAME").toString());
currentDoc.update(docDef);
else
currentFolder.addItem(newDoc);
baos.close();
//bais.close();
//dis.close();
As I said this only causes trouble if the file is already present in the ifs directory I'm placing it in.
Thanks for your time,
Cathal

Answer 1

try moving this line:
m_librarySession.createPublicObject(docDef);
into the "else" block of your condition:
else
// move the line here
currentFolder.addItem(newDoc);
the mistake you're making is that you always create a new document, whether or not you want to update an existing document or not.
this mistake causes two problems:
1. when you're updating (i.e. the "if" block of your condition will execute) you leave the new document you created floating around.
2. when you're updating, the existing document content gets set to 0 bytes. why? because you already sucked out all the bytes from your bais when you created the new document earlier! when you try to update the existing one later, the bais is already "at the end" and there's nothing left to read out. for more info about this you should read up on the java.io package.