[Python-bugs-list] [ python-Bugs-467924 ] Improve the ZipFile Interface
SourceForge.net
noreply@sourceforge.net
Sun, 05 Jan 2003 12:54:13 -0800
Bugs item #467924, was opened at 2001-10-04 17:54
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=105470&aid=467924&group_id=5470
Category: Python Library
Group: Feature Request
Status: Open
Resolution: None
Priority: 5
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Nobody/Anonymous (nobody)
Summary: Improve the ZipFile Interface
Initial Comment:
There exist two methods to write to a ZipFile
write(self, filename, arcname=None, compress_type=None)
writestr(self, zinfo, bytes)
but only one to read from it
read(self, name)
Additionally, the two 'write's behave differently with respect to compression.
---
(a) 'read' does not fit to 'write', since 'write' takes a file and adds it to a ZipFile,
but 'read' is not the reverse operation. 'read' should be called 'readstr' since it
much better matches to 'writestr'.
(b) It is confusing what 'write' and 'read' actually mean. Does 'write' write a file,
or into the ZipFile? It would be more obvious if ZipFile has 4 methods which
pair-wise fit together:
writestr (self, zinfo, bytes)
# same as now
readstr (self, name)
# returns bytes (as string), currently called 'read'
# 'read' could still live but should be deprecated
add (self, filename, arcname=None, compress_type=None)
# currently 'write'
# 'write' could still live but should be deprecated
extract (self, name, filename, arcname=None)
# new, desired functionality
(c) BOTH, 'writestr' and 'add' should by default use the 'compress_type' that was
passed to the constructor of 'ZipFile'. Currently, 'write' does it, 'writestr' via
zinfo does it not. 'ZipInfo' sets the compression strict to 'ZIP_STORED' :-(
It should not do that! It rather should:
- allow more parameters in the signature of the constructor
to also pass the compression type (and some other attributes, too)
- default to 'None', so that 'writestr' can see this, and then take
the default from the 'ZipFile' instance.
----------------------------------------------------------------------
>Comment By: Just van Rossum (jvr)
Date: 2003-01-05 21:54
Message:
Logged In: YES
user_id=92689
In Python 2.3, writestr() has an enhanced signature: the
first arg may now also be an archive name, in which case the
correct default settings are used (ie. the compression value
is taken from the file). See patch #651621.
extract() could be moderately useful (although I don't
understand the 'arcname' arg, how's that different from
'name'?) but would have to deal with file modes (bin/text).
The file mode isn't in the archive so would have to
(optionally) be supplied by the caller.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=105470&aid=467924&group_id=5470