Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Idea: allow traversing with pathlib semantics #1200

Open
gforcada opened this issue Mar 13, 2024 · 7 comments
Open

Idea: allow traversing with pathlib semantics #1200

gforcada opened this issue Mar 13, 2024 · 7 comments

Comments

@gforcada
Copy link
Contributor

In Zope/Plone we use traversal for almost everything:

root = somehow_get_root()
content = root.restrictedTraverse('and/now/get/some/content')

Wouldn't it be cool to be able to do instead:

root = somehow_get_root()
content = root / 'and' / 'now' / 'get' / 'some' / 'content'

@davisagli
Copy link
Member

How is that better than root['and']['now']['get']['some']['content']? It's for sure less obvious.

@ale-rt
Copy link
Member

ale-rt commented Mar 13, 2024

Cool :)

BTW it will be more convenient to do it like this:

content = root / "and/now/get/some/content"

See what we have in pathlib:

>>> from pathlib import Path
>>> Path("foo") / "bar/baz"
PosixPath('foo/bar/baz')

However, using restrictedTraverse and unrestrictedTraverse makes it clear the security implications.

Will the / operator traverse the object in a restricted or unrestricted way?

Theoretically, you could do:

content_path = root / "and/now/get/some/content"
content_path.restricted_get()
content_path.unrestricted_get()

With this it already starts to be less cool :D

@gforcada
Copy link
Contributor Author

@davisagli less obvious is you already know that you can do dictionary-like traversing, but with all the rage of using pathlib everywhere (maybe I'm biased here 😓 ) and that Zope/Plone always use the metaphor of storing objects like folders and files, well, it makes it easier for onboarding IMHO

@ale-rt good point on the (un)restricted that's not obvious at first sight... but maybe it could be a method of the object itself, i.e.:

obj_path = root / 'my/deep/nested/obj'
obj = obj_path.resolve()
# vs
obj = obj_path.unrestricted_resolve()

So one thing could be constructing the path to the object, and once you have that you need to decide how you want to use it.

Overriding the resolve method from upstream Python.

If I keep dreaming about this, Pathlib already has the idea of pure paths (objects with everything but I/O operations) and specific paths that have the I/O operations.

So, it would be a """"""matter"""""" of creating one such specific path meant for traversing a ZODB storage ✨

I don't have any specific need to implement it myself, but I thought that as an idea it would be worth sharing it here 😊

@d-maurer
Copy link
Contributor

d-maurer commented Mar 14, 2024 via email

@davisagli
Copy link
Member

davisagli commented Mar 14, 2024

I like @d-maurer's idea best of the ones listed here, but I'm not sure that I would actually use it much. It's already pretty convenient to construct string paths using f-strings.

Different topic, but something that I would use is an easier way to get the string path of a Zope object, instead of doing "/".join(obj.getPhysicalPath())

@d-maurer
Copy link
Contributor

d-maurer commented Mar 14, 2024 via email

@davisagli
Copy link
Member

@d-maurer Yep, absolute_url_path looks like what I want. I either wasn't aware of it or forgot. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants