collective / pas.plugins.ldap

Zope (and Plone) PAS Plugin providing users and groups from LDAP directory
http://pypi.python.org/pypi/pas.plugins.ldap
Other
13 stars 20 forks source link

Set the memcached TTW setting in the form definition to unicode #47

Closed fredvd closed 6 years ago

fredvd commented 6 years ago

So that you can save the controlpanel form if you change this field.

fredvd commented 6 years ago

@jensens can you check this one liner when you have time? I'm not YAFOWIL proficient, but this aligns the form field with zope.schema.textline. ATM the pas.plugins.ldap controlpanel form cannot be saved at all, only GS setup is working.

[edit] pasted traceback:

2017-10-18 23:29:05 ERROR Zope.SiteErrorLog 1508362145.50.971901967998 http://localhost:8082/plone2/plone_ldapcontrolpanel Traceback (innermost last): Module ZPublisher.Publish, line 138, in publish Module ZPublisher.mapply, line 77, in mapply Module ZPublisher.Publish, line 48, in call_object Module Products.Five.browser.metaconfigure, line 485, in call Module Products.Five.browser.pagetemplatefile, line 125, in call Module Products.Five.browser.pagetemplatefile, line 59, in call Module zope.pagetemplate.pagetemplate, line 132, in pt_render Module five.pt.engine, line 98, in call Module z3c.pt.pagetemplate, line 163, in render Module chameleon.zpt.template, line 261, in render Module chameleon.template, line 191, in render Module chameleon.template, line 171, in render Module 55db20ab52918e098ba295673d47c49a.py, line 367, in render Module ba170f520f0f1c4adb219c723fd48ca9.py, line 282, in render_master Module fe57acb06bb94404342f1128b0633886.py, line 1227, in render_master Module ba170f520f0f1c4adb219c723fd48ca9.py, line 264, in fill_content Module fe57acb06bb94404342f1128b0633886.py, line 458, in render_content Module ba170f520f0f1c4adb219c723fd48ca9.py, line 253, in fill_main Module 55db20ab52918e098ba295673d47c49a.py, line 340, in __fill_prefs_configlet_main Module pas.plugins.ldap.properties, line 76, in form Module yafowil.controller, line 32, in init Module pas.plugins.ldap.plonecontrolpanel.controlpanel, line 30, in save Module pas.plugins.ldap.properties, line 113, in save Module pas.plugins.ldap.properties, line 225, in memcached Module plone.registry.record, line 82, in _set_value Module zope.schema._bootstrapfields, line 181, in validate Module zope.schema._bootstrapfields, line 309, in _validate Module zope.schema._bootstrapfields, line 209, in _validate WrongType: ('127.0.0.1:11211', <type 'unicode'>, 'value')

rnixx commented 6 years ago

LGTM