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

Use latest z3c.form in Plone Classic UI #3459

Closed
5 tasks done
petschki opened this issue Mar 17, 2022 · 6 comments
Closed
5 tasks done

Use latest z3c.form in Plone Classic UI #3459

petschki opened this issue Mar 17, 2022 · 6 comments

Comments

@petschki
Copy link
Member

petschki commented Mar 17, 2022

PLIP (Plone Improvement Proposal)

Responsible Persons

Proposer: @petschki

Abstract

This PLIPs mission is to upgrade z3c.form to the latest available version in Plone Classic UI.

Assumption

Right now z3c.form is pinned to the latest 3.x version because of the dependencies to the subform functionality provided by z3c.form.object.SubformAdapter and z3c.form.object.ObjectSubForm. This has been removed in z3c.form since 4.x

Proposal & Implementation

To get the latest z3c.form we have to backport subform functionality to plone.z3cform and figure out which core packages needs to be refactored.

For addons we have to provide a deprecation message to the new plone.z3cform module

Deliverables

Participants

@petschki
Copy link
Member Author

petschki commented Mar 17, 2022

@plone/framework-team

I've created a branch with the updated z3c.form version and this what happened on jenkins ... plone/buildout.coredev#782

@petschki
Copy link
Member Author

plone.autoform and plone.app.portlets can safely be merged because there are only backwards compatible test fixes. CMFPlone has a backward compatible update to login form and can be merged too. The plone(.app).z3cform package Updates need z3c.form >= 4 and therefore an updated versions.cfg of buildout.coredev …

/cc @mauritsvanrees @jensens might wanna have a look?

@petschki
Copy link
Member Author

petschki commented Mar 18, 2022

PS: jenkins is down to 3 failing tests. I think I can fix them later today.

@tisto
Copy link
Sponsor Member

tisto commented Mar 21, 2022

@petschki I personally don't think this requires a PLIP. Though, for what it is worth I am +1.

@ale-rt
Copy link
Member

ale-rt commented Mar 22, 2022

+1 with this PLIP and what with Timo says about not having a PLIP for this!
Thanks @petschki

@petschki
Copy link
Member Author

Thanks for your comments. Indeed not really worth a PLIP ... I have merged the updates in plone.(app.)z3cform packages and bumped z3c.form version to 4.2 in cordev 6.0 today ... everything runs smooth.

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

3 participants