Jürgen Gmach

Results 156 comments of Jürgen Gmach

> @jugmac00, yes already registered to plone, forum seems deserted. The forum just has been created as the merge of Plone and Zope foundation was announced only last month. The...

+1 from me! Please also write a test for new code. > As json is not really a type.. When you have a look at the following list, the name...

While a general extension mechanism would be really great, I still think JSON is a common enough format to be supported in Zope core. There is always the question who...

I added it to Friday's sprint agenda, and I will try to come up with an answer afterwards.

Also, after reading the security chapter, I do not understand why in the following example the docstring matters, as there is an explicit security declaration: https://github.com/zopefoundation/Products.ZCatalog/pull/89#pullrequestreview-328032302

Thanks for your input. I guess I still need to read more about this topic and maybe play around in an instance to fully grasp that concept. - method with...

Hi @mauritsvanrees, thanks for investigating and sorry that my change caused you so much work. I have never used properties, but when I have a look at our documentation (...

:eyes: ``` ❯ all-repos-grep -C zope.json _updateProperty '*.py' output_zope/zopefoundation/Products.CMFCore:src/Products/CMFCore/exportimport/tests/test_memberdata.py: mdtool._updateProperty('email', 'value1') output_zope/zopefoundation/Products.CMFCore:src/Products/CMFCore/exportimport/tests/test_memberdata.py: mdtool._updateProperty('portal_skin', 'value2') output_zope/zopefoundation/Products.CMFCore:src/Products/CMFCore/exportimport/tests/test_memberdata.py: mdtool._updateProperty('listed', 'True') output_zope/zopefoundation/Products.CMFCore:src/Products/CMFCore/exportimport/tests/test_memberdata.py: mdtool._updateProperty('login_time', '2010/01/01') output_zope/zopefoundation/Products.CMFCore:src/Products/CMFCore/exportimport/tests/test_memberdata.py: mdtool._updateProperty('last_login_time', '2010/01/01') output_zope/zopefoundation/Products.CMFCore:src/Products/CMFCore/exportimport/tests/test_memberdata.py: mdtool._updateProperty('fullname', 'value3') output_zope/zopefoundation/Products.CMFCore:src/Products/CMFCore/exportimport/tests/test_properties.py: site._updateProperty('foo', 'Foo') output_zope/zopefoundation/Products.CMFCore:src/Products/CMFCore/exportimport/tests/test_properties.py:...

@leorochael's suggestion looks like a good compromise. Any objections or other opinions on this?

Once I finish chapter 4 of the Zope Dev guide, the very next topic to be revised is "5. Zope Products" / https://zope.readthedocs.io/en/latest/zdgbook/Products.html. The current state already explains how to...