BUGFIX XMLDocument upload tab
Hi there, zope-dev people and Amos, There was a small bug in XML Document that prevented file uploads from working. I think I fixed it. :) Diff at the bottom (do I do that right? I used diff -c, but I'm new to diffs). Regards, Martijn *** XMLDocument.py Mon Jul 26 13:07:11 1999 --- XMLDocument.py~ Thu Jul 22 19:09:11 1999 *************** *** 184,193 **** def manage_upload(self,file='', REQUEST=None): """Replace the contents of the document with the text in file.""" ! if type(file) is not type(''): file=file.read() ! ! try: ! self.parse(file) except ParseError, message: if REQUEST: return MessageDialog( title ='XML Parsing Error', --- 184,191 ---- def manage_upload(self,file='', REQUEST=None): """Replace the contents of the document with the text in file.""" ! try: ! self.parse(data) except ParseError, message: if REQUEST: return MessageDialog( title ='XML Parsing Error',
At 01:14 PM 7/26/99 +0200, Martijn Faassen wrote:
Hi there, zope-dev people and Amos,
There was a small bug in XML Document that prevented file uploads from working. I think I fixed it. :)
Thanks, I will check in this fix. BTW, XML Document underwent a major overhaul this weekend to bring it inline with the new ZDOM. -Amos
Amos Latteier wrote:
At 01:14 PM 7/26/99 +0200, Martijn Faassen wrote:
Hi there, zope-dev people and Amos,
There was a small bug in XML Document that prevented file uploads from working. I think I fixed it. :)
Thanks, I will check in this fix. BTW, XML Document underwent a major overhaul this weekend to bring it inline with the new ZDOM.
Is this in CVS somewhere? I've been looking but was unable to find it so far. Regards, Martijn
At 10:54 AM 7/27/99 +0200, Martijn Faassen wrote:
Thanks, I will check in this fix. BTW, XML Document underwent a major overhaul this weekend to bring it inline with the new ZDOM.
Is this in CVS somewhere? I've been looking but was unable to find it so far.
It is in CVS, but I don't think that it is visible to the public. I'll talk to Ken about changing this. Otherwise I'll probably put out another release with the next beta release of Zope. -Amos
participants (2)
-
Amos Latteier -
Martijn Faassen