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

Undocumented name space / import issues between Zope 2 and 4 #813

Open
jugmac00 opened this issue Apr 1, 2020 · 2 comments
Open

Undocumented name space / import issues between Zope 2 and 4 #813

jugmac00 opened this issue Apr 1, 2020 · 2 comments

Comments

@jugmac00
Copy link
Member

jugmac00 commented Apr 1, 2020

Hi,

with Zope 2 some symbols were readily available for use - even without importing them.

I recall DateTime and maybe others. (test?)

While this always confused me, and I am happy they are gone, could we please add a note to the upgrade instruction from Zope 2 to 4 / 5??

https://zope.readthedocs.io/en/latest/migrations/zope4/index.html

Maybe somebody knows what has changed exactly or where to look for.

I just stumbled over a NameError: name 'DateTime' is not defined in one untested and rarely used template of my app.

Thank you!

P.S.: Now, this is crazy.

I just created a test to reproduce the issue and the test works - with no modification for Python 3, but for Python 2 it fails with the same NameError.

@goschtl
Copy link

goschtl commented Apr 1, 2020

The namespace stuff itself is a really nice thing but "it fly's a bit under radar". I thought always it would be a nice idea to add an Adapter to modifiy/include stuff in the namespace. Think on the principal or a library to format values...

tal:content="principal.id"
tal:content="format.to_german(item.price)"
etc...

@d-maurer
Copy link
Contributor

d-maurer commented Apr 1, 2020 via email

@icemac icemac modified the milestone: 5.0 Apr 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants