Ticket #13708 (new Bug)

Opened 3 years ago

Last modified 3 years ago

Invalid property types in plip13350-edit-member-schema-ttw

Reported by: interra Owned by:
Priority: minor Milestone: 4.x
Component: Unknown Version:
Keywords: Cc:

Description

This bug report applies to  plone.app.users plip13350-edit-member-schema-ttw branch that is dedicated to implementing PLIP #13350. I'm submitting it here since Issues are disabled at Github for plone/plone.app.users repository.

Steps to reproduce:

  1. Add new Text field:
    1. on member-fields page click Add new field button.
    2. fill inputs (id: text_field, type: Text).
    3. select Text for Field type
    4. click Add
  2. login as plone user
  3. go to user's Personal Information form
  4. type test string on new property field
  5. click Save

One is getting following Traceback:

Traceback (innermost last):
 Module ZPublisher.Publish, line 126, in publish
 Module ZPublisher.mapply, line 77, in mapply
 Module ZPublisher.Publish, line 46, in call_object
 Module z3c.form.form, line 215, in __call__
 Module plone.z3cform.fieldsets.extensible, line 59, in update
 Module plone.z3cform.patch, line 30, in GroupForm_update
 Module z3c.form.group, line 141, in update
 Module plone.app.z3cform.csrf, line 21, in execute
 Module z3c.form.action, line 98, in execute
 Module z3c.form.button, line 315, in __call__
 Module z3c.form.button, line 170, in __call__
 Module plone.app.users.browser.z3cpersonalpreferences, line 160, in handleSave
 Module z3c.form.group, line 110, in applyChanges
 Module z3c.form.form, line 49, in applyChanges
 Module z3c.form.datamanager, line 91, in set
 Module plone.app.users.browser.z3cpersonalpreferences, line 76, in __setattr__
 Module plone.app.users.browser.z3cpersonalpreferences, line 63, in _setProperty
 Module Products.PlonePAS.tools.memberdata, line 264, in setMemberProperties
 Module Products.PlonePAS.sheet, line 85, in setProperty
 Module Products.PlonePAS.sheet, line 82, in validateProperty
PropertyValueError: Invalid value (test string) for property 'text_field' of type lines

One is expected to get personal information properly saved.

We do have fix for the issue and will submit pull request for it. The fix solves the issue only for standard filed types. There should be separate fix for extra field types that can potentially exist.

Change History

comment:1 Changed 3 years ago by interra

comment:2 Changed 3 years ago by interra

Pull request was merged in  608c9c6

Note: See TracTickets for help on using tickets.