Repository navigation
Add suffix property to zipfile.Path #88261
Description
| BPO | 44095 |
|---|---|
| Nosy | @jaraco, @miguendes, @MrQubo |
| PRs |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
assignee = None
closed_at = <Date 2021-05-14.18:00:17.721>
created_at = <Date 2021-05-10.05:29:23.213>
labels = ['type-feature', 'library', '3.11']
title = 'Add suffix property to zipfile.Path'
updated_at = <Date 2021-05-14.18:00:17.720>
user = 'https://github.com/MrQubo'bugs.python.org fields:
activity = <Date 2021-05-14.18:00:17.720>
actor = 'jaraco'
assignee = 'none'
closed = True
closed_date = <Date 2021-05-14.18:00:17.721>
closer = 'jaraco'
components = ['Library (Lib)']
creation = <Date 2021-05-10.05:29:23.213>
creator = 'MrQubo'
dependencies = []
files = []
hgrepos = []
issue_num = 44095
keywords = ['patch']
message_count = 6.0
messages = ['393352', '393637', '393650', '393671', '393682', '393683']
nosy_count = 3.0
nosy_names = ['jaraco', 'miguendes', 'MrQubo']
pr_nums = ['26129']
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'enhancement'
url = 'https://bugs.python.org/issue44095'
versions = ['Python 3.11']
Activity
MrQubo commentedon May 10, 2021
suffix property is present on pathlib.Path (specifically pathlib.PurePath) and it could also be added to zipfile.Path for consistency.
My use case is filtering files by suffix:
patch_files = list(filter(lambda file: file.suffix == '.patch', files))
Besides suffix also most of the other pathlib.PurePath properties and methods could be added.
miguendes commentedon May 14, 2021
+1.
The docstring says:
but only a few methods are supported. It'd be nice to have at least
stem,parents,suffixes,parts, which IMHO would make sense here.I'd love to work on this if the Core Devs think it makes sense and if OP is not doing so also.
MrQubo commentedon May 14, 2021
Thank for your interest Miguel. I have no experience with CPython codebase so I'll be glad if you work on this.
I think treating ZipFile's like a separate drive would be the best. If we agree on this then probably all of the properties and methods from pathlib could be implemented. But without this at least
suffix,suffixes,stem,with_name(),with_stem(), andwith_suffix()makes perfect sense.If some of the properties/methods won't be implemented on zipfile.Path then it would be useful to raise
NotImplementedError. Currently, trying to use suffix property, raises the errorAttributeError: 'Path' object has no attribute 'suffix'. This is confusing as the name of the class is the same as inpathlib.Pathso it's not immediately obvious that it meantzipfile.Path.The use-case for mimicking those two interfaces closely is to use both interchangeably (thanks to duck-typing).
miguendes commentedon May 14, 2021
I prototyped something by adding
suffix,suffixes, andstem. To me these are the most obvious ones and would be useful right away.Regarding the others I'm not so sure. It seems that zipfile.Path is just a convenient class to provide basic navigational operations. It's not meant to be 100% consistent with pathlib.Path.
In any case, it'd nice to discuss it.
Here it is: #26129
jaraco commentedon May 14, 2021
Sounds good to me. In general, it's easier to contribute first to https://github.com/jaraco/zipp and those changes can be readily merged into CPython. No worries, though. If these changes can be accepted here, I should be able to graft them onto jaraco/zipp as well, and thus provide the functionality for older Pythons.
At this point, Python 3.10 is frozen for features, which this change would be, so I'm updating the target version to 3.11.
That's right. It's meant to provide whatever consistency makes sense, but a subset of what's available on pathlib.Path. Happy to accept any contributions that would be useful.
Indeed, perhaps the documentation should be more clear about what is and isn't supported.
jaraco commentedon May 14, 2021
Thanks for the excellent patch. I've created jaraco/zipp#74 to track the backport of these changes.